Thread (4 messages) 4 messages, 4 authors, 2021-08-19

Re: What's cooking in git.git (Aug 2021, #06; Mon, 16)

From: Derrick Stolee <hidden>
Date: 2021-08-17 17:12:59

On 8/17/2021 12:47 PM, Junio C Hamano wrote:
Phillip Wood [off-list ref] writes:
...
quoted
quoted
* lh/systemd-timers (2021-07-02) 3 commits
  - maintenance: add support for systemd timers on Linux
  - maintenance: `git maintenance run` learned `--scheduler=<scheduler>`
  - cache.h: Introduce a generic "xdg_config_home_for(…)" function
  "git maintenance" scheduler learned to use systemd timers as a
  possible backend.
  Waiting for reviews.
I'd really like to see this merged, there was some discussion a couple
of weeks ago (cf [ref]), it 
petered out but no one seemed to be objecting to merging it.
Thanks for paying attention to this item.

My impression was that everybody likes the ability to use systemd
timers as an alternative to cron, but as Derrick summarised in the
message you pointed at, I think the implementation is not quite
there.

So "Waiting for reviews" is a mislabel; we are waiting for a reroll.
I think a lot of our comments were "We would hold an established
contributor to a higher standard, but this is sufficient for a new
contributor. The rest can be forward-fixed." We also have not heard
from the original contributor since that discussion.

Perhaps enough time has passed that we can expect Lénaïc will not
get back to us, I could give an attempt to forward-fix the comments
with a patch series on top of these. We could then hold
lh/systemd-timers in 'next' (or even 'seen') until it can be joined
with those fixes.

If preferred, I could restart the series by inserting some patches
and editing some of the ones in the list currently.

Thanks,
-Stolee
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help