[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

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: