<html>
<head>
<meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<div class="moz-cite-prefix">On 08/07/2013 10:18 PM, Chris Larson
wrote:<br>
</div>
<blockquote
cite="mid:CABcZAN=kKF1cLeCR+SMenSBG2bRxVd==GR727XN-h72FW2ZT2w@mail.gmail.com"
type="cite">
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
<div dir="ltr">
<div class="gmail_extra"><br>
<div class="gmail_quote">On Wed, Aug 7, 2013 at 1:08 AM, <span
dir="ltr"><<a moz-do-not-send="true"
href="mailto:Qi.Chen@windriver.com" target="_blank">Qi.Chen@windriver.com</a>></span>
wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0
.8ex;border-left:1px #ccc solid;padding-left:1ex">
<div id=":vd" style="overflow:hidden">From: Chen Qi <<a
moz-do-not-send="true"
href="mailto:Qi.Chen@windriver.com">Qi.Chen@windriver.com</a>><br>
<br>
The init script for irda writes configuration items to
/etc/sysconfig/irda<br>
if that file is not available in system. But it's
actually not necessary,<br>
the behavior doesn't change whether the init script
writes to the file or not.<br>
<br>
Considering it issues error messages in case of a
read-only rootfs, I delete<br>
the writing process.<br>
<br>
[YOCTO #4103]<br>
[YOCTO #4886]<br>
<br>
Signed-off-by: Chen Qi <<a moz-do-not-send="true"
href="mailto:Qi.Chen@windriver.com">Qi.Chen@windriver.com</a>></div>
</blockquote>
</div>
<br>
This is a start, but is incomplete, from what I can tell. As
mentioned in the other thread, the startup script shouldn't be
using /etc/sysconfig/ at all, in any form. We use
/etc/default/ for our startup script config files, not
/etc/sysconfig/. </div>
</div>
</blockquote>
Yeah, I agree with you.<br>
But when I checked the irda source code, I saw that it actually
could ship its own init script (irda-utils-xxx/etc/irda.rc). And I
think our init script is derived from this one. In its own init
script, /etc/sysconfig is used.<br>
(Of course, I still think /etc/default is a better location for its
configuration file.)<br>
<br>
<br>
<blockquote
cite="mid:CABcZAN=kKF1cLeCR+SMenSBG2bRxVd==GR727XN-h72FW2ZT2w@mail.gmail.com"
type="cite">
<div dir="ltr">
<div class="gmail_extra">Also, the purpose of this block was
clearly to implement a default configuration, yet the recipe
isn't altered to ship a default configuration to provide
equivalent functionality.<br>
</div>
</div>
</blockquote>
In its source code, there's a default configuration file, and we
don't use that. I think there might be a reason.<br>
So I'm not sure about this one.<br>
<br>
If you have a patch to fix the irda issue properly, send it out and
I'll rebase my remote branch and drop this one.<br>
<br>
Best Regards,<br>
Chen Qi<br>
<br>
<br>
<blockquote
cite="mid:CABcZAN=kKF1cLeCR+SMenSBG2bRxVd==GR727XN-h72FW2ZT2w@mail.gmail.com"
type="cite">
<div dir="ltr">
<div class="gmail_extra">
-- <br>
Christopher Larson<br>
clarson at kergoth dot com<br>
Founder - BitBake, OpenEmbedded, OpenZaurus<br>
Maintainer - Tslib<br>
Senior Software Engineer, Mentor Graphics
</div>
</div>
</blockquote>
<br>
</body>
</html>