svn commit: r359419 - head/usr.bin/ncal

Rodney W. Grimes freebsd at gndrsh.dnsmgr.net
Mon Mar 30 01:29:57 UTC 2020


> On Sunday, 29 March 2020 at  4:03:09 -0700, Rodney W. Grimes wrote:
> >> Author: grog
> >> Date: Sun Mar 29 04:18:27 2020
> >> New Revision: 359419
> >> URL: https://svnweb.freebsd.org/changeset/base/359419
> >>
> >> Log:
> >>   Remove spurious transition from Julian to Gregorian for country
> >>   "Latin".
> >>
> >>   Arguably the entire -p option should be removed.  It shows only a few
> >>   countries, and it doesn't have any relationship with the rest of the
> >>   program.
> >
> > This commit message and the diff below do not seem to match very well.
> > Is this actually what you intended to commit?
> 
> The material difference, yes.  Do you have an issue with that?

No, just that it seemed that an error may of happened,
but having you confirm this infact was the inteded change
I am quiet fine with it.

> 
> It seems that some trailing blanks in unrelated text were removed as
> well.  diff doesn't show these by default, so they crept through.  I
> know it's bad style to mix style commits with functional commits, but
> ultimately they had to go too.

Diff by default does show white space change,
one has to invoke it with -b or -w or one of the --ignore space
change options to cause it to not show them.

> Greg
-- 
Rod Grimes                                                 rgrimes at freebsd.org


More information about the svn-src-head mailing list