Skip Menu |

This queue is for tickets about the PAR-Packer CPAN distribution.

Report information
The Basics
Id: 60149
Status: resolved
Priority: 0/
Queue: PAR-Packer

People
Owner: Nobody in particular
Requestors: soreks [...] gmail.com
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



Subject: Fwd: strcasecmp is unresolved when linking on win32
Date: Fri, 6 Aug 2010 20:39:39 +0300
To: bug-PAR-Packer [...] rt.cpan.org
From: soreks <soreks [...] gmail.com>
The attached patch changes strcasecmp to stricmp on win32.

Message body is not shown because sender requested not to inline it.

On 2010-08-06 13:40:20, soreks@gmail.com wrote: Show quoted text
> The attached patch changes strcasecmp to stricmp on win32.
What version of PAR::Packer? And what C compiler are you using? Cheers, Roderich
Subject: Re: [rt.cpan.org #60149] Fwd: strcasecmp is unresolved when linking on win32
Date: Sun, 8 Aug 2010 19:43:00 +0300
To: bug-PAR-Packer [...] rt.cpan.org
From: soreks <soreks [...] gmail.com>
PAR::Packer 1.006 MS Visual Studio 2005 Changing strcasecmp to stricmp was the only way I got it linked. On Sun, Aug 8, 2010 at 19:36, Roderich Schupp via RT < bug-PAR-Packer@rt.cpan.org> wrote: Show quoted text
> <URL: https://rt.cpan.org/Ticket/Display.html?id=60149 > > > On 2010-08-06 13:40:20, soreks@gmail.com wrote:
> > The attached patch changes strcasecmp to stricmp on win32.
> > What version of PAR::Packer? And what C compiler are you using? > > Cheers, Roderich > > >
On 2010-08-08 12:43:40, soreks@gmail.com wrote: Show quoted text
> PAR::Packer 1.006 > MS Visual Studio 2005 > > Changing strcasecmp to stricmp was the only way I got it linked.
OK, in that case rev 1184 (after release 1.006) in the Subversion repo for PAR::Packer (see META.yml for the URL) should already have a slightly different fix. Cheers, Roderich
Subject: Re: [rt.cpan.org #60149] Fwd: strcasecmp is unresolved when linking on win32
Date: Sun, 8 Aug 2010 20:03:46 +0300
To: bug-PAR-Packer [...] rt.cpan.org
From: soreks <soreks [...] gmail.com>
Well, http://svn.openfoundry.org/par/PAR-Packer/trunk/ shows par - Revision 1200: /PAR-Packer/trunk, so it probably should. However, myldr/mktmpdir.c<http://svn.openfoundry.org/par/PAR-Packer/trunk/myldr/mktmpdir.c>still defines STREQ as strcasecmp on WIN32 so unless strcasecmp is somehow provided at the link time on win32, it'd hardly links with MS Visual Studio 2005 at least. On Sun, Aug 8, 2010 at 19:51, Roderich Schupp via RT < bug-PAR-Packer@rt.cpan.org> wrote: Show quoted text
> <URL: https://rt.cpan.org/Ticket/Display.html?id=60149 > > > On 2010-08-08 12:43:40, soreks@gmail.com wrote:
> > PAR::Packer 1.006 > > MS Visual Studio 2005 > > > > Changing strcasecmp to stricmp was the only way I got it linked.
> > OK, in that case rev 1184 (after release 1.006) in the > Subversion repo for PAR::Packer (see META.yml for the URL) > should already have a slightly different fix. > > Cheers, Roderich >
On 2010-08-08 13:04:29, soreks@gmail.com wrote: Show quoted text
> Well, http://svn.openfoundry.org/par/PAR-Packer/trunk/ shows par - > Revision > 1200: /PAR-Packer/trunk, so it probably should.
I wrote "a slightly different fix". Look at the diff for the mentioned rev. Cheers, Roderich
Subject: Re: [rt.cpan.org #60149] Fwd: strcasecmp is unresolved when linking on win32
Date: Sun, 8 Aug 2010 20:33:57 +0300
To: bug-PAR-Packer [...] rt.cpan.org
From: soreks <soreks [...] gmail.com>
Will you kindly provide a URL? http://svn.openfoundry.org/par/PAR-Packer/trunk/ shows only rev. 1200 and no diffs. I can see mktmpdir.c is rewritten in caching and fastio branches though. On Sun, Aug 8, 2010 at 20:17, Roderich Schupp via RT < bug-PAR-Packer@rt.cpan.org> wrote: Show quoted text
> <URL: https://rt.cpan.org/Ticket/Display.html?id=60149 > > > On 2010-08-08 13:04:29, soreks@gmail.com wrote:
> > Well, http://svn.openfoundry.org/par/PAR-Packer/trunk/ shows par - > > Revision > > 1200: /PAR-Packer/trunk, so it probably should.
> > I wrote "a slightly different fix". Look at the diff for > the mentioned rev. > > Cheers, Roderich >
On 2010-08-08 13:34:37, soreks@gmail.com wrote: Show quoted text
> Will you kindly provide a URL?
svn diff -c 1184 http://svn.openfoundry.org/par/PAR-Packer/trunk or simply check out trunk and test against that. Cgeers, Roderich
Fixed in PAR::Packer 1.010 (actually 1.006 - forgot to close this bug)