Page MenuHomeFreeBSD
Differential D19491

Document that cal(1) may not start a week with Monday
ClosedPublic
Actions

Authored by 0mp on Mar 7 2019, 9:53 AM.
Tags
None
Referenced Files
F170985631: D19491.id54797.diff
Tue, Sep 8, 12:44 AM
F170968041: D19491.id54798.diff
Mon, Sep 7, 10:14 PM
F170966529: D19491.id54799.diff
Mon, Sep 7, 10:03 PM
F170952690: D19491.id54794.diff
Mon, Sep 7, 8:06 PM
F170946735: D19491.id54795.diff
Mon, Sep 7, 7:11 PM
F170937626: D19491.id54797.diff
Mon, Sep 7, 5:46 PM
F170930700: D19491.id54795.diff
Mon, Sep 7, 4:37 PM
F170927550: D19491.id54796.diff
Mon, Sep 7, 4:09 PM
View All Files
Subscribers
imp

Details

Reviewers
bcr
eadler
Group Reviewers
manpages
Commits
rS344874: Document that cal(1) cannot start a week with Monday.
Summary
Document that cal(1) may not start a week with Monday

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 22932
Build 22015: arc lint + arc unit

Event Timeline

0mp created this revision.Mar 7 2019, 9:53 AM
Herald added a reviewer: manpages. · View Herald TranscriptMar 7 2019, 9:53 AM
Herald added a subscriber: imp. · View Herald Transcript
Harbormaster completed remote builds in B22929: Diff 54794.Mar 7 2019, 9:53 AM
0mp updated this revision to Diff 54795.Mar 7 2019, 9:54 AM

Fix a typo

Harbormaster completed remote builds in B22930: Diff 54795.Mar 7 2019, 9:54 AM
bcr accepted this revision.Mar 7 2019, 9:59 AM

Some countries start their week on Sunday, so I guess cal originated there and does not take other countries into account.
That being said, the change itself looks fine. Bumping the .Dd for this content change not withstanding. ;-)

This revision is now accepted and ready to land.Mar 7 2019, 9:59 AM
0mp updated this revision to Diff 54796.Mar 7 2019, 10:00 AM

Reference cal in ports as it offers a way to display Mondays first.

This revision now requires review to proceed.Mar 7 2019, 10:00 AM
Harbormaster completed remote builds in B22931: Diff 54796.Mar 7 2019, 10:00 AM
bcr added a comment.Mar 7 2019, 10:04 AM
This comment was removed by bcr.
bcr accepted this revision.Mar 7 2019, 10:12 AM

This newer version is better as it gives an alternative for people who need that Monday-starts-the-week functionality. Approved!

This revision is now accepted and ready to land.Mar 7 2019, 10:12 AM
0mp updated this revision to Diff 54797.Mar 7 2019, 10:15 AM

Clarify

This revision now requires review to proceed.Mar 7 2019, 10:15 AM
Harbormaster completed remote builds in B22932: Diff 54797.Mar 7 2019, 10:15 AM
bcr added a comment.Mar 7 2019, 10:16 AM

Clarify some more.

usr.bin/ncal/ncal.1
207

s/and have/and want to have/

0mp updated this revision to Diff 54798.Mar 7 2019, 10:17 AM

want to have

Harbormaster completed remote builds in B22933: Diff 54798.Mar 7 2019, 10:17 AM
This revision was not accepted when it landed; it landed in state Needs Review.Mar 7 2019, 10:19 AM
Closed by commit rS344874: Document that cal(1) cannot start a week with Monday. (authored by 0mp). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
usr.bin/
ncal/
11 lines

Diff 54797

View Options

usr.bin/ncal/ncal.1

Loading...