Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Joshua Lock <josh@linux.intel.com>
To: openembedded-core@lists.openembedded.org
Subject: Re: [PATCH 0/4] Add alsa-state from OE Classic v3
Date: Fri, 27 Jan 2012 10:49:06 -0800	[thread overview]
Message-ID: <4F22F1A2.7040602@linux.intel.com> (raw)
In-Reply-To: <E10B6388-7C66-4790-9666-A7C97D9EF238@dominion.thruhere.net>



On 27/01/12 00:00, Koen Kooi wrote:
>
> Op 27 jan. 2012, om 00:09 heeft Otavio Salvador het volgende geschreven:
>
>> On Tue, Jan 24, 2012 at 21:15, Joshua Lock<josh@linux.intel.com>  wrote:
>> I can't see any initscript, so I think switching to what's provided by alsa-utils would mean losing state saving for non-systemd folk but systemd users can probably just set VIRTUAL-RUNTIME_alsa-state = "alsa-utils-alsactl".
>>
>> You still needs the state. So I think we'd need to have a specific -systemd version of it when the configs and the units.

Quite right, we should separate the state files. I wasn't thinking 
straight there.

>
> I was thinking of splitting the existing recipe into alsa-state-conf for the conf files and keep the init stuff in ${PN}. As a follow on patch we can have alsa-utils-alsactl RRECOMMEND alsa-state-conf.
>
> thoughts?

Sounds good to me. I'd like to see what Martin thinks, as he rose the 
initial concern re: the VIRTUAL_RUNTIME. Hopefully SHR would also like 
the state files separate and more centrally maintained.

Cheers,
Joshua
-- 
Joshua Lock
         Yocto Project "Johannes factotum"
         Intel Open Source Technology Centre



      reply	other threads:[~2012-01-27 18:56 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-01-13 19:10 [PATCH 0/4] Add alsa-state from OE Classic v3 Joshua Lock
2012-01-13 19:10 ` [PATCH 1/4] task-base: move default definition of VIRTUAL-RUNTIME_apm Joshua Lock
2012-01-13 19:10 ` [PATCH 2/4] alsa-state: add alsa-state from oe classic Joshua Lock
2012-01-13 19:10 ` [PATCH 3/4] alsa-state: move state files to localstatedir Joshua Lock
2012-01-13 19:10 ` [PATCH 4/4] task-base: add VIRTUAL-RUNTIME_alsa-state to task-base-alsa Joshua Lock
2012-01-17 19:40 ` [PATCH 0/4] Add alsa-state from OE Classic v3 Saul Wold
2012-01-24 15:02 ` Koen Kooi
2012-01-24 23:15   ` Joshua Lock
2012-01-26 23:09     ` Otavio Salvador
2012-01-27  8:00       ` Koen Kooi
2012-01-27 18:49         ` Joshua Lock [this message]

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=4F22F1A2.7040602@linux.intel.com \
    --to=josh@linux.intel.com \
    --cc=openembedded-core@lists.openembedded.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.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox