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

Fixed gcc build errors #47

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

threadexio
Copy link

@threadexio threadexio commented Jan 16, 2022

  • All warnings treated as errors
  • Multiple definition of int verbose

GCC version: 11.1.0

Solutions:

  • Removed the -Werror from the Makefile
  • Fixed the multiple definitions of int verbose from error.h

- All warning treated as errors
- Multiple definition of "int verbose"
@Fenr1r-g
Copy link

Tested on my machine. Fixes the -werrors and successfully builds and installs sasquatch.

@int3rsys
Copy link

Thank you, I've downloaded your commit and it worked contrary to the master.

@jhilland0117
Copy link

jhilland0117 commented Jun 18, 2022

thanks! this fixes the build for ubuntu 20+

@agoodm88
Copy link

Thanks, this fixes the build for me on Ubuntu 22.04

@speidy
Copy link

speidy commented Sep 5, 2022

@devttys0 Please merge!

@amlamarra
Copy link

@devttys0 Would love to get this merged.

@jacopotediosi
Copy link

Merging this PR causes some files to be lost while extracting some squashfs images (e.g., see ReFirmLabs/binwalk#618), don't know why.
On Ubuntu 22.04.1LTS I suggest to compile using #51 instead of this.

stkw0 pushed a commit to OSPG/binwalk that referenced this pull request Feb 28, 2024
Apply devttys0/sasquatch#47 to fix build failure
with latest gcc

It should be noted that sasquatch has not been updated since March 2021

Fix ReFirmLabs#498

Signed-off-by: Fabrice Fontaine <[email protected]>
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.

9 participants