[Mesa-dev] Is it time to stop using the mailing list for patch review?

Dylan Baker dylan at pnwbakers.com
Wed Dec 11 18:44:32 UTC 2019


Quoting Nathan Kidd (2019-12-11 06:41:12)
> On 2019-12-10 7:10 p.m., Jason Ekstrand wrote:
> > Quoting Zebediah Figura (2019-12-10 10:58:45)
> > > On 12/10/19 12:21 PM, Matt Turner wrote:
> > > > On Mon, Dec 9, 2019 at 6:07 PM Dylan Baker <dylan at pnwbakers.com> wrote:
> > > > > I think its time we discussed whether we're going to continue to do patch review
> > > > > on the mailing list, or if it it should all go through gitlab. I think we should
> > > > > stop using the mailing list, here are some reasons:
> > > > >
> > > > > 1) Most development is happening on gitlab at this point, patches on the mailing
> > > > >    list are often overlooked
> > > > > 2) The mailing list bypasses CI which potentially breaks the build
> > > >
> > > > What concrete change would you propose?
> > >
> > > Removing mention of the mailing list from documentation would be nice.
> > > Also, currently the README implies that the mailing list is not only
> > > acceptable but preferred: "Note that Mesa uses email mailing-lists for
> > > patches submission, review and discussions."
> > 
> > I think updating README type docs is sufficient and I think it's
> > probably time we did that.  
> 
> As a recent drive-by patch submitter, updating the web page and other
> docs to say "use GitLab" would have been sufficient for me.  As they are
> now it wasn't clear which way is preferred, so I defaulted to using
> mesa-dev@ only to find that the dev focus seems to be in gitlab.fd.o and
> if it wasn't for Roland watching the mailing list my patch seemed likely
> to get missed.
> 
> I also went through the work of subscribing to mesa-stable@ to nominate
> there, but again it feels like I missed the de-facto channel for
> suggesting a stable backport patch.   Hint hint [1]. :)

Gahh, mesa-stable is a mailing list I *do* want to complete purge, MRs for
backports, "cc" or "fixes" in the commit message otherwise. I'll update the docs
to mention that.

Dylan
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 228 bytes
Desc: signature
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20191211/742ff54e/attachment.sig>


More information about the mesa-dev mailing list