From 06a32b319f065550efe0d2a9ff10ca6bdc32b893 Mon Sep 17 00:00:00 2001
From: Christoph Reiter <reiter.christoph@gmail.com>
Date: Sat, 23 Jul 2022 20:15:18 +0200
Subject: [PATCH] operon: hopefully better fix for flaky mtime tests

copy the mtime after we write everything, so there is no chance of
it changing before we note the initial value.
---
 quodlibet/operon/commands.py | 8 ++++----
 1 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/quodlibet/operon/commands.py b/quodlibet/operon/commands.py
index e0a5ef33a..af2dcfa8b 100644
--- a/quodlibet/operon/commands.py
+++ b/quodlibet/operon/commands.py
@@ -227,16 +227,16 @@ class EditCommand(Command):
         # write to tmp file
         fd, path = tempfile.mkstemp(suffix=".txt")
 
-        # XXX: copy mtime here so we can test for changes in tests by
-        # setting a out of date mtime on the source song file
-        copy_mtime(args[0], path)
-
         try:
             try:
                 os.write(fd, dump)
             finally:
                 os.close(fd)
 
+            # XXX: copy mtime here so we can test for changes in tests by
+            # setting a out of date mtime on the source song file
+            copy_mtime(args[0], path)
+
             # only parse the result if the editor returns 0 and the mtime has
             # changed
             old_mtime = mtime(path)
-- 
2.39.2

/diff/manifest.scm?id=c76bec3d93bc5008618fb0b9ff26f0e0f4323d18'>diff</a></td><td class='form'><form class='right' method='get' action='/guix/log/manifest.scm'>
<input type='hidden' name='id' value='c76bec3d93bc5008618fb0b9ff26f0e0f4323d18'/><select name='qt'>
<option value='grep'>log msg</option>
<option value='author'>author</option>
<option value='committer'>committer</option>
<option value='range'>range</option>
</select>
<input class='txt' type='search' size='10' name='q' value=''/>
<input type='submit' value='search'/>
</form>
</td></tr></table>
<div class='path'>path: <a href='/guix/log/?id=c76bec3d93bc5008618fb0b9ff26f0e0f4323d18'>root</a>/<a href='/guix/log/manifest.scm?id=c76bec3d93bc5008618fb0b9ff26f0e0f4323d18'>manifest.scm</a></div><div class='content'><table class='list nowrap'><tr class='nohover'><th class='left'>Age</th><th class='left'>Commit message (<a href='/guix/log/manifest.scm?id=c76bec3d93bc5008618fb0b9ff26f0e0f4323d18&amp;showmsg=1'>Expand</a>)</th><th class='left'>Author</th></tr>
<tr><td><span title='2024-11-11 07:28:34 +0100'>2024-11-11</span></td><td><a href='/guix/commit/manifest.scm?id=a84586dad60fe28c3b1832c197f1e0b1f6f12721'>maint: Add installer dependencies to the manifest.</a><span class='msg-avail'>...<span class='msg-tooltip'>* manifest.scm: Add guile-newt, guile-parted, guile-webutils.

Change-Id: Idcf46320d29c15f36da05f66e81b7779e37c1bf6
</span></span></td><td>Janneke Nieuwenhuizen</td></tr>
<tr><td><span title='2024-08-31 19:06:37 +0200'>2024-08-31</span></td><td><a href='/guix/commit/manifest.scm?id=9af235335e8f8baf7f434a76570a954999231879'>gnu: guix: Remove unnecessary dependencies.</a><span class='msg-avail'>...<span class='msg-tooltip'>This reverts 0fda0486523d67c2c464386c07a0c2800d8d8c20, instead moving
the dependencies needed for ‘make dist’ to ‘manifest.scm’.

Removing this extra dependencies make it simpler to build the ‘guix’
package in particular on less-capable systems like i586-gnu.

* gnu/packages/package-management.scm (guix)[native-inputs]: Remove
IMAGEMAGICK and PERL.  Use GRAPHVIZ-MINIMAL instead of GRAPHVIZ.
* manifest.scm: Use the full-blown ‘graphviz’ package; add ‘imagemagick’
and ‘perl’.

Reported-by: Janneke Nieuwenhuizen &lt;janneke@gnu.org&gt;
Change-Id: Ia9d05c699e3cc2f9a9235a67f8ec840c26b66a82
</span></span></td><td>Ludovic Courtès</td></tr>
<tr><td><span title='2024-06-24 08:51:43 -0400'>2024-06-24</span></td><td><a href='/guix/commit/manifest.scm?id=7f194852a46edfe47537d11bc39fb1dd20e18b37'>manifest: Streamline; add packages useful for patch review/submission.</a><span class='msg-avail'>...<span class='msg-tooltip'>* manifest.scm: Fix alternate command line invocation (which would not
consider 'manifest.scm').  Use specifications instead of packages.  Remove
perl (now in the guix package's native inputs).  Add b4, git, git:send-email,
mumi, nss-certs, openssl and patman to the manifest.

Change-Id: I49d92dda059856ce217cea9054a466a1754dcf94
Signed-off-by: Maxim Cournoyer &lt;maxim.cournoyer@gmail.com&gt;
</span></span></td><td>Maxim Cournoyer</td></tr>
<tr><td><span title='2024-04-02 23:28:58 +0200'>2024-04-02</span></td><td><a href='/guix/commit/manifest.scm?id=049273bb8c1ea3d6402db646312acdc4abf4678a'>maint: Add perl to the manifest.</a><span class='msg-avail'>...<span class='msg-tooltip'>* manifest.scm: Add perl.

Change-Id: Ie70cea1eb4d62f2a649292209391a92abdae2288
</span></span></td><td>Janneke Nieuwenhuizen</td></tr>
<tr><td><span title='2024-01-31 09:07:25 +0100'>2024-01-31</span></td><td><a href='/guix/commit/manifest.scm?id=66dd088d3c3404f5cbe2d4d4c5e8f93d184703b0'>maint: Add gnupg to the manifest.</a><span class='msg-avail'>...<span class='msg-tooltip'>* manifest.scm: Add gnupg.

Change-Id: Id5e90f87e5c3a07aa3e49afa97cc0b6d3ce71b41
</span></span></td><td>Ludovic Courtès</td></tr>