<div><br></div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, May 13, 2019 at 3:10 AM Luca Boccassi <<a href="mailto:luca.boccassi@gmail.com">luca.boccassi@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Fri, 2019-05-10 at 09:45 -0700, Khem Raj wrote:<br>
> On Fri, May 10, 2019 at 5:10 AM <<br>
> <a href="mailto:luca.boccassi@gmail.com" target="_blank">luca.boccassi@gmail.com</a><br>
> > wrote:<br>
> > From: Luca Boccassi <<br>
> > <a href="mailto:luca.boccassi@microsoft.com" target="_blank">luca.boccassi@microsoft.com</a><br>
> > ><br>
> > <br>
> > portable services have been declared production ready with v241, so<br>
> > enable<br>
> > by default and install the files in the main package.<br>
> > <br>
> <br>
> what is the rationale to enable it by default ?<br>
<br>
Hi,<br>
<br>
commit ada8a5d9175b disabled it by default because it was experimental:<br>
"Add a new PACKAGECONFIG item 'portabled'. Default to disable it<br>
because it's still experimental, according to the announcement letter."<br>
<br>
Now it's been declared as stable by upstream, and also given machined<br>
and nspawn are enabled by default I don't see a reason why this<br>
shouldn't be as well, as they all provide the same "kind" of<br>
feature/functionality, albeit on a different level.<br>
</blockquote><div dir="auto"><br></div><div dir="auto">Ok being stable is a good thing I agree but I still fail to understand why make it default I am for disabling nspawn and machined as well </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
> > Signed-off-by: Luca Boccassi <<br>
> > <a href="mailto:luca.boccassi@microsoft.com" target="_blank">luca.boccassi@microsoft.com</a><br>
> > ><br>
> > ---<br>
> > v2: Just enable by default, and let the files be installed in the<br>
> > main package<br>
> > as requested by reviewers.<br>
> > <br>
> > meta/recipes-core/systemd/<a href="http://systemd_242.bb" rel="noreferrer" target="_blank">systemd_242.bb</a> | 1 +<br>
> > 1 file changed, 1 insertion(+)<br>
> > <br>
> > diff --git a/meta/recipes-core/systemd/<a href="http://systemd_242.bb" rel="noreferrer" target="_blank">systemd_242.bb</a><br>
> > b/meta/recipes-core/systemd/<a href="http://systemd_242.bb" rel="noreferrer" target="_blank">systemd_242.bb</a><br>
> > index 73e03c7a77..f15d125bf8 100644<br>
> > --- a/meta/recipes-core/systemd/<a href="http://systemd_242.bb" rel="noreferrer" target="_blank">systemd_242.bb</a><br>
> > +++ b/meta/recipes-core/systemd/<a href="http://systemd_242.bb" rel="noreferrer" target="_blank">systemd_242.bb</a><br>
> > @@ -81,6 +81,7 @@ PACKAGECONFIG ??= " \<br>
> > nss \<br>
> > nss-mymachines \<br>
> > nss-resolve \<br>
> > + portabled \<br>
> > quotacheck \<br>
> > randomseed \<br>
> > resolved \<br>
> > --<br>
> > 2.20.1<br>
> > <br>
> > --<br>
> > _______________________________________________<br>
> > Openembedded-core mailing list<br>
> > <a href="mailto:Openembedded-core@lists.openembedded.org" target="_blank">Openembedded-core@lists.openembedded.org</a><br>
> > <br>
> > <a href="http://lists.openembedded.org/mailman/listinfo/openembedded-core" rel="noreferrer" target="_blank">http://lists.openembedded.org/mailman/listinfo/openembedded-core</a><br>
> > <br>
-- <br>
Kind regards,<br>
Luca Boccassi<br>
<br>
</blockquote></div></div>