On Wed, May 30, 2012 at 08:07:38PM -0700, David O'Brien wrote: > On Wed, May 30, 2012 at 07:34:33AM +0400, Sergey Kandaurov wrote: > > On 29 May 2012 23:49, David E. O'Brien <obr...@freebsd.org> wrote: > ... > > > Modified: head/contrib/groff/tmac/doc-common > ... > > > +.ds doc-operating-system-FreeBSD-8.3 � � 8.3 > > > +.ds doc-operating-system-FreeBSD-9.1 � � 9.1 > > > +.ds doc-operating-system-FreeBSD-10.0 � �10.0 > > This should be imported from savannah.gnu.org repository, > > and not changed directly. > > Unfortunately we cannot do that per r217595: > > This is a direct commit to contrib/ as we will no longer import > any newer groff snapshots, due to licensing issues. > > > For local changes not found in the (latest) contrib doc-common > > you should use gnu/usr.bin/groff/tmac/mdoc.local > > I followed r217595, was that commit the wrong approach?
Wrong. mdoc is still distributed under the BSD license, and changes to it should be cherry picked and merged via vendor/groff/dist/. "8.3" and "10.0" that you added were already present in mdoc.local. I've cleaned up this mess now while I was pulling up latest changes to mdoc. And BTW, I still recommend a pre-commit review for groff, as hinted in src/MAINTAINERS. _______________________________________________ svn-src-all@freebsd.org mailing list http://lists.freebsd.org/mailman/listinfo/svn-src-all To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"