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: 2229
Status: resolved
Worked: 15 min
Priority: 0/
Queue: ExtUtils-ParseXS
People
Owner: kwilliams [...] cpan.org
Requestors:
Cc:
AdminCc:
Bug Information
Severity: Critical
Broken in: 1.98_01
Fixed in: 2.01

History Show all quoted text
  Fri Mar 14 06:57:03 2003 PODMASTER [...] cpan.org - Ticket created
Subject: xsubpp is not compatible with the original (-C++ is missing)
xsubpp is not compatible with the original (-C++ is missing) Please visit http://search.cpan.org/author/GSAR/perl-5.6.1/lib/ExtUtils/xsubpp and/or http://search.cpan.org/author/JHI/perl-5.8.0/lib/ExtUtils/xsubpp and make sure that xsubpp has all the same options. I was compiling wxPerl [http://wxperl.sf.net/] when I ran into this issue. Thanks
  Fri Mar 14 10:10:56 2003 kwilliams [...] cpan.org - Correspondence added 5 min
As far as I can tell, the xsubpp 'C++' flag doesn't actually do anything. It causes a variable ($cplusplus) to be set, and then that variable is never used again. Can you confirm that from a look at http://search.cpan.org/author/JHI/perl-5.8.0/lib/ExtUtils/xsubpp ? Since it looked like the code ##ifdef __cplusplus #extern "C" ##endif was getting added regardless of what the 'C++' flag was set to, I just removed the flag. -Ken
  Fri Mar 14 10:10:57 2003 kwilliams [...] cpan.org - Status changed from 'new' to 'open'
  Fri Mar 14 10:10:57 2003 kwilliams [...] cpan.org - Given to KWILLIAMS
  Sun Mar 16 13:08:43 2003 Guest - Correspondence added
... Show quoted text
> was getting added regardless of what the 'C++' flag was set to, I just > removed the flag.
Yup, that appears to be the case, but xsubpp will die cause it doesn't recognize C++ as an option. I've done some sourcediving back to perl5.003_07, and it didn't do anything there either. It appears it is only there for some kind of backwards compatibility, and that smells like a good reason to put it back. So, your xsubpp currently dies if -C++ shows up (not good), and most folks using the option don't have a clue it's useless. You should ammend the documentation to discourage people from using the -C++ option ( and/or submit a patch to the same effect to p5p -- actually, a better idea might be to incorporate yer module into CORE ;)
  Sun Mar 16 22:03:35 2003 ken [...] mathforum.org - Correspondence added
Date: Sun, 16 Mar 2003 21:03:24 -0600
Subject: Re: [cpan #2229] xsubpp is not compatible with the original (-C++ is missing)
From: Ken Williams <ken [...] mathforum.org>
To: bug-ExtUtils-ParseXS [...] rt.cpan.org
RT-Send-Cc:
On Sunday, March 16, 2003, at 12:08 PM, Guest via RT wrote: Show quoted text
> Yup, that appears to be the case, > but xsubpp will die cause it doesn't > recognize C++ as an option.
Aha, indeed. I forgot that GetOpt would die in that situation. I'll add it back in so that it doesn't die. Show quoted text
> You should ammend the documentation to discourage people from using > the -C++ option > ( and/or submit a patch to the same effect to p5p -- > actually, a better idea might be to incorporate yer module into CORE > ;)
Yeah, I do want to get it back into core (this isn't a module I'm really eager to maintain!), but probably there's not time to get it properly tested for 5.8.1. I can write Hugo/p5p, maybe. -Ken
  Thu Mar 20 09:48:17 2003 kwilliams [...] cpan.org - Correspondence added 10 min
RT-Send-CC: crazyinsomniac [...] yahoo.com
This is now fixed in ExtUtils::ParseXS 2.01. -Ken
  Thu Mar 20 09:48:17 2003 kwilliams [...] cpan.org - Status changed from 'open' to 'resolved'
  Sat Mar 22 18:59:08 2003 kwilliams [...] cpan.org - Broken in 1.98_01 added
  Sat Mar 22 18:59:09 2003 kwilliams [...] cpan.org - Fixed in 2.01 added
  Sat Mar 22 18:59:09 2003 kwilliams [...] cpan.org - CustomField 2.00 deleted