svn commit: r307446 - head/www/xpi-colorfultabs

[Emanuel Haupt]

Florian Smeets <flo at smeets.im> wrote:
> On 11/15/2012 11:22, Emanuel Haupt wrote:
> > Florian Smeets <flo at FreeBSD.org> wrote:
> >> Author: flo
> >> Date: Thu Nov 15 10:16:41 2012
> >> New Revision: 307446
> >> URL: http://svnweb.freebsd.org/changeset/ports/307446
> >>
> >> Log:
> >>   Update to 16.4
> >>   
> >>   Feature safe:	yes
> >>
> >>
> >> Modified: head/www/xpi-colorfultabs/pkg-plist
> >> ==============================================================================
> >> --- head/www/xpi-colorfultabs/pkg-plist	Thu Nov 15 10:01:42
> >> 2012	(r307445) +++ head/www/xpi-colorfultabs/pkg-plist
> >> Thu Nov 15 10:16:41 2012	(r307446) @@ -1,3 +1,5 @@
> >> +%%XPI_XPIDIR%%/chrome/.DS_Store
> >> +%%XPI_XPIDIR%%/chrome/content/.DS_Store
> > 
> > These two files are probably not needed.
> > 
> 
> Yeah, i know. I saw them but decided to just add them to the plist,
> rather than hacking up the Makefile with something to use find on the
> WRKSRC and delete them.
> 
> If anybody sees this as a problem, i can hack the Makefile to delete
> them prior to installation.

This (untested) patch will probably do the trick:

Index: Makefile
===================================================================
--- Makefile    (revision 307446)
+++ Makefile    (working copy)
@@ -13,5 +13,8 @@
 XPI_ID=                {0545b830-f0aa-4d7e-8820-50a4629a56fe}
 XPI_NUM=       1368
 
+pre-patch:
+       @find ${WRKSRC} -name .DS_Store -exec rm -fv {} \;
+
 .include "${.CURDIR}/../xpi-adblock/Makefile.xpi"
 .include <bsd.port.mk>
Index: pkg-plist
===================================================================
--- pkg-plist   (revision 307446)
+++ pkg-plist   (working copy)
@@ -1,5 +1,3 @@
-%%XPI_XPIDIR%%/chrome/.DS_Store
-%%XPI_XPIDIR%%/chrome/content/.DS_Store
 %%XPI_XPIDIR%%/chrome/content/bak.png
 %%XPI_XPIDIR%%/chrome/content/bkg.png
 %%XPI_XPIDIR%%/chrome/content/border.png