Skip to content

[doxy] fix build when more aggressive -W options are used#410

Merged
chrissie-c merged 1 commit intomasterfrom
fix-doxybuild
Jul 15, 2020
Merged

[doxy] fix build when more aggressive -W options are used#410
chrissie-c merged 1 commit intomasterfrom
fix-doxybuild

Conversation

@fabbione
Copy link
Copy Markdown
Member

make[2]: Entering directory '/builddir/build/BUILD/kronosnet-1.17/man'
gcc -DHAVE_CONFIG_H -I. -I.. -O3 -ggdb3 -Werror -Wall -Wextra -Wno-unused-parameter -pthread -I/usr/include/libxml2 -c -o doxyxml-doxyxml.o test -f 'doxyxml.c' || echo './'doxyxml.c
doxyxml.c: In function 'allcaps':
doxyxml.c:414:13: error: comparison of integer expressions of different signedness: 'int' and 'size_t' {aka 'long unsigned int'} [-Werror=sign-compare]
414 | for (i=0; i< strlen(name); i++) {
| ^
cc1: all warnings being treated as errors
make[2]: *** [Makefile:623: doxyxml-doxyxml.o] Error 1
make[2]: Leaving directory '/builddir/build/BUILD/kronosnet-1.17/man'
make[1]: *** [Makefile:567: all-recursive] Error 1

Signed-off-by: Fabio M. Di Nitto fdinitto@redhat.com

make[2]: Entering directory '/builddir/build/BUILD/kronosnet-1.17/man'
gcc -DHAVE_CONFIG_H -I. -I..    -O3 -ggdb3 -Werror -Wall -Wextra -Wno-unused-parameter -pthread  -I/usr/include/libxml2   -c -o doxyxml-doxyxml.o `test -f 'doxyxml.c' || echo './'`doxyxml.c
doxyxml.c: In function 'allcaps':
doxyxml.c:414:13: error: comparison of integer expressions of different signedness: 'int' and 'size_t' {aka 'long unsigned int'} [-Werror=sign-compare]
  414 |  for (i=0; i< strlen(name); i++) {
      |             ^
cc1: all warnings being treated as errors
make[2]: *** [Makefile:623: doxyxml-doxyxml.o] Error 1
make[2]: Leaving directory '/builddir/build/BUILD/kronosnet-1.17/man'
make[1]: *** [Makefile:567: all-recursive] Error 1

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>
@fabbione fabbione requested a review from chrissie-c July 14, 2020 14:57
@chrissie-c chrissie-c merged commit caec56b into master Jul 15, 2020
@chrissie-c
Copy link
Copy Markdown
Contributor

Thanks

@chrissie-c chrissie-c deleted the fix-doxybuild branch July 15, 2020 07:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants