Re: new lmodern package
Hi everyone,
First, thanks for adopting lmodern and working on it while I wasn't
available.
I have a few remarks and questions on the new package:
1. IMHO, the Build-Depends -> Build-Depends-Indep change is wrong:
,----[ http://lists.debian.org/debian-devel/2003/debian-devel-200310/msg01460.html ]
| * If your package only builds binary-all packages, you can choose
| whether to use Build-Depends-Indep or Build-Depends. However many
| packages need Build-Dependencies for the clean target (dh_clean), if
| this applies you must use "Build-Depends".
`----
2. I'm not sure why you need tex-common (>> 0.9) in
Build-Depends-Indep. I suppose you need
the-first-version-where-dh_installtexfonts is usable, which would
imply a >= relation, wouldn't it?
3. What is the point of create-compat-links.sh if you don't install
it? I assume it is not for future use by the Debian package
since it touches /usr/local.
4. I tend to disagree with some of the settings you chose for new
fonts in lmodern.defoma-hints (for instance, the GeneralFamily set
to Roman for typewriter fonts, but I didn't check everything---the
defoma documentation says we can use our preferred value when
the predefined values don't fit, therefore I had chosen values like
Typewriter-SmallCaps).
[ a while later... ]
Ah, but I see from TODO.Debian that this is work in progress.
5. Why do you use such stuff as 'if [ "X$newloc" = "X" ]'?
AFAICT, this X thingie is useless nowadays.
6. In debian/rules, I see you are using dh_installtexfonts. OK.
However, I seem to recall having read here that dh_installtex
supersedes dh_installtexfonts. Am I mistaken?
7. Are you sure that the .cfg files for replacement maps should use
MixedMap instead of map ? I would say that, if you are using LM as
a replacement font, you want LM to be used, and therefore the
Type 1 fonts, even if CM, PL, etc. exist in PK format. Therefore, I
would use Map. What do you think?
When we agree, I can do the upload. Thank you.
--
Florent
Reply to: