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

maniread() should return a tied, case-insensitive hash on VMS [cpan #2746] #14006

Closed
p5pRT opened this issue Jul 28, 2014 · 6 comments
Closed

Comments

@p5pRT
Copy link

p5pRT commented Jul 28, 2014

Migrated from rt.perl.org#122420 (status was 'rejected')

Searchable as RT122420$

@p5pRT
Copy link
Author

p5pRT commented Jul 28, 2014

From @jkeenan

[Since ExtUtils-Manifest is now primarily maintained by Perl 5 Porters, I am moving this ticket here from https://rt.cpan.org/Ticket/Display.html?id=2746. -- jkeenan]

#####
http​://archive.develooper.com/makemaker@​perl.org/msg01210.html

maniread() needs to deal better with the non-case preserving nature of VMS.

It currently does this clumsily by downcasing the filenames. This is lossy. Also, VMS isn't always non-case preserving.

Instead, have maniread() return a tied case-insensitive hash on VMS.
#####

This issue *may* have been addressed by subsequent work on the VMS-specific parts of ExtUtils​::Manifest. Craig (or anyone with VMS chops), can you comment?

Thank you very much.
Jim Keenan

@p5pRT
Copy link
Author

p5pRT commented Jul 28, 2014

From @jkeenan

Summary of my perl5 (revision 5 version 21 subversion 3) configuration​:
  Commit id​: ce5b0b8
  Platform​:
  osname=linux, osvers=3.13.0-32-generic, archname=x86_64-linux
  uname='linux zareason 3.13.0-32-generic #57-ubuntu smp tue jul 15 03​:51​:08 utc 2014 x86_64 x86_64 x86_64 gnulinux '
  config_args='-des -Dusedevel'
  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 -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=8, ptrsize=8, doublesize=8, byteorder=12345678
  d_longlong=define, longlongsize=8, d_longdbl=define, longdblsize=16
  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 -L/usr/local/lib'
  libpth=/usr/local/lib /usr/lib/gcc/x86_64-linux-gnu/4.8/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
  libs=-lnsl -ldl -lm -lcrypt -lutil -lc
  perllibs=-lnsl -ldl -lm -lcrypt -lutil -lc
  libc=libc-2.19.so, so=so, useshrplib=false, libperl=libperl.a
  gnulibc_version='2.19'
  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'

Characteristics of this binary (from libperl)​:
  Compile-time options​: HAS_TIMES PERLIO_LAYERS PERL_DONT_CREATE_GVSV
  PERL_HASH_FUNC_ONE_AT_A_TIME_HARD PERL_MALLOC_WRAP
  PERL_NEW_COPY_ON_WRITE PERL_PRESERVE_IVUV
  PERL_USE_DEVEL USE_64_BIT_ALL USE_64_BIT_INT
  USE_LARGE_FILES USE_LOCALE USE_LOCALE_COLLATE
  USE_LOCALE_CTYPE USE_LOCALE_NUMERIC USE_LOCALE_TIME
  USE_PERLIO USE_PERL_ATOF
  Built under linux
  Compiled at Jul 27 2014 17​:53​:58
  %ENV​:
  PERL5LIB="/home/jkeenan/perl5/lib/perl5"
  PERLBREW_BASHRC_VERSION="0.67"
  PERLBREW_HOME="/home/jkeenan/.perlbrew"
  PERLBREW_MANPATH="/home/jkeenan/perl5/perlbrew/perls/perl-5.20.0/man"
  PERLBREW_PATH="/home/jkeenan/perl5/perlbrew/bin​:/home/jkeenan/perl5/perlbrew/perls/perl-5.20.0/bin"
  PERLBREW_PERL="perl-5.20.0"
  PERLBREW_ROOT="/home/jkeenan/perl5/perlbrew"
  PERLBREW_VERSION="0.67"
  PERL_LOCAL_LIB_ROOT="/home/jkeenan/perl5"
  PERL_MB_OPT="--install_base "/home/jkeenan/perl5""
  PERL_MM_OPT="INSTALL_BASE=/home/jkeenan/perl5"
  PERL_WORKDIR="gitwork/perl"
  @​INC​:
  lib
  /home/jkeenan/perl5/lib/perl5/x86_64-linux
  /home/jkeenan/perl5/lib/perl5
  /usr/local/lib/perl5/site_perl/5.21.3/x86_64-linux
  /usr/local/lib/perl5/site_perl/5.21.3
  /usr/local/lib/perl5/5.21.3/x86_64-linux
  /usr/local/lib/perl5/5.21.3
  .

@p5pRT
Copy link
Author

p5pRT commented Jul 28, 2014

From @craigberry

On Sun, Jul 27, 2014 at 8​:26 PM, James E Keenan
<perlbug-followup@​perl.org> wrote​:

# New Ticket Created by James E Keenan
# Please include the string​: [perl #122420]
# in the subject line of all future correspondence about this issue.
# <URL​: https://rt-archive.perl.org/perl5/Ticket/Display.html?id=122420 >

[Since ExtUtils-Manifest is now primarily maintained by Perl 5 Porters, I am moving this ticket here from https://rt.cpan.org/Ticket/Display.html?id=2746. -- jkeenan]

#####
http​://archive.develooper.com/makemaker@​perl.org/msg01210.html

maniread() needs to deal better with the non-case preserving nature of VMS.

It currently does this clumsily by downcasing the filenames. This is lossy. Also, VMS isn't always non-case preserving.

Instead, have maniread() return a tied case-insensitive hash on VMS.
#####

This issue *may* have been addressed by subsequent work on the VMS-specific parts of ExtUtils​::Manifest. Craig (or anyone with VMS chops), can you comment?

A quick glance at Manifest.pm suggests that quite a bit of the
VMS-specific hacks should just be ripped out. No promises about when
I'll get to it, though.

@p5pRT
Copy link
Author

p5pRT commented Jul 28, 2014

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

@p5pRT
Copy link
Author

p5pRT commented Sep 14, 2014

From @jkeenan

On Sun Jul 27 18​:26​:33 2014, jkeenan wrote​:

[Since ExtUtils-Manifest is now primarily maintained by Perl 5
Porters, I am moving this ticket here from
https://rt.cpan.org/Ticket/Display.html?id=2746. -- jkeenan]

#####
http​://archive.develooper.com/makemaker@​perl.org/msg01210.html

maniread() needs to deal better with the non-case preserving nature of
VMS.

Since it was subsequently resolved that ExtUtils-Manifest would be maintained outside of core by PerlToolchainGang, and since these issues are referenced at​: Perl-Toolchain-Gang/ExtUtils-Manifest#1, I am closing this rt.perl.org ticket.

Thank you very much.
Jim Keenan

@p5pRT
Copy link
Author

p5pRT commented Sep 14, 2014

@jkeenan - Status changed from 'open' to 'rejected'

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

No branches or pull requests

1 participant