Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 10 Apr 2017 17:05:27 -0700
From:      Ngie Cooper <yaneurabeya@gmail.com>
To:        Ed Maste <emaste@freebsd.org>
Cc:        "src-committers@freebsd.org" <src-committers@freebsd.org>,  "svn-src-all@freebsd.org" <svn-src-all@freebsd.org>,  "svn-src-head@freebsd.org" <svn-src-head@freebsd.org>
Subject:   Re: svn commit: r316688 - head/share/man/man5
Message-ID:  <CAGHfRMA-p3cxAvrbzfRD5hHACv%2BGwJyRCR7=z71_Bp0Xoogu7Q@mail.gmail.com>
In-Reply-To: <CAPyFy2CWz_YNd--8j69MOs_AaOJ8V8qERftRSZ8N3grn3A5RqQ@mail.gmail.com>
References:  <201704102100.v3AL0lJX041545@repo.freebsd.org> <CAPyFy2CWz_YNd--8j69MOs_AaOJ8V8qERftRSZ8N3grn3A5RqQ@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
On Mon, Apr 10, 2017 at 2:13 PM, Ed Maste <emaste@freebsd.org> wrote:
...
> It looks like several of these $FreeBSD$ entries on the individual
> WITH_/WITHOUT_ entries ended up with bogus rev numbers on the previous
> src.conf.5 regen so there are a number of spurious entries in this
> diff, but I confirmed that the content changes are as desired.
>
> I'd like to change makeman to eliminate the copied $FreeBSD$ content
> from each WITH_/WITHOUT_ block, leaving one $FreeBSD$ for the
> generated file as a whole. My initial motivation was to allow
> src.conf.5 to be generated in a git working tree or other VCS that
> does not expand the $FreeBSD$ tags, but it also eliminates spurious
> diffs when regenerating on a stable branch and would have avoided the
> spurious changes here.
>
> That change is in Phabricator review D7997: https://reviews.freebsd.org/D7997.

The reason for the bogus $FreeBSD$ entries is actually because the
files themselves are fixed in time (due to svn copy/svn switch'es and
the like). Admittedly, this is a good case for getting rid of the RCS
metadata altogether.
Thanks!
-Ngie



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAGHfRMA-p3cxAvrbzfRD5hHACv%2BGwJyRCR7=z71_Bp0Xoogu7Q>