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

Owner: Nobody
Requestors: me [at] xenu.pl
Cc:
AdminCc:

Operating System: mswin32
PatchStatus: (no value)
Severity: low
Type:
Perl Version: 5.24.1
Fixed In: (no value)



Date: Sun, 19 Feb 2017 18:28:49 +0100
To: perlbug [...] perl.org
Subject: [PATCH] Remove EPOC and MacOS Classic support from PathTools
From: Tomasz Konojacki <me [...] xenu.pl>
Download (untitled) / with headers
text/plain 3.2k
This is a bug report for perl from me@xenu.pl, generated with the help of perlbug 1.40 running under perl 5.24.1. ----------------------------------------------------------------- [Please describe your issue here] perl dropped MacOS classic and EPOC support many years ago. There is no point in keeping support for them in PathTools. This change will make any further work on PathTools a bit easier. [Please do not change anything below this line] ----------------------------------------------------------------- --- Flags: category=library severity=low module=File::Spec --- Site configuration information for perl 5.24.1: Configured by strawberry-perl at Mon Jan 16 02:03:13 2017. Summary of my perl5 (revision 5 version 24 subversion 1) configuration: Platform: osname=MSWin32, osvers=6.3, archname=MSWin32-x64-multi-thread uname='Win32 strawberry-perl 5.24.1.1 #1 Mon Jan 16 02:00:29 2017 x64' config_args='undef' hint=recommended, useposix=true, d_sigaction=undef useithreads=define, usemultiplicity=define use64bitint=define, use64bitall=undef, uselongdouble=undef usemymalloc=n, bincompat5005=undef Compiler: cc='gcc', ccflags =' -s -O2 -DWIN32 -DWIN64 -DCONSERVATIVE -DPERL_TEXTMODE_SCRIPTS -DPERL_IMPLICIT_CONTEXT -DPERL_IMPLICIT_SYS -fwrapv -fno-strict-aliasing -mms-bitfields', optimize='-s -O2', cppflags='-DWIN32' ccversion='', gccversion='4.9.2', gccosandvers='' intsize=4, longsize=4, ptrsize=8, doublesize=8, byteorder=12345678, doublekind=3 d_longlong=define, longlongsize=8, d_longdbl=define, longdblsize=16, longdblkind=3 ivtype='long long', ivsize=8, nvtype='double', nvsize=8, Off_t='long long', lseeksize=8 alignbytes=8, prototype=define Linker and Libraries: ld='g++', ldflags ='-s -L"C:\STRAWB~1\perl\lib\CORE" -L"C:\STRAWB~1\c\lib"' libpth=C:\STRAWB~1\c\lib C:\STRAWB~1\c\x86_64-w64-mingw32\lib C:\STRAWB~1\c\lib\gcc\x86_64-w64-mingw32\4.9.2 libs=-lmoldname -lkernel32 -luser32 -lgdi32 -lwinspool -lcomdlg32 -ladvapi32 -lshell32 -lole32 -loleaut32 -lnetapi32 -luuid -lws2_32 -lmpr -lwinmm -lversion -lodbc32 -lodbccp32 -lcomctl32 perllibs=-lmoldname -lkernel32 -luser32 -lgdi32 -lwinspool -lcomdlg32 -ladvapi32 -lshell32 -lole32 -loleaut32 -lnetapi32 -luuid -lws2_32 -lmpr -lwinmm -lversion -lodbc32 -lodbccp32 -lcomctl32 libc=, so=dll, useshrplib=true, libperl=libperl524.a gnulibc_version='' Dynamic Linking: dlsrc=dl_win32.xs, dlext=xs.dll, d_dlsymun=undef, ccdlflags=' ' cccdlflags=' ', lddlflags='-mdll -s -L"C:\STRAWB~1\perl\lib\CORE" -L"C:\STRAWB~1\c\lib"' --- @INC for perl 5.24.1: C:/Strawberry/perl/site/lib C:/Strawberry/perl/vendor/lib C:/Strawberry/perl/lib --- Environment for perl 5.24.1: HOME (unset) LANG (unset) LANGUAGE (unset) LD_LIBRARY_PATH (unset) LOGDIR (unset) PATH=C:\Program Files (x86)\Razer Chroma SDK\bin;C:\Program Files\Razer Chroma SDK\bin;C:\Windows\system32;C:\Windows;C:\Windows\System32\Wbem;C:\Windows\System32\WindowsPowerShell\v1.0\;C:\Windows\system32\config\systemprofile\AppData\Local\Microsoft\WindowsApps;C:\Program Files\Git\cmd;C:\Strawberry\c\bin;C:\Strawberry\perl\site\bin;C:\Strawberry\perl\bin;C:\Users\xenu\AppData\Local\Microsoft\WindowsApps; PERL_BADLANG (unset) SHELL (unset)
RT-Send-CC: perl5-porters [...] perl.org
the patch is attached
Subject: remove_epoc_macos.patch

Message body is not shown because it is too large.

Download (untitled) / with headers
text/plain 153b
Due to code freeze, we'll defer evaluation of this ticket until after release of perl-5.26.0. Thank you very much. -- James E Keenan (jkeenan@cpan.org)
Date: Wed, 29 Mar 2017 10:39:28 +0100
From: Dave Mitchell <davem [...] iabyn.com>
Subject: Re: [perl #130818] [PATCH] Remove EPOC and MacOS Classic support from PathTools
To: perl5-porters [...] perl.org
Download (untitled) / with headers
text/plain 642b
On Sun, Feb 19, 2017 at 09:36:46AM -0800, Tomasz Konojacki wrote: Show quoted text
> perl dropped MacOS classic and EPOC support many years ago. There > is no point in keeping support for them in PathTools. > > This change will make any further work on PathTools a bit easier.
But since PathTools is dual-lifed, is there any possibility that someone will want to use a newer CPAN PathTools release on an old perl running on such a platform? Or are MacOS classic and EPOC so old that they're not supported on any perl that supports PathTools? -- Hofstadter's Law: It always takes longer than you expect, even when you take into account Hofstadter's Law.
Subject: Re: [perl #130818] [PATCH] Remove EPOC and MacOS Classic support from PathTools
From: Leon Timmermans <fawaka [...] gmail.com>
To: Dave Mitchell <davem [...] iabyn.com>
Date: Wed, 29 Mar 2017 18:02:25 +0200
CC: Perl5 Porters <perl5-porters [...] perl.org>
Download (untitled) / with headers
text/plain 692b
On Wed, Mar 29, 2017 at 11:39 AM, Dave Mitchell <davem@iabyn.com> wrote: Show quoted text
> But since PathTools is dual-lifed, is there any possibility that > someone will want to use a newer CPAN PathTools release on an old perl > running on such a platform? Or are MacOS classic and EPOC so old that > they're not supported on any perl that supports PathTools?
MakeMaker dropped MacOS classic support in 2004, and the commit message says it "hasn't worked since 5.5x". I think we can safely assume it's dead. EPOC is the predecessor of Symbian, and apparently we're already using FIle::Spec::Win32 on Symbian so it also seems of little use (I have no idea if Perl on Symbian is in a working state). Leon
RT-Send-CC: perl5-porters [...] perl.org
Download (untitled) / with headers
text/plain 763b
On Wed, 29 Mar 2017 09:03:33 -0700, LeonT wrote: Show quoted text
> On Wed, Mar 29, 2017 at 11:39 AM, Dave Mitchell <davem@iabyn.com> wrote:
> > But since PathTools is dual-lifed, is there any possibility that > > someone will want to use a newer CPAN PathTools release on an old perl > > running on such a platform? Or are MacOS classic and EPOC so old that > > they're not supported on any perl that supports PathTools?
> > MakeMaker dropped MacOS classic support in 2004, and the commit > message says it "hasn't worked since 5.5x". I think we can safely > assume it's dead.
Perl may not run on MacOS classic any more, but many Mac OS X GUI applications still use old-style paths. The ability to manipulate them in a Perl script is still valuable. -- Father Chrysostomos
Date: Thu, 30 Mar 2017 01:36:48 +0200
CC: Perl5 Porters <perl5-porters [...] perl.org>
To: perlbug <perlbug-followup [...] perl.org>
Subject: Re: [perl #130818] [PATCH] Remove EPOC and MacOS Classic support from PathTools
From: Leon Timmermans <fawaka [...] gmail.com>
Download (untitled) / with headers
text/plain 430b
On Thu, Mar 30, 2017 at 12:39 AM, Father Chrysostomos via RT <perlbug-followup@perl.org> wrote: Show quoted text
> Perl may not run on MacOS classic any more, but many Mac OS X GUI applications still use old-style paths. The ability to manipulate them in a Perl script is still valuable.
That is an interesting point. It actually seems at least one module on CPAN does exactly this (Mac::iPod::Gnupod). Guess it's not as dead as thought. Leon
RT-Send-CC: perl5-porters [...] perl.org
Download (untitled) / with headers
text/plain 583b
On Wed, 29 Mar 2017 15:39:28 -0700, sprout wrote: Show quoted text
> Perl may not run on MacOS classic any more, but many Mac OS X GUI > applications still use old-style paths. The ability to manipulate > them in a Perl script is still valuable.
If it's the case and there's still some support for old-style paths in OS X, we indeed should consider keeping File::Spec::MacOS. However, I think that MacOS-specific code in Cwd.pm should be removed anyway. That code isn't platform independent and will work only when $^O eq 'MacOS', which means it's useless on other (i.e. supported) platfoms.
RT-Send-CC: perl5-porters [...] perl.org
Download (untitled) / with headers
text/plain 560b
On Wed, 29 Mar 2017 02:40:29 -0700, davem wrote: Show quoted text
> But since PathTools is dual-lifed, is there any possibility that > someone will want to use a newer CPAN PathTools release on an old perl > running on such a platform? Or are MacOS classic and EPOC so old that > they're not supported on any perl that supports PathTools?
The last EPOC device was released in 1999. EPOC perl port wasn't much more than a toy and I'd be surprised if anyone is still using it, let alone relying on it. Regarding MacOS, I have nothing to add to the other posts in this thread.
RT-Send-CC: perl5-porters [...] perl.org
Download (untitled) / with headers
text/plain 922b
On Thu, 30 Mar 2017 00:40:48 GMT, me@xenu.pl wrote: Show quoted text
> On Wed, 29 Mar 2017 02:40:29 -0700, davem wrote:
> > But since PathTools is dual-lifed, is there any possibility that > > someone will want to use a newer CPAN PathTools release on an old > > perl > > running on such a platform? Or are MacOS classic and EPOC so old that > > they're not supported on any perl that supports PathTools?
> > The last EPOC device was released in 1999. EPOC perl port wasn't much > more than a toy and I'd be surprised if anyone is still using it, let > alone relying on it. > > Regarding MacOS, I have nothing to add to the other posts in this > thread.
Tomasz: It seems there is more consensus for removing support for EPOC tool in PathTools than for MacOS. In order to move this RT toward resolution, would you be able to provide a patch that *only* removes EPOC support? Thank you very much. -- James E Keenan (jkeenan@cpan.org)
RT-Send-CC: perl5-porters [...] perl.org
Download (untitled) / with headers
text/plain 1.8k
On Tue, 16 May 2017 00:58:39 GMT, jkeenan wrote: Show quoted text
> On Thu, 30 Mar 2017 00:40:48 GMT, me@xenu.pl wrote:
> > On Wed, 29 Mar 2017 02:40:29 -0700, davem wrote:
> > > But since PathTools is dual-lifed, is there any possibility that > > > someone will want to use a newer CPAN PathTools release on an old > > > perl > > > running on such a platform? Or are MacOS classic and EPOC so old > > > that > > > they're not supported on any perl that supports PathTools?
> > > > The last EPOC device was released in 1999. EPOC perl port wasn't much > > more than a toy and I'd be surprised if anyone is still using it, let > > alone relying on it. > > > > Regarding MacOS, I have nothing to add to the other posts in this > > thread.
> > Tomasz: > > It seems there is more consensus for removing support for EPOC tool in > PathTools than for MacOS. > > In order to move this RT toward resolution, would you be able to > provide a patch that *only* removes EPOC support? >
I decided to try to do this myself. I started from the patch already submitted in this RT. However, I found several mentions of /EPOC/i in the core distribution apart from those addressed in the original patch. In addition, when you modify source code in, say, File::Spec::Unix, you have to bump $VERSION in all the *.pm files under dist/File-Spec. The results so far can be found in the smoke-me/jkeenan/130818-remove-epoc branch. That still needs more work, since: * There are 2 modules under cpan/ which refer to EPOC. Our protocol is that those should be corrected upstream first. * I made no attempt to revise the data in Module::CoreList to reflect removal of one .pm file and the modification of several others. All this, of course, presumes that we have actually made a final decision to remove EPOC support from the core distribution -- and we haven't done that yet. Thank you very much. -- James E Keenan (jkeenan@cpan.org)
RT-Send-CC: perl5-porters [...] perl.org
Download (untitled) / with headers
text/plain 2.1k
On Wed, 14 Jun 2017 01:34:57 GMT, jkeenan wrote: Show quoted text
> On Tue, 16 May 2017 00:58:39 GMT, jkeenan wrote:
> > On Thu, 30 Mar 2017 00:40:48 GMT, me@xenu.pl wrote:
> > > On Wed, 29 Mar 2017 02:40:29 -0700, davem wrote:
> > > > But since PathTools is dual-lifed, is there any possibility that > > > > someone will want to use a newer CPAN PathTools release on an old > > > > perl > > > > running on such a platform? Or are MacOS classic and EPOC so old > > > > that > > > > they're not supported on any perl that supports PathTools?
> > > > > > The last EPOC device was released in 1999. EPOC perl port wasn't > > > much > > > more than a toy and I'd be surprised if anyone is still using it, > > > let > > > alone relying on it. > > > > > > Regarding MacOS, I have nothing to add to the other posts in this > > > thread.
> > > > Tomasz: > > > > It seems there is more consensus for removing support for EPOC tool > > in > > PathTools than for MacOS. > > > > In order to move this RT toward resolution, would you be able to > > provide a patch that *only* removes EPOC support? > >
> > I decided to try to do this myself. I started from the patch already > submitted in this RT. However, I found several mentions of /EPOC/i in > the core distribution apart from those addressed in the original > patch. In addition, when you modify source code in, say, > File::Spec::Unix, you have to bump $VERSION in all the *.pm files > under dist/File-Spec. > > The results so far can be found in the smoke-me/jkeenan/130818-remove- > epoc branch. That still needs more work, since: > > * There are 2 modules under cpan/ which refer to EPOC. Our protocol > is that those should be corrected upstream first. > > * I made no attempt to revise the data in Module::CoreList to reflect > removal of one .pm file and the modification of several others. > > All this, of course, presumes that we have actually made a final > decision to remove EPOC support from the core distribution -- and we > haven't done that yet. >
Sawyer: Can we get a thumbs up for this removal of EPOC support from PathTools? If so, I'll proceed with the next steps. -- James E Keenan (jkeenan@cpan.org)
Date: Fri, 23 Jun 2017 19:23:19 -0400
From: Sawyer X <xsawyerx [...] gmail.com>
CC: perl5-porters [...] perl.org
Subject: Re: [perl #130818] [PATCH] Remove EPOC and MacOS Classic support from PathTools
To: perlbug-followup [...] perl.org
Download (untitled) / with headers
text/plain 2.1k
On 06/22/2017 05:28 PM, James E Keenan via RT wrote: Show quoted text
> On Wed, 14 Jun 2017 01:34:57 GMT, jkeenan wrote:
>> On Tue, 16 May 2017 00:58:39 GMT, jkeenan wrote:
>>> On Thu, 30 Mar 2017 00:40:48 GMT, me@xenu.pl wrote:
>>>> On Wed, 29 Mar 2017 02:40:29 -0700, davem wrote:
>>>>> But since PathTools is dual-lifed, is there any possibility that >>>>> someone will want to use a newer CPAN PathTools release on an old >>>>> perl >>>>> running on such a platform? Or are MacOS classic and EPOC so old >>>>> that >>>>> they're not supported on any perl that supports PathTools?
>>>> The last EPOC device was released in 1999. EPOC perl port wasn't >>>> much >>>> more than a toy and I'd be surprised if anyone is still using it, >>>> let >>>> alone relying on it. >>>> >>>> Regarding MacOS, I have nothing to add to the other posts in this >>>> thread.
>>> Tomasz: >>> >>> It seems there is more consensus for removing support for EPOC tool >>> in >>> PathTools than for MacOS. >>> >>> In order to move this RT toward resolution, would you be able to >>> provide a patch that *only* removes EPOC support? >>>
>> I decided to try to do this myself. I started from the patch already >> submitted in this RT. However, I found several mentions of /EPOC/i in >> the core distribution apart from those addressed in the original >> patch. In addition, when you modify source code in, say, >> File::Spec::Unix, you have to bump $VERSION in all the *.pm files >> under dist/File-Spec. >> >> The results so far can be found in the smoke-me/jkeenan/130818-remove- >> epoc branch. That still needs more work, since: >> >> * There are 2 modules under cpan/ which refer to EPOC. Our protocol >> is that those should be corrected upstream first. >> >> * I made no attempt to revise the data in Module::CoreList to reflect >> removal of one .pm file and the modification of several others. >> >> All this, of course, presumes that we have actually made a final >> decision to remove EPOC support from the core distribution -- and we >> haven't done that yet. >>
> Sawyer: Can we get a thumbs up for this removal of EPOC support from PathTools?
I agree with removing the Mac OS specific code from Cwd.pm, but not removing File::Spec::MacOS. (Just to be clear on it.)
To: perl5-porters [...] perl.org
Date: Mon, 18 Dec 2017 03:26:10 +0000
From: Zefram <zefram [...] fysh.org>
Subject: Re: [perl #130818] [PATCH] Remove EPOC and MacOS Classic support from PathTools
Download (untitled) / with headers
text/plain 291b
I've removed support for running PathTools on EPOC in commit e9a5b59fedc30cba91617c6b239f9568cf4ce08e, and on MacOS Classic in commit e229c0feaa828e8255bda950ed3456ef196226b3. There's no reason to remove support for processing paths of those platforms. This ticket can be closed. -zefram


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