Skip to content
This repository was archived by the owner on Jun 15, 2023. It is now read-only.

DOC: Fix wrong folder for python wrapping#353

Open
dekvall wants to merge 3 commits intoSuperElastix:developfrom
dekvall:fix-doc
Open

DOC: Fix wrong folder for python wrapping#353
dekvall wants to merge 3 commits intoSuperElastix:developfrom
dekvall:fix-doc

Conversation

@dekvall
Copy link

@dekvall dekvall commented Feb 11, 2020

No description provided.

@kaspermarstal
Copy link
Member

Hi! Thanks for you contribution!

When I look in the docs, this path has already been corrected: https://simpleelastix.readthedocs.io/GettingStarted.html. Would be okay with me closing this PR ?

@dekvall
Copy link
Author

dekvall commented Feb 17, 2020

Hi,

When i visit the link, the python install instructions for Windows look like this:
image

Though, when I click View on GitHub in the Read The Docs interface it links to the GettingStarted at this point in time https://github.com/SuperElastix/SimpleElastix/blob/de4ae202e93b477d953220a947c06ae61f45690d/Documentation/Sphinx/GettingStarted.rst
So maybe the docs didn't build properly.

de4ae20 removed sudo and added the Packaging/setup.py but Packaging was not removed from Line 175

${BUILD_DIRECTORY}/SimpleITK-build/Wrapping/Python/Packaging
which is what I wanted to remedy with this PR.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants