Add a CrBuild command to ninja-build.vim

tools/vim/ninja-build.vim

 " Copyright (c) 2012 The Chromium Authors. All rights reserved.
 " Use of this source code is governed by a BSD-style license that can be
 " found in the LICENSE file.
 "
 " Adds a "Compile this file" function, using ninja. On Mac, binds Cmd-k to
 " this command. On Windows, Ctrl-F7 (which is the same as the VS default).
 "
 " Requires that gyp has already generated build.ninja files, and that ninja is
 " in your path (which it is automatically if depot_tools is in your path).
 "

[Nico, 2012/10/15 22:41:35]

Can you mention this up here too?

[enne (OOO), 2012/10/16 00:09:17]

Done.

 " Add the following to your .vimrc file:
 "     so /path/to/src/tools/vim/ninja-build.vim
 
 python << endpython
 import os
 import vim
 
 
 def path_to_current_buffer():
   """Returns the absolute path of the current buffer."""
@@ skipping 28 matching lines @@
       rel_mtime = os.path.getmtime(os.path.join(root, 'Release',  test_path))
     except os.error:
       rel_mtime = 0
     return rel_mtime - debug_mtime >= 15
   configuration = 'Debug'
   if is_release_15s_newer('build.ninja') or is_release_15s_newer('protoc'):
     configuration = 'Release'
   return configuration
 
 
-def compute_ninja_command(configuration=None):
+def compute_ninja_command_for_current_buffer(configuration=None):
   """Returns the shell command to compile the file in the current buffer."""
   if not configuration: configuration = guess_configuration()
   build_dir = os.path.join(path_to_source_root(), 'out', configuration)
 
   # ninja needs filepaths for the ^ syntax to be relative to the
   # build directory.
   file_to_build = path_to_current_buffer()
   file_to_build = os.path.relpath(file_to_build, build_dir)
 
   return ' '.join(['ninja', '-C', build_dir, file_to_build + '^'])
 
 
 def set_makepgr_to_single_file_ninja():
-  build_cmd = compute_ninja_command()
+  build_cmd = compute_ninja_command_for_current_buffer()
   if sys.platform == 'win32':
     # Escape \ for Vim, and ^ for both Vim and shell.
     build_cmd = build_cmd.replace('\\', '\\\\').replace('^', '^^^^')
   vim.command('let &makeprg=\'%s\'' % build_cmd)
+
+
+def compute_ninja_command_for_target(target='', configuration=None):
+  if not configuration: configuration = guess_configuration()
+  build_dir = os.path.join(path_to_source_root(), 'out', configuration)
+  return ' '.join(['ninja', '-C', build_dir, target])
+
+
+def set_makepgr_to_target_ninja(target=''):
+  vim.command('let &makeprg=\'%s\'' % compute_ninja_command_for_target(target))
 endpython
 
 fun! CrCompileFile()
   let l:oldmakepgr = &makeprg
   python set_makepgr_to_single_file_ninja()
   silent make | cwindow
   if !has('gui_running')
     redraw!
   endif
   let &makeprg = l:oldmakepgr
 endfun
 
+fun! CrCompileTarget(...)

[Nico, 2012/10/15 22:41:35]

This does more than just compiling. Maybe CrBuildTarget? (Nice from an
autocompletion perspective too, since "CrB" is now a unique prefix.) And maybe
CrBuild is enough too, but I don't care too much about the "Target" suffix.

Finally, should this build 'chrome' if no target is mentioned explicitly? That's
probably the more common case?

[enne (OOO), 2012/10/16 00:09:17]

I have mixed feelings about default building chrome.  How do you build
everything, then?

+  let l:oldmakepgr = &makeprg
+  let l:target = a:0 > 0 ? a:1 : ''
+  python set_makepgr_to_target_ninja(vim.eval('l:target'))
+  silent make | cwindow
+  if !has('gui_running')
+    redraw!
+  endif
+  let &makeprg = l:oldmakepgr
+endfun

[Nico, 2012/10/15 22:41:35]

Can you think of a way to not make this function look like an almost-identical
copy of the previous one? (Maybe make both delegate to a new 3rd function that
contains the actual code, with the function to call being a parameter?)

[enne (OOO), 2012/10/16 00:09:17]

Yeah, yeah.  I was optimizing for minimal vimscript writing.  Now I understand
how to return values from python functions to Vim and my brain is broken.

+
 command! CrCompileFile call CrCompileFile()
+command! -nargs=? CrCompileTarget call CrCompileTarget(<args>)
 
 if has('mac')
   map <D-k> :CrCompileFile<cr>
   imap <D-k> <esc>:CrCompileFile<cr>
 elseif has('win32')
   map <C-F7> :CrCompileFile<cr>
   imap <C-F7> <esc>:CrCompileFile<cr>
 elseif has('unix')
   map <Leader>o :CrCompileFile<cr>
 endif