<html>
<head>
<meta content="text/html; charset=windows-1252"
http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<p><br>
</p>
<br>
<div class="moz-cite-prefix">On 11/29/2016 03:08 PM, Cal Sullivan
wrote:<br>
</div>
<blockquote
cite="mid:e91e2209-1d22-a53b-8cf4-e37eb86d9a14@intel.com"
type="cite">
<meta content="text/html; charset=windows-1252"
http-equiv="Content-Type">
<tt>Oops, accidentally dropped CCs. See my reply below.</tt><br>
<br>
<div class="moz-cite-prefix">On 11/29/2016 01:05 PM, Cal Sullivan
wrote:<br>
</div>
<blockquote
cite="mid:effe6688-33a7-7bda-8dea-471e58149824@intel.com"
type="cite">
<meta content="text/html; charset=windows-1252"
http-equiv="Content-Type">
<br>
<br>
<div class="moz-cite-prefix">On 11/29/2016 12:52 PM, Leonardo
Sandoval wrote:<br>
</div>
<blockquote
cite="mid:2a1fd093-8b07-1e04-4167-f52c0698446e@linux.intel.com"
type="cite">
<meta http-equiv="Content-Type" content="text/html;
charset=windows-1252">
<p><br>
</p>
<br>
<div class="moz-cite-prefix">On 11/29/2016 02:15 PM,
Christopher Larson wrote:<br>
</div>
<blockquote
cite="mid:CABcZANnQ87Q=h4-ponSyGP_-Q8bsRPfMOQ+UZWf0S7i85D2-GQ@mail.gmail.com"
type="cite">
<div dir="ltr">
<div class="gmail_extra"><br>
<div class="gmail_quote">On Tue, Nov 29, 2016 at 11:29
AM, <span dir="ltr"><<a moz-do-not-send="true"
href="mailto:leonardo.sandoval.gonzalez@linux.intel.com"
target="_blank">leonardo.sandoval.gonzalez@linux.intel.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=":2m0" class="a3s aXjCH m158b154f368e9e70">From:
Leonardo Sandoval <<a moz-do-not-send="true"
href="mailto:leonardo.sandoval.gonzalez@linux.intel.com">leonardo.sandoval.gonzalez@<wbr>linux.intel.com</a>><br>
<br>
getty devices were not being removed in some cases
because device name<br>
was not at the end of the line, for example a
ttyS1 device:<br>
<br>
S1:12345:respawn:/bin/start_<wbr>getty 115200
ttyS1 vt102<br>
<br>
Removing this limitation allows sed to remove any
line containing<br>
the device.<br>
<br>
Signed-off-by: Leonardo Sandoval <<a
moz-do-not-send="true"
href="mailto:leonardo.sandoval.gonzalez@linux.intel.com">leonardo.sandoval.gonzalez@<wbr>linux.intel.com</a>><br>
---<br>
meta/recipes-core/sysvinit/<a
moz-do-not-send="true"
href="http://sysvinit-inittab_2.88dsf.bb"
rel="noreferrer" target="_blank">sys<wbr>vinit-inittab_2.88dsf.bb</a>
| 2 +-<br>
1 file changed, 1 insertion(+), 1 deletion(-)<br>
<br>
diff --git a/meta/recipes-core/sysvinit/<a
moz-do-not-send="true"
href="http://sysvinit-inittab_2.88dsf.bb"
rel="noreferrer" target="_blank">s<wbr>ysvinit-inittab_2.88dsf.bb</a>
b/meta/recipes-core/sysvinit/<a
moz-do-not-send="true"
href="http://sysvinit-inittab_2.88dsf.bb"
rel="noreferrer" target="_blank">s<wbr>ysvinit-inittab_2.88dsf.bb</a><br>
index c219cbf..dd30f43 100644<br>
--- a/meta/recipes-core/sysvinit/<a
moz-do-not-send="true"
href="http://sysvinit-inittab_2.88dsf.bb"
rel="noreferrer" target="_blank">s<wbr>ysvinit-inittab_2.88dsf.bb</a><br>
+++ b/meta/recipes-core/sysvinit/<a
moz-do-not-send="true"
href="http://sysvinit-inittab_2.88dsf.bb"
rel="noreferrer" target="_blank">s<wbr>ysvinit-inittab_2.88dsf.bb</a><br>
@@ -62,7 +62,7 @@ if [ "x$D" = "x" ] && [
-e /proc/consoles ]; then<br>
k=`echo ${i} | sed s/^.*\://g`<br>
if [ -z "`grep ${j}
/proc/consoles`" ]; then<br>
if [ -z "${k}" ] || [ -z
"`grep ${k} /proc/consoles`" ] || [ ! -e /dev/${j}
]; then<br>
- sed -i /^.*${j}$/d
/etc/inittab<br>
+ sed -i /^.*${j}/d
/etc/inittab</div>
</blockquote>
</div>
<br>
This doesn’t check for the end of a word, so it’d match
a partial as well. Would that be a problem? i.e. ttyS1
vs ttyS10.<br>
</div>
</div>
</blockquote>
<br>
That is the intention, that line removal is not tied to
finding the device at the end of the line. With proposed
patch, it removes the device, no matter where is located.<br>
<br>
</blockquote>
Right, but look at this hypothetical example:<br>
<br>
S1:12345:respawn:/bin/start_getty 115200 ttyS1 vt102<br>
S10:12345:respawn:/bin/start_getty 115200 ttyS10<br>
<br>
Both would match, when you only want the top to match. This is
untested but likely more what we want:<br>
<br>
sed -i /^.*${j}[\ $]/d /etc/inittab<br>
<br>
That way we either match up to a space (end of word) or end of
line.<br>
</blockquote>
</blockquote>
<br>
Got it, that can be possible. Sending a v2<br>
<br>
<blockquote
cite="mid:e91e2209-1d22-a53b-8cf4-e37eb86d9a14@intel.com"
type="cite">
<blockquote
cite="mid:effe6688-33a7-7bda-8dea-471e58149824@intel.com"
type="cite"> <br>
---<br>
Cal<br>
<br>
<blockquote
cite="mid:2a1fd093-8b07-1e04-4167-f52c0698446e@linux.intel.com"
type="cite"> <br>
<blockquote
cite="mid:CABcZANnQ87Q=h4-ponSyGP_-Q8bsRPfMOQ+UZWf0S7i85D2-GQ@mail.gmail.com"
type="cite">
<div dir="ltr">
<div class="gmail_extra">-- <br>
<div class="gmail_signature"
data-smartmail="gmail_signature">Christopher Larson<br>
clarson at kergoth dot com<br>
Founder - BitBake, OpenEmbedded, OpenZaurus<br>
Maintainer - Tslib<br>
Senior Software Engineer, Mentor Graphics</div>
</div>
</div>
</blockquote>
<br>
<br>
<fieldset class="mimeAttachmentHeader"></fieldset>
<br>
</blockquote>
<br>
</blockquote>
<br>
</blockquote>
<br>
</body>
</html>