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
Rename CDash project from Slicer4 to SlicerStable #4644
Comments
@jcfr Did CDash developers ever answer your question about being able to rename a project? |
Indeed, there are few tasks to consider that would also require coordinating between different people:
For completeness, here is the comment of @zackgalbreath from Jan 17, 2019, 11:26 PM
|
Alternatively, we could simply create a new project called
That way creating the rewrite rule is not in the critical path to ensure the submission associated with Stable extension still work as expected. Since (1) keeping the historical build results is not relevant (they are only kept around for a limited time - details below), (2) duplicating the configuration is easy enough, this would be a reasonable path forward. @lassoan @sjh26 @jamesobutler What do you think ? Slicer4 CDash configuration ( Miscellaneous tab) |
We could leave Slicer4 as is, and add a new SlicerStable project. This can be done anytime, but if we want to avoid confusion then it should be done before Slicer5 is released. |
Yes, a new SlicerStable project is fine with me to be done prior to Slicer5 released. Then Slicer4 can be kept around for some time for debugging when certain things maybe began to fail, and then after awhile when we don't need to look back at the old results, then remove the Slicer4 project. |
Oh, @jcfr beat me by a few seconds! I agree with adding a new SlicerStable project. Why do we need the redirect? CTestConfig.cmake in extensions contain |
The setting of drop location when extensions are build is set in Extensions/CMake/SlicerExtensionsDashboardDriverScript.cmake, currently the project is hardcoded to |
This issue was created automatically from an original Mantis Issue. Further discussion may take place here.
The text was updated successfully, but these errors were encountered: