Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the ExtUtils-ParseXS CPAN distribution.

Report information
The Basics
Id: 54577
Status: resolved
Priority: 0/
Queue: ExtUtils-ParseXS
People
Owner: Nobody in particular
Requestors: avg [...] icyb.net.ua
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Fri Feb 12 19:33:29 2010 avg [...] icyb.net.ua - Ticket created
Subject: incorrect substitution regexps in ParseXS.pm?
Date: Sat, 13 Feb 2010 02:32:14 +0200
To: bug-ExtUtils-ParseXS [...] rt.cpan.org
From: Andriy Gapon <avg [...] icyb.net.ua>
I have Perl 5.10.1 installed here and I get errors like the following during build of p5-Cairo: Error: '_OUTLIST double' not in typemap in Cairo.xs Please note '_OUTLIST' in the diagnostics. I looked at ParseXS.pm and see instances of code like the following: $arg =~ s/^(IN|IN_OUTLIST|OUTLIST|OUT|IN_OUT)\s*//; I am not sure how that works for you / in other versions of perl, but here the first matching alternative wins. That is, if $arg is something like "IN_OUTLIST xxx", then the expression above will strip leading "IN" instead of "IN_OUTLIST" as must have been intended. I think that match alternatives in the substitute expressions must be sorted properly. Better yet, to avoid problems in the future, use of alternatives should be avoided in the substitute expression. Something more obvious has to be used here. -- Andriy Gapon
  Fri Feb 12 19:48:38 2010 avg [...] icyb.net.ua - Correspondence added
Subject: Re: [rt.cpan.org #54577] AutoReply: incorrect substitution regexps in ParseXS.pm?
Date: Sat, 13 Feb 2010 02:47:37 +0200
To: bug-ExtUtils-ParseXS [...] rt.cpan.org
From: Andriy Gapon <avg [...] icyb.net.ua>
on 13/02/2010 02:33 Bugs in ExtUtils-ParseXS via RT said the following: Show quoted text
> I looked at ParseXS.pm and see instances of code like the following: > $arg =~ s/^(IN|IN_OUTLIST|OUTLIST|OUT|IN_OUT)\s*//;
Or, more likely, was '\s*' actually supposed to be '\s+'? That is, at least one (white)space is required after those keywords? -- Andriy Gapon
  Sat Feb 13 19:51:57 2010 dagolden [...] cpan.org - Correspondence added
Latest version has updated those regexps to s/^(IN|IN_OUTLIST|OUTLIST|OUT|IN_OUT)\b\s*//;
  Sat Feb 13 19:51:59 2010 The RT System itself - Status changed from 'new' to 'open'
  Sat Feb 13 19:52:00 2010 dagolden [...] cpan.org - Status changed from 'open' to 'resolved'