This is a live mirror of the Perl 5 development currently hosted at https://github.com/perl/perl5
Remove obscure "attrib -r ..\t\*.*" command from minitest recipe
authorSteve Hay <steve.m.hay@googlemail.com>
Wed, 7 Jan 2015 14:24:14 +0000 (14:24 +0000)
committerSteve Hay <steve.m.hay@googlemail.com>
Wed, 7 Jan 2015 14:24:14 +0000 (14:24 +0000)
This command is not run for any other test targets, and there seems to be
no point in it, at least not now, since files under t\ are not read-only
anyway.

(It dates from the creation of the minitest target in commit 137443ea0a.
At that time, the same command was already used in the $(PERLEXE) target,
having been added there in the initial creation of win32/Makefile in
commit 0a753a7640. Needless to say, neither commit gives any details on
why the command was deemed necessary.)

win32/Makefile
win32/makefile.mk

index 3d80676..f8d8a69 100644 (file)
@@ -1290,7 +1290,6 @@ minitest : .\config.h $(MINIPERL) ..\git_version.h $(GLOBEXE) $(CONFIGPM) $(UNID
        if exist ..\t\perl.exe del /f ..\t\perl.exe
        rename ..\t\miniperl.exe perl.exe
        $(XCOPY) $(GLOBEXE) ..\t\$(NULL)
-       attrib -r ..\t\*.*
 # Note this perl.exe is miniperl
        cd ..\t && perl.exe TEST base/*.t comp/*.t cmd/*.t run/*.t io/*.t re/*.t opbasic/*.t op/*.t uni/*.t perf/*.t pragma/*.t
 
index a234813..9cfea1c 100644 (file)
@@ -1570,7 +1570,6 @@ minitest : .\config.h $(MINIPERL) ..\git_version.h $(GLOBEXE) $(CONFIGPM) $(UNID
        if exist ..\t\perl.exe del /f ..\t\perl.exe
        rename ..\t\miniperl.exe perl.exe
        $(XCOPY) $(GLOBEXE) ..\t\$(NULL)
-       attrib -r ..\t\*.*
 # Note this perl.exe is miniperl
        cd ..\t && perl.exe TEST base/*.t comp/*.t cmd/*.t run/*.t io/*.t re/*.t opbasic/*.t op/*.t uni/*.t perf/*.t pragma/*.t