Skip to content
Snippets Groups Projects

[python][cmake] fix handling of venv parameters

Merged Christian Engwer requested to merge fix-python-venv-debian-testing into master

cmake parameters are passed as a list. The "-p ${PYTHON_EXECUTABLE}" option was passed as string, which lead to problems setting up the venv on my Debian testing system.

We now pass all parameters as lists, just like it should be...

  1. @core: perhaps someone from the cmake and/or python crowd should have a look...
  2. I guess this wuld be god in 2.7?!

Merge request reports

Pipeline #23316 passed

Pipeline passed for b1864f5d on fix-python-venv-debian-testing

Approval is optional

Merged by Christian EngwerChristian Engwer 5 years ago (Jan 26, 2020 9:20pm UTC)

Merge details

  • Changes merged into master with 47c65b8e.
  • Deleted the source branch.

Pipeline #23936 passed

Pipeline passed for 47c65b8e on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading