Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Radu Moisan <radu.moisan@intel.com>
To: Samuel Stirtzel <s.stirtzel@googlemail.com>
Cc: Patches and discussions about the oe-core layer
	<openembedded-core@lists.openembedded.org>
Subject: Re: [PATCH v4] systemd: set default.target to multi-user.target
Date: Tue, 02 Apr 2013 16:35:31 +0300	[thread overview]
Message-ID: <515ADEA3.5030800@intel.com> (raw)
In-Reply-To: <CAGiJk9chnOh7F9koDo2kEP5rjJ97kZ9bL6uEKkWYegS9g4AUnw@mail.gmail.com>


On 04/02/2013 04:02 PM, Samuel Stirtzel wrote:
> 2013/4/2 Radu Moisan <radu.moisan@intel.com>:
>> This fixes a service dependency issue;
>> While graphical.target is the default mode, systemd
>> will try to start display-manager.service which is not
>> available.
>>
>> For xserver-nodm-init we would then have something like:
>> inherit update-alternatives
>> ALTERNATIVE_${PN} = "systemd-def-target"
>> ALTERNATIVE_TARGET[systemd-def-target] = "${systemd_unitdir}/system/graphical.target"
>> ALTERNATIVE_LINK_NAME[systemd-def-target] = "${systemd_unitdir}/system/default.target"
>> ALTERNATIVE_PRIORITY[systemd-def-target] ?= "10"
>>
>> Signed-off-by: Radu Moisan <radu.moisan@intel.com>
>> ---
>>   meta/recipes-core/systemd/systemd_199.bb |    2 ++
>>   1 file changed, 2 insertions(+)
>>
>> diff --git a/meta/recipes-core/systemd/systemd_199.bb b/meta/recipes-core/systemd/systemd_199.bb
>> index ba1d133..bf1eb39 100644
>> --- a/meta/recipes-core/systemd/systemd_199.bb
>> +++ b/meta/recipes-core/systemd/systemd_199.bb
>> @@ -248,6 +248,7 @@ update-alternatives --install ${base_sbindir}/halt halt ${base_bindir}/systemctl
>>   update-alternatives --install ${base_sbindir}/reboot reboot ${base_bindir}/systemctl 300
>>   update-alternatives --install ${base_sbindir}/shutdown shutdown ${base_bindir}/systemctl 300
>>   update-alternatives --install ${base_sbindir}/poweroff poweroff ${base_bindir}/systemctl 300
>> +update-alternatives --install ${systemd_unitdir}/system/default.target systemd-def-target ${systemd_unitdir}/system/multi-user.target 1
>>   }
>>
>>   pkg_prerm_systemd () {
>> @@ -256,6 +257,7 @@ update-alternatives --remove halt ${base_bindir}/systemctl
>>   update-alternatives --remove reboot ${base_bindir}/systemctl
>>   update-alternatives --remove shutdown ${base_bindir}/systemctl
>>   update-alternatives --remove poweroff ${base_bindir}/systemctl
>> +update-alternatives --remove systemd-def-target ${systemd_unitdir}/system/multi-user.target
>>   }
>>
>>   pkg_postinst_udev-hwdb () {
>> --
>> 1.7.9.5
>>
> Reliving a dejavu?
>
> This was already rejected before see [1], hope you remember this.
>
> [1] http://lists.linuxtogo.org/pipermail/openembedded-core/2013-January/034806.html
It was not rejected, have you seen a nack from Ross? Actually I just 
checked a few days ago with Ross and the reason it didn't merge was 
because it didn't apply anymore. And because systemd upgrade to v199 was 
pending I waited for that one to get pulled in. So now with systemd_199 
in, here is my rebased v3 as well.

There is a bug filled for this problem Bug#3816, do you have a better 
solution in mind?

Radu



  reply	other threads:[~2013-04-02 13:52 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-04-02 12:54 [PATCH v4] systemd: set default.target to multi-user.target Radu Moisan
2013-04-02 13:02 ` Samuel Stirtzel
2013-04-02 13:35   ` Radu Moisan [this message]
2013-04-02 14:15     ` Samuel Stirtzel
2013-04-03 14:15       ` Burton, Ross
2013-04-03 14:51         ` Samuel Stirtzel
2013-04-03 15:24           ` Richard Purdie
2013-04-04  6:24             ` Samuel Stirtzel
2013-04-03 15:27           ` Burton, Ross
2013-04-04  6:59             ` Samuel Stirtzel
2013-04-04  7:20               ` Martin Jansa
2013-04-04  7:37             ` Koen Kooi
2013-04-04  8:12               ` Samuel Stirtzel
2013-04-04 14:30               ` Burton, Ross
2013-04-04 15:17                 ` Andreas Müller
2013-04-04 15:20                   ` Burton, Ross
2013-04-04 15:58                 ` Koen Kooi

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=515ADEA3.5030800@intel.com \
    --to=radu.moisan@intel.com \
    --cc=openembedded-core@lists.openembedded.org \
    --cc=s.stirtzel@googlemail.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