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
vim_configurable: change python version detection #8125
Conversation
A better solution than #7619 |
cc me |
Any input on this? |
[ ./python_framework.patch ]; | ||
|
||
configureFlags | ||
= [ "--enable-gui=${args.gui}" "--with-features=${args.features}" ]; | ||
|
||
nativeBuildInputs | ||
= [ ncurses pkgconfig gtk libX11 libXext libSM libXpm libXt libXaw libXau | ||
libXmu glib libICE ]; | ||
= if args.gui != "no" then [ ncurses pkgconfig gtk libX11 libXext libSM libXpm libXt libXaw libXau |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can take out ncurses pkgconfig
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
From both?
Complete rewrite, much simpler, no changes to X. The python3 specific stuff (python3Support, etc) was removed entirely, in favor of a simple if/else statement using |
Instead of explicitly stating "I want version 3" or "I want version 2" you now simply specify what the python argument will be, and vim_configurable will set up the flags for you. config.vim.python must be set, still.
Pinging @jagajaga. |
Hmm. What does this statement do? |
places a 3 if |
The
|
So to enable python 3 we need to override pkg with |
Yes. Changing the |
vim_configurable: change python version detection
Instead of explicitly stating "I want version 3" or "I want version 2"
you now simply specify what the
python
argument will be, andvim_configurable will set up the flags for you.
config.vim.python
mustbe set, still.
This is a complete rewrite of the PR I submitted in June 2015.
(Branch name is a bit of a misnomer, but that's the name of the package I use to test.)