svn commit: r302141 - in head: audio/gnump3d audio/gnump3d/files audio/icecast2 audio/icecast2/files audio/ices0 audio/ices0/files audio/liquidsoap audio/liquidsoap/files audio/squeezeboxserver aud...

[Chris Rees]

On 21 Aug 2012 21:00, "Doug Barton" <dougb at freebsd.org> wrote:
>
> On 8/10/2012 5:51 AM, Chris Rees wrote:
> > On 10 August 2012 13:09, Pietro Cerutti <gahr at freebsd.org> wrote:
> >> Mind to fix this please?
> >>
> >>
http://pointyhat.freebsd.org/errorlogs/amd64-errorlogs/a.9.20120809134611/gnu-radius-1.6.1_4.log
> >>
> >> Thanks,
> >
> > Correct fix is at [1].
>
> When I saw this message originally it never occurred to me that you
> would go to all the trouble of producing a 1-line diff, put it up on a
> web page, and post it to the mailing list .... and then not just punch
> it in.
>
> Can you please explain why you would do this, and how this could
> possibly be beneficial to the project?
>

It hadn't been approved by the maintainer.  I would have linked to the
commit message had I committed it.

Perhaps it's not a big deal... but I always prefer to be the committer who
fixed my pointyhats rather than having the appearance of having to be
tidied up after.

Chris