develooper Front page | perl.perl5.porters | Postings from June 2009

[PATCH] limit what add-package.pl might try to delete

Thread Next
From:
David Golden
Date:
June 28, 2009 20:27
Subject:
[PATCH] limit what add-package.pl might try to delete
Message ID:
1246246030-29780-1-git-send-email-dagolden@cpan.org
Old way looked in perl source for files matching exclude
list to delete.  New way looks only in source distribution.

This means that files already in perl source will need to be
manually removed, but avoids the risk that a bad exclusion
regex blows away unrelated things in the source tree.
---
 Porting/add-package.pl |   10 +++++-----
 1 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/Porting/add-package.pl b/Porting/add-package.pl
index 20947c0..46d1a28 100644
--- a/Porting/add-package.pl
+++ b/Porting/add-package.pl
@@ -80,7 +80,7 @@ my @LibFiles;
                                 undef
                             }
                         : 1
-                     } `find $Repo/lib -type f`
+                     } `find lib -type f`
         or die "Could not detect library files\n";
       
     print "done\n" if $Verbose;
@@ -141,11 +141,11 @@ my @TestFiles;
                     ### should we get rid of this file?
                     grep { $ExcludeRe && $_ =~ $ExcludeRe
                         ? do {  warn "Removing $_\n";
-                                system("rm $_") and die "rm '$_' failed: $?";
+                                system("rm $TopDir/$_") and die "rm '$_' failed: $?";
                                 undef
                             }
                         : 1
-                     } `find $TopDir/t -type f`
+                     } `find t -type f`
         or die "Could not detect testfiles\n";
 
     print "done\n" if $Verbose;
@@ -173,11 +173,11 @@ BIN: {
                 ### should we get rid of this file?
                 grep { $ExcludeRe && $_ =~ $ExcludeRe
                     ? do {  warn "Removing $_\n";
-                            system("rm $_") and die "rm '$_' failed: $?";
+                            system("rm $TopDir/$_") and die "rm '$_' failed: $?";
                             undef
                         }
                     : 1
-                 } `find $TopBinDir -type f`
+                 } `find $BinDir -type f`
         or die "Could not detect binfiles\n";
 
     print "done\n" if $Verbose;
-- 
1.5.6.3


Thread Next


nntp.perl.org: Perl Programming lists via nntp and http.
Comments to Ask Bjørn Hansen at ask@perl.org | Group listing | About