New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

./Build alien_dry_run #102

Merged
merged 3 commits into from Jan 27, 2015

Conversation

Projects
None yet
4 participants
@plicease
Member

plicease commented Jan 25, 2015

Debugging a issue with Alien::FFI on MSWin32 and it was useful to be able to see the commands as they would actually be executed, without actually executing, so I added an ACTION_alien_dry_run.

Its not quite as clever as I would like it to be (ie, would be nice to probe the repositories and tell me exactly which one(s) it would download), but I think this is a good start.

@plicease

This comment has been minimized.

Member

plicease commented Jan 26, 2015

rebased and updated to reflect that commands can be array refs.

I recall now that there is a ./Build fakeinstall command so maybe a more suitable name is fakebuild or alien_fakebuild?

@zmughal

This comment has been minimized.

Member

zmughal commented Jan 26, 2015

Merge: Aye.

I like the name alien_fakebuild, but I'm not the best person to comment on that. :-P

@plicease

This comment has been minimized.

Member

plicease commented Jan 26, 2015

@zmughal that name is growing on me too, pending further input I will rename it before merging if this is approved.

@jberger

This comment has been minimized.

Member

jberger commented Jan 26, 2015

I have often used dry_run in things, but yeah I think fakebuild is more consistent. Aye

@mohawk2

This comment has been minimized.

Contributor

mohawk2 commented Jan 27, 2015

I say whatever name is considered best. Otherwise, "aye".

@plicease plicease merged commit edf7594 into Perl5-Alien:master Jan 27, 2015

1 check was pending

continuous-integration/travis-ci The Travis CI build is in progress
Details
@plicease

This comment has been minimized.

Member

plicease commented Jan 27, 2015

Sent to CPAN as 0.009_01

@plicease plicease deleted the plicease:dry_run branch Feb 16, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment