[Pkg-jed-commit] r580 - jed/trunk/debian

[Rafael Laboissiere]

* Jörg Sommer <jo-guest at alioth.debian.org> [2007-02-25 23:41]:

> Author: jo-guest
> Date: 2007-02-25 23:41:07 +0100 (Sun, 25 Feb 2007)
> New Revision: 580
> 
> Modified:
>    jed/trunk/debian/rules
> Log:
> ??? debian/rules
>   · hevea referenced the subfiles with an absolute path, which is not valid
>     after installation. Therefore, call hevea from the info directory.
> 
>   · dh_installinfo checks also for the right permission of the directory and
>     maybe something more. Therefore use this instead of mkdir to create
>     /usr/share/info.
> 
> 
> Modified: jed/trunk/debian/rules
> ===================================================================
> --- jed/trunk/debian/rules	2007-02-25 22:38:39 UTC (rev 579)
> +++ jed/trunk/debian/rules	2007-02-25 22:41:07 UTC (rev 580)
> @@ -84,8 +84,8 @@
>  
>  	# generate info files from jed.tex
>  	rmdir $(jed-common)/usr/share/jed/info
> -	mkdir -p $(jed-common)/usr/share/info
> -	hevea -fix -info -o $(jed-common)/usr/share/info/jed.info \
> +	dh_installinfo -pjed-common     # create the directory /u/s/info
> +	cd $(jed-common)/usr/share/info/; hevea -fix -info -o jed.info \
>  	    $(jed-common)/usr/share/doc/jed-common/manual/jed.tex

If you do not give an argument and if there is no debian/jed-common.info
file, then the dh_installinfo command does not create the usr/share/info
directory in the package installation area.  So, the changes above do not
work.

Besides that, dh_installinfo will create snipets for the postinst and prerm.
The prerm snipet is okay, but the postinst snipet will become duplicated in
the final postinst script, besides being wrong (lacking --section and
--description options to install-info).  Instead of using dh_installinfo, we
should create usr/share/info by hand. The code in dh_installinfo does the
following (translating for Perl to pseudo-shell):

    install -d $tmp/usr/share/info
    cp <info-files> $tmp/usr/share/info
    chmod -R go=rX $tmp/usr/share/info/
    chmod -R u+rw $tmp/usr/share/info/

Notice that the chmod commands are run recursively, so they will also affect
the *.info* files installed, not only the directory.  I do noth tink that
the chmod commands are necessary in our case.  We should replace the
dh_installinfo call by a call to mkdir, as before.  We should also create a
jed-common.prerm script with a call to install-info --remove.

-- 
Rafael