PRsWithPatches [FreeBSD WIKI]

This page is read only. You can view the source, but not change it. Ask your administrator if you think this is wrong.
====== PRs and DRs containing patches ======

Many submissions to %%FreeBSD%% contain a patch of some form.  From an end-users point of view, these may well be the most frustrating type of ignored submissions, for several reasons:

  - A user has actually gone to the effort of diagnosing the problem and coding a solution or workaround, only to have it ignored.

  - Users may feel frustrated as they perceive the actual process of committing the patch to be simple, compared with the effort of creating the patch.

  - Patches can quickly become outdated and unapplicable in their current form, and may regularly need to be updated by the submitter and/or other people wanting the functionality.

  - Bugs with patches (whether originally there or added later) that don't get acted on can acquire a will-never-get-fixed smell that can discourage others coming in later with the same problem, sometimes for years.

For these reasons and more, it would be good if we could go through PRs which contain patches and decide if they are ready to be committed as-is.  If not, then feedback should be added to the PR to explain why so, to give other people a clue as to how to get the patch into a committable shape.

Possible criteria to decide if it committable include:
  * Is the corrected behaviour actually a bug (or, do we want the functionality, for improvement requests)?
  * Does it fix the bug in question?
  * Does it apply cleanly to -HEAD?
  * Does it follow style(9), or can it be made to easily?

There are a lot of submitted patches that do fix real bugs, and do fit the above criteria for being ready to commit.  Let's get some of them pushed into the tree!

===== PRs containing patches in Bugzilla =====

Around a third of all open PRs in [[/Bugzilla|FreeBSD's Bugzilla]] contain a patch of some form.  Here are the ones for individual ports:

  * [[https://bugs.freebsd.org/bugzilla/buglist.cgi?component=Individual%20Port%28s%29&email1=ports-bugs%40FreeBSD.org&emailassigned_to1=1&emailtype1=equals&f1=attachments.ispatch&f2=OP&f3=attachments.filename&f4=attachments.filename&f5=attachments.filename&f6=CP&j2=OR&list_id=374864&o1=equals&o3=substring&o4=substring&o5=substring&product=Ports%20%26%20Packages&query_format=advanced&resolution=---&v1=1&v3=diff&v4=patch&v5=shar|unassigned ports PRs containing patches]]
  * [[https://bugs.freebsd.org/bugzilla/buglist.cgi?component=Individual%20Port%28s%29&f1=attachments.ispatch&f2=OP&f3=attachments.filename&f4=attachments.filename&f5=attachments.filename&f6=CP&j2=OR&limit=0&list_id=374860&o1=equals&o3=substring&o4=substring&o5=substring&order=bug_id%20DESC&product=Ports%20%26%20Packages&query_format=advanced&resolution=---&v1=1&v3=diff&v4=patch&v5=shar|all ports PRs containing patches (whether assigned or not)]]

Similarly, for base system PRs:

  * [[https://bugs.freebsd.org/bugzilla/buglist.cgi?email1=bugs%40FreeBSD.org&emailassigned_to1=1&emailtype1=equals&f1=attachments.ispatch&f2=OP&f3=attachments.filename&f4=attachments.filename&f5=attachments.filename&f6=CP&j2=OR&limit=0&list_id=374866&o1=equals&o3=substring&o4=substring&o5=substring&order=bug_id%20DESC&product=Base%20System&query_format=advanced&resolution=---&v1=1&v3=diff&v4=patch&v5=shar|unassigned src PRs containing patches]]
  * [[https://bugs.freebsd.org/bugzilla/buglist.cgi?f1=attachments.ispatch&f2=OP&f3=attachments.filename&f4=attachments.filename&f5=attachments.filename&f6=CP&j2=OR&limit=0&list_id=374867&o1=equals&o3=substring&o4=substring&o5=substring&order=bug_id%20DESC&product=Base%20System&query_format=advanced&resolution=---&v1=1&v3=diff&v4=patch&v5=shar|all src PRs containing patches (whether assigned or not)]]

And for documentation PRs:

  * [[https://bugs.freebsd.org/bugzilla/buglist.cgi?email1=%28doc%7Cbugs%29%40FreeBSD.org&emailassigned_to1=1&emailtype1=regexp&f1=attachments.ispatch&f2=OP&f3=attachments.filename&f4=attachments.filename&f5=attachments.filename&f6=CP&j2=OR&list_id=374921&o1=equals&o3=substring&o4=substring&o5=substring&product=Documentation&query_format=advanced&resolution=---&v1=1&v3=diff&v4=patch&v5=shar|unassigned doc PRs containing patches]]
  * [[https://bugs.freebsd.org/bugzilla/buglist.cgi?f1=attachments.ispatch&f2=OP&f3=attachments.filename&f4=attachments.filename&f5=attachments.filename&f6=CP&j2=OR&list_id=374916&o1=equals&o3=substring&o4=substring&o5=substring&product=Documentation&query_format=advanced&resolution=---&v1=1&v3=diff&v4=patch&v5=shar|all doc PRs containing patches (whether assigned or not)]]

===== DRs containing patches in Phabricator =====

In general, most DRs in [[/Phabricator]] contain patches.

===== A caveat =====

In [[/MarkLinimon|MarkLinimon]]'s experience, PRs/DRs should be worked on "newest first".  While this is counter-intuitive, many of the older submissions are either controversial, are going to require some hard work, or both.  These can be quite frustrating and thus the easiest ones to discourage new volunteers.  So, the above links are sorted newest-first.