Front page | perl.perl5.porters |
Postings from December 2009
2009/12/10 Tony Cook <tony@develop-help.com>:
> On Wed, Dec 09, 2009 at 03:29:03PM -0800, Jan Dubois wrote:
>> On Tue, 08 Dec 2009, Rafael Garcia-Suarez wrote:
>> >
>> > In perl.git, the branch blead has been updated
>> >
>> > <http://perl5.git.perl.org/perl.git/commitdiff/13290fcd82823e130109fca5199c99fb886e29eb?hp=958e6d4b516d826d7139a8ead
>> > 2b1054c5f8b9eda>
>> >
>> > - Log -----------------------------------------------------------------
>> > commit 13290fcd82823e130109fca5199c99fb886e29eb
>> > Author: Tony Cook <tony@develop-help.com>
>> > + $^O =~ /^(MSWin32|cygwin|uwin|dos|epoc)$/
>> > + and skip("Can't inplace edit without backups on $^O", 4);
>>
>> I suspect OS/2 belongs in that list as well.
>
> You're right, from hints/os2.sh:
>
> aout_ccflags="-DDOSISH -DPERL_IS_AOUT -DOS2=2 -DEMBED -I. $_defemxcrtrev -D__ST_MT_ERRNO__"
>
> The DOSISH define is used to decide whether to implement no-backup
> inplace editing.
>
> I"ll make a patch (tonight probably) if someone doesn't beat me to it.
These test doesn't look right to me, as the first two tests -i
(without *) pass fine on cygwin.
mingw and MSVC probably also.
The messsage says "Can't inplace edit without backups"
What it really tests is if -i and -i* works, but it doesn't check
the intermediate backups files which are created on those platforms for -i.
-i* doesn't work.
Tests are needed if
1. -i works (dosish ok)
2. -i works without backups (dosish nok)
3. -i* works (dosish nok)
4. -i* works without backups (dosish nok)
And we need the list of .bak extensions per platform on -i here.
And in the pod also, as from time to time I here someone saying
that -i doesn't work on windows. It does fine, just not inplace.
--
Reini Urban
http://phpwiki.org/ http://murbreak.at/
Thread Previous
|
Thread Next