This repository has been archived by the owner on Jul 8, 2022. It is now read-only.
Use python2 (if available) to run cxxtestgen #523
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
mliszcz
force-pushed
the
use-python2
branch
2 times, most recently
from
f78586d
to
755cb01
Compare
|
Hi, Can I get some comments on this? The current solution (hardcoded |
|
Did you try to replace cxxtestgen.py with a more recent version from https://github.com/CxxTest/cxxtest github repository? |
|
I think the PR is first step in the right direction i.e. a pure enhancement. |
bourtemb
approved these changes
Feb 5, 2019
bourtemb
added a commit
to bourtemb/cppTango
that referenced
this pull request
Mar 1, 2019
bourtemb
added a commit
to bourtemb/cppTango
that referenced
this pull request
Mar 1, 2019
bourtemb
added a commit
to bourtemb/cppTango
that referenced
this pull request
Mar 26, 2019
bourtemb
added a commit
to bourtemb/cppTango
that referenced
this pull request
Mar 28, 2019
Ingvord
pushed a commit
that referenced
this pull request
Mar 29, 2019
* Added CHANGELOG.md This file is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/). List changes since cppTango 9.2.5. Changes with potential compatibility issues are documented as suggested in #509 and are in bold. * Add links to issues and PRs in CHANGELOG.md * Update CHANGELOG.md with references to #508,#516,#518,#520,#521,#522,#523,#529,#531,#533,#537 * Update CHANGELOG.md with #517 * Update CHANGELOG.md. Add link for issue 395 * Moved 2 items from Fixed to Changed section Added a warning about the event subscription change sleep reduction * First version of the release notes for cppTango 9.3.3 Rename RELEASE_NOTES in RELEASE_NOTES.md. The previous RELEASE_NOTES content has been moved to CHANGELOG.md file. * Added reference to cppTango#539 Preparation for cppTango 9.3.3 release * Update CHANGELOG and RELEASE_NOTES with #541 and #542 * Reword one item in CHANGELOG.md and RELEASE_NOTES.md * Update CHANGELOG.md with #544, #546 and #549 * Add reference to #532 Add 9.3.3 release date * Add 9.3.3 release date
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
As python 2.x is reaching EOL soon (PEP 373) and some distros already switched to use python 3.x as a default python interpreter, I propose to follow PEP 394 and use
python2command/link instead ofpythonwhen invoking cxxtestgen.If
python2is missing, CMake will fall-back to usingpython(if it's on$PATH).