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.
Download strcasecmp.patch
text/x-diff 294b

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

Download (untitled) / with headers
text/plain 193b
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>
Download (untitled) / with headers
text/plain 467b
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 > > >
Download (untitled) / with headers
text/plain 342b
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>
Download (untitled) / with headers
text/plain 897b
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 >
Download (untitled) / with headers
text/plain 273b
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>
Download (untitled) / with headers
text/plain 634b
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 >
Download (untitled) / with headers
text/plain 216b
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)


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.