From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wm1-x332.google.com (mail-wm1-x332.google.com [IPv6:2a00:1450:4864:20::332]) by sourceware.org (Postfix) with ESMTPS id 84C343849ACF for ; Fri, 19 Apr 2024 19:11:34 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.2 sourceware.org 84C343849ACF Authentication-Results: sourceware.org; dmarc=pass (p=quarantine dis=none) header.from=adacore.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=adacore.com ARC-Filter: OpenARC Filter v1.0.0 sourceware.org 84C343849ACF Authentication-Results: server2.sourceware.org; arc=none smtp.remote-ip=2a00:1450:4864:20::332 ARC-Seal: i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1713553897; cv=none; b=vF4xgwDhU1q8TK2bWLYpAFEYUjovT9W6cd7T695EsI+xLjuJRv9b+R7WUw06zc8enZucDiZTIoGgiOL3zlhmezv7zW4Z74i07KmbHeo0w96YUrJAoy/8wf+qtP+L9w0M7eAoGs5UUdz3RmTDI6ngtVfSbOi1O0tJE5kCNEiI43Q= ARC-Message-Signature: i=1; a=rsa-sha256; d=sourceware.org; s=key; t=1713553897; c=relaxed/simple; bh=uZQR5gAJWxxTRomGWaAi+ND97WDdKx767+eNE447nz4=; h=DKIM-Signature:Date:From:To:Subject:Message-ID:MIME-Version; b=uJTLlI/gGeZpP1qws4DJyEdaVjzEM03/LGX7oe1taSczI46hOY/qddeblxz5N80DI+7vB6mjHJ2fkpXZQiI1lJ3NIAvMSe+i8CsGbWoJCHPtxPoBxGw3AJYkD+8GufMAzBUjYu2NPRXNVH1tj2dUL6LdcGb8nhGG2rI9C7QeSaI= ARC-Authentication-Results: i=1; server2.sourceware.org Received: by mail-wm1-x332.google.com with SMTP id 5b1f17b1804b1-417f5268b12so25194935e9.1 for ; Fri, 19 Apr 2024 12:11:34 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=adacore.com; s=google; t=1713553893; x=1714158693; darn=sourceware.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=9Qr6H2rGHJBT77arFG47kL01cXxq/qUcPfIr0ORwKcg=; b=ZdWJzvDDvzsC/EX+9P8Fv6aWaI2gct7aQRzO0Ee9ljoy7TdKYYHBRQpg64O2ggc3U1 Hm9nX8CQpY6se6xuTKd1w6GHCJZkn71z4CcKZCBtNuUpC7ym6zDPh8U/dE/mHh3OqxPy lPaf1PqXph/7/RR+YLpSOGPlk6rAwMbOmVVFuQr99rvmAX4w5JEXwFZ+tjC8B0/arOXQ k5DSHDzIG9Pt5241llpNUDg8SBDdrvyfKHDJWYkW4tFFO+ZSLTG2Fgo16ibXOHRozL1Q VcSTQ0BHHnx7A+4z3YeM7nPHlcEeZ0HWU2v4ReZ7gALycSTbhg5r0oGuVH28VfmtGwR2 ONmw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1713553893; x=1714158693; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=9Qr6H2rGHJBT77arFG47kL01cXxq/qUcPfIr0ORwKcg=; b=O0YvQStZFldtzbAciLMzH36a55slCULAQ1qoN+tKvzi9Q970ErKpR9H0msfJBDBe6Y 4WbOXURjpFo1xjlQfa6y4pe8T3rF0n4hmxM4/vkmDNmjbx4WvVpDCTR+ZMbO5OyctH0w SCPQmyTZ39fvfio5sNd1GdZqq1CJXR/h2DQw8Hb5kX/nd1nUZaomz0wXy+7qYn8Tks/X xeQHsDpX0anB2eNLBb9zgh/FQDf6JVkM/3XcI73KGF77Zc9yi/sWnaPG27cuU+xugcLq 8Kk9KSzqqxPRTSCIml8hQ5aN2+ccyWV767IoVLzDFFPO1HnuV/jWZJXf+lpT8DVUu2mh lTjg== X-Forwarded-Encrypted: i=1; AJvYcCUU8HRN2lqDkncqKjDe+FN2YmH4fRMSqzsLFbZsrpaIMyjSlr8ZQHTpOxG2ivp4VeVp2L+XHRE8kJNnDNViZeb2gr0= X-Gm-Message-State: AOJu0YygYQRZMfj5jORqB1qpDvma9Iw0edMzh22W78itanihBl+0omp4 JrK6d2Hj99P9uQakdGUU+KJP3cWzkoPgbcwU+R8UfbgjQQsAEy7B/1WZguJJ X-Google-Smtp-Source: AGHT+IFQFVd3LgDZ2c4H0C4IVnc48Tqm2Rz1N1iN3ghBPnz6q19cxzZxq1YtOTOLIxkG7B09M4dkRg== X-Received: by 2002:a05:600c:4ecb:b0:418:e7b6:21de with SMTP id g11-20020a05600c4ecb00b00418e7b621demr2649628wmq.23.1713553892743; Fri, 19 Apr 2024 12:11:32 -0700 (PDT) Received: from takamaka.gnat.com ([37.169.59.190]) by smtp.gmail.com with ESMTPSA id s9-20020a05600c45c900b004170e0aff7asm11229011wmo.35.2024.04.19.12.11.32 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 19 Apr 2024 12:11:32 -0700 (PDT) Received: by takamaka.gnat.com (Postfix, from userid 1000) id B445F8188D; Fri, 19 Apr 2024 12:11:26 -0700 (PDT) Date: Fri, 19 Apr 2024 12:11:26 -0700 From: Joel Brobecker To: "Frank Ch. Eigler" Cc: brobecker@adacore.com, gdb@sourceware.org, gcc@gcc.gnu.org Subject: Re: adacore git-hooks - daemon-mode email vs. systemd-logind linger Message-ID: References: <20240416105622.GD1423@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20240416105622.GD1423@redhat.com> X-Spam-Status: No, score=-3.8 required=5.0 tests=BAYES_00,DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE,SPF_HELO_NONE,SPF_PASS,TXREP autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on server2.sourceware.org List-Id: Hi Frank, > - just send the emails immediately, without the daemon stuff; if the > delays are there to try to sequentialize them, consider instead > getting the hooks to emit Message-Id:/In-Reply-To:/Date: headers to > let MUA's sort properly at reception We can certainly add a non-daemon mode to the hooks. I don't know when I'd have time to work on this enhancement, unfortunately. > - invoke the email sending wrapped in a "systemd-run --user" > deferred execution gadget, including a "sleep XXX" if you must > keep time-hope-based sequencing > > - move email stuff entirely out of the hooks; these repos are "public > property" anyhow, and we can put cron jobs in place elsewhere to > trigger email notifications about commits; heck, they could run the > hook code itself later, just feed it retrospective git-hook lines You can configure the hooks to turn emailing off completely. From memory, this triggers a warning message on the user end when they do the push, just to let them know to not expect emails. The downside of going this way, other than the minor annoyance above, is that you lose track of who did the push. Right now, the emails allows you to derive the information from the emails' from field. Thinking more long term, I think there are talks about using a more comprehensive system for source and contribution management, similar to products such as GitLab or GitHub. Those will likely come with they own set of tooling for things such as alerting, pre-commit checks, keeping track of who submitted, who reviewed and approved, and who pushed. That's why I tend to think that, for the moment, I'd go the simplest route overall: In this case, I think the simplest is to just have a way to disable the daemon mode, and just have the users wait. In the vast majority of cases, the wait will be a few seconds. Even with a patch-series of a 100 patches, it would mean a 1min40s wait. Very bearable. If you agree, we can create an issue for it on the git-hooks repoisitory on GitHub. -- Joel