mpdaf icon indicating copy to clipboard operation
mpdaf copied to clipboard

source tarball for building packages?

Open alexmyczko opened this issue 4 years ago • 15 comments

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/

alexmyczko avatar Feb 17 '21 17:02 alexmyczko

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

saimn avatar Feb 17 '21 17:02 saimn

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

alexmyczko avatar Feb 17 '21 18:02 alexmyczko

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

saimn avatar Feb 17 '21 18:02 saimn

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

alexmyczko avatar Feb 17 '21 18:02 alexmyczko

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

saimn avatar Feb 17 '21 19:02 saimn

root@sid:~# cython3 --version
Cython version 0.29.21

alexmyczko avatar Feb 18 '21 20:02 alexmyczko

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.

saimn avatar Feb 18 '21 21:02 saimn

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

alexmyczko avatar Feb 19 '21 06:02 alexmyczko

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.

saimn avatar Feb 19 '21 14:02 saimn

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 avatar Feb 19 '21 21:02 alexmyczko

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

alexmyczko avatar Feb 25 '21 07:02 alexmyczko

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

saimn avatar Feb 27 '21 21:02 saimn

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

alexmyczko avatar Feb 28 '21 13:02 alexmyczko

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

saimn avatar Feb 28 '21 17:02 saimn

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

alexmyczko avatar Feb 28 '21 17:02 alexmyczko