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

Update PKGBUILD #2067

Merged
merged 1 commit into from
Sep 6, 2018
Merged

Update PKGBUILD #2067

merged 1 commit into from
Sep 6, 2018

Conversation

sevu
Copy link
Contributor

@sevu sevu commented Aug 7, 2018

updating to the AUR version.

@eamanu you have a packaging related PR, would you have any changes to propose here once #2047 is merged? Most python packages don't specify the directory for site-packages in the PKGBUILD.

Switch to python 3.7, and apparently the pkgver() calculation wasn't added here yet.
@centaurialpha
Copy link
Member

Sorry, I'm a little lost, it must be because I do not understand very well how AUR works or things like that. Could you explain a little more what you want to say?

@sevu
Copy link
Contributor Author

sevu commented Aug 7, 2018

Python stuff can mostly be installed with such a command, and depend, for the installation only, on python-setuptools.
python3 setup.py install --root="/" --optimize=1

I don't know how to set that up and instead copied the files with cp
The PR looks like it has sth. to do with the usual way to install python things.

@sevu
Copy link
Contributor Author

sevu commented Sep 5, 2018

Feel free to merge, it's not dependent on the other one, this was just room for further improvement.

As for the pkgver() function, packages with the -git suffix build usually a new version than mentioned in the pkgbuild, and this one takes the last commit date as version.

@centaurialpha centaurialpha merged commit ac69541 into ninja-ide:master Sep 6, 2018
@centaurialpha
Copy link
Member

Thanks! @sevu

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants