Page MenuHomeFreeBSD

git-arc: with "create" allow to specify parent of the first commit
AbandonedPublic

Authored by glebius on Nov 17 2021, 6:28 AM.
Tags
None
Referenced Files
Unknown Object (File)
Jan 30 2024, 4:50 PM
Unknown Object (File)
Dec 23 2023, 12:19 AM
Unknown Object (File)
Nov 21 2023, 7:37 AM
Unknown Object (File)
Nov 14 2023, 3:03 AM
Unknown Object (File)
Nov 13 2023, 2:12 PM
Unknown Object (File)
Nov 11 2023, 12:49 PM
Unknown Object (File)
Nov 6 2023, 7:43 PM
Unknown Object (File)
Oct 25 2023, 12:52 AM

Details

Reviewers
markj
imp
Group Reviewers
manpages

Diff Detail

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

Event Timeline

Adding explanations of reviewer* and subscriber* may help as well for people not familiar with the phabricator workflow, but that's plausibly out of scope for this.

tools/tools/git/git-arc.1
39

Unlike reviewer* and subscriber*, I have no idea what parent is for or why/when/how to use it. Since not all contributors will be able to read the actual wrapper well enough to figure it, it needs elaboration.

The change itself looks good, thanks.

tools/tools/git/git-arc.1
39

Indeed, I'd write "parent revision" at least. And a sentence or two of exposition would be good as well.

This revision is now accepted and ready to land.Nov 17 2021, 2:31 PM
debdrup added inline comments.
tools/tools/git/git-arc.1
39

Unlike reviewer* and subscriber*, I have no idea what parent is for or why/when/how to use it. Since not all contributors will be able to read the actual wrapper well enough to figure it, it needs elaboration.

The syntax should at the very least show if it's only possible to have one parent, or if there can be more - the mdoc can optionally be copied almost verbatum from the above examples if that's the case, or I can assist with creating markup for if there's only one parent, if requested.

Change looks good, modulo the other comments. Tagged a spot that would be the logical place to explain in a setence what reviewer, subscriber and parent revision(s) are.

tools/tools/git/git-arc.1
140

I'd add a .Pp here and explain about parent here. It wouldn't hurt to explain about subscriber and reviewer too, since those are also not explained.

tools/tools/git/git-arc.1
39

There can be only one parent specified. What would be proper mandoc for that? Thanks

This revision now requires review to proceed.Nov 17 2021, 9:40 PM

Minor grammar nit, but it looks good to me overall with that addition.

tools/tools/git/git-arc.1
82–83

Address pauamma_gundo.com comment