On Tue, 26 Apr 2011, Landry Breuil wrote:

> On Tue, Apr 26, 2011 at 11:46:34AM +0200, David Coppa wrote:
> > On Tue, 26 Apr 2011, David Coppa wrote:
> > 
> > because they're symbolic links to tab-new.png
> > 
> > All of the aforementioned icons don't exist in the standard gtk+2 
> > stock-icons.
> 
> pkg_locate stock_new-tab says that xfce|tango|gnome-icon-theme provides
> those... so if you really want to add a run_depends, make it conditional
> on those.

Like thunar port does?

Index: Makefile
===================================================================
RCS file: /cvs/ports/www/midori/Makefile,v
retrieving revision 1.32
diff -u -p -r1.32 Makefile
--- Makefile    21 Apr 2011 07:06:43 -0000      1.32
+++ Makefile    26 Apr 2011 10:58:08 -0000
@@ -5,7 +5,7 @@ COMMENT =       lightweight web browser
 XFCE_VERSION = 0.3.3
 XFCE_GOODIE =  midori
 
-REVISION =     0
+REVISION =     1
 
 MAINTAINER =   Landry Breuil <[email protected]>
 CATEGORIES =   www
@@ -26,6 +26,9 @@ LIB_DEPENDS = www/webkit>=1.1.15v0 \
                databases/sqlite3 \
                devel/libunique \
                devel/libsoup>=2.28.0
+
+RUN_DEPENDS =  x11/xfce4/xfce4-icon-theme \
+               tango-icon-theme-*|gnome-icon-theme-*:x11/tango-icon-theme
 
 # pick sqlite
 CONFIGURE_ENV+=        CPPFLAGS="-I${LOCALBASE}/include"

Reply via email to