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

Owner: jkeenan <jkeenan [at] cpan.org>
Requestors: kes-kes [at] yandex.ru
Cc:
AdminCc:

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



To: perlbug <perlbug [...] perl.org>, perl5-porters <perl5-porters [...] perl.org>
Subject: Typo for PERL5LIB doc in perlrun
Date: Fri, 30 Mar 2018 20:20:31 +0300
From: KES <kes-kes [...] yandex.ru>
Download (untitled) / with headers
text/plain 3.8k
This is a bug report for perl from kes-kes@yandex.ru, generated with the help of perlbug 1.40 running under perl 5.24.0. ----------------------------------------------------------------- [Please describe your issue here] Please fix the typo for PERL5LIB doc in perlrun Show quoted text
>Only the -[CDIMUdmtwW] switches are allowed
and later in text: Show quoted text
>If PERL5OPT begins with -T
so -T option is allowed and should be listed as allowed [Please do not change anything below this line] ----------------------------------------------------------------- --- Flags: category=docs severity=low --- Site configuration information for perl 5.24.0: Configured by kes at Wed Oct 19 14:07:47 EEST 2016. Summary of my perl5 (revision 5 version 24 subversion 0) configuration: Platform: osname=linux, osvers=4.4.0-43-generic, archname=x86_64-linux uname='linux work 4.4.0-43-generic #63-ubuntu smp wed oct 12 13:48:03 utc 2016 x86_64 x86_64 x86_64 gnulinux ' config_args='-de -Dprefix=/home/kes/perl5/perlbrew/perls/perl-5.24.0 -Aeval:scriptdir=/home/kes/perl5/perlbrew/perls/perl-5.24.0/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 ='-fwrapv -fno-strict-aliasing -pipe -fstack-protector-strong -I/usr/local/include -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64', optimize='-O2', cppflags='-fwrapv -fno-strict-aliasing -pipe -fstack-protector-strong -I/usr/local/include' ccversion='', gccversion='5.4.0 20160609', 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 =' -fstack-protector-strong -L/usr/local/lib' libpth=/usr/local/lib /usr/lib/gcc/x86_64-linux-gnu/5/include-fixed /usr/include/x86_64-linux-gnu /usr/lib /lib/x86_64-linux-gnu /lib/../lib /usr/lib/x86_64-linux-gnu /usr/lib/../lib /lib /lib64 /usr/lib64 libs=-lpthread -lnsl -ldl -lm -lcrypt -lutil -lc perllibs=-lpthread -lnsl -ldl -lm -lcrypt -lutil -lc libc=libc-2.23.so, so=so, useshrplib=false, libperl=libperl.a gnulibc_version='2.23' 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-strong' Locally applied patches: Devel::PatchPerl 1.38 --- @INC for perl 5.24.0: /home/kes/perl5/perlbrew/perls/perl-5.24.0/lib/site_perl/5.24.0/x86_64-linux /home/kes/perl5/perlbrew/perls/perl-5.24.0/lib/site_perl/5.24.0 /home/kes/perl5/perlbrew/perls/perl-5.24.0/lib/5.24.0/x86_64-linux /home/kes/perl5/perlbrew/perls/perl-5.24.0/lib/5.24.0 . --- Environment for perl 5.24.0: HOME=/home/kes LANG=en_US.UTF-8 LANGUAGE=en LC_ADDRESS=uk_UA.UTF-8 LC_IDENTIFICATION=uk_UA.UTF-8 LC_MEASUREMENT=uk_UA.UTF-8 LC_MESSAGES=en_US.UTF-8 LC_MONETARY=uk_UA.UTF-8 LC_NAME=uk_UA.UTF-8 LC_NUMERIC=uk_UA.UTF-8 LC_PAPER=uk_UA.UTF-8 LC_TELEPHONE=uk_UA.UTF-8 LD_LIBRARY_PATH (unset) LOGDIR (unset) PATH=/home/kes/perl5/perlbrew/bin:/home/kes/perl5/perlbrew/perls/perl-5.24.0/bin:/home/kes/bin:/home/kes/bin:/home/kes/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games PERLBREW=command perlbrew PERLBREW_BASHRC_VERSION=0.78 PERLBREW_HOME=/home/kes/.perlbrew PERLBREW_MANPATH=/home/kes/perl5/perlbrew/perls/perl-5.24.0/man PERLBREW_PATH=/home/kes/perl5/perlbrew/bin:/home/kes/perl5/perlbrew/perls/perl-5.24.0/bin PERLBREW_PERL=perl-5.24.0 PERLBREW_ROOT=/home/kes/perl5/perlbrew PERLBREW_VERSION=0.78 PERL_BADLANG (unset) SHELL=/bin/bash
RT-Send-CC: perl5-porters [...] perl.org
Download (untitled) / with headers
text/plain 1.8k
On Fri, 30 Mar 2018 17:22:57 GMT, kes-kes@yandex.ru wrote: Show quoted text
> This is a bug report for perl from kes-kes@yandex.ru, > generated with the help of perlbug 1.40 running under perl 5.24.0. > > > ----------------------------------------------------------------- > [Please describe your issue here] > > Please fix the typo for PERL5LIB doc in perlrun
> > Only the -[CDIMUdmtwW] switches are allowed
> and later in text:
> > If PERL5OPT begins with -T
> so -T option is allowed and should be listed as allowed > >
I'm confused about this report, because neither of the two text fragments you quote appear in the PERL5LIB section. Here is the entire content of the PERL5LIB section of pod/perlrun.pod (blead) (line lengths trimmed to look better in RT): ##### =item PERL5LIB X<PERL5LIB> A list of directories in which to look for Perl library files before looking in the standard library. Any architecture-specific and version-specific directories, such as F<version/archname/>, F<version/>, or F<archname/> under the specified locations are automatically included if they exist, with this lookup done at interpreter startup time. In addition, any directories matching the entries in C<$Config{inc_version_list}> are added. (These typically would be for older compatible perl versions installed in the same directory tree.) If PERL5LIB is not defined, PERLLIB is used. Directories are separated (like in PATH) by a colon on Unixish platforms and by a semicolon on Windows (the proper path separator being given by the command C<perl -V:I<path_sep>>). When running taint checks, either because the program was running setuid or setgid, or the B<-T> or B<-t> switch was specified, neither PERL5LIB nor PERLLIB is consulted. The program should instead say: use lib "/my/directory"; ##### Is there anything amiss in *that* section? Thank you very much. -- James E Keenan (jkeenan@cpan.org)
RT-Send-CC: perl5-porters [...] perl.org
Download (untitled) / with headers
text/plain 778b
I am sorry. This section: PERL5OPT Command-line options (switches). Switches in this variable are treated as if they were on every Perl command line. Only the -[CDIMUdmtwW] switches are allowed. When running taint checks (either because the program was running setuid or setgid, or because the -T or -t switch was used), this variable is ignored. If PERL5OPT begins with -T, tainting will be enabled and subsequent options ignored. If PERL5OPT begins with -t, tainting will be enabled, a writable dot removed from @INC, and subsequent options honored. As you can see -T option is not listed as allowed: [CDIMUdmtwW] switches are allowed
RT-Send-CC: perl5-porters [...] perl.org
Download (untitled) / with headers
text/plain 1.1k
On Tue, 04 Sep 2018 07:48:10 GMT, kes-kes@yandex.ru wrote: Show quoted text
> I am sorry. This section: > > PERL5OPT Command-line options (switches). Switches in this variable > are > treated as if they were on every Perl command line. Only > the > -[CDIMUdmtwW] switches are allowed. When running taint > checks > (either because the program was running setuid or setgid, > or > because the -T or -t switch was used), this variable is > ignored. If PERL5OPT begins with -T, tainting will be > enabled > and subsequent options ignored. If PERL5OPT begins with > -t, > tainting will be enabled, a writable dot removed from > @INC, > and subsequent options honored. > > As you can see -T option is not listed as allowed: > > [CDIMUdmtwW] switches are allowed
Thanks. I fixed this in commit 3809fbedec3d42a2a277c56a53c17eec96ca5c54 and sorted the switches to group upper-case and lower-case switches separately. Please review. Am taking this RT for the purpose of closing it in a few days unless someone objects. Thank you very much. -- James E Keenan (jkeenan@cpan.org)
RT-Send-CC: perl5-porters [...] perl.org
Download (untitled) / with headers
text/plain 1.4k
On Tue, 04 Sep 2018 13:58:57 GMT, jkeenan wrote: Show quoted text
> On Tue, 04 Sep 2018 07:48:10 GMT, kes-kes@yandex.ru wrote:
> > I am sorry. This section: > > > > PERL5OPT Command-line options (switches). Switches in this > > variable > > are > > treated as if they were on every Perl command line. Only > > the > > -[CDIMUdmtwW] switches are allowed. When running taint > > checks > > (either because the program was running setuid or setgid, > > or > > because the -T or -t switch was used), this variable is > > ignored. If PERL5OPT begins with -T, tainting will be > > enabled > > and subsequent options ignored. If PERL5OPT begins with > > -t, > > tainting will be enabled, a writable dot removed from > > @INC, > > and subsequent options honored. > > > > As you can see -T option is not listed as allowed: > > > > [CDIMUdmtwW] switches are allowed
> > Thanks. I fixed this in commit > 3809fbedec3d42a2a277c56a53c17eec96ca5c54 and sorted the switches to > group upper-case and lower-case switches separately. > > Please review. Am taking this RT for the purpose of closing it in a > few days unless someone objects. > > Thank you very much.
No objections heard, so I'm marking this ticket resolved. I believe it should be considered for backporting to perl-5.26.3 and perl-5.28.1, so I'm linking to the respective meta-tickets. Thank you very much. -- 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