Page MenuHomeFreeBSD

Add Tom Jones (thj) as a new src committer.
ClosedPublic

Authored by thj on Apr 11 2018, 3:05 PM.
Tags
None
Referenced Files
Unknown Object (File)
Dec 22 2023, 9:58 PM
Unknown Object (File)
Jun 30 2023, 4:41 AM
Unknown Object (File)
Jun 29 2023, 3:52 AM
Unknown Object (File)
May 8 2023, 4:30 AM
Unknown Object (File)
Apr 25 2023, 9:33 AM
Unknown Object (File)
Apr 7 2023, 4:22 PM
Unknown Object (File)
Feb 9 2023, 3:35 AM
Unknown Object (File)
Jan 5 2023, 12:23 AM
Subscribers
None

Details

Summary

Add Tom Jones (thj) as a new src committer.

  • Add authors.ent entity
  • Add email
  • Add news item
  • Add PGP key

Approved by: jtl (mentor)

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

jtl requested changes to this revision.Apr 11 2018, 5:06 PM
jtl added inline comments.
en_US.ISO8859-1/articles/contributors/contrib.committers.xml
585

I think this should go above mjoras to meet the criteria (alphabetical order by last name).

share/pgpkeys/pgpkeys-developers.xml
1014

Again, I believe this should be above mjoras.

share/xml/news.xml
34

This needs to move into the <month> block. Because things are in reverse-chronological order, the <day> block for 7 April should be before the block for 4 April.

This revision now requires changes to proceed.Apr 11 2018, 5:06 PM

Move my entries to above mjoras

Move news.xml entry into the month block

Looks good. Please check on two things before committing:

  1. I don't see a svn:keywords property change for head/share/pgpkeys/thj.key. I expected to see one, although this could just be a Phabricator display issue. Can you confirm you've followed the instructions in the Note at the start of 5.3.7 of the Committer's Guide ("Adding and Removing Files")? If not, please do so and then svn remove and svn add the file again. You can check that the property was correctly applied by running this command: svn propget svn:keywords thj.key. You should see FreeBSD=%H.
  2. Please check on the indentation in news.xml. (See in-line note.)
share/xml/news.xml
44

The indenting looks funky here (for the </event> line), but that could be a Phabricator display issue. Can you double check that it uses the same indenting (including tabs vs. spaces) as the <event> line prior to committing?

Thanks!

This revision is now accepted and ready to land.Apr 11 2018, 5:45 PM

Make the indentation in for "event" blocks news.xml match. It was equal number of spaces to tabs, now is tabs.

The "svn:keywords" property was not being set as there is no auto-props entry for key files in the example from 5.3.7 of the committers guide.

Set "svn:keywords" property to "" (empty string) rather than "FreeBSD=%H". As that is what the majority of the other keys in share/pgpkeys have.

This revision now requires review to proceed.Apr 11 2018, 8:28 PM
This revision is now accepted and ready to land.Apr 11 2018, 10:54 PM
This revision was automatically updated to reflect the committed changes.