search

scriptotek / mc2skos

Command line script for converting Marc21 Classification and Authority records to SKOS/RDF
The Unlicense
21 stars 4 forks source link

Fix release with YAML and vocabularies.yml #39

Closed nichtich closed 6 years ago

nichtich commented 6 years ago

The current release 0.7.1 does not require pyaml and does not include vocabularies.yml so it throws an error if installed on a new system (at least this just happened to my colleague).

danmichaelo commented 6 years ago

Hm, that's strange. It should have been included in that release:

https://github.com/scriptotek/mc2skos/blob/v0.7.1/setup.py

But I can reproduce the problem on Python 2.7:

pyenv shell 2.7.13
virtualenv ENV --no-site-packages
pip install mc2skos

Seems like both pyaml and future are not included as requirements in the py2.7.egg for some reason.

danmichaelo commented 6 years ago

Ah, perhaps I should move some stuff from setup_requires to install_requires. I've never been very good at packaging python packages..

danmichaelo commented 6 years ago

Still a bit of a mystery to me why rdflib-jsonld fails when rdflib isn't included in setup_requires, but at least the current setup in 0.7.2 seems to work fine.