#1252: Thorn configuration scripts should not be run if there are missing thorns -------------------------+-------------------------------------------------- Reporter: hinder | Owner: Type: enhancement | Status: new Priority: minor | Milestone: Component: Cactus | Version: Keywords: | -------------------------+-------------------------------------------------- If there are thorns present in the thornlist which are not present in the source tree, Cactus currently displays the corresponding error message, and then runs the rest of the CST including thorn configuration scripts. Since these can build large external libraries (e.g. LORENE), it can be a long time before the user notices that the build has failed. I would prefer if Cactus aborted before running the CST scripts.
#1252: Thorn configuration scripts should not be run if there are missing thorns --------------------------+------------------------------------------------- Reporter: hinder | Owner: Type: enhancement | Status: new Priority: minor | Milestone: Component: Cactus | Version: Resolution: | Keywords: postrelease --------------------------+------------------------------------------------- Changes (by knarf):
* keywords: => postrelease
#1252: Thorn configuration scripts should not be run if there are missing thorns --------------------------+------------------------------------------------- Reporter: hinder | Owner: knarf Type: enhancement | Status: assigned Priority: minor | Milestone: Component: Cactus | Version: Resolution: | Keywords: postrelease --------------------------+------------------------------------------------- Changes (by anonymous):
* owner: => knarf * status: new => assigned
#1252: Thorn configuration scripts should not be run if there are missing thorns --------------------------+------------------------------------------------- Reporter: hinder | Owner: knarf Type: enhancement | Status: assigned Priority: minor | Milestone: Component: Cactus | Version: Resolution: | Keywords: postrelease --------------------------+-------------------------------------------------
Comment (by eschnett):
See https://trac.einsteintoolkit.org/ticket/1707.
#1252: Thorn configuration scripts should not be run if there are missing thorns --------------------------+------------------------------------------------- Reporter: hinder | Owner: knarf Type: enhancement | Status: review Priority: minor | Milestone: Component: Cactus | Version: Resolution: | Keywords: postrelease --------------------------+------------------------------------------------- Changes (by knarf):
* status: assigned => review
Comment:
https://bitbucket.org/cactuscode/cactus/branch/early_CST_abort
#1252: Thorn configuration scripts should not be run if there are missing thorns --------------------------+------------------------------------------------- Reporter: hinder | Owner: knarf Type: enhancement | Status: review Priority: minor | Milestone: Component: Cactus | Version: Resolution: | Keywords: postrelease --------------------------+-------------------------------------------------
Comment (by rhaas):
Erik is working on making the ExternalLibraries build themselves at the same time as the other thorns in ticket 1707 which is also up for review (the branch https://svn.cactuscode.org/projects/ExternalLibraries/GSL/branches/eschnett /delayed-build seems to have stabilized, at least there were no changes over the weekend).
Are those two approaches both applicable (ie does this patch still make sense assuming that all ExternelLibraries would soon do what GSL is trying to do in Erik's branch)?
#1252: Thorn configuration scripts should not be run if there are missing thorns --------------------------+------------------------------------------------- Reporter: hinder | Owner: knarf Type: enhancement | Status: review Priority: minor | Milestone: Component: Cactus | Version: Resolution: | Keywords: postrelease --------------------------+-------------------------------------------------
Comment (by knarf):
Assuming all external libraries are changed (even outside the ET) I don't see the immediate need for this change anymore, true. The only harm I can see, on the other hand, is that Cactus would bail out before it detected all possible errors - but then if it already got errors in that stage, the likelihood that following errors are simply caused by the earlier ones is high.
#1252: Thorn configuration scripts should not be run if there are missing thorns --------------------------+------------------------------------------------- Reporter: hinder | Owner: somebody Type: enhancement | Status: assigned Priority: minor | Milestone: Component: Cactus | Version: Resolution: | Keywords: postrelease --------------------------+------------------------------------------------- Changes (by knarf):
* owner: knarf => somebody * status: review => assigned
Comment:
testing assignment emails
#1252: Thorn configuration scripts should not be run if there are missing thorns --------------------------+------------------------------------------------- Reporter: hinder | Owner: knarf Type: enhancement | Status: assigned Priority: minor | Milestone: Component: Cactus | Version: Resolution: | Keywords: postrelease --------------------------+------------------------------------------------- Changes (by knarf):
* owner: somebody => knarf
Comment:
testing assignment emails
#1252: Thorn configuration scripts should not be run if there are missing thorns --------------------------+------------------------------------------------- Reporter: hinder | Owner: somebody Type: enhancement | Status: assigned Priority: minor | Milestone: Component: Cactus | Version: Resolution: | Keywords: postrelease --------------------------+------------------------------------------------- Changes (by knarf):
* owner: knarf => somebody
#1252: Thorn configuration scripts should not be run if there are missing thorns --------------------------+------------------------------------------------- Reporter: hinder | Owner: knarf Type: enhancement | Status: assigned Priority: minor | Milestone: Component: Cactus | Version: Resolution: | Keywords: postrelease --------------------------+------------------------------------------------- Changes (by knarf):
* owner: somebody => knarf
#1252: Thorn configuration scripts should not be run if there are missing thorns --------------------------+------------------------------------------------- Reporter: hinder | Owner: somebody Type: enhancement | Status: assigned Priority: minor | Milestone: Component: Cactus | Version: Resolution: | Keywords: postrelease --------------------------+------------------------------------------------- Changes (by knarf):
* owner: knarf => somebody
#1252: Thorn configuration scripts should not be run if there are missing thorns --------------------------+------------------------------------------------- Reporter: hinder | Owner: knarf Type: enhancement | Status: assigned Priority: minor | Milestone: Component: Cactus | Version: Resolution: | Keywords: postrelease --------------------------+------------------------------------------------- Changes (by knarf):
* owner: somebody => knarf
#1252: Thorn configuration scripts should not be run if there are missing thorns --------------------------+------------------------------------------------- Reporter: hinder | Owner: Type: enhancement | Status: assigned Priority: minor | Milestone: Component: Cactus | Version: Resolution: | Keywords: postrelease --------------------------+------------------------------------------------- Changes (by knarf):
* owner: knarf =>
#1252: Thorn configuration scripts should not be run if there are missing thorns --------------------------+------------------------------------------------- Reporter: hinder | Owner: Type: enhancement | Status: assigned Priority: minor | Milestone: Component: Cactus | Version: Resolution: | Keywords: postrelease --------------------------+-------------------------------------------------
Comment (by rhaas):
Can this be closed (see comment:6) due to the changes to ExternalLibraries separationg detection from building?
#1252: Thorn configuration scripts should not be run if there are missing thorns --------------------------+------------------------------------------------- Reporter: hinder | Owner: Type: enhancement | Status: closed Priority: minor | Milestone: Component: Cactus | Version: Resolution: fixed | Keywords: postrelease --------------------------+------------------------------------------------- Changes (by hinder):
* status: assigned => closed * resolution: => fixed
#1252: Thorn configuration scripts should not be run if there are missing thorns --------------------------+------------------------------------------------- Reporter: hinder | Owner: Type: enhancement | Status: closed Priority: minor | Milestone: Component: Cactus | Version: Resolution: fixed | Keywords: postrelease --------------------------+-------------------------------------------------
Comment (by knarf):
Yes. Just for the record: the CST aborts early in this case now. I just couldn't find this ticket when I merged the pull request a while back.
trac@lists.einsteintoolkit.org