Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

warning for [.posix.] classes too eager #15191

Closed
p5pRT opened this issue Feb 21, 2016 · 10 comments
Closed

warning for [.posix.] classes too eager #15191

p5pRT opened this issue Feb 21, 2016 · 10 comments

Comments

@p5pRT
Copy link

p5pRT commented Feb 21, 2016

Migrated from rt.perl.org#127582 (status was 'resolved')

Searchable as RT127582$

@p5pRT
Copy link
Author

p5pRT commented Feb 21, 2016

From @tonycoz

tony@​mars​:.../git/perl$ ./perl -v | grep subv
This is perl 5, version 23, subversion 8 (v5.23.8-6-g8571c4b) built for x86_64-linux
tony@​mars​:.../git/perl$ ./perl -Wce '/([.].*)[.]/'
POSIX syntax [. .] belongs inside character classes (but this one isn't implemented) in regex; marked by <-- HERE in m/([.].*)[.] <-- HERE / at -e line 1.
-e syntax OK

The heuristic here might be a bit too eager.

First seen in​:

POSIX syntax [. .] belongs inside character classes (but this one isn't implemented) in regex; marked by <-- HERE in m/([.].*)[.] <-- HERE / at /home/tony/perl/blead/lib/site_perl/5.23.8/Lingua/EN/Inflect.pm line 1419.

Tony

@p5pRT
Copy link
Author

p5pRT commented Feb 24, 2016

From Mark.Martinec@ijs.si

Created by Mark.Martinec@ijs.si

Came across the following warning in Net/DNS/Resolver/Base.pm,
issued by perl 5.23.8 (test case minimized) :

  $ perl -e 'use strict; use warnings; "test" =~ /[.].*[.]/'

  POSIX syntax [. .] belongs inside character classes
  (but this one isn't implemented) in regex;
  marked by <-- HERE in m/[.].*[.] <-- HERE
  / at -e line 1.

Although this regexp is unnecessarily complicated, it seems
the perl is confused about what the syntax means.

Perl Info

Flags:
    category=core
    severity=low

Site configuration information for perl 5.23.8:

Configured by mark at Tue Feb 23 19:43:47 CET 2016.

Summary of my perl5 (revision 5 version 23 subversion 8) configuration:
   
  Platform:
    osname=freebsd, osvers=10.3-beta2, archname=amd64-freebsd
    uname='freebsd neli.ijs.si 10.3-beta2 freebsd 10.3-beta2 #0 r295700: wed feb 17 16:32:27 cet 2016 mark@neli.ijs.si:usrobjusrsrcsysneli amd64 '
    config_args='-de -Dprefix=/home/mark/perl5/perlbrew/perls/perl-5.23.8 -Dusedevel -Aeval:scriptdir=/home/mark/perl5/perlbrew/perls/perl-5.23.8/bin'
    hint=recommended, useposix=true, d_sigaction=define
    useithreads=undef, usemultiplicity=undef
    use64bitint=define, use64bitall=define, uselongdouble=undef
    usemymalloc=n, bincompat5005=undef
  Compiler:
    cc='cc', ccflags ='-DHAS_FPSETMASK -DHAS_FLOATINGPOINT_H -fno-strict-aliasing -pipe -fstack-protector -I/usr/local/include -D_FORTIFY_SOURCE=2',
    optimize='-O',
    cppflags='-DHAS_FPSETMASK -DHAS_FLOATINGPOINT_H -fno-strict-aliasing -pipe -fstack-protector -I/usr/local/include'
    ccversion='', gccversion='4.2.1 Compatible FreeBSD Clang 3.4.1 (tags/RELEASE_34/dot1-final 208032)', gccosandvers=''
    intsize=4, longsize=8, ptrsize=8, doublesize=8, byteorder=12345678, doublekind=3
    d_longlong=define, longlongsize=8, d_longdbl=define, longdblsize=16, longdblkind=3
    ivtype='long', ivsize=8, nvtype='double', nvsize=8, Off_t='off_t', lseeksize=8
    alignbytes=8, prototype=define
  Linker and Libraries:
    ld='cc', ldflags ='-Wl,-E  -fstack-protector -L/usr/local/lib'
    libpth=/usr/lib /usr/local/lib /usr/include/clang/3.4.1 /usr/lib
    libs=-lpthread -lgdbm -lm -lcrypt -lutil -lc
    perllibs=-lpthread -lm -lcrypt -lutil -lc
    libc=, so=so, useshrplib=false, libperl=libperl.a
    gnulibc_version=''
  Dynamic Linking:
    dlsrc=dl_dlopen.xs, dlext=so, d_dlsymun=undef, ccdlflags=' '
    cccdlflags='-DPIC -fPIC', lddlflags='-shared  -L/usr/local/lib -fstack-protector'



@INC for perl 5.23.8:
    /home/mark/perl5/perlbrew/perls/perl-5.23.8/lib/site_perl/5.23.8/amd64-freebsd
    /home/mark/perl5/perlbrew/perls/perl-5.23.8/lib/site_perl/5.23.8
    /home/mark/perl5/perlbrew/perls/perl-5.23.8/lib/5.23.8/amd64-freebsd
    /home/mark/perl5/perlbrew/perls/perl-5.23.8/lib/5.23.8
    .


Environment for perl 5.23.8:
    HOME=/home/mark
    LANG (unset)
    LANGUAGE (unset)
    LC_ALL=en_US.UTF-8
    LD_LIBRARY_PATH (unset)
    LOGDIR (unset)
    PATH=/home/mark/perl5/perlbrew/bin:/home/mark/perl5/perlbrew/perls/perl-5.23.8/bin:/usr/local/bin:/usr/local/sbin:/usr/bin:/usr/sbin:/bin:/sbin:/home/mark/go/bin
    PERLBREW_BASHRC_VERSION=0.59
    PERLBREW_HOME=/home/mark/.perlbrew
    PERLBREW_MANPATH=/home/mark/perl5/perlbrew/perls/perl-5.23.8/man
    PERLBREW_PATH=/home/mark/perl5/perlbrew/bin:/home/mark/perl5/perlbrew/perls/perl-5.23.8/bin
    PERLBREW_PERL=perl-5.23.8
    PERLBREW_ROOT=/home/mark/perl5/perlbrew
    PERLBREW_VERSION=0.73
    PERL_BADLANG (unset)
    SHELL=/usr/local/bin/bash

@p5pRT
Copy link
Author

p5pRT commented Feb 24, 2016

From @tonycoz

On Tue Feb 23 16​:07​:11 2016, mmartinec wrote​:

Came across the following warning in Net/DNS/Resolver/Base.pm,
issued by perl 5.23.8 (test case minimized) :

$ perl -e 'use strict; use warnings; "test" =~ /[.].*[.]/'

POSIX syntax [. .] belongs inside character classes
(but this one isn't implemented) in regex;
marked by <-- HERE in m/[.].*[.] <-- HERE
/ at -e line 1.

Although this regexp is unnecessarily complicated, it seems
the perl is confused about what the syntax means.

This looks the same as #127582.

Tony

@p5pRT
Copy link
Author

p5pRT commented Feb 24, 2016

The RT System itself - Status changed from 'new' to 'open'

@p5pRT
Copy link
Author

p5pRT commented Mar 1, 2016

From @khwilliamson

This looks the same as #127582.

Tony

Confirmed. I'm now merging the tickets
--
Karl Williamson

@p5pRT
Copy link
Author

p5pRT commented Mar 1, 2016

@khwilliamson - Status changed from 'new' to 'open'

@p5pRT
Copy link
Author

p5pRT commented Mar 1, 2016

From @khwilliamson

Fixed by a9149df
--
Karl Williamson

@p5pRT
Copy link
Author

p5pRT commented Mar 1, 2016

@khwilliamson - Status changed from 'open' to 'pending release'

@p5pRT
Copy link
Author

p5pRT commented May 13, 2016

From @khwilliamson

Thank you for submitting this report. You have helped make Perl better.
 
With the release of Perl 5.24.0 on May 9, 2016, this and 149 other issues have been resolved.

Perl 5.24.0 may be downloaded via https://metacpan.org/release/RJBS/perl-5.24.0

@p5pRT p5pRT closed this as completed May 13, 2016
@p5pRT
Copy link
Author

p5pRT commented May 13, 2016

@khwilliamson - Status changed from 'pending release' to 'resolved'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant