-
Notifications
You must be signed in to change notification settings - Fork 6
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
Missing installation requirements #63
Comments
There's no C code in keg. It could be a module from the dependency chain that pip tries to install. |
regardless, you get an error if you follow the docs unless you already have gcc installed. |
Sorry for leaving this for so long. Yes, it appears something in the dependency chain ( |
I found a few others as well that were needed while playing with keg zypper in jing |
Both are used by kiwi not keg directly, so you would expect them to be installed automatically in the tox env. Maybe not jing, this might be optional. |
I ran into needs them before ever using kiwi. I did not think kiwi was a hard requriment if you are only building the kiwi files and not using kiwi to build the image. |
Keg uses bits of kiwi to validate the generated description, so it is a hard requirement. |
hmm, mine runs without kiwi installed.
|
It should be installed under |
I guess this can be closed, or is there anything that needs fixing from your POV @KeithMnemonic ? |
I do not think you should close it until all of these extra bits are documented. I will try on a fresh system and then list anything needed that was not found in the tox env. |
Using the current readme https://github.com/SUSE-Enceladus/keg#readme it does not provide correct requirements to develop with keg. The readme only mentions "Make sure the Python development headers are installed (e.g. python36-devel)" The understood expectation of a README is to provide correct steps to begin using/developing the project. I have fresh system with only python3-devel installed simply running tox -devel , I get this error `ERROR: Command errored out with exit status 1: /home/vagrant/repos/github/keg/.tox/3/bin/python3 -u -c 'import io, os, sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-install-r8q93rz3/pyxattr_b09b4c8f9c4248ec86e00addac8efd92/setup.py'"'"'; file='"'"'/tmp/pip-install-r8q93rz3/pyxattr_b09b4c8f9c4248ec86e00addac8efd92/setup.py'"'"';f = getattr(tokenize, '"'"'open'"'"', open)(file) if os.path.exists(file) else io.StringIO('"'"'from setuptools import setup; setup()'"'"');code = f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, file, '"'"'exec'"'"'))' install --record /tmp/pip-record-b7f6ipis/install-record.txt --single-version-externally-managed --compile --install-headers /home/vagrant/repos/github/keg/.tox/3/include/site/python3.6/pyxattr Check the logs for full command output. ERROR: could not install deps [-r.virtualenv.dev-requirements.txt]; v = InvocationError('/home/vagrant/repos/github/keg/.tox/3/bin/pip install -r.virtualenv.dev-requirements.txt (see /home/vagrant/repos/github/keg/.tox/3/log/devel-1.log)', 1) If I install gcc, the tox envs runs ok. If you continue follow the steps in the readme, everything runs as expected. The next set of missing requirements is found when you go to build the docs as listed in the Makefile. make latexpdf : make[1]: latexmk: Command not found Also some of the build targets in the makefile show "not registered" errors make doctest Sphinx error: where is [doctest, coverage, spell] I logged a new issue on this. To Summarize, the current README should list the following as additional requirements. gcc Also if developing on python3.8, you would not install python36-devel. I think the "alternative" python3-devel enables zypper to install the correct devel package for the installed python version. If only python3.6 is supported then this should be clearly stated in the README |
The keg documentation only mentions needing kiwi installed but it also requires gcc and the python3X-devel package. Without these you get the following errors when installing from pip
The text was updated successfully, but these errors were encountered: