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

blacklist pythontex in the buildsystem #708

Closed
zuntrax opened this issue Jan 17, 2017 · 1 comment · Fixed by #723
Closed

blacklist pythontex in the buildsystem #708

zuntrax opened this issue Jan 17, 2017 · 1 comment · Fixed by #723
Labels
area: buildsystem Related to our cmake/python buildsystem good first issue Suitable for newcomers improvement Enhancement of an existing component just do it You can start working on this, there should be nothing left to discuss
Milestone

Comments

@zuntrax
Copy link
Contributor

zuntrax commented Jan 17, 2017

Pythontex is some kind of a python interpreter bundled with texlive. CMake finds it, but it's not suitable for what we are doing (obviously). We should blacklist it to get rid of clutter in the configure output. Also the "dropping invalid python interpreter" message (for the remaining possible cases) should not be a warning, but a normal message without modifiers.

Configure output
[...]
Testing /usr/bin/python3
Testing /usr/bin/python3.6
Testing /usr/bin/python2.7
Testing /usr/bin/python3.6m
Testing /usr/share/texmf-dist/scripts/pythontex/pythontex.py
usage: pythontex.py [-h] [--version] [--encoding ENCODING]
                    [--error-exit-code {true,false}] [--runall [{true,false}]
                    | --rerun {never,modified,errors,warnings,always}]
                    [--hashdependencies [{true,false}]] [-j N] [-v]
                    [--interpreter INTERPRETER]
                    [--debug [::] | --interactive
                    [::]]
                    TEXNAME
pythontex.py: error: unrecognized arguments: -c
CMake Warning at buildsystem/modules/FindPython.cmake:149 (message):
  Dropping invalid python interpreter
  '/usr/share/texmf-dist/scripts/pythontex/pythontex.py'
Call Stack (most recent call first):
  buildsystem/HandlePythonOptions.cmake:6 (find_package)
  CMakeLists.txt:69 (include)
[...]
@zuntrax zuntrax added area: buildsystem Related to our cmake/python buildsystem improvement Enhancement of an existing component labels Jan 17, 2017
@TheJJ TheJJ added just do it You can start working on this, there should be nothing left to discuss good first issue Suitable for newcomers labels Jan 19, 2017
@zuntrax zuntrax added this to the v0.4.0 milestone Jan 23, 2017
@TheJJ TheJJ mentioned this issue Jan 27, 2017
30 tasks
@TheJJ
Copy link
Member

TheJJ commented Feb 13, 2017

Done in #723

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: buildsystem Related to our cmake/python buildsystem good first issue Suitable for newcomers improvement Enhancement of an existing component just do it You can start working on this, there should be nothing left to discuss
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants