I'm not sure wether I'm happy with splitting xpdf at all (expect
people asking where pdftotext and friends has been gone after the
update), but anyway, here are some comments:

On Tue, Jan 15, 2008 at 10:24:29AM +0100, Landry Breuil wrote:
> Ah, yes, bernd@ sent me a diff a while ago about this one, with a
> @pkgpath marker to solve the update/conflicts. It also moves pdftops to
> -main package, i'm not sure if it's needed.. and -main doesn't
> run_depends on -utils.

The code for writing postscript is linked into both xpdf and pdftops, so
I don't think the latter is needed for the -main package.

> diff -u -p -r1.57 Makefile
> --- Makefile  9 Nov 2007 07:15:12 -0000       1.57
> +++ Makefile  23 Dec 2007 15:42:37 -0000
[...]
> -LIB_DEPENDS+=        t1.>=3::devel/t1lib
> -RUN_DEPENDS+=        ::print/ghostscript/gnu-fonts
> +LIB_DEPENDS-main=    t1.>=3::devel/t1lib ${LIB_DEPENDS}
> +LIB_DEPENDS-utils=
> +RUN_DEPENDS-utils=   ::print/ghostscript/gnu-fonts

AFAIK, xpdf itself also makes use of the ghostscript fonts.

> diff -N patches/patch-Makefile_in
> --- /dev/null 1 Jan 1970 00:00:00 -0000
> +++ patches/patch-Makefile_in 12 Nov 2007 18:48:43 -0000
> @@ -0,0 +1,17 @@
> +$OpenBSD$
> +--- Makefile.in.orig Mon Nov 12 19:48:32 2007
> ++++ Makefile.in      Mon Nov 12 19:48:32 2007
> +@@ -94,13 +94,6 @@ install: dummy
> +     $(INSTALL_DATA) $(srcdir)/doc/pdfimages.1 
> $(DESTDIR)@mandir@/man1/pdfimages.1
> +     -mkdir -p $(DESTDIR)@mandir@/man5
> +     $(INSTALL_DATA) $(srcdir)/doc/xpdfrc.5 $(DESTDIR)@mandir@/man5/xpdfrc.5
> +-    -mkdir -p $(DESTDIR)@sysconfdir@
> +-    @if test ! -f $(DESTDIR)@sysconfdir@/xpdfrc; then \
> +-            echo "$(INSTALL_DATA) $(srcdir)/doc/sample-xpdfrc 
> $(DESTDIR)@sysconfdir@/xpdfrc"; \
> +-            $(INSTALL_DATA) $(srcdir)/doc/sample-xpdfrc 
> $(DESTDIR)@sysconfdir@/xpdfrc; \
> +-    else \
> +-            echo "# not overwriting the existing 
> $(DESTDIR)@sysconfdir@/xpdfrc"; \
> +-    fi
> + 
> + clean:
> +     -cd goo; $(MAKE) clean

A patch just to shutup the update-plist warnings? If you want to
do this, you can just rm the offending file (xpdfrc) in post-install.

Ciao,
        Kili

Reply via email to