[Buildroot] [Bug 10176] New: Rsyslog's S01logging is deleted by Busybox.mk from TARGET

[Buildroot] [Bug 10176] New: Rsyslog's S01logging is deleted by Busybox.mk from TARGET

[bugzilla at busybox.net]

https://bugs.busybox.net/show_bug.cgi?id=10176

            Bug ID: 10176
           Summary: Rsyslog's S01logging is deleted by Busybox.mk from
                    TARGET
           Product: buildroot
           Version: 2017.02
          Hardware: All
                OS: Linux
            Status: NEW
          Severity: normal
          Priority: P5
         Component: Other
          Assignee: unassigned at buildroot.uclibc.org
          Reporter: mail at kkrach.de
                CC: buildroot at uclibc.org
  Target Milestone: ---

Created attachment 7166
  --> https://bugs.busybox.net/attachment.cgi?id=7166&action=edit
Do not remove S01logging

It seems that when no SYSLOGD is configured in Busybox, the file
$(TARGET_DIR)/etc/init.d/S01logging is removed during installation of Busybox.
But in case that rsyslog also has installed a S01logging, changing the Busybox
configuration will cause that the startscript of rsyslog is removed.

I ve checked busybox.mk and S01logging of rsyslog - and both seem to have the
problem also in the current Buildroot master. I'm working with 2017.02 and do
reproduce it only there.

Further more, I ve attached the patch with solves the problem for me. Normally,
the packages do not remove their startscript - so I ve removed the
startscript-removal as well.

BR, Karl

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Buildroot] [Bug 10176] Rsyslog's S01logging is deleted by Busybox.mk from TARGET

[bugzilla at busybox.net]

https://bugs.busybox.net/show_bug.cgi?id=10176

--- Comment #1 from Arnout Vandecappelle <arnout@mind.be> ---
Your analysis seems correct. Could you send the patch to the mailing list, as
per the contribution guidelines? See
https://buildroot.org/downloads/manual/manual.html#submitting-patches

You also need to do the following.

- Add your Signed-off-by line. This is a short way for you to assert that you
are entitled to contribute the patch under Buildroot's GPL license.  See 
http://elinux.org/Developer_Certificate_Of_Origin for more details.

- Make it a git-formatted patch.

- Add a note "Fixes https://bugs.busybox.net/show_bug.cgi?id=10176"

Thanks!

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Buildroot] [Bug 10176] Rsyslog's S01logging is deleted by Busybox.mk from TARGET

[bugzilla at busybox.net]

https://bugs.busybox.net/show_bug.cgi?id=10176

Thomas Petazzoni <thomas.petazzoni@free-electrons.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
             Status|NEW                         |RESOLVED

--- Comment #2 from Thomas Petazzoni <thomas.petazzoni@free-electrons.com> ---
Fixed by
https://git.buildroot.org/buildroot/commit/?id=84e835ea9261b3e844f1a18489dd89253e3eb839.
Thanks for reporting the bug in the first place, and sorry for the long delay!

-- 
You are receiving this mail because:
You are on the CC list for the bug.