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

Re: new lmodern package



Dear Florent!

Good to hear that you are back!

On Fre, 10 Feb 2006, Florent Rougon wrote:
> I have a few remarks and questions on the new package:

Good, you are welcome:

>   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".
> `----

This I definitely do not udnerstand. The dh_clean does nothing more in
our case than remove the debian/lmodern directory and the generated
debhelper scripts, why should this force the use of Build-Depends.

Maybe I don't understand the sentence above:
  However many packages need Build-Dependencies for the clean target
  (dh_clean), ...
Does this mean: 
  If you call dh_clean, you have to use Build-Depends
I can't believe this, and also lintian complains about the
Build-Depends.

More explanations please ...

>   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?

Ahh, good point, left over from when I first adapted the package and
there was a stupid error in the 0.9 version of the debhelper scripts.,
so I built my own 0.9.1 version.

Yes, it should be >= 0.10

>   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.

Please see the discussion on the debian-tetex-main ML. We include it
now only in the source package. For etch+1 we will remove the
compatibility links, but install this script in doc for users
convenience.

>   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.

Defnitely, I haven't checked the defoma stuff at all, the list is AFAIR
automatically generated.

In fact I have *NO* experience with defoma etc, so if you or someone
wants to improve this, you are welcome.

>   5. Why do you use such stuff as 'if [ "X$newloc" = "X" ]'?
>      AFAICT, this X thingie is useless nowadays.

habit? bad habit? I don't know...

>   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?

dh_installtex will be included in tex-common only on 0.16, so now I
cannot use it.

But yes, you are right, I want to switch to dh_installtex as soon as it
is released with tex-common 0.16.

>   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?

Misunderstanding of Map vs MixedMap. 

MixedMap only tells you that there are pk sources and type1 sources.

Since we have
	dvipsPreferOutline true
in updmap.cfg normally the type1 sources are used. Furthermore, the maps
in these config files are *NOT* activated.

If we would write Map instead of MixedMap, nobody would ever again be
able to use the cmr pk fonts, even if he tells dvips that he does not
wan to have type1 fonts!


I think the only open problem is the Build-Depends stuff, I don't
understand. If we stay with Build-Depend-Indep, I suggest we leave the
pacakge as is, and for the next version I switch to dhinstalltex and the
correct tex-common >= 0.16.
(the >> 0.9 is correct anyway for now, say me as mathematician ;-))

Any other urgend matters?

Florent, do you want to have write access to the lmodern svn repo, i.e
pkg-texlive svn?

Best wishes

Norbert

-------------------------------------------------------------------------------
Dr. Norbert Preining <preining AT logic DOT at>             Università di Siena
gpg DSA: 0x09C5B094      fp: 14DF 2E6C 0307 BE6D AD76  A9C0 D2BF 4AA3 09C5 B094
-------------------------------------------------------------------------------
AHENNY (adj.)
The way people stand when examining other people's bookshelves.
			--- Douglas Adams, The Meaning of Liff



Reply to: