discussion and development of piem
 help / color / mirror / code / Atom feed
From: Kyle Meyer <kyle@kyleam.com>
To: Leo <sourcehut@relevant-information.com>
Cc: piem@inbox.kyleam.com, Sean Whitton <spwhitton@spwhitton.name>,
	Xinglu Chen <public@yoctocell.xyz>
Subject: Re: [PATCH 1/1] Use notmuch-extract-patch if available
Date: Sun, 12 Dec 2021 13:45:48 -0500	[thread overview]
Message-ID: <87ilvt64rn.fsf@kyleam.com> (raw)
In-Reply-To: <87k0ga85yf.fsf@relevant-information.com>

[ +cc Sean for awareness, more context at
  https://inbox.kyleam.com/piem/20211209204319.168897-1-sourcehut@relevant-information.com/T/#u ]

Leo writes:

> Kyle Meyer <kyle@kyleam.com> writes:
>
>> It's been a little while since I've looked over mailscripts, but I think
>> it's got a lot of neat functionality (and in general am excited to see
>> work in this space).  I obviously decided to focus piem's patch
>> extraction functionality around b4, but I'm happy to consider some
>> support for notmuch-extract-patch if there are people that 1) want to
>> use notmuch-extract-patch and 2) for whatever reason, prefer to use piem
>> rather than mailscripts.el.
>
> I've looked a bit at mailscripts.el and it serves as a thin wrapper
> around the perl scripts in the same repo.  It has two features: 1) some
> light integration with debbugs and 2) apply the patch in this
> message/thread to a repo chosen by project.el/projectile. 2) is
> basically the same as piem except that it prompts the user for a project
> (every time).
>
> I feel like there is an opportunity here to reduce
> fragmentation and make piem the main interface to mailscripts.
> mailscripts.el is very short and just with this patch it supports half
> the functionality of mailscripts.el.  This would of course require
> communication with the maintainers of mailscripts and a non-trivial
> amount of work.

Hmm.  When thinking about unifying anything here, I suppose a core issue
is the same thing that came up in another thread
(<87y2gs374y.fsf@kyleam.com>), where I said (slightly edited):

  A related, larger issue here is my choice to focus piem on public-inbox
  even though many users may only be interested in the functionality for
  applying patches.  Working on projects without a public-inbox archive is
  supported to some degree [4], and notmuch users can even process patches
  with b4, but that just kind of fell together rather than being a goal
  (even more so because mailscripts already covered this).  My primary
  interest with piem is public-inbox, and there's some very exciting work
  upstream on a local client [5] that I plan to support.
  
  [...]

  [4] https://docs.kyleam.com/piem/Applying-patches-without-a-public_002dinbox-archive.html
  [5] https://public-inbox.org/meta/20201215114722.27400-1-e@80x24.org/

Given this focus, I doubt mailscripts would want to pull in piem to use
its functionality for applying patches.  And at the same time, there
doesn't seem to be quite enough meat or complexity in the patch
application code that it'd be worth having an upstream library that both
piem and mailscripts could depend on.

My view on it at this point is that piem and mailscripts have a
different focus, and I don't think users having a few options in this
area is a bad thing.

  reply	other threads:[~2021-12-12 18:45 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-12-09 20:43 [PATCH 0/1] Use notmuch-extract-patch if available sourcehut
2021-12-09 20:43 ` [PATCH 1/1] " sourcehut
2021-12-11 21:44   ` Kyle Meyer
2021-12-12  9:44     ` Leo
2021-12-12 18:45       ` Kyle Meyer [this message]
2021-12-12 19:33         ` Sean Whitton
2021-12-12 19:49           ` Kyle Meyer
2021-12-14 19:12             ` Sean Whitton
2021-12-13 11:45           ` Leo
2021-12-14 11:36     ` Leo
2021-12-20  6:45       ` Sean Whitton
2021-12-12 19:26   ` Sean Whitton
2021-12-13 11:53     ` Leo

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

  List information: https://git.kyleam.com/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87ilvt64rn.fsf@kyleam.com \
    --to=kyle@kyleam.com \
    --cc=piem@inbox.kyleam.com \
    --cc=public@yoctocell.xyz \
    --cc=sourcehut@relevant-information.com \
    --cc=spwhitton@spwhitton.name \
    /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.
Code repositories for project(s) associated with this public inbox

	https://git.kyleam.com/piem/

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).