From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from outgoing.selfhost.de ([82.98.82.6]) by ewsd; Sat Feb  1 07:24:50 EST 2020
Received: (qmail 26222 invoked from network); 1 Feb 2020 12:24:27 -0000
Received: from unknown (HELO mx03.bss-wf.de) (postmaster@emdtgvmf.mail.selfhost.de@84.150.47.200)
  by mailout.selfhost.de with ESMTPA; 1 Feb 2020 12:24:27 -0000
Received: by mx03.bss-wf.de (Postfix, from userid 1000)
	id 2DF223DDC5; Sat,  1 Feb 2020 13:24:27 +0100 (CET)
X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on mx03
X-Spam-Level: 
X-Spam-Status: No, score=-1.0 required=5.0 tests=ALL_TRUSTED autolearn=ham
	autolearn_force=no version=3.4.2
Received: from 9pi.pala (p4FD3F2D5.dip0.t-ipconnect.de [79.211.242.213])
	by mx03.bss-wf.de (Postfix) with ESMTPSA id 6226E3DDAD
	for <9front@9front.org>; Sat,  1 Feb 2020 13:24:26 +0100 (CET)
Message-ID: <C544019E2842D505054CDED1920449AE@yourdomain.dom>
To: 9front@9front.org
Subject: upas/fs plumb modify for self-made flag changes
Date: Sat, 1 Feb 2020 13:24:25 +0100
From: theinicke@bss-wf.de
MIME-Version: 1.0
Content-Type: multipart/mixed;
	boundary="upas-htaovoypysyyrxxeqpimhdmjow"
List-ID: <9front.9front.org>
List-Help: <http://lists.9front.org>
X-Glyph: ➈
X-Bullshit: ISO-certified plugin-oriented factory GPU solution 

This is a multi-part message in MIME format.
--upas-htaovoypysyyrxxeqpimhdmjow
Content-Disposition: inline
Content-Type: text/plain; charset="US-ASCII"
Content-Transfer-Encoding: 7bit

Currently upas/fs plumbs modify messages only if the flag changes are made by another imap connection. If the flag changes are made within the running upas/fs no modify message is plumbed.

Assuming this is not intentionally and we want to get a modify plumb message after flags were changed in running upas/fs the attached patch fixes this by making sure to send a modify for pending changes upon mbox sync.

The modify could be plumbed earlier, but this way it costs us almost nothing and changed flags from outside upas/fs are noticed in the same time interval anyway.

--
Tobias Heinicke

--upas-htaovoypysyyrxxeqpimhdmjow
Content-Type: multipart/mixed;
	boundary="upas-umdudpmpojxyuvldpzcfdozagy"
Content-Disposition: inline

This is a multi-part message in MIME format.
--upas-umdudpmpojxyuvldpzcfdozagy
Content-Disposition: inline
Content-Type: text/plain; charset="US-ASCII"
Content-Transfer-Encoding: 7bit

from postmaster@9pi:
The following attachment had content that we can't
prove to be harmless.  To avoid possible automatic
execution, we changed the content headers.
The original header was:

	Content-Disposition: attachment; filename=upasfs.patch
	Content-Type: text/plain; charset="US-ASCII"
	Content-Transfer-Encoding: 7bit
--upas-umdudpmpojxyuvldpzcfdozagy
Content-Type: application/octet-stream
Content-Disposition: attachment; filename="upasfs.patch.suspect"

diff -r 8857059a1805 sys/src/cmd/upas/fs/fs.c
--- a/sys/src/cmd/upas/fs/fs.c	Fri Jan 31 09:25:39 2020 -0800
+++ b/sys/src/cmd/upas/fs/fs.c	Sat Feb 01 13:22:38 2020 +0100
@@ -1197,6 +1197,7 @@
 			mb->modflags(mb, m, f);
 		m->flags = f;
 		m->cstate |= Cidxstale;
+		m->cstate |= Cmod;
 	}
 	return nil;
 }
--upas-umdudpmpojxyuvldpzcfdozagy--