[FS#259] LEDE build error with OSX since commit: "firmware-utils: mkwrggimg: new tool for D-Link DAP-2695"

[LEDE Bugs]

A new Flyspray task has been opened.  Details are below. 

User who did this - Bluse-Blue (Bluse) 

Attached to Project - LEDE Project
Summary - LEDE build error with OSX since commit: "firmware-utils: mkwrggimg: new tool for D-Link DAP-2695"
Task Type - Bug Report
Category - Base system
Status - Unconfirmed
Assigned To - 
Operating System - All
Severity - Low
Priority - Very Low
Reported Version - Trunk
Due in Version - Undecided
Due Date - Undecided
Details - Hi all,

The commit "firmware-utils: mkwrggimg: new tool for D-Link DAP-2695", (55eb6ed0615683eb2d407794db8629e12920f58c) brakes LEDE compilation under OS X ( tested El Capitan).

Lede version: latest trunk
target: ar71xxx

Error: 

mkdir -p /Users/bluse/Desktop/Coding-Area/LEDE/build_dir/host/firmware-utils/bin
gcc -O2 -I/Users/bluse/Desktop/Coding-Area/LEDE/staging_dir/host/include -I/Users/bluse/Desktop/Coding-Area/LEDE/staging_dir/host/usr/include  -I/usr/local/opt/openssl/include -include endian.h -L/Users/bluse/Desktop/Coding-Area/LEDE/staging_dir/host/lib -L/U
sers/bluse/Desktop/Coding-Area/LEDE/staging_dir/host/usr/lib  -L/usr/local/opt/openssl/lib -o /Users/bluse/Desktop/Coding-Area/LEDE/build_dir/host/firmware-utils/bin/addpattern src/addpattern.c
ld: warning: directory not found for option '-L/Users/bluse/Desktop/Coding-Area/LEDE/staging_dir/host/usr/lib'
gcc -O2 -I/Users/bluse/Desktop/Coding-Area/LEDE/staging_dir/host/include -I/Users/bluse/Desktop/Coding-Area/LEDE/staging_dir/host/usr/include  -I/usr/local/opt/openssl/include -include endian.h -L/Users/bluse/Desktop/Coding-Area/LEDE/staging_dir/host/lib -L/U
sers/bluse/Desktop/Coding-Area/LEDE/staging_dir/host/usr/lib  -L/usr/local/opt/openssl/lib -o /Users/bluse/Desktop/Coding-Area/LEDE/build_dir/host/firmware-utils/bin/asustrx src/asustrx.c
src/asustrx.c:135:42: warning: format specifies type 'unsigned short *' but the argument has type 'uint8_t *'
      (aka 'unsigned char *') [-Wformat]
                        if (sscanf(optarg, "%hu.%hu.%hu.%hu", &version[0], &version[1], &version[2], &version[3]) != 4)
                                            ~~~               ^~~~~~~~~~~
                                            %s
src/asustrx.c:135:55: warning: format specifies type 'unsigned short *' but the argument has type 'uint8_t *'
      (aka 'unsigned char *') [-Wformat]
                        if (sscanf(optarg, "%hu.%hu.%hu.%hu", &version[0], &version[1], &version[2], &version[3]) != 4)
                                                ~~~                        ^~~~~~~~~~~
                                                %s
src/asustrx.c:135:68: warning: format specifies type 'unsigned short *' but the argument has type 'uint8_t *'
      (aka 'unsigned char *') [-Wformat]
                        if (sscanf(optarg, "%hu.%hu.%hu.%hu", &version[0], &version[1], &version[2], &version[3]) != 4)
                                                    ~~~                                 ^~~~~~~~~~~
                                                    %s
src/asustrx.c:135:81: warning: format specifies type 'unsigned short *' but the argument has type 'uint8_t *'
      (aka 'unsigned char *') [-Wformat]
                        if (sscanf(optarg, "%hu.%hu.%hu.%hu", &version[0], &version[1], &version[2], &version[3]) != 4)
                                                        ~~~                                          ^~~~~~~~~~~
                                                        %s
4 warnings generated.


Steps to reproduce:
use the mentioned commit and build new LEDE image.

Workaround:
revert mentioned commit via: git revert --strategy resolve 55eb6ed0615683eb2d407794db8629e12920f58c


Greetings Bluse
   

 

More information can be found at the following URL:
https://bugs.lede-project.org/index.php?do=details&task_id=259