On Tue, May 12, 2015 at 06:09:28PM +0200, Jérémie Courrèges-Anglas wrote:
> Alexandr Shadchin <[email protected]> writes:
> 
> > On Sun, Mar 29, 2015 at 03:54:39PM +0500, Alexandr Shadchin wrote:
> >> Hi,
> >> 
> >> This diff updates djview4 to the latest release.
> >> Tested on amd64.
> >> 
> >> Comments ? OK ?
> >> 
> >
> > Update to 4.10.3.
> 
> Nits: you may want to set SHARED_ONLY to stop the build early on old
> vaxen (no need to process deps). Also you could remove the useless .la
> file.  It's a shame that the build is now mostly silent (there's too
> much black magic going on with Makefile.qmake).
> 
> Otherwise, djview4 and the firefox plugin seem to work fine here.
> 
> -- 
> jca | PGP : 0x1524E7EE / 5135 92C1 AD36 5293 2BDF  DDCC 0DFA 74AE 1524 E7EE

I was a bit hasty, and has already made a commit :-)

Changes:
* Add SHARED_ONLY
* Remove .la from package
* Verbose on

Do you know need bump or not?

-- 
Alexandr Shadchin

Index: Makefile
===================================================================
RCS file: /cvs/ports/graphics/djview4/Makefile,v
retrieving revision 1.23
diff -u -p -r1.23 Makefile
--- Makefile    12 May 2015 16:11:54 -0000      1.23
+++ Makefile    12 May 2015 16:28:56 -0000
@@ -1,5 +1,7 @@
 # $OpenBSD: Makefile,v 1.23 2015/05/12 16:11:54 shadchin Exp $
 
+SHARED_ONLY=   Yes
+
 COMMENT=       portable DjVu viewer and browser plugin
 
 DISTNAME=      djview-4.10.3
@@ -32,6 +34,8 @@ CONFIGURE_STYLE=gnu
 CONFIGURE_ARGS=        --with-tiff=${LOCALBASE}
 CONFIGURE_ENV+=        QTDIR="${MODQT4_QTDIR}" \
                CPPFLAGS="-I${X11BASE}/include"
+
+MAKE_ENV+=     V=1
 
 NO_TEST=       Yes
 
Index: pkg/PLIST
===================================================================
RCS file: /cvs/ports/graphics/djview4/pkg/PLIST,v
retrieving revision 1.7
diff -u -p -r1.7 PLIST
--- pkg/PLIST   12 May 2015 16:11:54 -0000      1.7
+++ pkg/PLIST   12 May 2015 16:28:56 -0000
@@ -2,12 +2,11 @@
 @bin bin/djview
 lib/mozilla/
 lib/mozilla/plugins/
-lib/mozilla/plugins/nsdejavu.la
+@comment lib/mozilla/plugins/nsdejavu.la
 lib/mozilla/plugins/nsdejavu.so
 @man man/man1/djview.1
 @man man/man1/nsdejavu.1
 share/applications/djvulibre-djview4.desktop
-share/djvu/
 share/djvu/djview4/
 share/djvu/djview4/djview_cs.qm
 share/djvu/djview4/djview_de.qm

Reply via email to