Gnus development mailing list
 help / color / mirror / Atom feed
From: Eric Abrahamsen <eric@ericabrahamsen.net>
To: ding@gnus.org
Subject: Re: Calling for testers: scratch/gnus-decoded branch
Date: Thu, 06 Jun 2019 16:47:59 -0700	[thread overview]
Message-ID: <87muiuff80.fsf@ericabrahamsen.net> (raw)
In-Reply-To: <86zhmvgjnx.fsf@gmail.com>

Andy Moreton <andrewjmoreton@gmail.com> writes:

> On Wed 29 May 2019, Eric Abrahamsen wrote:
>
>> Hello reckless Gnus users!
>>
>> I'm working on the second half of the refactoring that broke everyone's
>> Gnus last time -- changing the hash tables to real hash tables, and
>> group names to decoded strings.
>>
>> In order to prevent a repeat performance of the last patch, I have a)
>> been much more careful about testing this one, and b) hope I can talk a
>> handful of you into helping me test.
>>
>> Basically you would check out a local copy of the scratch/gnus-decoded
>> branch, run "make" in it, BACK UP YOUR .newsrc.eld FILE, then restart
>> Emacs and do what you normally do in Gnus, and see if anything looks
>> funny. When you're tired of that, switch back to master, run "make"
>> again, and replace your backup .newsrc.eld file.
>
> If this set of changes modifies the file format of .newsrc.eld then you
> need a different strategy to ensure that users can continue to use their
> gnus data on both old and new gnus.

While I agree with the spirit of this, it's not how Gnus has done things
in the past. There is a `gnus-newsrc-file-version' variable and a
`gnus-convert-old-newsrc' function precisely for changing the file
format following a code upgrade. It's also generally understood that
syncing .newsrc.eld files between machines is risky and unpredictable.
Nor does keeping two copies of the data in different formats sound very
practical -- I don't know of any elisp code that does that.

However! I do agree that this change should probably use the existing
upgrade machinery, which means it would make most sense to hold off
until Gnus ups a version tick. All the tweaking I have in mind for Gnus
will probably warrant some sort of new version, at which point the
upgrade machinery actually prompts to do the conversion, and it should
be very obvious to users that they have a new .newsrc.eld version
(though actually I don't have anything else in mind that will affect
the file format).

So I'll go back to writing non-ascii group names in the gross old
encoding, slightly disappointing as that may be...

Eric




  reply	other threads:[~2019-06-06 23:47 UTC|newest]

Thread overview: 31+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-05-30  0:59 Eric Abrahamsen
2019-06-04  5:24 ` Eric Abrahamsen
2019-06-06  2:38   ` Michael Heerdegen
2019-06-06  2:48     ` Emanuel Berg
2019-06-06 23:07       ` Eric Abrahamsen
2019-06-07  3:24         ` Emanuel Berg
2019-06-07  3:58           ` Eric Abrahamsen
2019-06-07  4:15             ` Emanuel Berg
2019-06-07  5:44               ` Eric Abrahamsen
2019-06-07 15:22                 ` Emanuel Berg
2019-06-07 16:52                   ` Eric Abrahamsen
2019-06-08 21:01                     ` Emanuel Berg
2019-06-06  9:14 ` Andy Moreton
2019-06-06 23:47   ` Eric Abrahamsen [this message]
2019-06-07  9:48     ` Tassilo Horn
2019-06-07 16:46       ` Eric Abrahamsen
2019-06-07 22:41 ` Adam Sjøgren
2019-06-10  4:18   ` Eric Abrahamsen
2019-06-16 13:24     ` Adam Sjøgren
2019-06-16 16:53       ` Eric Abrahamsen
2019-06-16 19:06         ` Adam Sjøgren
2019-06-16 19:45           ` Eric Abrahamsen
2019-06-16 20:01             ` Adam Sjøgren
2019-06-16 20:57               ` Eric Abrahamsen
2019-06-16 21:25                 ` Eric Abrahamsen
2019-07-20  9:22                 ` Adam Sjøgren
2019-07-21  3:19                   ` Eric Abrahamsen
2019-07-21 10:08                     ` Adam Sjøgren
2019-07-22 17:05                       ` Eric Abrahamsen
2019-07-22 17:08                         ` Adam Sjøgren
2019-06-16 19:45           ` Eric Abrahamsen

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=87muiuff80.fsf@ericabrahamsen.net \
    --to=eric@ericabrahamsen.net \
    --cc=ding@gnus.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;
as well as URLs for NNTP newsgroup(s).