This is the mail archive of the cygwin mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

RE: binutils 20080523-1: version string breaks python2.5 distutils


Christopher Faylor wrote on 29 May 2008 03:02:

> On Thu, May 29, 2008 at 01:49:55AM +0000, Joe Pham wrote:
>> As part of the binutils 20080523-1 package, both ld (ld -v) and dllwrap
>> (dllwrap --version) report their version as "2.18.50.20080523".  This
>> version string format causes the python2.5 distutils tool to abort with
>> error.  On Linux, the format would have been something like "2.18.50
>> 20080523".  Any chance to make this consistent with Linux?
> 
> The version string is straight from a standard binutils distribution.
> Nothing has been done to modify it.  It comes from this line in the
> bfd Makefile:
> 
> ./bfd/Makefile:  
> bfd_version_string="\"$(VERSION).$${bfd_version_date}\"" ;\ 
> 
> So, no.  I'd suggest modifying python.  It pretty clearly doesn't adapt
> correctly to a standard binutils version string.

  To be fair, it looks like the bfd version format may have changed over
time.  The space was introduced by bje four years ago:

http://sourceware.org/cgi-bin/cvsweb.cgi/src/bfd/Makefile.am.diff?r1=1.133&r
2=1.134&cvsroot=src

Then it was removed again, last year, in 

http://sourceware.org/cgi-bin/cvsweb.cgi/src/bfd/Makefile.am.diff?r1=1.187&r
2=1.188&cvsroot=src

with the comment

	* Makefile.am (bfdver.h): Use "." not " " between version number
	and date.

  So, yes, Python should definitely use a pattern match to ignore the
separator character.

    cheers,
      DaveK
-- 
Can't think of a witty .sigline today....


--
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple
Problem reports:       http://cygwin.com/problems.html
Documentation:         http://cygwin.com/docs.html
FAQ:                   http://cygwin.com/faq/


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]