alema/hkd
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

build fails on arm on case label range #3

New Issue
Closed
opened 2020-09-22 02:24:45 +02:00 by Ghost · 2 comments
Ghost commented 2020-09-22 02:24:45 +02:00

I've tested compiling on a x86_64 machine and it works properly, but when I attempt to do it on a armv7h it fails; perhaps it's a difference in default variable size for the two archs?

$ make
gcc -Wall -Werror -pedantic --std=c99 -O2 -D_FORTIFY_SOURCE=2 -Wl,-O1,--sort-common,--as-needed,-z,relro,-z,now  hkd.c   -o hkd
hkd.c: In function 'parse_config_file':
hkd.c:745:5: error: case label value is less than minimum value for type [-Werror=switch-outside-range]
745 |     case EOF:
|     ^~~~
cc1: all warnings being treated as errors
make: *** [<builtin>: hkd] Error 1
$ gcc --version
gcc (GCC) 10.2.0
Copyright (C) 2020 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
I've tested compiling on a x86_64 machine and it works properly, but when I attempt to do it on a armv7h it fails; perhaps it's a difference in default variable size for the two archs? ``` $ make gcc -Wall -Werror -pedantic --std=c99 -O2 -D_FORTIFY_SOURCE=2 -Wl,-O1,--sort-common,--as-needed,-z,relro,-z,now hkd.c -o hkd hkd.c: In function 'parse_config_file': hkd.c:745:5: error: case label value is less than minimum value for type [-Werror=switch-outside-range] 745 | case EOF: | ^~~~ cc1: all warnings being treated as errors make: *** [<builtin>: hkd] Error 1 $ gcc --version gcc (GCC) 10.2.0 Copyright (C) 2020 Free Software Foundation, Inc. This is free software; see the source for copying conditions. There is NO warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. ```
alema commented 2020-10-04 23:26:42 +02:00
Owner

Hi, sorry for the inactivity, this error is due to EOF being defined as a negative signed int of a value exceeding the minimum value permitted by char on arm (which seems to be unsigned).

I should have fixed the problem with a preprocessor check to enable that case only on x86 architectures, check if it compiles and I will (hopefully) be abled to close this issue.

Again sorry for the long wait

Hi, sorry for the inactivity, this error is due to EOF being defined as a negative signed int of a value exceeding the minimum value permitted by char on arm (which seems to be unsigned). I should have fixed the problem with a preprocessor check to enable that case only on x86 architectures, check if it compiles and I will (hopefully) be abled to close this issue. Again sorry for the long wait
Ghost commented 2020-10-16 00:45:45 +02:00
Author

Hello,
sorry for my own delay in responding.
The preprocessor directive works and it builds ok, moreover, it parses the file ok according to the -d option

Hello, sorry for my own delay in responding. The preprocessor directive works and it builds ok, moreover, it parses the file ok according to the -d option
Ghost closed this issue 2020-10-16 00:45:50 +02:00
Sign in to join this conversation.
No Label
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: alema/hkd#3
No description provided.
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?