- removed comment
Cactus: output actual path to config-info on failure
Pull request is here:
Keyword: None
Comments (6)
-
reporter -
- removed comment
What is the problem that this is trying to fix?
-
reporter - removed comment
Without it, if there is an error during configuration the output is (literally):
installation or configuration problem: C compiler cannot create executables (see configs/<configname>/config-data/config.log for details).
ie
<configname>
is shown as such. This is unnecessary annoying since to then actually look at the file one has to copy&paste the path then edit it by hand to replace<configname>
with the actual configuration name.This is made (much) worse by simfactory have a default simulation name "sim" so that people who run "simfactory/bin/sim build" for the first time and receive this error do not even (always) know what the configuration name is to begin with.
-
- removed comment
OK, this looks good to me. If you've tested it, please apply.
-
reporter - changed status to resolved
- removed comment
Hmm, apparently this was merged into master in git hash 387fae8c "Cactus: output actual path to config-info on failure" about three months ago. Apparently by myself.
Oh well. Closing this ticket.
-
reporter - edited description
- changed status to closed
- Log in to comment
Now that the Wu release is done, it is time to review and include or reject this.