Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 15 Jun 2007 18:39:58 +0000
From:      Alexey Dokuchaev <danfe@FreeBSD.org>
To:        infofarmer@FreeBSD.org
Cc:        cvs-ports@freebsd.org, cvs-all@freebsd.org, ports-committers@freebsd.org
Subject:   Re: cvs commit: ports/graphics/sng Makefile pkg-descr
Message-ID:  <20070615183958.GA1681@FreeBSD.org>
In-Reply-To: <cb5206420706151130v7a5e687g31cfeac49f2ca1ab@mail.gmail.com>
References:  <200706151800.l5FI0BZD087745@repoman.freebsd.org> <20070615180818.EEBAE16A484@hub.freebsd.org> <cb5206420706151130v7a5e687g31cfeac49f2ca1ab@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
On Fri, Jun 15, 2007 at 10:30:34PM +0400, Andrew Pantyukhin wrote:
> On 6/15/07, Alexey Dokuchaev <danfe@freebsd.org> wrote:
> >danfe       2007-06-15 18:00:11 UTC
> >
> >  FreeBSD ports repository
> >
> >  Modified files:
> >    graphics/sng         Makefile pkg-descr
> >  Log:
> >  - Fix the build with recent GCC [1]
> >  - Correctly assign RUN_DEPENDS
> >  - Minor indentation nit in Makefile
> >  - Kill EOL space in pkg-descr
> >
> >  Reported by:    pointyhat (logs) [1]
> >
> >  Revision  Changes    Path
> >  1.13      +3 -2      ports/graphics/sng/Makefile
> >  1.2       +1 -1      ports/graphics/sng/pkg-descr
> >_______________________________________________
> > pre-patch:
> >        @${REINPLACE_CMD} -e 's|-static||g' ${WRKSRC}/Makefile.in
> >+       @${REINPLACE_CMD} -e '33,34s|^static ||' ${WRKSRC}/sngd.c
> 
> I'd warn you against these numbered sed substitutions.
> For one, they're hard to maintain. They also can cause
> silent damage to sources which is very hard to track.

Well, since (luckily for us!) our REINPLACE_CMD implementation leaves
.bak-files, it is really not that hard to run diff(1) over them.

> 
> In this very case, it use this:
> '/^static png_/s|^static ||'

I'd buy your reasoning here.  It's just that I love sed(1)+conciseness too
freaking much :-) and want to avoid adding trivial patch files.

I'll try to think more about my regexps next time, thanks!

./danfe



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