From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail.windriver.com (mail.windriver.com [147.11.1.11]) by mail.openembedded.org (Postfix) with ESMTP id 584D46B91D for ; Thu, 22 Aug 2013 07:25:06 +0000 (UTC) Received: from ALA-HCA.corp.ad.wrs.com (ala-hca.corp.ad.wrs.com [147.11.189.40]) by mail.windriver.com (8.14.5/8.14.3) with ESMTP id r7M7P425029002 (version=TLSv1/SSLv3 cipher=AES128-SHA bits=128 verify=FAIL); Thu, 22 Aug 2013 00:25:04 -0700 (PDT) Received: from [128.224.162.231] (128.224.162.231) by ALA-HCA.corp.ad.wrs.com (147.11.189.50) with Microsoft SMTP Server (TLS) id 14.2.342.3; Thu, 22 Aug 2013 00:25:03 -0700 Message-ID: <5215BCD0.7080902@windriver.com> Date: Thu, 22 Aug 2013 15:25:04 +0800 From: Kang Kai User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130623 Thunderbird/17.0.7 MIME-Version: 1.0 To: Khem Raj References: <5a0310813a9df4c60eaf5ba31e2de12bf3d08ccf.1376965065.git.kai.kang@windriver.com> <760CDDA0-416D-4E16-8FC2-6205ED265506@gmail.com> <5215B4A7.8070604@windriver.com> <9A96060F-4FEA-4785-A90F-B71A615976D3@gmail.com> In-Reply-To: <9A96060F-4FEA-4785-A90F-B71A615976D3@gmail.com> X-Originating-IP: [128.224.162.231] Cc: openembedded-core@lists.openembedded.org, Zhangle.Yang@windriver.com Subject: Re: [PATCH 1/1] bc: fix segmentation fault X-BeenThere: openembedded-core@lists.openembedded.org X-Mailman-Version: 2.1.12 Precedence: list List-Id: Patches and discussions about the oe-core layer List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 22 Aug 2013 07:25:06 -0000 Content-Type: text/plain; charset="UTF-8"; format=flowed Content-Transfer-Encoding: 8bit On 2013年08月22日 15:15, Khem Raj wrote: > Kai > > On Aug 21, 2013, at 11:50 PM, Kang Kai wrote: > >>> what does adding this include file fix ? >> To be honest, I am not sure why add this include. After build it is harmless, so I didn't want to modify it because I think LFS is trustable. >> > Its matter of understanding the change and may be explaining a bit better what it fixes and make the patch better for upstream too so try to build it without this include added and see if you get some warnings/errors that it fixes and then add that information to patch header. Thanks. I'll check it and be more careful next time. Regards, Kai > >> Regards, >> Kai > > -- Regards, Neil | Kai Kang