From: Phil Blundell <pb@pbcl.net>
To: Khem Raj <raj.khem@gmail.com>
Cc: openembedded-core@lists.openembedded.org
Subject: Re: [PATCH 2/4] rootfs: Do not uninstall update-rc.d
Date: Mon, 07 Sep 2015 22:48:10 +0100 [thread overview]
Message-ID: <1441662490.1680.82.camel@pbcl.net> (raw)
In-Reply-To: <EC219B31-F312-41D6-81C9-D13213CB2759@gmail.com>
On Mon, 2015-09-07 at 14:15 -0700, Khem Raj wrote:
> > On Sep 7, 2015, at 12:36 PM, Phil Blundell <pb@pbcl.net> wrote:
> >
> > On Sun, 2015-09-06 at 15:25 +0000, Khem Raj wrote:
> >> update-rc.d is now needed by systemd to interact with sysv
> >> scripts, so if we have a mixed system, then we can not uninstall
> >> update-rc.d as it is going to break systemd functionality
> >
> > The description above implies that you will only uninstall update-rc.d
> > if systemd is present, but that doesn't seem to be what the patch does.
>
> I meant to not say its only for systemd. but that systemd now needs it.
> so update-rc.d is no longer forcefully removed from image depchain.
Well, right. But wouldn't it be better only to leave it in there if
it's actually necessary, i.e. if you have systemd and you also have sysv
scripts and the rootfs isn't read-only? As it stands, your patch will
cause update-rc.d to be left around for people who aren't using systemd,
which seems like a retrograde step.
p.
next prev parent reply other threads:[~2015-09-07 21:48 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-09-06 15:25 [PATCH 0/4] Systemd Upgrade Khem Raj
2015-09-06 15:25 ` [PATCH 1/4] busybox: Enable getopt applet Khem Raj
2015-09-06 15:25 ` [PATCH 2/4] rootfs: Do not uninstall update-rc.d Khem Raj
2015-09-07 19:36 ` Phil Blundell
2015-09-07 21:15 ` Khem Raj
2015-09-07 21:48 ` Phil Blundell [this message]
2015-09-08 7:03 ` Khem Raj
2015-09-08 19:38 ` Khem Raj
2015-09-10 15:48 ` Richard Purdie
2015-09-10 23:35 ` Khem Raj
2015-09-10 23:39 ` Khem Raj
2015-09-06 15:25 ` [PATCH 3/4] systemd: Upgrade 219 -> 225 Khem Raj
2015-09-06 15:25 ` [PATCH 4/4] systemd: Implement OE-Specific systemd-sysv-install Khem Raj
2015-09-07 19:26 ` [PATCH 0/4] Systemd Upgrade Richard Purdie
2015-09-07 20:53 ` Khem Raj
2015-09-07 20:56 ` Richard Purdie
2015-09-07 21:00 ` Khem Raj
2015-09-07 21:09 ` Richard Purdie
2015-09-07 21:12 ` Khem Raj
2015-09-07 21:15 ` Richard Purdie
2015-09-08 10:31 ` Burton, Ross
2015-09-08 18:12 ` Khem Raj
2015-09-08 18:09 ` Khem Raj
2015-09-09 20:13 ` Burton, Ross
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
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1441662490.1680.82.camel@pbcl.net \
--to=pb@pbcl.net \
--cc=openembedded-core@lists.openembedded.org \
--cc=raj.khem@gmail.com \
/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.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox