Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 21 Aug 2011 11:44:54 +0000
From:      Alexey Dokuchaev <danfe@FreeBSD.org>
To:        Chris Rees <crees@freebsd.org>
Cc:        Rene Ladan <rene@freebsd.org>, cvs-ports@freebsd.org, cvs-all@freebsd.org, ports-committers@freebsd.org
Subject:   Re: cvs commit: ports/www/chromium Makefile
Message-ID:  <20110821114454.GA77777@FreeBSD.org>
In-Reply-To: <CADLo838r7ksFKTjB24NBVTtnyDaf54jdz9aQ9Ggb22CtsvA9ng@mail.gmail.com>
References:  <201108211100.p7LB0cB7060742@repoman.freebsd.org> <CADLo838r7ksFKTjB24NBVTtnyDaf54jdz9aQ9Ggb22CtsvA9ng@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
On Sun, Aug 21, 2011 at 12:04:37PM +0100, Chris Rees wrote:
> On 21 August 2011 12:00, Rene Ladan <rene@freebsd.org> wrote:
> > rene š š š š2011-08-21 11:00:38 UTC
> >
> > šModified files:
> > š šwww/chromium š š š š Makefile
> > šLog:
> > šDo not hardcode the patch directory as "files", use ${FILESDIR} instead.
> 
> Shouldn't it be PATCHDIR?

Ideally, only one should stay, I don't really care which one (although
FILESDIR is maybe a bit better as it's more generic -- there might be not
only patches, but drop-in files, etc).

I wanted to look at merging FILESDIR/PATCHDIR logic in bpm several years
ago, but didn't go much further.

./danfe



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20110821114454.GA77777>