Re: xerces-c, xerces-c-devel, xerces-c-doc 2.1.0-2 available for review/upload
Sorry about that guys... I don't like making excuses, so I'll just blame it on the hallucinogenic drugs :) I'll have a -1 version back there soon and I'll remove the -2 version. It'll contain the fix for the problem with the wrong dll (my tar.incl file still had the libxerces instead of cygxerces). Also, Gareth mentioned that there is a one line fix patch for xerces-c 2.1.0. I'd like to get that in as well if I can dig it up. src/xercesc/dom/impl/DOMDocumentTypeImpl.cpp line 236 should be internalSubset = docImpl->cloneString(internalSubset); should be pretty obvious when you see it. Gareth _ Surf the Web without missing calls! Get MSN Broadband. http://resourcecenter.msn.com/access/plans/freeactivation.asp
Re: xerces-c, xerces-c-devel, xerces-c-doc 2.1.0-2 available for review/upload
Sorry about that guys... I don't like making excuses, so I'll just blame it on the hallucinogenic drugs :) I'll have a -1 version back there soon and I'll remove the -2 version. It'll contain the fix for the problem with the wrong dll (my tar.incl file still had the libxerces instead of cygxerces). Also, Gareth mentioned that there is a one line fix patch for xerces-c 2.1.0. I'd like to get that in as well if I can dig it up. Charles, To address the dll naming with the version inside (cygxerces-c2_1_0.dll), I had some similar concerns when I started mucking with this package. I had decided not to change the dll name because xerces-c exports C++ types and thought it to be a big mess if there wasn't some type of versioning embedded in the name, which is probably why they did it this way (my assumption, could be wrong). At build time, there are three different libs to link against (when xerces-c-devel is installed): /usr/lib/libxerces-c2_1_0.dll.a /usr/lib/libxerces-c.dll.a (symlink to libxerces-c2_1_0.dll.a) /usr/lib/libxerces.dll.a (also symlink to libxerces-c2_1_0.dll.a) - the symlinks are created with a postinstall script So if the hypothetical Bob wants to link with the most recent cygwin-packaged xerces-c-devel on his machine and not have to type -lxerces-c2_1_0 in his link line, he can instead type -lxerces-c or -lxerces (Thanks Gerrit for the patch to make this happen). Please let me have any feedback/complaints/concerns. I'll get this new package out later today... I've got a few issues at my day job to wrap up so that I can spend the next three days at OOPSLA :) thanks! -Abe - Original Message - > I don't recall people bumping the -x numbers previously. I don't see > why that's necessary. Theoretically, we are a small enough and > intelligent enough group that we won't be confused when the contents of > a file are updated without updating the file version. > > I think, in general, a new cygwin package should start out with a -1. > It's hardly worth starting a long discussion about but, IMO, the -x > numbers are incremented when there is a new cygwin release not when > someone is providing packages for review.
Re: xerces-c, xerces-c-devel, xerces-c-doc 2.1.0-2 available for review/upload
On Tue, Nov 05, 2002 at 03:11:44PM +0100, Pavel Tsekov wrote: > > >On 6 Nov 2002, Robert Collins wrote: > >> On Wed, 2002-11-06 at 01:01, Pavel Tsekov wrote: >> > On 6 Nov 2002, Robert Collins wrote: >> > >> > > Consider this: if the test copy we review was -3, and you approve that >> > > version, you should not rename it to -1, else when -2 comes out, all the >> > > testers will fail to upgrade. So -3 in review, stays as -3 when >> > > uploading. Likewise, updating -1 to -2 to -3 during the review process >> > > allows the testers to update properly. Does that make sense? >> > >> > Ok, but if one releases a new Cygwin package shouldn't it get -1, which >> > will make it first Cygwin release of this package ? And I'm under the >> > impression that xerces is a new package :) >> >> No, it should get *a* number that will then monotonically increase >> within the same vendor version number. > >Ok, I've just checked the package naming conventions to refresh my >memory. Check this: > >http://cygwin.com/setup.html#naming > >Maybe it should be updated. I don't recall people bumping the -x numbers previously. I don't see why that's necessary. Theoretically, we are a small enough and intelligent enough group that we won't be confused when the contents of a file are updated without updating the file version. I think, in general, a new cygwin package should start out with a -1. It's hardly worth starting a long discussion about but, IMO, the -x numbers are incremented when there is a new cygwin release not when someone is providing packages for review. cgf
Re: xerces-c, xerces-c-devel, xerces-c-doc 2.1.0-2 available for review/upload
> On Sun, 3 Nov 2002, Gareth Pearce wrote: > > > >Anyone wishing to review these packages can point setup.exe to > > >http://abackus.imagineis.com. > > > > I'll definitely review these when i get home ... hopefully they will solve > > my xerces-c related problems. > > Ping. Have you had a chance to review this package ? As already posted - I had a chance for a 'quick' review - which found the current version had the wrong library - Charles has also sent some comments offlist he gave the usual split the library into a seperate package suggestion as well he pointed out that my supposition that the library containing functions of the same name as stdio functions was a property of libstdc++.a and shouldnt be a problem. (which means i am back to square one with reasons why linking the library is causing my application to crash where it doesnt otherwise) Finally I heard today that the 2.1.0 version has a sort of nasty bug in it. Cant remember offhand what it is, but its a one line patch to fix... Gareth
Re: xerces-c, xerces-c-devel, xerces-c-doc 2.1.0-2 available for review/upload
> >Anyone wishing to review these packages can point setup.exe to > >http://abackus.imagineis.com. > > I'll definitely review these when i get home ... hopefully they will solve > my xerces-c related problems. > > Gareth - still doesnt understand why hes got xerces-c related problems in > the first place. Only looked into binary packages since thats what i was after... will look into the rest if i have some time... Okay well xerces-c-2.1.0-2.tar.bz2 - contains libxerces instead of cygxerces easy fix there... The rest 'looks' okay not so good news is my application which uses xerces-c stil dies horribly in an fopen call. I tried to make a reduced testcase - but nope it dosnt like me I 'dont like' the fact that the libxerces-c dll appears to contain a stack of cstdio as 'T' when you nm the non-striped version. Surely that potentially can play havoc - all the stdio calls need to be handled by cygwin1.dll interestingly NONE of these come from the object files which get linked togeather... I am a little worried that g++ --shared is doing silly stuff. Ofcourse my problem is probably unique - I am special that way... Gareth
Re: xerces-c, xerces-c-devel, xerces-c-doc 2.1.0-2 available for review/upload
I added a patch to follow some recommendations from Gerrit. 1) have the library name be "cygxerces-c2_1_0.dll" rather than "libxerces-c2_1_0.dll" (following conventions used by other libraries in cygwin). 2) the symlinks created in the build structure during "make" more accurately represent the links created during "make install" 3) in "make distclean" and "make clean", libraries and symlinks are removed http://abackus.imagineis.com/xerces-c/setup.hint http://abackus.imagineis.com/xerces-c/xerces-c-2.1.0-2.tar.bz2 http://abackus.imagineis.com/xerces-c/xerces-c-2.1.0-2-src.tar.bz2 http://abackus.imagineis.com/xerces-c-devel/setup.hint http://abackus.imagineis.com/xerces-c-devel/xerces-c-devel-2.1.0-2.tar.bz2 http://abackus.imagineis.com/xerces-c-doc/setup.hint http://abackus.imagineis.com/xerces-c-doc/xerces-c-doc-2.1.0-2.tar.bz2 Anyone wishing to review these packages can point setup.exe to http://abackus.imagineis.com. I'll definitely review these when i get home ... hopefully they will solve my xerces-c related problems. Gareth - still doesnt understand why hes got xerces-c related problems in the first place. _ Surf the Web without missing calls! Get MSN Broadband. http://resourcecenter.msn.com/access/plans/freeactivation.asp
xerces-c, xerces-c-devel, xerces-c-doc 2.1.0-2 available for review/upload
I added a patch to follow some recommendations from Gerrit. 1) have the library name be "cygxerces-c2_1_0.dll" rather than "libxerces-c2_1_0.dll" (following conventions used by other libraries in cygwin). 2) the symlinks created in the build structure during "make" more accurately represent the links created during "make install" 3) in "make distclean" and "make clean", libraries and symlinks are removed http://abackus.imagineis.com/xerces-c/setup.hint http://abackus.imagineis.com/xerces-c/xerces-c-2.1.0-2.tar.bz2 http://abackus.imagineis.com/xerces-c/xerces-c-2.1.0-2-src.tar.bz2 http://abackus.imagineis.com/xerces-c-devel/setup.hint http://abackus.imagineis.com/xerces-c-devel/xerces-c-devel-2.1.0-2.tar.bz2 http://abackus.imagineis.com/xerces-c-doc/setup.hint http://abackus.imagineis.com/xerces-c-doc/xerces-c-doc-2.1.0-2.tar.bz2 Anyone wishing to review these packages can point setup.exe to http://abackus.imagineis.com. -Abe