Re: 7.3.967 fails on Tiger PPC due to Python problem

2013-05-27 Thread dv1445
björn, on 05/27/13 at 08:22:23 +0200, wrote:
> On Mon, May 27, 2013 at 2:02 AM,  wrote:
> 
> > björn, on 05/24/13 at 15:35:52 +0200, wrote:
> > > I had a look and there are two issues:
> > >
> > > 1. Py_VISIT was introduced in Python v2.4, perhaps Mac OS X Tiger is
> > > including an older Python version (otherwise I don't know why the linker
> > > would fail)
> >
> > I don't think it's this; I have 2.7.5 installed, which MacVim prior to
> > 7.3.965 works fine with.  And the offending commit (and later ones) appear
> > to be recognizing 2.7.5, rather than the old one built into OSX.
> 
> 
> I had a look -- Tiger ships with 2.3.x.  It would seem that you are
> compiling with the 2.7.x headers but linking against the 2.3.x library.
>  Most likely because the linker gets the "-framework Python" flag, which
> causes it to look at the system default version.  I've never built against
> a non-default Python version myself so I can't be of much help with it.

I just pulled the latest MacVim (7.3.1020) on my Tiger PPC box.  It builds!  
Thanks for the fix.

I also scanned the old config.log from when it wouldn't build, and I can find 
no indication that 2.3.x was being picked up.  Everything mentioned either 
/Library/frameworks/python.framework/blah-blah-2.7-blah, or said "-framework 
Python".  The latter is also what config.log says on the working versions and 
on other archs too.

-gmn

-- 
-- 
You received this message from the "vim_mac" 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_mac" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to vim_mac+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.




Re: 7.3.967 fails on Tiger PPC due to Python problem

2013-05-27 Thread dv1445
Now that I'm back from out of town, I will very soon be able to try to build on 
Tiger PPC again after the recent changes.

> > > 1. Py_VISIT was introduced in Python v2.4, perhaps Mac OS X Tiger is
> > > including an older Python version (otherwise I don't know why the linker
> > > would fail)
> > I don't think it's this; I have 2.7.5 installed, which MacVim prior to
> > 7.3.965 works fine with.  And the offending commit (and later ones) appear
> > to be recognizing 2.7.5, rather than the old one built into OSX.
> I had a look -- Tiger ships with 2.3.x.  It would seem that you are
> compiling with the 2.7.x headers but linking against the 2.3.x library.
>  Most likely because the linker gets the "-framework Python" flag, which
> causes it to look at the system default version.  I've never built against
> a non-default Python version myself so I can't be of much help with it.

I'll check my build logs when I get a hold of my TigerPPC box, to see if 2.3.x 
is entering the picture.  But if this were the case, wouldn't it really be 
weird that (a) everything was fine as of 7.3.806 and probably until 7.3.965, 
(b) everything always was and is fine with plain BramVim?  I'm no programmer, 
but why would BramVim vs. MacVim make a difference to the linker?

-gmn

-- 
-- 
You received this message from the "vim_mac" 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_mac" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to vim_mac+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.




Re: 7.3.967 fails on Tiger PPC due to Python problem

2013-05-26 Thread björn
On Mon, May 27, 2013 at 2:02 AM,  wrote:

> björn, on 05/24/13 at 15:35:52 +0200, wrote:
> > I had a look and there are two issues:
> >
> > 1. Py_VISIT was introduced in Python v2.4, perhaps Mac OS X Tiger is
> > including an older Python version (otherwise I don't know why the linker
> > would fail)
>
> I don't think it's this; I have 2.7.5 installed, which MacVim prior to
> 7.3.965 works fine with.  And the offending commit (and later ones) appear
> to be recognizing 2.7.5, rather than the old one built into OSX.


I had a look -- Tiger ships with 2.3.x.  It would seem that you are
compiling with the 2.7.x headers but linking against the 2.3.x library.
 Most likely because the linker gets the "-framework Python" flag, which
causes it to look at the system default version.  I've never built against
a non-default Python version myself so I can't be of much help with it.

Björn

-- 
-- 
You received this message from the "vim_mac" 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_mac" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to vim_mac+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.




Re: 7.3.967 fails on Tiger PPC due to Python problem

2013-05-26 Thread dv1445
Sorry for the delay; been out of town a few days with no computer.

björn, on 05/24/13 at 15:35:52 +0200, wrote:
> I had a look and there are two issues:
> 
> 1. Py_VISIT was introduced in Python v2.4, perhaps Mac OS X Tiger is
> including an older Python version (otherwise I don't know why the linker
> would fail)

I don't think it's this; I have 2.7.5 installed, which MacVim prior to 7.3.965 
works fine with.  And the offending commit (and later ones) appear to be 
recognizing 2.7.5, rather than the old one built into OSX.

-gmn

-- 
-- 
You received this message from the "vim_mac" 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_mac" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to vim_mac+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.




Re: 7.3.967 fails on Tiger PPC due to Python problem

2013-05-24 Thread Bram Moolenaar

Björn Winckler wrote:

> On Thu, May 23, 2013 at 1:26 PM, Phil Dobbin  wrote:
> 
> > On 23/05/2013 04:53, dv1...@wayne.edu wrote:
> >
> > > Phil Dobbin, on 05/23/13 at 01:50:52 +0100, wrote:
> > >> On 19/05/2013 06:52, dv1...@wayne.edu wrote:
> > >>
> > >>> dv1...@wayne.edu, on 05/19/13 at 01:11:47 -0400, wrote:
> >  MacVim failed to build on OS X 10.4.11 (PPC) after pulling to 7.3.967.
> >  Everything as of the 7.3.806 pull from February was just fine.  Seems
> >  the recent fiddling with Python in BramLand has messed things up for
> >  poor Tiger.
> > >>>
> > >>> I forgot to mention that on Tiger PPC, BramVim 7.3.967 (and indeed
> > 7.3.969) built fine.  Just a point of comparison.
> > >>
> > >> I apologise for not being to help with your dilemma but I've just
> > >> acquired a couple of legacy G4 PowerMacs running 10.4.11 & if there is a
> > >> link to a tarball of 7.3.806 available, I would greatly appreciate
> > >> having it. Snapshot 55's Python is unable to cope with a lot of my
> > plugins.
> > >
> > > The best I can do is to point you towards a zipped archive of the source
> > code for 7.3.806, which comes from Björn's repo:
> > https://github.com/b4winckler/macvim/archive/1ba220ad3da69e3607a7e36b97c66e43bca68790.zip
> > .
> > >
> > > I discovered (reported in an earlier post) that it's patch 7.3.965 that
> > introduces what the build process's error messages claim to be the
> > offending lines.  It seems really weird that BramVim would build without
> > problem while MacVim would choke.
> >
> > Many thanks for that.
> >
> > It may be worth your while asking Bram on vim_use about this. Although,
> > saying that, he may have his hands full vim of Vim 7.4 at the moment.
> >
> 
> I had a look and there are two issues:
> 
> 1. Py_VISIT was introduced in Python v2.4, perhaps Mac OS X Tiger is
> including an older Python version (otherwise I don't know why the linker
> would fail)

if_python.c contains:

#ifndef Py_VISIT
# define Py_VISIT(obj) visit(obj, arg)
#endif

Why doesn't that work?

> 2. The "suggest explicit braces to avoid ambiguous ‘else’" warning is
> because of code like "if (cond) Py_DECREF(...)" and Py_DECREF() is a macro
> which expands to "if (..) ... else ...".  So the code will work as intended
> by there is a warning.

I'll add a patch for that.

> So, #1 is the only serious problem.  I guess Vim no longer supports older
> versions of Python.  I'm not sure what the policy on being backwards
> compatible with older Python versions is.

We try to support Python 2.2.  Should work.

-- 
CRONE:  Who sent you?
ARTHUR: The Knights Who Say GNU!
CRONE:  Aaaagh!  (she looks around in rear) No!  We have no licenses here.
   "Monty Python and the Holy editor wars" PYTHON (MONTY) SOFTWARE LTD

 /// 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_mac" 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_mac" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to vim_mac+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.




Re: 7.3.967 fails on Tiger PPC due to Python problem

2013-05-24 Thread björn
On Thu, May 23, 2013 at 1:26 PM, Phil Dobbin  wrote:

> On 23/05/2013 04:53, dv1...@wayne.edu wrote:
>
> > Phil Dobbin, on 05/23/13 at 01:50:52 +0100, wrote:
> >> On 19/05/2013 06:52, dv1...@wayne.edu wrote:
> >>
> >>> dv1...@wayne.edu, on 05/19/13 at 01:11:47 -0400, wrote:
>  MacVim failed to build on OS X 10.4.11 (PPC) after pulling to 7.3.967.
>  Everything as of the 7.3.806 pull from February was just fine.  Seems
>  the recent fiddling with Python in BramLand has messed things up for
>  poor Tiger.
> >>>
> >>> I forgot to mention that on Tiger PPC, BramVim 7.3.967 (and indeed
> 7.3.969) built fine.  Just a point of comparison.
> >>
> >> I apologise for not being to help with your dilemma but I've just
> >> acquired a couple of legacy G4 PowerMacs running 10.4.11 & if there is a
> >> link to a tarball of 7.3.806 available, I would greatly appreciate
> >> having it. Snapshot 55's Python is unable to cope with a lot of my
> plugins.
> >
> > The best I can do is to point you towards a zipped archive of the source
> code for 7.3.806, which comes from Björn's repo:
> https://github.com/b4winckler/macvim/archive/1ba220ad3da69e3607a7e36b97c66e43bca68790.zip
> .
> >
> > I discovered (reported in an earlier post) that it's patch 7.3.965 that
> introduces what the build process's error messages claim to be the
> offending lines.  It seems really weird that BramVim would build without
> problem while MacVim would choke.
>
> Many thanks for that.
>
> It may be worth your while asking Bram on vim_use about this. Although,
> saying that, he may have his hands full vim of Vim 7.4 at the moment.
>

I had a look and there are two issues:

1. Py_VISIT was introduced in Python v2.4, perhaps Mac OS X Tiger is
including an older Python version (otherwise I don't know why the linker
would fail)

2. The "suggest explicit braces to avoid ambiguous ‘else’" warning is
because of code like "if (cond) Py_DECREF(...)" and Py_DECREF() is a macro
which expands to "if (..) ... else ...".  So the code will work as intended
by there is a warning.

So, #1 is the only serious problem.  I guess Vim no longer supports older
versions of Python.  I'm not sure what the policy on being backwards
compatible with older Python versions is.

Björn

-- 
-- 
You received this message from the "vim_mac" 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_mac" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to vim_mac+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.




Re: 7.3.967 fails on Tiger PPC due to Python problem

2013-05-23 Thread Phil Dobbin
On 23/05/2013 04:53, dv1...@wayne.edu wrote:

> Phil Dobbin, on 05/23/13 at 01:50:52 +0100, wrote:
>> On 19/05/2013 06:52, dv1...@wayne.edu wrote:
>>
>>> dv1...@wayne.edu, on 05/19/13 at 01:11:47 -0400, wrote:
 MacVim failed to build on OS X 10.4.11 (PPC) after pulling to 7.3.967.
 Everything as of the 7.3.806 pull from February was just fine.  Seems
 the recent fiddling with Python in BramLand has messed things up for
 poor Tiger.
>>>
>>> I forgot to mention that on Tiger PPC, BramVim 7.3.967 (and indeed 7.3.969) 
>>> built fine.  Just a point of comparison.
>>
>> I apologise for not being to help with your dilemma but I've just
>> acquired a couple of legacy G4 PowerMacs running 10.4.11 & if there is a
>> link to a tarball of 7.3.806 available, I would greatly appreciate
>> having it. Snapshot 55's Python is unable to cope with a lot of my plugins.
> 
> The best I can do is to point you towards a zipped archive of the source code 
> for 7.3.806, which comes from Björn's repo: 
> https://github.com/b4winckler/macvim/archive/1ba220ad3da69e3607a7e36b97c66e43bca68790.zip.
> 
> I discovered (reported in an earlier post) that it's patch 7.3.965 that 
> introduces what the build process's error messages claim to be the offending 
> lines.  It seems really weird that BramVim would build without problem while 
> MacVim would choke.

Many thanks for that.

It may be worth your while asking Bram on vim_use about this. Although,
saying that, he may have his hands full vim of Vim 7.4 at the moment.

Cheers,

  Phil...

-- 
currently (ab)using
CentOS 5.9 & 6.4, Debian Squeeze & Wheezy, Fedora Beefy, Spherical &
That Damn Cat, Lubuntu 12.10, OS X Snow Leopard & Tiger, Ubuntu Precise,
Quantal & Raring
GnuGPG Key : http://phildobbin.org/publickey.asc


-- 
-- 
You received this message from the "vim_mac" 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_mac" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to vim_mac+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.




Re: 7.3.967 fails on Tiger PPC due to Python problem

2013-05-22 Thread dv1445
Phil Dobbin, on 05/23/13 at 01:50:52 +0100, wrote:
> On 19/05/2013 06:52, dv1...@wayne.edu wrote:
> 
> > dv1...@wayne.edu, on 05/19/13 at 01:11:47 -0400, wrote:
> >> MacVim failed to build on OS X 10.4.11 (PPC) after pulling to 7.3.967.
> >> Everything as of the 7.3.806 pull from February was just fine.  Seems
> >> the recent fiddling with Python in BramLand has messed things up for
> >> poor Tiger.
> > 
> > I forgot to mention that on Tiger PPC, BramVim 7.3.967 (and indeed 7.3.969) 
> > built fine.  Just a point of comparison.
> 
> I apologise for not being to help with your dilemma but I've just
> acquired a couple of legacy G4 PowerMacs running 10.4.11 & if there is a
> link to a tarball of 7.3.806 available, I would greatly appreciate
> having it. Snapshot 55's Python is unable to cope with a lot of my plugins.

The best I can do is to point you towards a zipped archive of the source code 
for 7.3.806, which comes from Björn's repo: 
https://github.com/b4winckler/macvim/archive/1ba220ad3da69e3607a7e36b97c66e43bca68790.zip.

I discovered (reported in an earlier post) that it's patch 7.3.965 that 
introduces what the build process's error messages claim to be the offending 
lines.  It seems really weird that BramVim would build without problem while 
MacVim would choke.

-gmn

-- 
-- 
You received this message from the "vim_mac" 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_mac" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to vim_mac+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.




Re: 7.3.967 fails on Tiger PPC due to Python problem

2013-05-22 Thread Phil Dobbin
On 19/05/2013 06:52, dv1...@wayne.edu wrote:

> dv1...@wayne.edu, on 05/19/13 at 01:11:47 -0400, wrote:
>> MacVim failed to build on OS X 10.4.11 (PPC) after pulling to 7.3.967.
>> Everything as of the 7.3.806 pull from February was just fine.  Seems
>> the recent fiddling with Python in BramLand has messed things up for
>> poor Tiger.
> 
> I forgot to mention that on Tiger PPC, BramVim 7.3.967 (and indeed 7.3.969) 
> built fine.  Just a point of comparison.

I apologise for not being to help with your dilemma but I've just
acquired a couple of legacy G4 PowerMacs running 10.4.11 & if there is a
link to a tarball of 7.3.806 available, I would greatly appreciate
having it. Snapshot 55's Python is unable to cope with a lot of my plugins.

Any help appreciated.

Cheers,

  Phil...

-- 
currently (ab)using
CentOS 5.9 & 6.4, Debian Squeeze & Wheezy, Fedora Beefy, Spherical &
That Damn Cat, Lubuntu 12.10, OS X Snow Leopard & Tiger, Ubuntu Precise,
Quantal & Raring
GnuGPG Key : http://phildobbin.org/publickey.asc


-- 
-- 
You received this message from the "vim_mac" 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_mac" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to vim_mac+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.




Re: 7.3.967 fails on Tiger PPC due to Python problem

2013-05-22 Thread dv1445
dv1...@wayne.edu, on 05/19/13 at 01:11:47 -0400, wrote:
> Scrolling up through the terminal output reveals some warnings
> that are potentially useful:
> 
> -pipe  -DMACOS_X_UNIX -no-cpp-precomp  -g -O -U_FORTIFY_SOURCE 
> -D_FORTIFY_SOURCE=1   
> -I/Library/Frameworks/Python.framework/Versions/2.7/include/python2.7 
> -DPYTHON_HOME=\"/Library/Frameworks/Python.framework/Versions/2.7\"  -o 
> objects/if_python.o if_python.c
> In file included from if_python.c:660:
> if_py_both.h: In function ‘OptionsDestructor’:
> if_py_both.h:1415: warning: suggest explicit braces to avoid ambiguous ‘else’
> if_py_both.h: In function ‘OptionsTraverse’:
> if_py_both.h:1423: warning: implicit declaration of function ‘Py_VISIT’
> if_py_both.h: In function ‘OptionsClear’:
> if_py_both.h:1430: warning: implicit declaration of function ‘Py_CLEAR’
> if_py_both.h: In function ‘WinListDestructor’:
> if_py_both.h:2112: warning: suggest explicit braces to avoid ambiguous ‘else’

Since the choking was on the functions Py_{VISIT,CLEAR}, and the
warnings reflected the lines 1423 and 1430 in if_py_both.h, I decided
to do an 'hg blame' on that file.  The result is that the lines in
question were most recently touched by patch 7.3.965 (the python
garbage collection patch).

I cannot bisect to test commits prior to that, because updating to
such a commit (e.g., 7.3.930) deletes all the MacVim parts and leaves
the working directory as plain BramVim.  As I noted earlier, BramVim
doesn't have the problem in question.

-gmn

-- 
-- 
You received this message from the "vim_mac" 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_mac" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to vim_mac+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.




Re: 7.3.967 fails on Tiger PPC due to Python problem

2013-05-18 Thread dv1445
dv1...@wayne.edu, on 05/19/13 at 01:11:47 -0400, wrote:
> MacVim failed to build on OS X 10.4.11 (PPC) after pulling to 7.3.967.
> Everything as of the 7.3.806 pull from February was just fine.  Seems
> the recent fiddling with Python in BramLand has messed things up for
> poor Tiger.

I forgot to mention that on Tiger PPC, BramVim 7.3.967 (and indeed 7.3.969) 
built fine.  Just a point of comparison.

-gmn

-- 
-- 
You received this message from the "vim_mac" 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_mac" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to vim_mac+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/groups/opt_out.