Date: Fri, 19 Jul 2013 13:01:50 -0400 From: Adam Weinberger <adamw@FreeBSD.org> To: Alexey Dokuchaev <danfe@FreeBSD.org> Cc: svn-ports-head@freebsd.org, Johan van Selst <johans@FreeBSD.org>, svn-ports-all@freebsd.org, ports-committers@freebsd.org Subject: Re: svn commit: r323298 - head/games/ltris Message-ID: <20130719170150.GA85858@apnoea.adamw.org> In-Reply-To: <20130719143745.GA81905@FreeBSD.org> References: <201307191429.r6JETuEb004689@svn.freebsd.org> <20130719143745.GA81905@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
>> (2013/07/19 @ 1037 EST): Alexey Dokuchaev said, in 1.1K: <<
> > @${REINPLACE_CMD} -e 's|/icons|/pixmaps|g' ${WRKSRC}/Makefile.in
> > @${REINPLACE_CMD} -e 's|= @mixer_flag@|= @INTLLIBS@ @mixer_flag@|g ; \
> > s|--mode=666||g' ${WRKSRC}/src/Makefile.in
> > + @${REINPLACE_CMD} -e 's/^inline //' \
> > + ${WRKSRC}/src/sdl.h ${WRKSRC}/src/tools.h
>
> I might be nit picking, but it's generally advised and preferred to use the
> same separators for regex' that were previously used (that is, bars instead
> of slashes here) in the Makefile, for consistency's sake.
>> end of "Re: svn commit: r323298 - head/games/ltris" from Alexey Dokuchaev <<
Don't be That Guy. When you pick every style nit, it detracts from the
importance of your substance nits.
# Adam
--
Adam Weinberger
adamw@adamw.org
http://www.adamw.org
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20130719170150.GA85858>