Skip to content
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

ensure that PYTHON_INSTALL_DIR is initialized for generated .dsv file #190

Merged
merged 2 commits into from
Aug 23, 2019

Conversation

dirk-thomas
Copy link
Contributor

Follow up of #187.

Signed-off-by: Dirk Thomas <dirk-thomas@users.noreply.github.com>
@dirk-thomas dirk-thomas added bug Something isn't working in review Waiting for review (Kanban column) labels Aug 22, 2019
@dirk-thomas dirk-thomas self-assigned this Aug 22, 2019
Signed-off-by: Dirk Thomas <dirk-thomas@users.noreply.github.com>
@dirk-thomas
Copy link
Contributor Author

I update the patch to use the native path (with backslashes on Windows) rather than the CMake path (using forward slashes). That way the paths are consistent (prefix and relative subdirectory as well with the classic environment hooks).

@dirk-thomas dirk-thomas merged commit 5857419 into master Aug 23, 2019
@delete-merged-branch delete-merged-branch bot deleted the dirk-thomas/setup-performance-fix branch August 23, 2019 16:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working in review Waiting for review (Kanban column)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants