svn commit: r307782 - head/audio/asunder

[Erwin Lansing]

On Mon, Nov 26, 2012 at 11:57:37AM -0500, Eitan Adler wrote:
> On 26 November 2012 11:11, Alexey Dokuchaev <danfe at freebsd.org> wrote:
> > On Mon, Nov 26, 2012 at 04:57:23PM +0100, Guido Falsi wrote:
> >> On 11/26/12 14:56, Alexey Dokuchaev wrote:
> >> >We have standard knob for this one (APE_DESC).
> >> >
> >> >>  PORTDOCS= AUTHORS COPYING ChangeLog INSTALL NEWS README TODO
> >> >
> >> >Normally we do not install COPYING as part of the docs, LICENSE should be
> >> >set instead.  If software expects to find this file (e.g. for displaying in
> >> >the about dialog), it should be redirected to share/licenses/${PKGNAME}
> >> >instead (see `graphics/luminance' port for an example).
> >>
> >> Thanks for these, I'll fix them and commit after maintainer approval.
> >
> > Technically speaking, you don't have to have maintainer's approval for these
> > things, as they don't affect functionality, just make the port more adhering
> > to our standards (and common sense).
> 
> It is good to ping the maintainer though unless there is specific reason not to.
> - Since they signed up to maintain the port its nice not to change
> things without asking
> - It teaches the maintainers about our standards :)
> 
Actually, if you strictly follow our rules, the maintainer has the final
say and you have to ask beforehand, with the usual timeouts.  For
trivial changes, e.g., updating CONFLICTS for a slave port, you may not
want to wait for that and commit right away, but the least you can do is
to send a mail afterwards to the maintainer as a courtesy so they don't
have to find out by having conflicts in the their local repository if
their working on another update or similar.

Being polite really doesn't cost much.

Erwin

-- 
Erwin Lansing                                    http://droso.dk
erwin at FreeBSD.org                        http:// www.FreeBSD.org