-
Notifications
You must be signed in to change notification settings - Fork 17
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
fcm make: allow multiple runs in same directory #188
Conversation
bd1d6bb
to
7428e89
Compare
@benfitzpatrick please review. |
A make can now be named, so multiple non-overlapping makes can work in the same directory.
use = /path/to/succeeded/make/ | ||
</pre> | ||
|
||
<p>For more information on how an inheritance behave, see:</p> |
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.
behaves
Doc typo fixed. |
@@ -36,10 +36,11 @@ my $E = 'FCM::System::Exception'; | |||
|
|||
# Configuration parser label to action map | |||
my %CONFIG_PARSER_OF = ( | |||
'dest' => \&_parse_dest, |
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.
I preferred this way of doing it
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.
The old way has been restored.
end program hello | ||
__FORTRAN__ | ||
|
||
fcm make -q -C "${PWD}/hello" -n '-friend' |
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.
Might as well do a 'run_pass'?
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.
Done.
build.source=$HERE/src | ||
build.target{task}=link | ||
__CFG__ | ||
run_fail "${TEST_KEY_BASE}-snub" fcm make -C "${PWD}/snub" -n '-no-friend' |
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.
What is the error message like? Can we have a grep?
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.
Done.
One remaining comment (the 'grep' one). |
We will have to be careful to match versions across platforms - a new-fcm's extract breaks old-fcm's build. |
fcm make: allow multiple runs in same directory
A make can now be named, so multiple non-overlapping makes can work in
the same directory.
Close #126.