Google Groups no longer supports new Usenet posts or subscriptions. Historical content remains viewable.
Dismiss

5.8.4 RC2

2 views
Skip to first unread message

Nicholas Clark

unread,
Apr 16, 2004, 10:16:45 AM4/16/04
to perl5-...@perl.org
Macavity's a Mystery Cat: he's called the Hidden Paw --
For he's the master criminal who can defy the Law.
He's the bafflement of Scotland Yard, the Flying Squad's despair:
For when they reach the scene of crime -- /Macavity's not there/!

Macavity, Macavity, there's no one like Macavity,
He's broken every human law, he breaks the law of gravity.
His powers of levitation would make a fakir stare,
And when you reach the scene of crime -- /Macavity's not there/!
You may seek him in the basement, you may look up in the air --
But I tell you once and once again, /Macavity's not there/!

from 'Macavity: The Mystery Cat' by T.S. Eliot

All this suidperl changing makes me uneasy - it means that the RC1 wasn't
actually release worthy. So try again - here's release candidate 2.
Get it now from

http://opensource.fotango.com/~nclark/perl-5.8.4-RC2.tar.bz2

(or s/bz2$/gz/ if you really want a 25% larger download.)

coming soon to a CPAN mirror near you soon as

ftp://ftp.cpan.org/pub/CPAN/authors/id/N/NW/NWCLARK/perl-5.8.4-RC2.tar.bz2

I'll make an announcement on use.perl this evening (or at least feed it to
the lions^Wsubmission queue)

Nicholas Clark

Stas Bekman

unread,
Apr 16, 2004, 5:11:00 PM4/16/04
to Nicholas Clark, perl5-...@perl.org
Building the maint branch I've noticed:

cp ../../regcomp.c re_comp.c
cc -c -D_REENTRANT -D_GNU_SOURCE -DTHREADS_HAVE_PIDS -DDEBUGGING
-fno-strict-aliasing -I/usr/local/include -D_LARGEFILE_SOURCE
-D_FILE_OFFSET_BITS=64 -I/usr/include/gdbm -g -DVERSION=\"0.04\"
-DXS_VERSION=\"0.04\" -fpic "-I../.." -DPERL_EXT_RE_BUILD -DPERL_EXT_RE_DEBUG
-DPERL_EXT re_comp.c
re_comp.c: In function `S_study_chunk':
re_comp.c:724: warning: comparison is always false due to limited range of
data type

I have never looked at the warnings before, so it may have been there for a
long time, but may be it's important.

FWIW, in the mod_perl 2 land we build with -Werror under the maintainer mode,
which forces us to fix warnings right away.

All tests successful on the other hand.

__________________________________________________________________
Stas Bekman JAm_pH ------> Just Another mod_perl Hacker
http://stason.org/ mod_perl Guide ---> http://perl.apache.org
mailto:st...@stason.org http://use.perl.org http://apacheweek.com
http://modperlbook.org http://apache.org http://ticketmaster.com

Stas Bekman

unread,
Apr 16, 2004, 9:49:07 PM4/16/04
to Nicholas Clark, perl5-...@perl.org
Stas Bekman wrote:
> Building the maint branch I've noticed:
>
> cp ../../regcomp.c re_comp.c
> cc -c -D_REENTRANT -D_GNU_SOURCE -DTHREADS_HAVE_PIDS -DDEBUGGING
> -fno-strict-aliasing -I/usr/local/include -D_LARGEFILE_SOURCE
> -D_FILE_OFFSET_BITS=64 -I/usr/include/gdbm -g -DVERSION=\"0.04\"
> -DXS_VERSION=\"0.04\" -fpic "-I../.." -DPERL_EXT_RE_BUILD
> -DPERL_EXT_RE_DEBUG -DPERL_EXT re_comp.c
> re_comp.c: In function `S_study_chunk':
> re_comp.c:724: warning: comparison is always false due to limited range
> of data type

one more:

cp ByteLoader.pm ../../lib/ByteLoader.pm
cc -c -D_REENTRANT -D_GNU_SOURCE -DTHREADS_HAVE_PIDS -DDEBUG_LEAKING_SCALARS
-DDEBUGGING -fno-strict-aliasing -I/usr/local/include -D_LARGEFILE_SOURCE
-D_FILE_OFFSET_BITS=64 -I/usr/include/gdbm -g -DVERSION=\"0.05\"
-DXS_VERSION=\"0.05\" -fpic "-I../.." byterun.c
In function `byterun':
byterun.c:898: warning: comparison is always false due to limited range of
data type

that's with gcc-3.3.2

Marcus Holland-Moritz

unread,
Apr 17, 2004, 3:57:37 AM4/17/04
to Stas Bekman, Nicholas Clark, perl5-...@perl.org
On 2004-04-16, at 14:11:00 -0700, Stas Bekman wrote:

> Building the maint branch I've noticed:
>
> cp ../../regcomp.c re_comp.c
> cc -c -D_REENTRANT -D_GNU_SOURCE -DTHREADS_HAVE_PIDS -DDEBUGGING
> -fno-strict-aliasing -I/usr/local/include -D_LARGEFILE_SOURCE
> -D_FILE_OFFSET_BITS=64 -I/usr/include/gdbm -g -DVERSION=\"0.04\"
> -DXS_VERSION=\"0.04\" -fpic "-I../.." -DPERL_EXT_RE_BUILD -DPERL_EXT_RE_DEBUG
> -DPERL_EXT re_comp.c
> re_comp.c: In function `S_study_chunk':
> re_comp.c:724: warning: comparison is always false due to limited range of
> data type
>
> I have never looked at the warnings before, so it may have been there for a
> long time, but may be it's important.

This one is due to the MEM_WRAP_CHECK macro that is used when
PERL_MALLOC_WRAP is defined. It contains

(n)>((MEM_SIZE)~0)/sizeof(t)

and in this particular case, sizeof(t) is 1 and n is of type U8,
which is apparently smaller than MEM_SIZE.

Not sure what the best fix would be. Simply adding a cast is not
enough.

> FWIW, in the mod_perl 2 land we build with -Werror under the maintainer mode,
> which forces us to fix warnings right away.
>
> All tests successful on the other hand.
>
> __________________________________________________________________
> Stas Bekman JAm_pH ------> Just Another mod_perl Hacker
> http://stason.org/ mod_perl Guide ---> http://perl.apache.org
> mailto:st...@stason.org http://use.perl.org http://apacheweek.com
> http://modperlbook.org http://apache.org http://ticketmaster.com
>


--
We must believe that it is the darkest before the dawn of a beautiful
new world. We will see it when we believe it.
-- Saul Alinsky

0 new messages