Re: [meta-networking][PATCH 2/4] atftp: build with gcc 5.2

[Kang Kai <Kai.Kang@windriver.com>]

On 2015年08月22日 00:08, Khem Raj wrote:
> On Fri, Aug 21, 2015 at 1:37 AM,  <kai.kang@windriver.com> wrote:
>> From: Kai Kang <kai.kang@windriver.com>
>>
>> atftp fails to build with gcc 5.2:
>>
>> tftp_def.h:54:14: warning: inline function 'Strncpy' declared but never defined
>>
>> GCC 5 defaults to -std=gnu11 instead of -std=gnu89. The semantics of
>> inline function changes. Pass '-std=gnu89' to gcc to compile atftp.
>>
>> Ref:
>> https://gcc.gnu.org/gcc-5/porting_to.html
>>
>> Signed-off-by: Kai Kang <kai.kang@windriver.com>
>> ---
>>   meta-networking/recipes-daemons/atftp/atftp_git.bb | 2 ++
>>   1 file changed, 2 insertions(+)
>>
>> diff --git a/meta-networking/recipes-daemons/atftp/atftp_git.bb b/meta-networking/recipes-daemons/atftp/atftp_git.bb
>> index b53f637..0ece58f 100644
>> --- a/meta-networking/recipes-daemons/atftp/atftp_git.bb
>> +++ b/meta-networking/recipes-daemons/atftp/atftp_git.bb
>> @@ -32,6 +32,8 @@ USERADD_PACKAGES = "${PN}d"
>>   USERADD_PARAM_${PN}d = "--system --no-create-home --shell /bin/false \
>>                           --user-group nobody"
>>
>> +EXTRA_OEMAKE = "CFLAGS='${CFLAGS} -std=gnu89'"
> this is a woraround and will work, however if you are at and fix it
> once for all that will be desirable. eg. this file you could move the
> definition of
> inline function into the relevant .h file and make it a static inline

Hi Raj,

Thanks. I'll try it.

--Kai

>
>> +
>>   do_install_append() {
>>       install -d ${D}${sysconfdir}/init.d
>>       install -m 0755 ${WORKDIR}/atftpd.init ${D}${sysconfdir}/init.d/atftpd
>> --
>> 1.9.1
>>
>> --
>> _______________________________________________
>> Openembedded-devel mailing list
>> Openembedded-devel@lists.openembedded.org
>> http://lists.openembedded.org/mailman/listinfo/openembedded-devel


-- 
Regards,
Neil | Kai Kang