Page MenuHomeFreeBSD

committers-doc.dot: add grahamperrin
ClosedPublic

Authored by grahamperrin on Jul 7 2022, 6:43 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Nov 6, 7:41 PM
Unknown Object (File)
Oct 20 2024, 2:36 PM
Unknown Object (File)
Oct 20 2024, 2:36 PM
Unknown Object (File)
Oct 20 2024, 2:36 PM
Unknown Object (File)
Oct 20 2024, 2:36 PM
Unknown Object (File)
Oct 20 2024, 2:07 PM
Unknown Object (File)
Oct 11 2024, 4:59 PM
Unknown Object (File)
Sep 27 2024, 3:05 AM
Subscribers

Details

Summary

Committer's Guide, steps for new committers, step 5.

Mentors: Glen Barber (gjb) and Sergio Carlavilla (carlavilla).

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision is now accepted and ready to land.Jul 7 2022, 6:50 PM

View OptionsShow Path in Repository

Path Does Not Exist

Also, https://reviews.freebsd.org/D35746#UQ0_10 above, doc should have been src … I don't imagine this being correctable (by me), should I abandon this D35746 and start afresh?

The raw diff from Phabricator didn't work for me with git-apply(1).

Instead, here's a diff produced by GitLab.

share/misc/committers-doc.dot
165–169

I put myself alphabetically between crees and philip, however I see that crees is preceded – not alphabetically – by rene and dru.

Is order negligible, here?

https://cgit.freebsd.org/src/tree/share/misc/committers-doc.dot?id=71e1ff237ed6bdfa2a394e0d2b121aed981865fa#n35 was simply:

# Keep the list sorted by mentor login.

https://reviews.freebsd.org/D35746#UQ0_10 above, doc should have been src … I don't imagine this being correctable (by me), …

☑ Now I see rG FreeBSD src repository – I guess, correction occurred when I uploaded the file for diff 107981.

This revision is now accepted and ready to land.Jul 15 2022, 6:30 PM
share/misc/committers-doc.dot
165–169

dot -T png -o /tmp/commiters-doc.png /usr/src/share/misc/committers-doc.dot at a glance, this part of the result looks OK:

image.png (313×1 px, 109 KB)

– so I guess that order is negligible.

Approved

Thanks. Not noted in the commit, because I authored the commit around the time of acceptance of an earlier revision.


Today's push succeeded:


Attempts yesterday failed, somewhat mysteriously. From an attempt at 21:09 BST:

% time git -C /usr/src push freebsd HEAD:main
Enter passphrase for key '/home/grahamperrin/.ssh/id_rsa':  
load: 0.95  cmd: ssh 22002 [select] 242.59r 0.18u 0.00s 0% 7844k
load: 0.68  cmd: ssh 22002 [select] 645.63r 0.18u 0.00s 0% 7900k
load: 0.64  cmd: ssh 22002 [select] 677.54r 0.18u 0.00s 0% 7900k
^C0.019u 0.000s 11:20.05 0.0%   2472+332k 0+0io 0pf+0w
%
  • for a few minutes, whilst awaiting a response, I occasionally keyed Control-T
  • Control-C abandoned the attempt.

There's additional context, which I'll not echo here. All things considered, I'm happy for yesterday's events to remain a mystery ☑


Success today was partly thanks to a simple git primer (for ports) https://people.freebsd.org/~danfe/git.html (2021-04-16, @danfe), which I stumbled across whilst seeking something completely unrelated to Git. Step 6 there drops a hint that seems to be absent from the Committer's Guide:

  • git show HEAD

A transcript (other people might learn from my mistakes):

– key points:


FreeBSD bug 261812 – Miscellaneous small changes to the Committer's Guide.