Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 28 Mar 2018 18:58:41 +0000
From:      Alexey Dokuchaev <danfe@FreeBSD.org>
To:        Mathieu Arnold <mat@FreeBSD.org>
Cc:        ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org
Subject:   Re: svn commit: r465830 - head/astro/stellarium
Message-ID:  <20180328185841.GA93245@FreeBSD.org>
In-Reply-To: <20180328180646.aqml7rdmr773xiot@ogg.in.absolight.net>
References:  <201803281720.w2SHKmHU086890@repo.freebsd.org> <20180328172437.GA20977@FreeBSD.org> <20180328180646.aqml7rdmr773xiot@ogg.in.absolight.net>

next in thread | previous in thread | raw e-mail | index | archive | help
On Wed, Mar 28, 2018 at 08:06:46PM +0200, Mathieu Arnold wrote:
> On Wed, Mar 28, 2018 at 05:24:37PM +0000, Alexey Dokuchaev wrote:
> > On Wed, Mar 28, 2018 at 05:20:48PM +0000, Alexey Dokuchaev wrote:
> > > New Revision: 465830
> > > URL: https://svnweb.freebsd.org/changeset/ports/465830
> > > 
> > > Log:
> > >   - Update `astro/stellarium' to version 0.18.0
> > >   - TIMESTAMP (Stellarium-stellarium-v0.17.0_GH0.tar.gz) = 1521967282
> >                                         ^^^^^^
> > Ah, sorry, I've fucked up, it should read 0.18.0.  TIMESTAMP logic should
> > really be fixed so we won't have to reiterate it in the commit logs. :-(
> 
> It was decided that the TIMESTAMP logic would work as it currently does,
> so these lines you add to your commit messages are really not needed.

Current logic is both confusing and redundant, as it adds yet another
timestamp to already existing distfile(s) timestamp and commit timestamp.

Per my understanding, it was only put in place because of Git, which does
not support keyword expansion.  Personally I don't understand why would we
want to support a crippled VCS, but before I figure a way out of this, I
feel that proper TIMESTAMP metainfo should either go to commit message, or
be recorded elsewhere.

./danfe



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20180328185841.GA93245>