search

musevlt / mpdaf

MUSE Python Data Analysis Framework
BSD 3-Clause "New" or "Revised" License
15 stars 4 forks source link

source tarball for building packages? #18

Open alexmyczko opened 3 years ago

alexmyczko commented 3 years ago 
Make sure you're either building from a fully intact git repository or PyPI tarballs. Most other sources (such as GitHub's tarballs, a git checkout without the .git folder) don't contain the necessary metadata and will not work.

I'm aware of your message, but how hard would it be to create proper source tarballs?

full logs:

https://sid.ethz.ch/debian/mpdaf/

saimn commented 3 years ago

What do you mean by proper source tarballs ? There is a source archive (Python sdist) on PyPI (https://pypi.org/project/mpdaf/#files).

alexmyczko commented 3 years ago

true will try that 60 mb source tarball. sorry for the noise

saimn commented 3 years ago

Yes it's quite big, because it contains some test files.

alexmyczko commented 3 years ago

looks much better, if you want to hint me why it fails? https://sid.ethz.ch/debian/mpdaf/mpdaf_3.5-1_amd64.build

saimn commented 3 years ago

It could be the version of cython, which one are you using ?

alexmyczko commented 3 years ago 
root@sid:~# cython3 --version
Cython version 0.29.21
saimn commented 3 years ago

Ok, I think I know: the lib/mpdaf/obj/merging.c was generated with an older version of Cython, which doesn't work with Python 3.9. Could you try to delete this file? It would then be generated again with your recent Cython.

alexmyczko commented 3 years ago

updated https://sid.ethz.ch/debian/mpdaf/mpdaf_3.5-1_amd64.build

saimn commented 3 years ago

Much better ! For the test failure, it seems that __version__ is None, which could happen if you didn't installed the package ? Not sure what is the process for Debian packaging. The version is determined with pkg_resources from the metadata created in site-packages when installing the package.

alexmyczko commented 3 years ago

Ah, so I disabled the tests, and we've got Debian packages, Ubuntu, Devuan, Raspberry or any deb based distro can have that packages. Ok they need some more improvement, but other than that, an easy way to install/remove/upgrade for mpdaf.

alexmyczko commented 3 years ago

https://mentors.debian.net/package/mpdaf/ looks much better, care to fix those manpage typos listed there?

saimn commented 3 years ago

@alexmyczko - Sure, but it's not clear to me what needs to be fixed.

alexmyczko commented 3 years ago

this part:

I: typo-in-manual-page
usr/share/man/man1/mpdaf.1.gz "Allow to" "Allow one to"
usr/share/man/man1/mpdaf.1.gz "Allow to" "Allow one to"
usr/share/man/man1/mpdaf.1.gz "Allow to" "Allow one to"
usr/share/man/man1/mpdaf.1.gz "Allow to" "Allow one to"
usr/share/man/man1/mpdaf.1.gz "Allow to" "Allow one to"
usr/share/man/man1/mpdaf.1.gz "Allow to" "Allow one to"
usr/share/man/man1/mpdaf.1.gz "Allow to" "Allow one to"
usr/share/man/man1/mpdaf.1.gz "Allow to" "Allow one to"
usr/share/man/man1/mpdaf.1.gz "Allow to" "Allow one to"
usr/share/man/man1/mpdaf.1.gz "Allow to" "Allow one to"
usr/share/man/man1/mpdaf.1.gz "Allow to" "Allow one to"
usr/share/man/man1/mpdaf.1.gz "Allow to" "Allow one to"
usr/share/man/man1/mpdaf.1.gz "Allow to" "Allow one to"
usr/share/man/man1/mpdaf.1.gz "Allow to" "Allow one to"
usr/share/man/man1/mpdaf.1.gz "Allow to" "Allow one to"
usr/share/man/man1/mpdaf.1.gz "Allow to" "Allow one to"
usr/share/man/man1/mpdaf.1.gz "Allow to" "Allow one to"
usr/share/man/man1/mpdaf.1.gz "Allow to" "Allow one to"
usr/share/man/man1/mpdaf.1.gz "Allow to" "Allow one to"
usr/share/man/man1/mpdaf.1.gz "Allow to" "Allow one to"
usr/share/man/man1/mpdaf.1.gz "Allow to" "Allow one to"
usr/share/man/man1/mpdaf.1.gz "Allow to" "Allow one to"
usr/share/man/man1/mpdaf.1.gz "Allow to" "Allow one to"
usr/share/man/man1/mpdaf.1.gz "Allow to" "Allow one to"
usr/share/man/man1/mpdaf.1.gz "Allow to" "Allow one to"
usr/share/man/man1/mpdaf.1.gz "allow to" "allow one to"
usr/share/man/man1/mpdaf.1.gz "allow to" "allow one to"
usr/share/man/man1/mpdaf.1.gz "allow to" "allow one to"
usr/share/man/man1/mpdaf.1.gz "allow to" "allow one to"
usr/share/man/man1/mpdaf.1.gz "allows to" "allows one to"
usr/share/man/man1/mpdaf.1.gz "allows to" "allows one to"
usr/share/man/man1/mpdaf.1.gz "allows to" "allows one to"
usr/share/man/man1/mpdaf.1.gz "allows to" "allows one to"
usr/share/man/man1/mpdaf.1.gz "an other" "another"
usr/share/man/man1/mpdaf.1.gz "an other" "another"
usr/share/man/man1/mpdaf.1.gz "an other" "another"
usr/share/man/man1/mpdaf.1.gz "an other" "another"
usr/share/man/man1/mpdaf.1.gz "an other" "another"
usr/share/man/man1/mpdaf.1.gz "an other" "another"
usr/share/man/man1/mpdaf.1.gz "an other" "another"
usr/share/man/man1/mpdaf.1.gz Informations Information
usr/share/man/man1/mpdaf.1.gz Udpate Update
usr/share/man/man1/mpdaf.1.gz accomodate accommodate
usr/share/man/man1/mpdaf.1.gz coresponding corresponding
usr/share/man/man1/mpdaf.1.gz coresponding corresponding
usr/share/man/man1/mpdaf.1.gz corresponing corresponding
usr/share/man/man1/mpdaf.1.gz corresponing corresponding
usr/share/man/man1/mpdaf.1.gz differents different
usr/share/man/man1/mpdaf.1.gz independantly independently
usr/share/man/man1/mpdaf.1.gz informations information
usr/share/man/man1/mpdaf.1.gz informations information
usr/share/man/man1/mpdaf.1.gz informations information
usr/share/man/man1/mpdaf.1.gz informations information
usr/share/man/man1/mpdaf.1.gz instanciate instantiate
usr/share/man/man1/mpdaf.1.gz ot to
usr/share/man/man1/mpdaf.1.gz paramater parameter
usr/share/man/man1/mpdaf.1.gz paramters parameters
usr/share/man/man1/mpdaf.1.gz ploting plotting
usr/share/man/man1/mpdaf.1.gz prefered preferred
usr/share/man/man1/mpdaf.1.gz prefered preferred
usr/share/man/man1/mpdaf.1.gz substraction subtraction
saimn commented 3 years ago

Ok, done. So you are building the whole documentation as a manpage ?

alexmyczko commented 3 years ago

Well for the moment yes as manpages are a should exist for Debian but I should probably also build some other formats...