Kazunobu Kuriyama wrote:
> Thanks to many, Vim now passes test_channel successfully on OS X.
>
> Unfortunately, this is not the case when Vim is built with the option
> --disable-darwin.
>
> This is not because darwin-disabled Vim is inherently incapable of the
> channel feature, but just because there's no has() variable to represent it
> properly.
>
> Please look at line 31 of test_channel.vim:
>
> let s:chopt = has('macunix') ? {'waittime' : 1} : {}
>
> This is the key to make test_channel work successfully on OS X.
>
> In eval.c, the value 'macunix' is defined only if --disable-darwin is not
> specified at build time.
>
> As a result, when the option is specified at build time, has('macunix') is
> to be evaluated false at runtime and thus causing test_channel to get
> suprious failure.
>
> The attached patch fixes this issue and includes the proposal of
> introducing a new has() variable, 'osx'.
>
> This new variable is to be defined in eval.c when the compiler preset macro
> __APPLE__ is defined at build time. As a result, for any build on OS X,
> the value is always defined regardless of the presence of --disable-darwin.
>
> With this patch, I confirmed that even darwin-disabled Vim passes
> test_channel successfully.
>
> This is quite natural: What 'macunix' represents is some Mac specific code
> which is relevant to paste board and text conversion; it has nothing to do
> with IPC.
>
> So, IMHO, the use of 'macunix' to check IPC capabilities is an abuse. The
> new variable would help to represent the runtime characteristics of OS X.
Thanks! Do we have a better description for what the "mac" feature
stands for? Is that for pre-OS-X or also defined for OS-X?
I still wonder why we need this waittime at all. I believe you said
before that adding a short sleep does not help. We probably have to
tweak the code around where connect() is called.
--
MAN: Fetchez la vache!
GUARD: Quoi?
MAN: Fetchez la vache!
"Monty Python and the Holy Grail" PYTHON (MONTY) PICTURES LTD
/// Bram Moolenaar -- [email protected] -- http://www.Moolenaar.net \\\
/// sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\ an exciting new programming language -- http://www.Zimbu.org ///
\\\ help me help AIDS victims -- http://ICCF-Holland.org ///
--
--
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php
---
You received this message because you are subscribed to the Google Groups
"vim_dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.