In D28281#632703, @fernape wrote:

LGTM.

Just in case you need help with git: https://github.com/bsdimp/freebsd-git-docs/blob/main/src-cvt.md

If you have two commits (the original and the updated with the sorted fields) try to squash them as one.

Thanks. Just committed, although the log format is not as I expected. I'm guessing that arc land alters the commit log and that your typical git push is preferred?

In D28281#632709, @lcook wrote:

In D28281#632703, @fernape wrote:

LGTM.

Just in case you need help with git: https://github.com/bsdimp/freebsd-git-docs/blob/main/src-cvt.md

If you have two commits (the original and the updated with the sorted fields) try to squash them as one.

Thanks. Just committed, although the log format is not as I expected. I'm guessing that arc land alters the commit log and that your typical git push is preferred?

Do not use arc land as it does not do exactly what we want. Have a look at: https://wiki.freebsd.org/action/show/Phabricator?action=show&redirect=CodeReview

It is a bit outdated now that 2/3 repos are in git, but still.

Also, you can have a look at this in case it helps: https://wiki.freebsd.org/FernandoApesteguia/MyReviewWorkflow

In D28281#632710, @fernape wrote:

In D28281#632709, @lcook wrote:

In D28281#632703, @fernape wrote:

LGTM.

Just in case you need help with git: https://github.com/bsdimp/freebsd-git-docs/blob/main/src-cvt.md

If you have two commits (the original and the updated with the sorted fields) try to squash them as one.

Thanks. Just committed, although the log format is not as I expected. I'm guessing that arc land alters the commit log and that your typical git push is preferred?

Do not use arc land as it does not do exactly what we want. Have a look at: https://wiki.freebsd.org/action/show/Phabricator?action=show&redirect=CodeReview

It is a bit outdated now that 2/3 repos are in git, but still.

Also, you can have a look at this in case it helps: https://wiki.freebsd.org/FernandoApesteguia/MyReviewWorkflow

Duly noted! Thank you for the clarification. :-)