Updating contributing article to include git instead of tar/shar

[freebsd_ny-central.org]

Summary

This patch originated out of PR 278284, which indicated that tar and shar are outdated processes for change submissions.

There may probably be better people to speak to this to me, so I hope this is a step in the right direction. Feedback is very welcome.

I have

• added reference to git format-patch as well as git diff
• removed tar and shar in terms of the "de-facto" submission path
• added Phabricator as review option
• removed the hosting question to be asked from project members, since we now have Phabricator

[fernape]

LGTM.

Thanks!

[brooks]

For non-ports contributions we should really discourage submissions via bugzilla unless it's what a specific committer you're working with wants. Generally, bugzilla is where patches go to die.

documentation/content/en/articles/contributing/_index.adoc

265 ↗

(On Diff #138739)

I think we should drop the mention of tar entirely. For things that Phabricator doesn't handle well (lots of renames, etc) GitHub PRs are more appropriate.

286 ↗

(On Diff #138739)

I'd drop FTP while you're here, it's archaic.

[fernape]

documentation/content/en/articles/contributing/_index.adoc

265 ↗

(On Diff #138739)

I think we should drop the mention of tar entirely. For things that Phabricator doesn't handle well (lots of renames, etc) GitHub PRs are more appropriate.

Agree.

286 ↗

(On Diff #138739)

Probably worth mentioning "... a web server or other online storage solution." so we cover google drive, dropbox, etc...

[brooks]

documentation/content/en/articles/contributing/_index.adoc

286 ↗

(On Diff #138739)

I guess so. Personally I want Phab or a GitHub PR. Anything else and the interest bar for looking at such submissions becomes very high.

[freebsd_ny-central.org]

I've done some additional updates based on your inputs. Additional feedback very
welcome, of course.

I've switched to arcanist now - if something's off with the patch, please let me
know - I might still have some of the config screwed up...

Suggestion for preliminary commit message (once we get there):

Instead of sending tar files and only filing patches as PR on
Bugzilla, contributions should instead by uploaded to Phabricator
or provided as pull requests on Github.

PR:             278274
Reported by:    brooks
Reviewed by:    brooks, fernape

[freebsd_ny-central.org]

documentation/content/en/articles/contributing/_index.adoc

286 ↗

(On Diff #138739)

I've removed any form of website and recommend Phabricator and Github only - is this a working compromise or should we mention these still?

[brooks]

This is an improvement.

I'm not convinced we should discuss any non-git contribution methods, but can certainly be committed as is.

[jrm]

documentation/content/en/articles/contributing/_index.adoc

252 ↗

(On Diff #139231)

Would linking directly to man:git-format-patch [1] here make sense? We already link to man:git[1] a few lines above.

261 ↗

(On Diff #139231)

We could also mention man:git-apply[1] here for when the patch was generated with git format-patch.

...(which you may test with man:patch[1] or man:git-format-patch[1])...

[freebsd_ny-central.org]

[articles][committers-guide]: streamlining stable/N and releng/N.n syntax

As outlined in https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=279503 there are
various ways we are listing stable/N and releng/N.n - sometimes with numbers,
sometimes not.

I'm suggesting to migrate everything into one common format. What I have not
resolved yet (and would appreciate feedback on): there are still numerous
examples with i.e. stable/13 (I updated from stable/12). It would make sense to
simply change them all into a stable/N and releng/N.n format.

Ed made the point that he received issue reports for people misunderstanding the
placeholder syntax, so maybe it makes sense to leave a handful of examples with
numbers?

PR: 279503

[freebsd_ny-central.org]

Mistakenly had another file in there... removed.

[freebsd_ny-central.org]

And this happens when I don't use arc more regularly. I push a new patch into an old revision. Apologies and please ignore this one.