Re: Patch 7.4.2118
2016-07-30 21:12 GMT+09:00 Bram Moolenaar : > > Kent Sibilev wrote: > > > On Friday, July 29, 2016 at 3:01:36 PM UTC-4, Bram Moolenaar wrote: > > > Patch 7.4.2118 > > > Problem:Mac: can't build with tiny features. > > > Solution: Don't define FEAT_CLIPBOARD unconditionally. (Kazunobu > Kuriyama) > > > Files: src/vim.h > > > > After this patch the FEAT_CLIPBOARD is not enabled when building with > huge feature set. The issue is that this condition > > > > # if defined(FEAT_SMALL) && !defined(FEAT_CLIPBOARD) > > # define FEAT_CLIPBOARD > > # endif > > > > is evaluated before the following #include > > > > #include "feature.h" > > > > which actually defines FEAT_SMALL. > > > > It looks like that the following patch fixes it: > > > > diff --git a/src/vim.h b/src/vim.h > > index b785527..157be28 100644 > > --- a/src/vim.h > > +++ b/src/vim.h > > @@ -84,6 +84,8 @@ > > # define ROOT_UID 0 > > #endif > > > > +#include "feature.h" /* #defines for optionals and features */ > > + > > /* > > * MACOS_CLASSIC compiling for MacOS prior to MacOS X > > * MACOS_X_UNIX compiling for MacOS X (using os_unix.c) > > @@ -180,8 +182,6 @@ > > #endif > > > > > > -#include "feature.h" /* #defines for optionals and features */ > > - > > /* +x11 is only enabled when it's both available and wanted. */ > > #if defined(HAVE_X11) && defined(WANT_X11) > > # define FEAT_X11 > > Ah, so this never worked properly. > > Moving feature.h has other problems. I'll move the part that depends on > FEAT_SMALL down. > Hi, I confirm that, thanks to Patch 7.4.2122, all the feature sets are now available on OS X with appropriate clipboard feature inclusion, i.e., +clipboard for all but tiny. Regards, Kazunobu > > -- > A parent can be arrested if his child cannot hold back a burp during a > church > service. > [real standing law in Nebraska, United States of America] > > /// Bram Moolenaar -- b...@moolenaar.net -- 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 vim_dev+unsubscr...@googlegroups.com. > For more options, visit https://groups.google.com/d/optout. > -- -- 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 vim_dev+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.
Re: Patch 7.4.2118
Kent Sibilev wrote: > On Friday, July 29, 2016 at 3:01:36 PM UTC-4, Bram Moolenaar wrote: > > Patch 7.4.2118 > > Problem:Mac: can't build with tiny features. > > Solution: Don't define FEAT_CLIPBOARD unconditionally. (Kazunobu Kuriyama) > > Files: src/vim.h > > After this patch the FEAT_CLIPBOARD is not enabled when building with huge > feature set. The issue is that this condition > > # if defined(FEAT_SMALL) && !defined(FEAT_CLIPBOARD) > # define FEAT_CLIPBOARD > # endif > > is evaluated before the following #include > > #include "feature.h" > > which actually defines FEAT_SMALL. > > It looks like that the following patch fixes it: > > diff --git a/src/vim.h b/src/vim.h > index b785527..157be28 100644 > --- a/src/vim.h > +++ b/src/vim.h > @@ -84,6 +84,8 @@ > # define ROOT_UID 0 > #endif > > +#include "feature.h" /* #defines for optionals and features */ > + > /* > * MACOS_CLASSIC compiling for MacOS prior to MacOS X > * MACOS_X_UNIX compiling for MacOS X (using os_unix.c) > @@ -180,8 +182,6 @@ > #endif > > > -#include "feature.h" /* #defines for optionals and features */ > - > /* +x11 is only enabled when it's both available and wanted. */ > #if defined(HAVE_X11) && defined(WANT_X11) > # define FEAT_X11 Ah, so this never worked properly. Moving feature.h has other problems. I'll move the part that depends on FEAT_SMALL down. -- A parent can be arrested if his child cannot hold back a burp during a church service. [real standing law in Nebraska, United States of America] /// Bram Moolenaar -- b...@moolenaar.net -- 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 vim_dev+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.
Re: Patch 7.4.2118
Hi, I confirm that Kent Sibilev's patch fixes the issue. For some reason (perhaps, due to whitespace conversion), the second hunk of the patch was rejected when I tried to include it into my repo. Attached is another version of his patch which fixes that whitespace issue. Regards, Kazunobu 2016-07-30 14:53 GMT+09:00 Kent Sibilev : > On Friday, July 29, 2016 at 3:01:36 PM UTC-4, Bram Moolenaar wrote: > > Patch 7.4.2118 > > Problem:Mac: can't build with tiny features. > > Solution: Don't define FEAT_CLIPBOARD unconditionally. (Kazunobu > Kuriyama) > > Files: src/vim.h > > After this patch the FEAT_CLIPBOARD is not enabled when building with huge > feature set. The issue is that this condition > > # if defined(FEAT_SMALL) && !defined(FEAT_CLIPBOARD) > # define FEAT_CLIPBOARD > # endif > > is evaluated before the following #include > > #include "feature.h" > > which actually defines FEAT_SMALL. > > It looks like that the following patch fixes it: > > diff --git a/src/vim.h b/src/vim.h > index b785527..157be28 100644 > --- a/src/vim.h > +++ b/src/vim.h > @@ -84,6 +84,8 @@ > # define ROOT_UID 0 > #endif > > +#include "feature.h" /* #defines for optionals and features */ > + > /* > * MACOS_CLASSIC compiling for MacOS prior to MacOS X > * MACOS_X_UNIX compiling for MacOS X (using os_unix.c) > @@ -180,8 +182,6 @@ > #endif > > > -#include "feature.h" /* #defines for optionals and features */ > - > /* +x11 is only enabled when it's both available and wanted. */ > #if defined(HAVE_X11) && defined(WANT_X11) > # define FEAT_X11 > > > Regards, > Kent. > > -- > -- > 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 vim_dev+unsubscr...@googlegroups.com. > For more options, visit https://groups.google.com/d/optout. > -- -- 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 vim_dev+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout. kent-sibilev-vim-h.patch Description: Binary data
Re: Patch 7.4.2118
On Friday, July 29, 2016 at 3:01:36 PM UTC-4, Bram Moolenaar wrote: > Patch 7.4.2118 > Problem:Mac: can't build with tiny features. > Solution: Don't define FEAT_CLIPBOARD unconditionally. (Kazunobu Kuriyama) > Files: src/vim.h After this patch the FEAT_CLIPBOARD is not enabled when building with huge feature set. The issue is that this condition # if defined(FEAT_SMALL) && !defined(FEAT_CLIPBOARD) # define FEAT_CLIPBOARD # endif is evaluated before the following #include #include "feature.h" which actually defines FEAT_SMALL. It looks like that the following patch fixes it: diff --git a/src/vim.h b/src/vim.h index b785527..157be28 100644 --- a/src/vim.h +++ b/src/vim.h @@ -84,6 +84,8 @@ # define ROOT_UID 0 #endif +#include "feature.h" /* #defines for optionals and features */ + /* * MACOS_CLASSIC compiling for MacOS prior to MacOS X * MACOS_X_UNIX compiling for MacOS X (using os_unix.c) @@ -180,8 +182,6 @@ #endif -#include "feature.h" /* #defines for optionals and features */ - /* +x11 is only enabled when it's both available and wanted. */ #if defined(HAVE_X11) && defined(WANT_X11) # define FEAT_X11 Regards, Kent. -- -- 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 vim_dev+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.
Patch 7.4.2118
Patch 7.4.2118 Problem:Mac: can't build with tiny features. Solution: Don't define FEAT_CLIPBOARD unconditionally. (Kazunobu Kuriyama) Files: src/vim.h *** ../vim-7.4.2117/src/vim.h 2016-07-26 22:02:50.147766335 +0200 --- src/vim.h 2016-07-29 20:58:47.734859474 +0200 *** *** 98,108 # ifndef HAVE_CONFIG_H # define UNIX # endif ! # ifndef FEAT_CLIPBOARD # define FEAT_CLIPBOARD ! # if defined(FEAT_SMALL) && !defined(FEAT_MOUSE) ! # define FEAT_MOUSE ! # endif # endif #endif #if defined(MACOS_X) || defined(MACOS_CLASSIC) --- 98,108 # ifndef HAVE_CONFIG_H # define UNIX # endif ! # if defined(FEAT_SMALL) && !defined(FEAT_CLIPBOARD) # define FEAT_CLIPBOARD ! # endif ! # if defined(FEAT_SMALL) && !defined(FEAT_MOUSE) ! # define FEAT_MOUSE # endif #endif #if defined(MACOS_X) || defined(MACOS_CLASSIC) *** ../vim-7.4.2117/src/version.c 2016-07-29 20:50:19.863640329 +0200 --- src/version.c 2016-07-29 20:59:24.086517317 +0200 *** *** 760,761 --- 760,763 { /* Add new patch number below this line */ + /**/ + 2118, /**/ -- Snoring is prohibited unless all bedroom windows are closed and securely locked. [real standing law in Massachusetts, United States of America] /// Bram Moolenaar -- b...@moolenaar.net -- 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 vim_dev+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.