Re: Bug#320944: Things to improve on pts-tetex-cm-super
On 03.08.05 Mickael Profeta (mike@alezan.org) wrote:
Hi,
> >1. In postinst you have code to make the package installing on teTeX
> >1.0.7. sarge has 2.0.2 and etch will have 3.0.x. Hence I don't think
> >it does make sense to keep that code. The postrm script is
> >incompatible to the teTeX packages from woody anyway.
> >
> >
> That's true. When I start building the package Peter ask me to keep
> compatibility with woody. Yet, now that sarge is out, this may be
> useless.
>
...and is dangerous, as your postrm is incompatible to teTeX of
woody.
> > 1. Simply remove that /usr/bin/t1c2pfb and reupload with arch. all.
> > The postinst script will care and call /usr/bin/t1c2pfb.pl if
> > t1c2pfb is not present.
> >
> >If that is not possible due to perfomance issues (I assume t1c2pfb
> >and t1c2pfb.pl to be functionally equal.):
> >
> I just tried that tonight, and it is MUCH slower. It take about 5
> min on my machine (AMD 3200+) to install the fonts. I find it too
> long. I think it is preferable to split the package in two, and
> provide the C utilities. (your point 2)
>
OK, so please split the files off into a separate package!
I guess the ${shlibs:Depends} from debian/control can be dropped
after removing any arch dependent files.
In http://www.alezan.org/debian/pts-tetex-cm-super_0.3.3-7_i386.build
I see some lintian warnings:
W: pts-tetex-cm-super source: maintainer-script-lacks-debhelper-token debian/postrm
W: pts-tetex-cm-super source: maintainer-script-lacks-debhelper-token debian/postinst
E: pts-tetex-cm-super source: package-lacks-versioned-build-depends-on-debhelper 3
W: pts-tetex-cm-super: binary-without-manpage t1c2pfb.pl
E: pts-tetex-cm-super: shell-script-fails-syntax-check ./usr/bin/t1c2pfb.pl
I guess the first three can be fixed easily.
> Anyway, if you want to have a look, I have an unofficial, not fully
> tested version 0.3.3-7 here:
>
> www.alezan.org/debian
>
Well, that should be all. IMHO the next step could be to upload the
separated pfb2t1c.c and t1c2pfb.c, let that package conflict with
pts-tetex-cm-super <= 0.3.3-6 and wait until it made its way through
the new packages queue.
H.
--
We don't really understand it, so we'll give it to the programmers.
http://hilmarpreusse.forum-rheinland.de/
Reply to: