On 9/7/2021 12:48 PM, Derrick Stolee wrote: > On 9/4/2021 4:54 PM, Lénaïc Huard wrote: >> Hello, >> >> Please find hereafter my updated patchset to add support for systemd >> timers on Linux for the `git maintenance start` command. >> >> The only changes compared to the previous version are fixes for the >> two typos in a comment that Ramsay Jones pointed out [1] >> >> [1] https://lore.kernel.org/git/51246c10-fe0b-b8e5-cdc3-54bdc6c8054e@xxxxxxxxxxxxxxxxxxxx/ > > The changes in the most recent two versions look good to me. I recently tested the 'seen' branch for an unrelated reason, but found that the t7900-maintenance.sh test failed for me. It was during test 34, 'start and stop Linux/systemd maintenance' with the following issue: + systemd-analyze verify systemd/user/git-maintenance@.service Failed to create /user.slice/user-1000.slice/session-44.scope/init.scope control group: Permission denied Failed to initialize manager: Permission denied Now, this test has the prereq SYSTEMD_ANALYZE, but for some reason this later command fails for permission issues. I'm running Ubuntu, if that helps. Thanks, -Stolee