Date: Wed, 10 Feb 2010 09:24:07 +0000 From: Alexey Dokuchaev <danfe@FreeBSD.org> To: Dmitry Marakasov <amdmi3@FreeBSD.org> Cc: cvs-ports@FreeBSD.org, cvs-all@FreeBSD.org, ports-committers@FreeBSD.org Subject: Re: cvs commit: ports/audio Makefile ports/audio/deadbeef Makefile distinfo pkg-descr pkg-plist ports/audio/deadbeef/files patch-Makefile.am patch-Makefile.in patch-configure patch-configure.ac Message-ID: <20100210092407.GA30717@FreeBSD.org> In-Reply-To: <201002092323.o19NNRvB090976@repoman.freebsd.org> References: <201002092323.o19NNRvB090976@repoman.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Tue, Feb 09, 2010 at 11:23:27PM +0000, Dmitry Marakasov wrote: > amdmi3 2010-02-09 23:23:27 UTC > > FreeBSD ports repository > > Modified files: > audio Makefile > Added files: > audio/deadbeef Makefile distinfo pkg-descr pkg-plist > audio/deadbeef/files patch-Makefile.am patch-Makefile.in > patch-configure patch-configure.ac > Log: > DeaDBeeF (as in 0xDEADBEEF) is an audio player for GNU/Linux systems with > X11 (though now it also runs in plain console without X, in FreeBSD, > and in OpenSolaris). > > PR: 143670 > Submitted by: zloidemon <g.veniamin@googlemail.com> ^^^^^^^^^ Fully qualified person id (full name) should be used when available (Whom: line of Makefile header). > > Revision Changes Path > 1.1092 +1 -0 ports/audio/Makefile > 1.1 +35 -0 ports/audio/deadbeef/Makefile (new) COMMENT= DeaDBeeF is an audio player for GNU/Linux systems with X11 ^^^^^^^^^^^^ Mentioning program name in COMMENT is redundant; simple "An audio player..." would be enough. "GNU/Linux" also looks strange on BSD system. "Unix[-like]" would be more appropriate. > 1.1 +13 -0 ports/audio/deadbeef/files/patch-Makefile.am (new) > 1.1 +21 -0 ports/audio/deadbeef/files/patch-Makefile.in (new) Weird. Can you explain why both of these patches are required, since .in is normally generated from .am (that is, patching .in is enough unless the port needs more of autotools magic than simple "./configure")? > 1.1 +57 -0 ports/audio/deadbeef/files/patch-configure (new) > 1.1 +34 -0 ports/audio/deadbeef/files/patch-configure.ac (new) Same here (if no autoconf required, patching .ac is useless). ./danfe
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20100210092407.GA30717>