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

DOC: Fix wrong folder for python wrapping #353

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

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