From: Jean-Baptiste Note <jean-baptiste.note@m4x.org>
To: "Ludovic Courtès" <ludo@gnu.org>
Cc: 70677-done@debbugs.gnu.org
Subject: [bug#70677] [PATCH 0/2] Improve syslog service flexibility
Date: Mon, 23 Dec 2024 23:34:30 +0100 [thread overview]
Message-ID: <87jzbqxc3t.fsf@m4x.org> (raw)
In-Reply-To: <874j2u9vag.fsf@gnu.org> ("Ludovic Courtès"'s message of "Mon, 23 Dec 2024 18:13:59 +0100")
[-- Attachment #1: Type: text/plain, Size: 1093 bytes --]
Hi Ludovic,
First, thank you for devoting some previous Christmas time to this. I'm
reading this out of office, and i'm suddenly assailed by a doubt. It's
been a while, but I think the second patch, amending the commandline,
was completely untested and completely wrong.
Indeed the string "-f syslog.conf" is not a proper argument and won't be
parsed as such, it should probably be something like "-f"
#$(syslog.conf) (two separate arguments).
I can't easily assess the situation from where I am, or access a
possibly updated patch version from work, but maybe the continuous
integration is showing the problem.
I'm very sorry for the situation, and I think you may want to revert
this part of the series to avoid nuisances to people who would do a
system upgrade.
I will send a fixed patch as soon as possible, which may take a few
days. The problem doesn't show upon system build but on system reboot,
and manifests itself with an early fail during the herd startup
sequence.
I'm very sorry about the situation, and even more so about wasting your
time.
Kind regards,
Jean-Baptiste
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 869 bytes --]
next prev parent reply other threads:[~2024-12-23 22:37 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-04-30 15:34 [bug#70677] [PATCH 0/2] Improve syslog service flexibility Jean-Baptiste Note
2024-04-30 15:36 ` [bug#70677] [PATCH 1/2] services: syslog: Add extra-options argument to syslog service Jean-Baptiste Note
2024-04-30 15:36 ` [bug#70677] [PATCH 2/2] services: syslog: Adjust service for rsyslog compatibility Jean-Baptiste Note
2024-12-23 17:13 ` bug#70677: [PATCH 0/2] Improve syslog service flexibility Ludovic Courtès
2024-12-23 22:34 ` Jean-Baptiste Note [this message]
2024-12-25 17:14 ` [bug#75091] [PATCH] services: syslog: fix configuration file argument 45mg
2024-12-25 21:49 ` bug#75091: " Ludovic Courtès
2024-12-26 11:30 ` [bug#70677] [PATCH 0/2] Improve syslog service flexibility Ludovic Courtès
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://guix.gnu.org/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87jzbqxc3t.fsf@m4x.org \
--to=jean-baptiste.note@m4x.org \
--cc=70677-done@debbugs.gnu.org \
--cc=ludo@gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/guix.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).