* [Buildroot] [Bug 6128] New: [patch] Fix S01logging pidfile handling
@ 2013-03-20 13:58 bugzilla at busybox.net
2013-03-20 14:00 ` [Buildroot] [Bug 6128] " bugzilla at busybox.net
` (3 more replies)
0 siblings, 4 replies; 7+ messages in thread
From: bugzilla at busybox.net @ 2013-03-20 13:58 UTC (permalink / raw)
To: buildroot
https://bugs.busybox.net/show_bug.cgi?id=6128
Summary: [patch] Fix S01logging pidfile handling
Product: buildroot
Version: unspecified
Platform: PC
OS/Version: Linux
Status: NEW
Severity: normal
Priority: P5
Component: Other
AssignedTo: unassigned at buildroot.uclibc.org
ReportedBy: arved at arved.at
CC: buildroot at uclibc.org
Estimated Hours: 0.0
There are two issues with S01logging
1. The initscript relies on the daemon creating a Pidfile, but the relevant
option has been disabled in the busybox-1.20.x-config.
2. The name contains a typo syslogd vs. syslog.
This is fixed with the attached patch.
--
Configure bugmail: https://bugs.busybox.net/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Buildroot] [Bug 6128] [patch] Fix S01logging pidfile handling
2013-03-20 13:58 [Buildroot] [Bug 6128] New: [patch] Fix S01logging pidfile handling bugzilla at busybox.net
@ 2013-03-20 14:00 ` bugzilla at busybox.net
2013-03-20 14:29 ` bugzilla at busybox.net
` (2 subsequent siblings)
3 siblings, 0 replies; 7+ messages in thread
From: bugzilla at busybox.net @ 2013-03-20 14:00 UTC (permalink / raw)
To: buildroot
https://bugs.busybox.net/show_bug.cgi?id=6128
--- Comment #1 from Tilman Keskin?z <arved@arved.at> 2013-03-20 14:00:49 UTC ---
Created attachment 4820
--> https://bugs.busybox.net/attachment.cgi?id=4820
Fix syslog initscript
--
Configure bugmail: https://bugs.busybox.net/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Buildroot] [Bug 6128] [patch] Fix S01logging pidfile handling
2013-03-20 13:58 [Buildroot] [Bug 6128] New: [patch] Fix S01logging pidfile handling bugzilla at busybox.net
2013-03-20 14:00 ` [Buildroot] [Bug 6128] " bugzilla at busybox.net
@ 2013-03-20 14:29 ` bugzilla at busybox.net
2013-03-25 11:09 ` Tilman Keskinöz
2013-03-20 15:41 ` bugzilla at busybox.net
2013-05-26 11:37 ` bugzilla at busybox.net
3 siblings, 1 reply; 7+ messages in thread
From: bugzilla at busybox.net @ 2013-03-20 14:29 UTC (permalink / raw)
To: buildroot
https://bugs.busybox.net/show_bug.cgi?id=6128
--- Comment #2 from Thomas Petazzoni <thomas.petazzoni@free-electrons.com> 2013-03-20 14:30:20 UTC ---
Thanks. Can you send this as two proper separate patches on the Buildroot
mailing list?
--
Configure bugmail: https://bugs.busybox.net/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Buildroot] [Bug 6128] [patch] Fix S01logging pidfile handling
2013-03-20 13:58 [Buildroot] [Bug 6128] New: [patch] Fix S01logging pidfile handling bugzilla at busybox.net
2013-03-20 14:00 ` [Buildroot] [Bug 6128] " bugzilla at busybox.net
2013-03-20 14:29 ` bugzilla at busybox.net
@ 2013-03-20 15:41 ` bugzilla at busybox.net
2013-05-26 11:37 ` bugzilla at busybox.net
3 siblings, 0 replies; 7+ messages in thread
From: bugzilla at busybox.net @ 2013-03-20 15:41 UTC (permalink / raw)
To: buildroot
https://bugs.busybox.net/show_bug.cgi?id=6128
--- Comment #3 from Tilman Keskin?z <arved@arved.at> 2013-03-20 15:42:06 UTC ---
done, mail waits now for moderator approval
--
Configure bugmail: https://bugs.busybox.net/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Buildroot] [Bug 6128] [patch] Fix S01logging pidfile handling
2013-03-20 14:29 ` bugzilla at busybox.net
@ 2013-03-25 11:09 ` Tilman Keskinöz
2013-03-25 12:25 ` Thomas Petazzoni
0 siblings, 1 reply; 7+ messages in thread
From: Tilman Keskinöz @ 2013-03-25 11:09 UTC (permalink / raw)
To: buildroot
* bugzilla [Wed, 20 Mar 2013 14:29:36 +0000 (UTC)]:
> https://bugs.busybox.net/show_bug.cgi?id=6128
>
> --- Comment #2 from Thomas Petazzoni <thomas.petazzoni@free-electrons.com> 2013-03-20 14:30:20 UTC ---
> Thanks. Can you send this as two proper separate patches on the Buildroot
> mailing list?
>
-------------- next part --------------
--- S01logging.orig 2013-03-20 14:42:18.313615967 +0100
+++ S01logging 2013-03-20 14:42:35.373615539 +0100
@@ -6,13 +6,13 @@
case "$1" in
start)
echo -n "Starting logging: "
- start-stop-daemon -S -q -p /var/run/syslog.pid --exec /sbin/syslogd -- -m 0
+ start-stop-daemon -S -q -p /var/run/syslogd.pid --exec /sbin/syslogd -- -m 0
start-stop-daemon -S -q -p /var/run/klogd.pid --exec /sbin/klogd
echo "OK"
;;
stop)
echo -n "Stopping logging: "
- start-stop-daemon -K -q -p /var/run/syslog.pid
+ start-stop-daemon -K -q -p /var/run/syslogd.pid
start-stop-daemon -K -q -p /var/run/klogd.pid
echo "OK"
;;
-------------- next part --------------
--- busybox-1.20.x.config.orig 2013-03-20 14:42:31.297615642 +0100
+++ busybox-1.20.x.config 2013-03-20 14:42:51.045615146 +0100
@@ -41,7 +41,7 @@
CONFIG_FEATURE_CLEAN_UP=y
CONFIG_FEATURE_UTMP=y
CONFIG_FEATURE_WTMP=y
-# CONFIG_FEATURE_PIDFILE is not set
+CONFIG_FEATURE_PIDFILE=y
CONFIG_FEATURE_SUID=y
# CONFIG_FEATURE_SUID_CONFIG is not set
# CONFIG_FEATURE_SUID_CONFIG_QUIET is not set
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: OpenPGP digital signature
URL: <http://lists.busybox.net/pipermail/buildroot/attachments/20130325/3d5bb88f/attachment-0001.asc>
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Buildroot] [Bug 6128] [patch] Fix S01logging pidfile handling
2013-03-25 11:09 ` Tilman Keskinöz
@ 2013-03-25 12:25 ` Thomas Petazzoni
0 siblings, 0 replies; 7+ messages in thread
From: Thomas Petazzoni @ 2013-03-25 12:25 UTC (permalink / raw)
To: buildroot
Dear Tilman Keskin?z,
On Mon, 25 Mar 2013 12:09:00 +0100, Tilman Keskin?z wrote:
> * bugzilla [Wed, 20 Mar 2013 14:29:36 +0000 (UTC)]:
> > https://bugs.busybox.net/show_bug.cgi?id=6128
> >
> > --- Comment #2 from Thomas Petazzoni
> > <thomas.petazzoni@free-electrons.com> 2013-03-20 14:30:20 UTC ---
> > Thanks. Can you send this as two proper separate patches on the
> > Buildroot mailing list?
Thanks, but unfortunately, those patches can't be applied as is. They
should be Git patches, with a commit log that explains why the patch
is needed and what it is doing, a Signed-off-by line, and sent inline
(and not as attachements) using git send-email.
Could you do this?
Thanks a lot for your contribution!
Thomas
--
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
^ permalink raw reply [flat|nested] 7+ messages in thread
* [Buildroot] [Bug 6128] [patch] Fix S01logging pidfile handling
2013-03-20 13:58 [Buildroot] [Bug 6128] New: [patch] Fix S01logging pidfile handling bugzilla at busybox.net
` (2 preceding siblings ...)
2013-03-20 15:41 ` bugzilla at busybox.net
@ 2013-05-26 11:37 ` bugzilla at busybox.net
3 siblings, 0 replies; 7+ messages in thread
From: bugzilla at busybox.net @ 2013-05-26 11:37 UTC (permalink / raw)
To: buildroot
https://bugs.busybox.net/show_bug.cgi?id=6128
Samuel Martin <s.martin49@gmail.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution| |FIXED
--- Comment #4 from Samuel Martin <s.martin49@gmail.com> 2013-05-26 11:36:07 UTC ---
The S01logging script typo has been merged.
The pidfile management remains deferred to start-stop-daemon, so the change in
the busybox configs is not needed; though any user can enable it.
--
Configure bugmail: https://bugs.busybox.net/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2013-05-26 11:37 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-03-20 13:58 [Buildroot] [Bug 6128] New: [patch] Fix S01logging pidfile handling bugzilla at busybox.net
2013-03-20 14:00 ` [Buildroot] [Bug 6128] " bugzilla at busybox.net
2013-03-20 14:29 ` bugzilla at busybox.net
2013-03-25 11:09 ` Tilman Keskinöz
2013-03-25 12:25 ` Thomas Petazzoni
2013-03-20 15:41 ` bugzilla at busybox.net
2013-05-26 11:37 ` bugzilla at busybox.net
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.