Skip Menu |
Report information
Id: 122675
Status: resolved
Priority: 0/
Queue: perl5

Owner: jkeenan <jkeenan [at] cpan.org>
Requestors: ambs [at] cpan.org
Cc:
AdminCc:

Operating System: Linux
PatchStatus: (no value)
Severity: low
Type:
Perl Version: 5.20.0
Fixed In: (no value)



Date: Mon, 1 Sep 2014 11:45:41 +0100 (WEST)
Subject: [PATCH] Update ExtUtilsÇ::CBuilder to CPAN version
To: perlbug [...] perl.org
From: ambs [...] cpan.org
Download (untitled) / with headers
text/plain 2.7k
This is a bug report for perl from ambs@cpan.org, generated with the help of perlbug 1.40 running under perl 5.20.0. ----------------------------------------------------------------- [Please describe your issue here] A fix was included in EU::CBuilder, and a version released to CPAN. The attached patch updates perl tree to latest EU::CB version. Cheers ambs [Please do not change anything below this line] ----------------------------------------------------------------- --- Flags: category=library severity=low Type=Patch PatchStatus=HasPatch module=ExtUtils::CBuilder --- Site configuration information for perl 5.20.0: Configured by root at Wed Aug 20 14:19:28 WEST 2014. Summary of my perl5 (revision 5 version 20 subversion 0) configuration: Platform: osname=linux, osvers=3.10.17, archname=i686-linux uname='linux eremita 3.10.17 #2 wed oct 23 17:46:52 cdt 2013 i686 intel(r) core(tm)2 duo cpu e7500 @ 2.93ghz genuineintel gnulinux ' config_args='-ds -e -Dprefix=/opt/perl-5.20.0' hint=recommended, useposix=true, d_sigaction=define useithreads=undef, usemultiplicity=undef use64bitint=undef, use64bitall=undef, uselongdouble=undef usemymalloc=n, bincompat5005=undef Compiler: cc='cc', ccflags ='-fwrapv -fno-strict-aliasing -pipe -fstack-protector -I/usr/local/include -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64', optimize='-O2', cppflags='-fwrapv -fno-strict-aliasing -pipe -fstack-protector -I/usr/local/include' ccversion='', gccversion='4.8.2', gccosandvers='' intsize=4, longsize=4, ptrsize=4, doublesize=8, byteorder=1234 d_longlong=define, longlongsize=8, d_longdbl=define, longdblsize=12 ivtype='long', ivsize=4, nvtype='double', nvsize=8, Off_t='off_t', lseeksize=8 alignbytes=4, prototype=define Linker and Libraries: ld='cc', ldflags =' -fstack-protector -L/usr/local/lib' libpth=/usr/local/lib /usr/lib/gcc/i486-slackware-linux/4.8.2/include-fixed /usr/lib /lib libs=-lnsl -lgdbm -ldb -ldl -lm -lcrypt -lutil -lc perllibs=-lnsl -ldl -lm -lcrypt -lutil -lc libc=libc-2.17.so, so=so, useshrplib=false, libperl=libperl.a gnulibc_version='2.17' Dynamic Linking: dlsrc=dl_dlopen.xs, dlext=so, d_dlsymun=undef, ccdlflags='-Wl,-E' cccdlflags='-fPIC', lddlflags='-shared -O2 -L/usr/local/lib -fstack-protector' --- @INC for perl 5.20.0: /opt/perl-5.20.0/lib/site_perl/5.20.0/i686-linux /opt/perl-5.20.0/lib/site_perl/5.20.0 /opt/perl-5.20.0/lib/5.20.0/i686-linux /opt/perl-5.20.0/lib/5.20.0 . --- Environment for perl 5.20.0: HOME=/home/ambs LANG=en_US LANGUAGE (unset) LC_COLLATE=C LD_LIBRARY_PATH (unset) LOGDIR (unset) PATH=/usr/local/bin:/usr/bin:/bin:/usr/games:/usr/lib/qt/bin PERL_BADLANG (unset) SHELL=/bin/bash
Download EUCB.patch
text/plain 807b

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

RT-Send-CC: perl5-porters [...] perl.org
Download (untitled) / with headers
text/plain 3.3k
On Mon Sep 01 03:46:05 2014, ambs@cpan.org wrote: Show quoted text
> This is a bug report for perl from ambs@cpan.org, > generated with the help of perlbug 1.40 running under perl 5.20.0. > > > ----------------------------------------------------------------- > [Please describe your issue here] > > A fix was included in EU::CBuilder, and a version released to > CPAN. The attached patch updates perl tree to latest EU::CB version. > > Cheers > ambs > > > > [Please do not change anything below this line] > ----------------------------------------------------------------- > --- > Flags: > category=library > severity=low > Type=Patch > PatchStatus=HasPatch > module=ExtUtils::CBuilder > --- > Site configuration information for perl 5.20.0: > > Configured by root at Wed Aug 20 14:19:28 WEST 2014. > > Summary of my perl5 (revision 5 version 20 subversion 0) > configuration: > > Platform: > osname=linux, osvers=3.10.17, archname=i686-linux > uname='linux eremita 3.10.17 #2 wed oct 23 17:46:52 cdt 2013 i686 > intel(r) core(tm)2 duo cpu e7500 @ 2.93ghz genuineintel gnulinux ' > config_args='-ds -e -Dprefix=/opt/perl-5.20.0' > hint=recommended, useposix=true, d_sigaction=define > useithreads=undef, usemultiplicity=undef > use64bitint=undef, use64bitall=undef, uselongdouble=undef > usemymalloc=n, bincompat5005=undef > Compiler: > cc='cc', ccflags ='-fwrapv -fno-strict-aliasing -pipe -fstack- > protector -I/usr/local/include -D_LARGEFILE_SOURCE > -D_FILE_OFFSET_BITS=64', > optimize='-O2', > cppflags='-fwrapv -fno-strict-aliasing -pipe -fstack-protector > -I/usr/local/include' > ccversion='', gccversion='4.8.2', gccosandvers='' > intsize=4, longsize=4, ptrsize=4, doublesize=8, byteorder=1234 > d_longlong=define, longlongsize=8, d_longdbl=define, longdblsize=12 > ivtype='long', ivsize=4, nvtype='double', nvsize=8, Off_t='off_t', > lseeksize=8 > alignbytes=4, prototype=define > Linker and Libraries: > ld='cc', ldflags =' -fstack-protector -L/usr/local/lib' > libpth=/usr/local/lib /usr/lib/gcc/i486-slackware- > linux/4.8.2/include-fixed /usr/lib /lib > libs=-lnsl -lgdbm -ldb -ldl -lm -lcrypt -lutil -lc > perllibs=-lnsl -ldl -lm -lcrypt -lutil -lc > libc=libc-2.17.so, so=so, useshrplib=false, libperl=libperl.a > gnulibc_version='2.17' > Dynamic Linking: > dlsrc=dl_dlopen.xs, dlext=so, d_dlsymun=undef, ccdlflags='-Wl,-E' > cccdlflags='-fPIC', lddlflags='-shared -O2 -L/usr/local/lib -fstack- > protector' > > > --- > @INC for perl 5.20.0: > /opt/perl-5.20.0/lib/site_perl/5.20.0/i686-linux > /opt/perl-5.20.0/lib/site_perl/5.20.0 > /opt/perl-5.20.0/lib/5.20.0/i686-linux > /opt/perl-5.20.0/lib/5.20.0 > . > > --- > Environment for perl 5.20.0: > HOME=/home/ambs > LANG=en_US > LANGUAGE (unset) > LC_COLLATE=C > LD_LIBRARY_PATH (unset) > LOGDIR (unset) > PATH=/usr/local/bin:/usr/bin:/bin:/usr/games:/usr/lib/qt/bin > PERL_BADLANG (unset) > SHELL=/bin/bash
ambs: Thank you for the patch. When I went to apply it, however, I noticed that it updated only one test file and Changes. It did not update Makefile.PL, which is where VERSION is declared in this distribution. Since we have the Makefile.PL in core, shouldn't we update VERSION there as well? Would it be possible to re-draw the patch to include changes to Makefile.PL? Thank you very much. Jim Keenan
CC: perl5-porters [...] perl.org
Subject: Re: [perl #122675] [PATCH] Update ExtUtils::CBuilder to CPAN version
From: Alberto Simões <albie [...] alfarrabio.di.uminho.pt>
To: perlbug-followup [...] perl.org
Date: Mon, 01 Sep 2014 14:45:21 +0100
Download (untitled) / with headers
text/plain 3.7k
On 01/09/14, 13:25, James E Keenan via RT wrote: Show quoted text
> On Mon Sep 01 03:46:05 2014, ambs@cpan.org wrote:
>> This is a bug report for perl from ambs@cpan.org, >> generated with the help of perlbug 1.40 running under perl 5.20.0. >> >> >> ----------------------------------------------------------------- >> [Please describe your issue here] >> >> A fix was included in EU::CBuilder, and a version released to >> CPAN. The attached patch updates perl tree to latest EU::CB version. >> >> Cheers >> ambs >> >> >> >> [Please do not change anything below this line] >> ----------------------------------------------------------------- >> --- >> Flags: >> category=library >> severity=low >> Type=Patch >> PatchStatus=HasPatch >> module=ExtUtils::CBuilder >> --- >> Site configuration information for perl 5.20.0: >> >> Configured by root at Wed Aug 20 14:19:28 WEST 2014. >> >> Summary of my perl5 (revision 5 version 20 subversion 0) >> configuration: >> >> Platform: >> osname=linux, osvers=3.10.17, archname=i686-linux >> uname='linux eremita 3.10.17 #2 wed oct 23 17:46:52 cdt 2013 i686 >> intel(r) core(tm)2 duo cpu e7500 @ 2.93ghz genuineintel gnulinux ' >> config_args='-ds -e -Dprefix=/opt/perl-5.20.0' >> hint=recommended, useposix=true, d_sigaction=define >> useithreads=undef, usemultiplicity=undef >> use64bitint=undef, use64bitall=undef, uselongdouble=undef >> usemymalloc=n, bincompat5005=undef >> Compiler: >> cc='cc', ccflags ='-fwrapv -fno-strict-aliasing -pipe -fstack- >> protector -I/usr/local/include -D_LARGEFILE_SOURCE >> -D_FILE_OFFSET_BITS=64', >> optimize='-O2', >> cppflags='-fwrapv -fno-strict-aliasing -pipe -fstack-protector >> -I/usr/local/include' >> ccversion='', gccversion='4.8.2', gccosandvers='' >> intsize=4, longsize=4, ptrsize=4, doublesize=8, byteorder=1234 >> d_longlong=define, longlongsize=8, d_longdbl=define, longdblsize=12 >> ivtype='long', ivsize=4, nvtype='double', nvsize=8, Off_t='off_t', >> lseeksize=8 >> alignbytes=4, prototype=define >> Linker and Libraries: >> ld='cc', ldflags =' -fstack-protector -L/usr/local/lib' >> libpth=/usr/local/lib /usr/lib/gcc/i486-slackware- >> linux/4.8.2/include-fixed /usr/lib /lib >> libs=-lnsl -lgdbm -ldb -ldl -lm -lcrypt -lutil -lc >> perllibs=-lnsl -ldl -lm -lcrypt -lutil -lc >> libc=libc-2.17.so, so=so, useshrplib=false, libperl=libperl.a >> gnulibc_version='2.17' >> Dynamic Linking: >> dlsrc=dl_dlopen.xs, dlext=so, d_dlsymun=undef, ccdlflags='-Wl,-E' >> cccdlflags='-fPIC', lddlflags='-shared -O2 -L/usr/local/lib -fstack- >> protector' >> >> >> --- >> @INC for perl 5.20.0: >> /opt/perl-5.20.0/lib/site_perl/5.20.0/i686-linux >> /opt/perl-5.20.0/lib/site_perl/5.20.0 >> /opt/perl-5.20.0/lib/5.20.0/i686-linux >> /opt/perl-5.20.0/lib/5.20.0 >> . >> >> --- >> Environment for perl 5.20.0: >> HOME=/home/ambs >> LANG=en_US >> LANGUAGE (unset) >> LC_COLLATE=C >> LD_LIBRARY_PATH (unset) >> LOGDIR (unset) >> PATH=/usr/local/bin:/usr/bin:/bin:/usr/games:/usr/lib/qt/bin >> PERL_BADLANG (unset) >> SHELL=/bin/bash
> > ambs: > > Thank you for the patch. When I went to apply it, however, I noticed that it updated only one test file and Changes. It did not update Makefile.PL, which is where VERSION is declared in this distribution. Since we have the Makefile.PL in core, shouldn't we update VERSION there as well? > > Would it be possible to re-draw the patch to include changes to Makefile.PL? > > Thank you very much. > Jim Keenan
Please check if this one is better. It changes a lot more files, given that I am using dzil to generate versions on all modules. If by some reason this is not good for the core, let me know, and I'll revert my changes. Thanks ambs
Download CBuilder.patch
text/plain 10.6k

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

Subject: Re: [perl #122675] [PATCH] Update ExtUtils::CBuilder to CPAN version
CC: Brian Fraser via RT <perlbug-followup [...] perl.org>, Perl5 Porters Mailing List <perl5-porters [...] perl.org>
To: albie [...] alfarrabio.di.uminho.pt
Date: Mon, 1 Sep 2014 16:13:36 +0200
From: Brian Fraser <fraserbn [...] gmail.com>
Download (untitled) / with headers
text/plain 719b
On Mon, Sep 1, 2014 at 3:45 PM, Alberto Simões <albie@alfarrabio.di.uminho.pt> wrote: Show quoted text
> > > On 01/09/14, 13:25, James E Keenan via RT wrote:
>> >> On Mon Sep 01 03:46:05 2014, ambs@cpan.org wrote:
>>> >>> This is a bug report for perl from ambs@cpan.org, >>> generated with the help of perlbug 1.40 running under perl 5.20.0. >>> >>> >>> ----------------------------------------------------------------- >>> [Please describe your issue here] >>> >>> A fix was included in EU::CBuilder, and a version released to >>> CPAN. The attached patch updates perl tree to latest EU::CB version.
The patch undoes one of the fixes for Android that I pushed to bleed some weeks back; is CPAN now the correct upstream for EU::CB?
To: Brian Fraser <fraserbn [...] gmail.com>
CC: Brian Fraser via RT <perlbug-followup [...] perl.org>, Perl5 Porters Mailing List <perl5-porters [...] perl.org>
From: Alberto Simões <albie [...] alfarrabio.di.uminho.pt>
Date: Mon, 01 Sep 2014 15:18:03 +0100
Subject: Re: [perl #122675] [PATCH] Update ExtUtils::CBuilder to CPAN version
Download (untitled) / with headers
text/plain 989b
Hello On 01/09/14, 15:13, Brian Fraser wrote: Show quoted text
> On Mon, Sep 1, 2014 at 3:45 PM, Alberto Simões > <albie@alfarrabio.di.uminho.pt> wrote:
>> >> >> On 01/09/14, 13:25, James E Keenan via RT wrote:
>>> >>> On Mon Sep 01 03:46:05 2014, ambs@cpan.org wrote:
>>>> >>>> This is a bug report for perl from ambs@cpan.org, >>>> generated with the help of perlbug 1.40 running under perl 5.20.0. >>>> >>>> >>>> ----------------------------------------------------------------- >>>> [Please describe your issue here] >>>> >>>> A fix was included in EU::CBuilder, and a version released to >>>> CPAN. The attached patch updates perl tree to latest EU::CB version.
> > The patch undoes one of the fixes for Android that I pushed to bleed > some weeks back; is CPAN now the correct upstream for EU::CB?
Erm, not really. I tried to see if there were changes. I looked again to the patch and I see what you mean. I'll merge it, prepare a new release, and send a new patch. Thanks for the notice! ambs
To: Brian Fraser <fraserbn [...] gmail.com>
Date: Mon, 01 Sep 2014 15:27:02 +0100
From: Alberto Simões <albie [...] alfarrabio.di.uminho.pt>
Subject: Re: [perl #122675] [PATCH] Update ExtUtils::CBuilder to CPAN version
CC: Brian Fraser via RT <perlbug-followup [...] perl.org>, Perl5 Porters Mailing List <perl5-porters [...] perl.org>
Download (untitled) / with headers
text/plain 914b
On 01/09/14, 15:18, Alberto Simões wrote: Show quoted text
> Hello > > On 01/09/14, 15:13, Brian Fraser wrote:
>> On Mon, Sep 1, 2014 at 3:45 PM, Alberto Simões >> <albie@alfarrabio.di.uminho.pt> wrote:
>>> >>> >>> On 01/09/14, 13:25, James E Keenan via RT wrote:
>>>> >>>> On Mon Sep 01 03:46:05 2014, ambs@cpan.org wrote:
>>>>> >>>>> This is a bug report for perl from ambs@cpan.org, >>>>> generated with the help of perlbug 1.40 running under perl 5.20.0. >>>>> >>>>> >>>>> ----------------------------------------------------------------- >>>>> [Please describe your issue here] >>>>> >>>>> A fix was included in EU::CBuilder, and a version released to >>>>> CPAN. The attached patch updates perl tree to latest EU::CB version.
>> >> The patch undoes one of the fixes for Android that I pushed to bleed >> some weeks back; is CPAN now the correct upstream for EU::CB?
I think I fixed it now. Thanks, one more time. ambs
Download CBuilder.patch
text/plain 10.5k

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

RT-Send-CC: perl5-porters [...] perl.org
Download (untitled) / with headers
text/plain 1.2k
On Mon Sep 01 07:14:01 2014, Hugmeir wrote: Show quoted text
> On Mon, Sep 1, 2014 at 3:45 PM, Alberto Simões > <albie@alfarrabio.di.uminho.pt> wrote:
> > > > > > On 01/09/14, 13:25, James E Keenan via RT wrote:
> >> > >> On Mon Sep 01 03:46:05 2014, ambs@cpan.org wrote:
> >>> > >>> This is a bug report for perl from ambs@cpan.org, > >>> generated with the help of perlbug 1.40 running under perl 5.20.0. > >>> > >>> > >>> ----------------------------------------------------------------- > >>> [Please describe your issue here] > >>> > >>> A fix was included in EU::CBuilder, and a version released to > >>> CPAN. The attached patch updates perl tree to latest EU::CB version.
> > The patch undoes one of the fixes for Android that I pushed to bleed > some weeks back; is CPAN now the correct upstream for EU::CB? >
Good point. ExtUtils-CBuilder is currently located under dist/. According to pod/perlsource.pod, ##### [dist/] is for dual-life modules where the blead source is canonical. ... Modules under F<dist/> should make an effort to work with earlier versions of Perl 5. ##### We could move it to cpan/, where the CPAN version is canonical -- BUT that should be the subject of a DIFFERENT RT ticket. -- James E Keenan (jkeenan@cpan.org)
From: Alberto Simões <albie [...] alfarrabio.di.uminho.pt>
Date: Mon, 01 Sep 2014 19:35:29 +0100
To: perl5-porters [...] perl.org
Subject: Re: [perl #122675] [PATCH] Update ExtUtils::CBuilder to CPAN version
Download (untitled) / with headers
text/plain 1.3k
On 01/09/14, 19:31, James E Keenan via RT wrote: Show quoted text
> On Mon Sep 01 07:14:01 2014, Hugmeir wrote:
>> On Mon, Sep 1, 2014 at 3:45 PM, Alberto Simões >> <albie@alfarrabio.di.uminho.pt> wrote:
>>> >>> >>> On 01/09/14, 13:25, James E Keenan via RT wrote:
>>>> >>>> On Mon Sep 01 03:46:05 2014, ambs@cpan.org wrote:
>>>>> >>>>> This is a bug report for perl from ambs@cpan.org, >>>>> generated with the help of perlbug 1.40 running under perl 5.20.0. >>>>> >>>>> >>>>> ----------------------------------------------------------------- >>>>> [Please describe your issue here] >>>>> >>>>> A fix was included in EU::CBuilder, and a version released to >>>>> CPAN. The attached patch updates perl tree to latest EU::CB version.
>> >> The patch undoes one of the fixes for Android that I pushed to bleed >> some weeks back; is CPAN now the correct upstream for EU::CB? >>
> > Good point. ExtUtils-CBuilder is currently located under dist/. According to pod/perlsource.pod, > > ##### > [dist/] is for dual-life modules where the blead source is canonical. ... Modules under F<dist/> should make an effort to work with earlier versions of Perl 5. > ##### > > We could move it to cpan/, where the CPAN version is canonical -- BUT that should be the subject of a DIFFERENT RT ticket. >
Happy to keep it as it is. It was a mistake. Best ambs
RT-Send-CC: perl5-porters [...] perl.org
Download (untitled) / with headers
text/plain 1.3k
On Mon Sep 01 07:27:44 2014, ambs wrote: Show quoted text
> > > On 01/09/14, 15:18, Alberto Simões wrote:
> > Hello > > > > On 01/09/14, 15:13, Brian Fraser wrote:
> >> On Mon, Sep 1, 2014 at 3:45 PM, Alberto Simões > >> <albie@alfarrabio.di.uminho.pt> wrote:
> >>> > >>> > >>> On 01/09/14, 13:25, James E Keenan via RT wrote:
> >>>> > >>>> On Mon Sep 01 03:46:05 2014, ambs@cpan.org wrote:
> >>>>> > >>>>> This is a bug report for perl from ambs@cpan.org, > >>>>> generated with the help of perlbug 1.40 running under perl 5.20.0. > >>>>> > >>>>> > >>>>> ----------------------------------------------------------------- > >>>>> [Please describe your issue here] > >>>>> > >>>>> A fix was included in EU::CBuilder, and a version released to > >>>>> CPAN. The attached patch updates perl tree to latest EU::CB version.
> >> > >> The patch undoes one of the fixes for Android that I pushed to bleed > >> some weeks back; is CPAN now the correct upstream for EU::CB?
> > I think I fixed it now. > > Thanks, one more time. > > ambs
I applied the patch to blead in commit 74d5bfab5a03c46777cd031b635f872b88dc1fef hugmeir: I checked to make sure that your most recent commit to the android module is included, but feel free to double-check. I'll close this ticket within 7 days unless there are further complaints. -- James E Keenan (jkeenan@cpan.org)
RT-Send-CC: perl5-porters [...] perl.org
Download (untitled) / with headers
text/plain 1.4k
On Mon Sep 01 11:40:46 2014, jkeenan wrote: Show quoted text
> On Mon Sep 01 07:27:44 2014, ambs wrote:
> > > > > > On 01/09/14, 15:18, Alberto Simões wrote:
> > > Hello > > > > > > On 01/09/14, 15:13, Brian Fraser wrote:
> > >> On Mon, Sep 1, 2014 at 3:45 PM, Alberto Simões > > >> <albie@alfarrabio.di.uminho.pt> wrote:
> > >>> > > >>> > > >>> On 01/09/14, 13:25, James E Keenan via RT wrote:
> > >>>> > > >>>> On Mon Sep 01 03:46:05 2014, ambs@cpan.org wrote:
> > >>>>> > > >>>>> This is a bug report for perl from ambs@cpan.org, > > >>>>> generated with the help of perlbug 1.40 running under perl > > >>>>> 5.20.0. > > >>>>> > > >>>>> > > >>>>> ----------------------------------------------------------------- > > >>>>> [Please describe your issue here] > > >>>>> > > >>>>> A fix was included in EU::CBuilder, and a version released to > > >>>>> CPAN. The attached patch updates perl tree to latest EU::CB > > >>>>> version.
> > >> > > >> The patch undoes one of the fixes for Android that I pushed to > > >> bleed > > >> some weeks back; is CPAN now the correct upstream for EU::CB?
> > > > I think I fixed it now. > > > > Thanks, one more time. > > > > ambs
> > I applied the patch to blead in commit > 74d5bfab5a03c46777cd031b635f872b88dc1fef > > hugmeir: I checked to make sure that your most recent commit to the > android module is included, but feel free to double-check. > > I'll close this ticket within 7 days unless there are further > complaints.
-- James E Keenan (jkeenan@cpan.org)


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

For issues related to this RT instance (aka "perlbug"), please contact perlbug-admin at perl.org