svn commit: r472498 - head/archivers/py-borgbackup

[Mathieu Arnold]

On Mon, Jun 18, 2018 at 10:26:42PM +0200, Carlos Jacobo Puga Medina wrote:
> > Enviar: lunes 18 de junio de 2018 a las 22:22
> > De: "Mathieu Arnold" <mat at FreeBSD.org>
> > Para: "Carlos J. Puga Medina" <cpm at FreeBSD.org>
> > CC: 
> > Asunto: Re: svn commit: r472498 - head/archivers/py-borgbackup
> >
> > On Fri, Jun 15, 2018 at 10:06:39PM +0000, Carlos J. Puga Medina wrote:
> > >   - Enable FUSE option by default
> > 
> > Why?
> >
> 
> The port maintainer requested this change. I'm missing something?

Commit messages are the place where you explain WHY you are doing
things.  Rewriting the diff in plain English only adds noise and should
be avoided.

The maintainer is the main caretaker of the port yes, but the maintainer
is not the only caretaker.  The port is maintained by the community.
And as such, the community needs to be able to remember, as a collective
why changes are made.  To be able to achieve this, writing WHY changes
are made in the commit messages is paramount.

Now, nobody has any idea why the option changed to default on, maybe it
is because the port is mostly useless without it, or because user may
need it, or because the maintainer decided he wanted to change something
else in the Makefile so that it looked more real.

-- 
Mathieu Arnold
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 949 bytes
Desc: not available
URL: <http://lists.freebsd.org/pipermail/svn-ports-all/attachments/20180619/73b20b5c/attachment.sig>