On Tue, Apr 26, 2011 at 01:02:23PM +0200, David Coppa wrote:
> 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

I was more thinking about
xfce4-icon-theme-*|tango-icon-theme-*|gnome-icon-theme-*:x11/tango-icon-theme

Thunar is slightly different here.
Landry

Reply via email to