Permalink
Switch branches/tags
Commits on Jun 16, 2011
  1. Merge remote-tracking branch 'greyblake/master'

    spf13 committed Jun 16, 2011
    Conflicts:
    	README.markdown
    	plugin/preview.vim
Commits on May 19, 2011
  1. Merge pull request #11 from lukaszkorecki/file_name

    greyblake committed May 19, 2011
    Adds name to the generated file so that preview works with QuickLook
  2. adds name to the generated file so that preview works with QuickLook …

    lukaszkorecki committed May 19, 2011
    …(and
    
    potentialy other browsers which require full file name not starting with a .)
Commits on Apr 1, 2011
Commits on Mar 22, 2011
  1. v0.6

    greyblake committed Mar 22, 2011
  2. Removed a space in README

    greyblake committed Mar 22, 2011
Commits on Mar 16, 2011
  1. updating readme

    spf13 committed Mar 16, 2011
  2. Fixing to work on Mac OSX

    spf13 committed Mar 16, 2011
Commits on Feb 14, 2011
  1. fixed typos

    greyblake committed Feb 14, 2011
Commits on Jan 14, 2011
Commits on Jan 8, 2011
Commits on Jan 7, 2011
  1. Version 0.5 is out

    greyblake committed Jan 7, 2011
  2. Merged decurtis-master

    greyblake committed Jan 7, 2011
  3. Another fix for programs with arguments.

    milkypostman committed Jan 7, 2011
    exec wants either multiple arguments, i.e., exec "prog", "arg1", "arg2".
    OR it wants a single string "prog arg1 arg2 ...".  On OSX and when you
    want a program to use arguments when loading, you need to consider that
    the value of `app` is not just a single word (representing an
    application).  In my case, `app="open -g -a Safari" because of OSX and
    because I want it to load in the background (-g).
  4. Double fork to ensure reaping thread via Process#detach is successful

    guns
    guns committed Jan 7, 2011
    The last attempt against zombie creation was unsuccessful because the
    reaping thread was aborted along with the main vim-ruby thread.
    
    If we double fork the browser program we can:
    
     * ensure that Process#detach will correctly setup and reap the
       grandchild process
     * create browser instances that are not children of the current vim
    
    This is essentially what Process#detach does when it works correctly
Commits on Jan 6, 2011
Commits on Jan 5, 2011
  1. no, STDOUT and STDERR, not $stdout and $stderr

    guns
    guns committed Jan 5, 2011
    Dang, I guess $stdout and $stderr no longer point to STDOUT and STDERR
    in vim. My mistake.
  2. Prevent zombies; use proper parameter escaping for exec; silence both…

    guns
    guns committed Jan 5, 2011
    … $stdout, $stderr
    
     * Beware: if you fork, you must always reap, lest the dead rise again
       and clutter your process tables!
    
     * Regexp.escape is the wrong tool for shell escaping. Kernel#exec
       properly escapes parameters when given as separate arguments
    
     * Silence both $stdout and $stderr in the fork (not STDOUT and STDERR)
    
    Ruby 1.9 can handle all of the above in a nice little call:
    
        Process.detach spawn(program, arg1, arg2, [:out, :err] => '/dev/null')
Commits on Jan 4, 2011
  1. Calling the view program was not properly redirected.

    milkypostman committed Jan 4, 2011
    There was some missing redirection code.  Here is how the current commit
    redirects output.
    
    When searching for an app to run all output is redirected to /dev/null
    as the ruby "system" command return true/false based on the return code
    fo the application.  Since we are looking at the 'which' command it will
    return 0 (true) if it finds the app and 1 (false) if it doesn't.
    
    When executing the viewer, stderr is redirected to stdout and stdout is
    redirected to /dev/null.  Note that stderr is _not_ redirected to
    /dev/null because of order of redirection.  However, this has the nice
    consequence of printing an error in the VIM window if the preview
    application cannot be called or has any err output.
  2. Support filenames with spaces.

    milkypostman committed Jan 4, 2011
    Filenames need to be escaped when sending them to exec.
Commits on Jan 2, 2011
  1. Update to allow this to work in OSX with Safari.

    milkypostman committed Jan 2, 2011
    Without an html extension, Safari will not open the resulting file.
    
    Also, in testing if the browser exists, it doesn't allow the addition of
    arguments on the call.  Thus, there is also a change to the way
    existance of executable is checked.  Splits the browser string and
    checks the first item.