legatoproject / legato-af

Legato Application Framework
Mozilla Public License 2.0
153 stars 118 forks source link

Building project on another linux systems #40

Closed MaksVal closed 5 years ago

MaksVal commented 5 years ago

Hi! Thank for your project. Please, give me a more info about requirements. I want to build this projects on The Arch linux, but I can not. Because I get a lot of mistakes.

$ legato_repo]$ ninja --version
1.9.0
$ gcc --version
gcc (GCC) 8.2.1 20181127

Thanks!

CoRfr commented 5 years ago

Hi @MaksVal ,

I am also building Legato on a Arch system, and although it usually fails more often because the gcc version is more recent, we usually try to fix these issues (especially since newer gcc versions ends up generally on Ubuntu OS that we officially support anyway).

But at this point it doesn't build it seems. It fails on my machine with

/home/broussel/Legato/src/repo/qa/legato/components/httpClientLibrary/le_httpClientLib.c: In function ‘BuildAndSendCredential’:
/home/broussel/Legato/src/repo/qa/legato/components/httpClientLibrary/le_httpClientLib.c:542:40: error: argument to ‘sizeof’ in ‘strncpy’ call is the same expression as the source; did you mean to use the size of the destination? [-Werror=sizeof-pointer-memaccess]
     strncpy(buffer+length, CRLF, sizeof(CRLF));
                                        ^
/home/broussel/Legato/src/repo/qa/legato/components/httpClientLibrary/le_httpClientLib.c: In function ‘BuildAndSendResource’:
/home/broussel/Legato/src/repo/qa/legato/components/httpClientLibrary/le_httpClientLib.c:628:44: error: argument to ‘sizeof’ in ‘strncpy’ call is the same expression as the source; did you mean to use the size of the destination? [-Werror=sizeof-pointer-memaccess]
         strncpy(buffer+length, CRLF, sizeof(CRLF));
                                            ^
cc1: all warnings being treated as errors

I'm gonna have a look at fixing it.

Otherwise a base for dependency would be https://github.com/legatoproject/docker-images/blob/master/build-archlinux-latest-x86_64/Dockerfile It looks like this is not entirely up-to-date.

CoRfr commented 5 years ago

Build issue with gcc 8.2.1 has been fixed, should be good now.