unofficial mirror of notmuch@notmuchmail.org
 help / color / mirror / code / Atom feed
From: Anton Khirnov <anton@khirnov.net>
To: Patrick Totzke <patricktotzke@gmail.com>,
	Michael J Gruber <git@grubix.eu>,
	notmuch@notmuchmail.org
Subject: Re: announce: my fork of alot
Date: Sun, 16 May 2021 19:47:24 +0200	[thread overview]
Message-ID: <162118724451.29687.13284058079630704324@lain.red.khirnov.net> (raw)
In-Reply-To: <162117970935.470262.11733111308148672432@piu>

Quoting Patrick Totzke (2021-05-16 17:41:49)
> Hi everyone,
> 
> All this sounds very exciting and I'd be very happy to see these features in
> (mainline) alot!
> 
> I agree that some of alot's underlying code is ready for refactoring
> and urwid in particular has been a big drag on quickly implementing things.
> Also, I'd be interested in hearing your thoughts on deprecating some "unworthy"
> features in order to reduce the maintenance effort!

That is largely a matter of perspective and personal preference. E.g.
among the things gone in my tree are:
- removing messages - I dropped that because I considered that code
  potentially dangerous, had no use for it myself, and just didn't want
  to tiptoe around it; someone actually using RemoveCommand in their
  workflow would have a different opinion
- switching to split-window layout for thread view made it simpler to
  implement quote folding, but also made sense to me since I never want
  to see more than one message at once;
  again, someone who prefers collapsing messages would see this as loss
  of functionality

https://xkcd.com/1172/ is very much in effect

> > > Why did I not submit all this as PRs to upstream alot? The main reasons
> > > were my lack of time and disagreement with the upstream about project
> > > status. From what I can tell, alot maintainers consider the project to
> > > be mature, so they prioritize stability and small incremental changes.
> > > From my perspective, alot is lacking some critical features -- some
> > > implemented in my fork already, some planned -- which makes it
> > > borderline-unusable for me. As implementing those features required
> > > large-scale architectural changes and my free time was quite limited, I
> > > prioritized quickly implementing the things I cared about over
> > > progressing in small incremental stable easily-reviewable steps.
> > 
> > I have a similar impression about the project status. I'm curious: What
> > are the architectural changes that you made?
> 
> 
> Yes, the speed at which alot progresses is borderline problematic. This is of
> course down to the small number of core contributors and the fact that for all
> of us life goes on an priorities change.
> 
> One problem is that the project attracts many users interested in pushing what
> I'd call "hotfixes" to address missing features: Often people would present
> a (nicely working) proof-of concept that is not well documented, tested, and
> doesn't adhere to common code conventions, only not to follow up on their
> promises to "clean things up", for all too understandable reasons.
> Still, I believe that just merging everything will quickly kill the project as
> a) this leads to code that is very difficult and time-consuming to maintain and
> b) broken features are very damaging to user's perception of the software, much
> more so than missing ones.
> 
> I am not accusing you of anything here, Anton. I just wish to point out
> potential long term difficulties and clarify that I tried to err on the side of
> cautiousness to keep alot afloat in a usable state for most (potential) users.

You would be very correct to accuse me of taking various shortcuts. I
would not call my changes "hotfixes", as I tried to keep continuous
future improvements in mind (and in fact see many of my changes as
cleanup and simplification). But I did make an explicit decision to
prioritise rapidly adding new functionality, at the cost of potential
regressions and loss of some features I did not need.

And again, this is a matter of perspective. If alot does what you want
it to do then of course you will value stability and consistency. But if
the lack of certain features makes it barely usable, then it makes sense
to be more radical.

> > > At this point my tree has over 200 new commits and some ~4k changed
> > > lines, so it's looking increasingly unlikely that I'll ever find the
> > > free time and motivation to upstream it -- especially given alot's
> > > glacial pace of development recently. If people are interested in using
> > > this, I'll probably fork it "properly" under a new name.
> > > 
> > > Any comments or questions are very much welcome. I can also be reached
> > > on IRC as elenril.
> > 
> > Have you tried raising these concerns with upstream before your fork?
> > Have you tried gathering a team around an idea and starting something
> > new together?
> > 
> > Frankly, upstream is borderline small already, and the way you started
> > your fork probably will not attract a team of people who want to make
> > that new fork their (common) own or are looking for a stronger team.
> 
> I share Michael's concerns about further splintering the small group of
> developers and believe that this would be to the detriment of both projects.
> 
> It's no secret that I am ready to give the helm to others. I have been
> maintaining this project for a while now, mainly for personal usage and as
> a fun distraction. I have tried to squeeze in time to review pull requests when
> possible and am grateful for the many code contributions over the years, most
> notably the big steps towards pgp/mime, python3 and notmuch2, all of which I'd
> have never found the time to implement myself.
> 
> It has so far been a successful, albeit slow, strategy to try and coordinate
> efforts and I would very much like to see this going on, but without
> sacrificing the quality of the code or the relative mature user experience.

And here is precisely the crux of the problem. My changes are pretty
drastic and 1) there WILL be bugs 2) someone WILL find them to degrade
their user experience. You cannot always satisfy everyone.

Combined with the fact that I also have a lot on my plate and don't see
myself reshaping my tree into nicely packaged atomic changes with a
ribbon on top (at least not any time soon).

> 
> To be clear: I still do not consider alot "mature" in the sense that I'd oppose
> radical refactoring. This is reflected in its version number :)

If you can find the time for it, maybe try to look at the individual
changes in my tree. Try it out, see what makes sense to you, what
doesn't, etc. I would be happy to see it all merged into alot, just
don't see how it can practically be done through the normal channels.

E.g. one thing I expect to be contentious is the removal of urwidtrees
use. I understand you are its author, so it may be unpleasant to hear,
but I found it to be a major obstacle to implementing quote folding.

-- 
Anton Khirnov

  reply	other threads:[~2021-05-16 17:47 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-05-16 10:19 announce: my fork of alot Anton Khirnov
2021-05-16 11:15 ` Michael J Gruber
2021-05-16 14:42   ` Anton Khirnov
2021-05-16 15:41   ` Patrick Totzke
2021-05-16 17:47     ` Anton Khirnov [this message]
2021-05-16 18:23       ` Patrick Totzke
2021-05-17  7:08         ` Patrick Totzke
2021-05-17  7:33           ` Michael J Gruber
2021-05-17  7:54             ` Michael J Gruber
2021-05-17  8:25             ` Anton Khirnov
2021-05-17  8:19           ` Anton Khirnov
2021-05-17  8:28             ` Patrick Totzke
2021-05-17  8:39               ` Anton Khirnov
2021-05-17  9:02         ` Anton Khirnov
2021-05-16 22:09 ` Johannes Larsen
2021-05-16 22:32   ` Felipe Contreras
2021-05-16 23:13     ` Johannes Larsen
2021-05-17  0:23       ` Felipe Contreras
2021-05-17  8:54   ` Anton Khirnov
2021-05-17 12:32     ` Johannes Larsen
2021-05-17 12:55       ` Anton Khirnov
2021-05-17 19:03         ` Johannes Larsen

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://notmuchmail.org/

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

  git send-email \
    --in-reply-to=162118724451.29687.13284058079630704324@lain.red.khirnov.net \
    --to=anton@khirnov.net \
    --cc=git@grubix.eu \
    --cc=notmuch@notmuchmail.org \
    --cc=patricktotzke@gmail.com \
    /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://yhetil.org/notmuch.git/

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