Conversation
unlink doesn't do anything to a directory / folder on most platforms so use rmtree. This is probably overkill anyway since the directories shouldn't exist if the test passes
Looks ok. Aye. |
use File::Copy qw( copy ); | ||
|
||
my $cmd = shift; | ||
@ARGV = grep { s/DESTDIR/$ENV{DESTDIR}/eg; $_ } @ARGV; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is modifying $_
in a grep
. Might be nicer as for
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually... should that be a /e? It seems like perl-evaluating the env var isn't right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be a map
not grep
, will update the branch.
Merge: Aye Looks good and that RPM is awesome! I kinda feel that I'd like to have a note in |
also the /e is unnecessary in this case
@zmughal added some commentary. |
Alien-Base: Alien-Base: DESTDIR support
This should resolve #39
also noted a documentation error when I was reviewing the documentation in writing the test.
In addition to the unit test, I was able to build rpms in OBS for both Acme::Alien::DontPanic and Acme::Ford::Prefect
https://build.opensuse.org/project/show/home:platypus:Alien-Base