From: Daniel Axtens <dja@axtens.net>
To: Eric Wong <e@80x24.org>, "Theodore Y. Ts'o" <tytso@mit.edu>
Cc: "Rafael J. Wysocki" <rjw@rjwysocki.net>,
workflows@vger.kernel.org, Shuah Khan <skhan@linuxfoundation.org>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Bjorn Helgaas <helgaas@kernel.org>,
Jiri Kosina <jikos@kernel.org>,
Konstantin Ryabitsev <konstantin@linuxfoundation.org>
Subject: Re: Kernel development collaboration platform wish list
Date: Tue, 01 Oct 2019 13:58:35 +1000 [thread overview]
Message-ID: <878sq53yms.fsf@dja-thinkpad.axtens.net> (raw)
In-Reply-To: <20190924013920.GA22698@dcvr>
Eric Wong <e@80x24.org> writes:
> "Theodore Y. Ts'o" <tytso@mit.edu> wrote:
>> One of the things I'd like to add as a strong desire is the ability to
>> review patches via web if that's what users would like to do. There
>> are some real benefits for web-based review. It means that if you
>> need to see greater context, it's relatively easy to do this. It also
>> is convenient to be able to see the conversation for a particular hunk
>> of code right alongside the code.
>
> I've already started on that with public-inbox; it's not enabled
> on lore, yet(*); but public-inbox.org/git can use "git apply"
> and dfpre:/dfpost: search prefixes to reconstruct git blobs out
> of emailed patches.
>
> For example, a self-rejected patch I posted in 2016:
>
> https://public-inbox.org/git/20160711210243.GA1604@whir/
>
> If you follow the link at the hunk header offset "+3,7",
> it'll bring you to:
>
> https://public-inbox.org/git/8d27707/s/?b=http-walker.c#n4
>
> The bottom of that page has a a "debug log:" which tells
> how the blob is reconstructed using the email and pre-existing
> blob.
>
> Since it can recreate blobs using patches, the next step is to
> support showing "git diff" against reconstructed blobs with for
> arbitrary contexts; but I got side-tracked from that earlier
> this year...
>
> (*) lore could configure coderepo associations, but that's a
> bear with the amount of repos kernel.org has...
>
>> It's clear that whatever we do, it needs to be compatible with e-mail.
>> That's very clear. But it would be useful if we can support both the
>> e-mail and web-based review. There have some prototypes that have
>> been floated which shows that it is at least possible; perhaps
>> imperfectly, but something which provides a bidrectional gateway
>> between those who perfer to use e-mail and those that prefer to use a
>> web-based UI would be able to do it.
>
> A hacker-oriented a webmail UI could be derived from
> public-inbox and completely interoperable with other mail
> servers (it would also support SMTP/IMAP, of course).
>
>> There will be many potential kernel contributors who will be used to
>> web-based UI's such as those that are available on github. So while
>> remaining e-mail compatible, having some way of allow as many
>> operations to be done via web interfaces might help us get some newer
>> developers who are more comfortable to living on the web than some of
>> us more senior developers who remember when "gopher" was a text-based
>> search engine, and not a mascot for the Go programming langauge. :-)
>
> Unfortunate, but yeah. Any web-based UIs ought to include some
> subtle (or maybe not-so-subtle) hints/pointers towards
> cheaper-to-run-and-likely-more-powerful tools and practices.
>
> I can't prove it, but it seems like public-inbox has been
> successful at promoting the use of Message-IDs in URLs and
> git-send-email :)
FYI, the next release of Patchwork will move to message-id based URLs
rather than numeric IDs. (Old URLs will still work of course! They will
redirect to new URLs.)
Regards,
Daniel
next prev parent reply other threads:[~2019-10-01 3:58 UTC|newest]
Thread overview: 33+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-09-13 8:22 Rafael J. Wysocki
2019-09-13 11:13 ` Jani Nikula
2019-09-17 21:58 ` Rafael J. Wysocki
2019-09-18 8:05 ` Jani Nikula
2019-09-18 13:43 ` Steven Rostedt
2019-09-13 11:37 ` Laurent Pinchart
2019-09-13 12:02 ` Geert Uytterhoeven
2019-09-13 12:15 ` Laurent Pinchart
2019-09-13 12:23 ` Geert Uytterhoeven
2019-09-17 22:00 ` Rafael J. Wysocki
2019-10-02 9:30 ` Rafael J. Wysocki
2019-09-17 18:40 ` Stefan Schmidt
2019-09-24 9:43 ` Paolo Bonzini
2019-09-24 12:20 ` Stefan Schmidt
2019-09-17 22:12 ` Rafael J. Wysocki
2019-10-01 3:52 ` Daniel Axtens
2019-10-01 4:50 ` Andrew Donnellan
2019-10-01 5:04 ` Daniel Axtens
2019-09-23 16:20 ` Theodore Y. Ts'o
2019-09-24 1:39 ` Eric Wong
2019-09-24 5:06 ` Greg Kroah-Hartman
2019-10-01 3:58 ` Daniel Axtens [this message]
2019-09-26 10:18 ` Geert Uytterhoeven
2019-09-26 11:40 ` Steven Rostedt
2019-09-26 13:39 ` Geert Uytterhoeven
2019-09-28 22:16 ` Steven Rostedt
2019-09-29 16:03 ` Dmitry Vyukov
2019-09-26 13:10 ` Jani Nikula
2019-09-26 13:41 ` Geert Uytterhoeven
2019-09-26 14:40 ` Jani Nikula
2019-10-02 9:55 ` Rafael J. Wysocki
2019-10-02 14:22 ` Daniel Axtens
2019-10-03 9:23 ` Rafael J. Wysocki
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=878sq53yms.fsf@dja-thinkpad.axtens.net \
--to=dja@axtens.net \
--cc=e@80x24.org \
--cc=gregkh@linuxfoundation.org \
--cc=helgaas@kernel.org \
--cc=jikos@kernel.org \
--cc=konstantin@linuxfoundation.org \
--cc=rjw@rjwysocki.net \
--cc=skhan@linuxfoundation.org \
--cc=tytso@mit.edu \
--cc=workflows@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox