search

melmothx / amusewiki

Text::Amuse-based publishing platform
https://amusewiki.org
Other
169 stars 26 forks source link

Which fonts do I need to run the tests? #425

Closed guest20 closed 1 year ago

guest20 commented 1 year ago

I'm not exactly sure how these packages and the tests fit together, but it seem like t/00-prepare-tests.t doesn't pass without fonts-linuxlibertine installed.

https://github.com/melmothx/amusewiki/blob/6e2d073f669e32a9af57ff0c784d8416f4900ed6/t/00-prepare-tests.t#L132 and if this font isn't installed, this test fails: https://github.com/melmothx/amusewiki/blob/6e2d073f669e32a9af57ff0c784d8416f4900ed6/t/00-prepare-tests.t#L184

https://amusewiki.org/library/custom-fonts

it seems like the tests want fonts-dejavu-extra fonts-dejavu-core fonts-linuxlibertine, so maybe they should also be in Build-Depends-Indep?

melmothx commented 1 year ago

guest20 @.***> writes:

I'm not exactly sure how these packages and the tests fit together, but it seem like t/00-prepare-tests.t doesn't pass without fonts-linuxlibertine installed.

https://github.com/melmothx/amusewiki/blob/6e2d073f669e32a9af57ff0c784d8416f4900ed6/t/00-prepare-tests.t#L132 and if this font isn't installed, this test fails: https://github.com/melmothx/amusewiki/blob/6e2d073f669e32a9af57ff0c784d8416f4900ed6/t/00-prepare-tests.t#L184

https://amusewiki.org/library/custom-fonts

it seems like the tests want fonts-dejavu-extra fonts-dejavu-core fonts-linuxlibertine, so maybe they should also be in Build-Depends-Indep?

Hm, I'm pretty sure that those are supposed to be pulled from libtext-amuse-compile-perl, which has all the needed fonts listed in the deps. Do you have libtext-amuse-compile-perl installed?

-- Marco

guest20 commented 1 year ago

I'm installing perl modules from cpanfile.snapshot and then manually adding packages to the list as tests fail more and more. I got my original list by just skipping the -perl entries in the debian/control, but obviously that misses second level dependencies, like the things libtext-amuse-compile-perl depends upon.

This might not be the ideal approach.

melmothx commented 1 year ago

guest20 @.***> writes:

I'm installing perl modules from cpanfile.snapshot and then manually adding packages to the list as tests fail more and more. I got my original list by just skipping the -perl entries in the debian/control, but obviously that misses second level dependencies, like the things libtext-amuse-compile-perl depends upon.

This might not be the ideal approach.

The matter is a bit complicated by the fact that there's the debian-way to install it and the cpan way. The script/install_fonts.sh was supposed to take care of the fonts, but from a quick look it seems outdated, so I think this matter is still worth digging and to be amended.

Thanks for the report, I'll get this straight shortly.

-- Marco