-
Notifications
You must be signed in to change notification settings - Fork 357
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
Fix lalsuite links #4101
Fix lalsuite links #4101
Conversation
docs/install_lalsuite.rst
Outdated
@@ -34,7 +34,7 @@ changing the path to the activate script appropriately. Clone the lalsuite repo | |||
Note that this checks out a read-only repository. If you want a git repository | |||
that you can edit, you can either fork this repository to your own GitHub | |||
account or, if you have ``LIGO.ORG`` credentials, you can follow the | |||
`instructions on the DASWG pages for cloning lalsuite <https://www.lsc-group.phys.uwm.edu/daswg/docs/howto/advanced-lalsuite-git.html#clone>`_. | |||
`instructions on the DASWG pages for cloning lalsuite <https://wiki.ligo.org/Computing/LALSuiteInstall>`_. |
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.
There are no 'DASWG pages' any more. They are LSC Computing/Software pages
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.
OK modulo 'DASWG' -> CompSoft or similar
Thanks Tom! DASWG has been changed to LSC Computing as requested. |
* Update install_lalsuite.rst * Update install_lalsuite.rst
* Update install_lalsuite.rst * Update install_lalsuite.rst
This fixes #3974 by updating links to lalsuite in the install lalsuite documentation page to that recommended by @duncanmmacleod.