Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Snoopy Fails to build from source with gcc-7 #123

Closed
MarcosFouces opened this issue Aug 7, 2017 · 3 comments · Fixed by #122
Closed

Snoopy Fails to build from source with gcc-7 #123

MarcosFouces opened this issue Aug 7, 2017 · 3 comments · Fixed by #122

Comments

@MarcosFouces
Copy link
Contributor

Checklist before I submit this issue report

I confirm that:

Issue description - Environment basics

(replace 'FILLIN' with actual values from your environment)

Key Value Comments
Architecture: amd64 x86 is not supported anymore
Linux distribution: Debian
Distribution version: Buster (testing)
Snoopy version: 2.4.6
Snoopy config file was used: yes snoopy-install.sh enables this by default
Snoopy threading support enabled: no not enabled by default as of v2.4.4

Issue description - How was Snoopy installed

This is not a runtime issue but a compile time one.

Issue description - Describe the problem you are having

Snoopy fails to build from source when trying to build it with gcc-7. It is now the default compiler version on Debian and the package will be removed on 20 august if a fix is not provided.

Here you can check the build log obtained with the previous version of snoopy packaged for Debian (2.3.1):
http://people.debian.org/~doko/logs/gcc7-20170126/snoopy_2.3.1-2_unstable_gcc7.log

You can also check the Debian bug report:
https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=853664

I also attached the build log with 2.4.6 version i obtained on my machine.

Expected result

A succesful build! :-)

Actual result

FTBS (attached the build log i obtained on my machine).

Steps to reproduce

Just us gcc-7 instead of the version you currently use to build Snoopy (surely gcc-6)
snoopy_gcc7-log.txt

Thanks for your time.
Greetings.

@lukas227
Copy link
Contributor

Just for reference: this problem is fixed by pull request #122 .

@ageis
Copy link

ageis commented Feb 1, 2018

Also failed to build with GCC 7.3.0:

cmdline.c:71:79: error: comparison between pointer and zero character constant [-Werror=pointer-compare]
     for (cmdLineArgCount=0 ; *(snoopy_inputdatastorage->argv+cmdLineArgCount) != '\0' ; cmdLineArgCount++);
                                                                               ^~
cmdline.c:71:30: note: did you mean to dereference the pointer?
     for (cmdLineArgCount=0 ; *(snoopy_inputdatastorage->argv+cmdLineArgCount) != '\0' ; cmdLineArgCount++);
                              ^

Edit: confirmed fixed by #122

@bostjan bostjan added this to the 2.4.NEXT milestone Apr 12, 2018
@bostjan
Copy link
Member

bostjan commented Apr 12, 2018

Fixed by merging pull request #122. This will be included in the next release.

@bostjan bostjan closed this as completed Apr 12, 2018
@bostjan bostjan modified the milestones: 2.4.10, 2.4.7 Nov 26, 2020
@bostjan bostjan linked a pull request Nov 26, 2020 that will close this issue
15 tasks
@bostjan bostjan self-assigned this Nov 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging a pull request may close this issue.

4 participants