all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* bug#52274: Gnus gcc header tokenization ignores space breaks multiple folder archive setup
@ 2021-12-04 10:00 Mats Lidell
  2021-12-04 19:23 ` Lars Ingebrigtsen
  0 siblings, 1 reply; 6+ messages in thread
From: Mats Lidell @ 2021-12-04 10:00 UTC (permalink / raw)
  To: 52274

Hi,

In commit 08ce17c2c0d32e200af3984d59f0b78ec500dc2c "Fix Gnus gcc header tokenization" space is removed form the tokenization of Gcc.

My setup with saving out going mails in two folders seems to have broken after that. My Gcc-header is setup like this:

    Gcc: <sent-folder><SPACE><mail-folder>

After the change "<sent-folder><SPACE><mail-folder>" seems to used for archiving causing some odd results.

I have configured gnus-message-archive-group to return a list of <sent-folder> and <mail-folder> so I am not tampering with the Gcc header directly.

Yours
-- 
%% Mats





^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#52274: Gnus gcc header tokenization ignores space breaks multiple folder archive setup
  2021-12-04 10:00 bug#52274: Gnus gcc header tokenization ignores space breaks multiple folder archive setup Mats Lidell
@ 2021-12-04 19:23 ` Lars Ingebrigtsen
  2021-12-04 22:34   ` Mats Lidell
  0 siblings, 1 reply; 6+ messages in thread
From: Lars Ingebrigtsen @ 2021-12-04 19:23 UTC (permalink / raw)
  To: Mats Lidell; +Cc: 52274

Mats Lidell <matsl@gnu.org> writes:

> In commit 08ce17c2c0d32e200af3984d59f0b78ec500dc2c "Fix Gnus gcc
> header tokenization" space is removed form the tokenization of Gcc.
>
> My setup with saving out going mails in two folders seems to have
> broken after that. My Gcc-header is setup like this:
>
>     Gcc: <sent-folder><SPACE><mail-folder>
>
> After the change "<sent-folder><SPACE><mail-folder>" seems to used for
> archiving causing some odd results.
>
> I have configured gnus-message-archive-group to return a list of
> <sent-folder> and <mail-folder> so I am not tampering with the Gcc
> header directly.

I've now changed gnus-inews-insert-gcc on master to insert a comma
between groups, so I think that should fix the problem.  Can you check?

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no





^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#52274: Gnus gcc header tokenization ignores space breaks multiple folder archive setup
  2021-12-04 19:23 ` Lars Ingebrigtsen
@ 2021-12-04 22:34   ` Mats Lidell
  2021-12-04 22:49     ` Lars Ingebrigtsen
  0 siblings, 1 reply; 6+ messages in thread
From: Mats Lidell @ 2021-12-04 22:34 UTC (permalink / raw)
  To: Lars Ingebrigtsen; +Cc: Mats Lidell, 52274

> Lars Ingebrigtsen writes:
> I've now changed gnus-inews-insert-gcc on master to insert a comma
> between groups, so I think that should fix the problem.  Can you check?

It does not quite fix it since the comma was inserted together with a space. The tokenizer leaves the leading space on the second folder. Which in my case makes the server method be " nnimap" which gnus-check-server does not understand.

(message-tokenize-header "folder1, folder2" ",\n\t") => ("folder1" " folder2")

%% Mats
PS. Curious, what was the problem with having space as separator? DS.





^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#52274: Gnus gcc header tokenization ignores space breaks multiple folder archive setup
  2021-12-04 22:34   ` Mats Lidell
@ 2021-12-04 22:49     ` Lars Ingebrigtsen
  2021-12-05  9:33       ` Mats Lidell
  0 siblings, 1 reply; 6+ messages in thread
From: Lars Ingebrigtsen @ 2021-12-04 22:49 UTC (permalink / raw)
  To: Mats Lidell; +Cc: 52274

Mats Lidell <matsl@gnu.org> writes:

> It does not quite fix it since the comma was inserted together with a
> space. The tokenizer leaves the leading space on the second
> folder. Which in my case makes the server method be " nnimap" which
> gnus-check-server does not understand.
>
> (message-tokenize-header "folder1, folder2" ",\n\t") => ("folder1" " folder2")

Darn.  I've now made it insert just a single comma.

> PS. Curious, what was the problem with having space as separator? DS.

Uhm...  it had something to do with group names containing spaces (which
is valid in nnimap).  Unfortunately the checkin doesn't refer to a bug
report, for some reason...

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no





^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#52274: Gnus gcc header tokenization ignores space breaks multiple folder archive setup
  2021-12-04 22:49     ` Lars Ingebrigtsen
@ 2021-12-05  9:33       ` Mats Lidell
  2021-12-05 20:12         ` Lars Ingebrigtsen
  0 siblings, 1 reply; 6+ messages in thread
From: Mats Lidell @ 2021-12-05  9:33 UTC (permalink / raw)
  To: Lars Ingebrigtsen; +Cc: Mats Lidell, 52274

> Lars Ingebrigtsen writes:
> Darn.  I've now made it insert just a single comma.

Great. Works fine now.

> > PS. Curious, what was the problem with having space as separator? DS.
>
> Uhm...  it had something to do with group names containing spaces (which
> is valid in nnimap).  Unfortunately the checkin doesn't refer to a bug
> report, for some reason...

I see. Makes sense.

%% Mats





^ permalink raw reply	[flat|nested] 6+ messages in thread

* bug#52274: Gnus gcc header tokenization ignores space breaks multiple folder archive setup
  2021-12-05  9:33       ` Mats Lidell
@ 2021-12-05 20:12         ` Lars Ingebrigtsen
  0 siblings, 0 replies; 6+ messages in thread
From: Lars Ingebrigtsen @ 2021-12-05 20:12 UTC (permalink / raw)
  To: Mats Lidell; +Cc: 52274

Mats Lidell <matsl@gnu.org> writes:

>> Darn.  I've now made it insert just a single comma.
>
> Great. Works fine now.

Thanks for checking; I'm closing this bug report, then.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no





^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2021-12-05 20:12 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-12-04 10:00 bug#52274: Gnus gcc header tokenization ignores space breaks multiple folder archive setup Mats Lidell
2021-12-04 19:23 ` Lars Ingebrigtsen
2021-12-04 22:34   ` Mats Lidell
2021-12-04 22:49     ` Lars Ingebrigtsen
2021-12-05  9:33       ` Mats Lidell
2021-12-05 20:12         ` Lars Ingebrigtsen

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.