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

264–265

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.

287

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

[fernape]

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

264–265

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.

287

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

287

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

287

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

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

261

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])...