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

Included -std=gnu90 option in compilation flags to fix compilation on GCC version >=5 #58

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

Conversation

caioycosta
Copy link

Compilation using newest versions (5+) of GCC was giving an error while using GCC <=4 was working fine.
According to https://stackoverflow.com/a/14737642/1152602 , in GCC 5 the default was changed from -std=gnu90 to -std=gnu11.
Among other things this changes the behavior of the "inline" directive, which was giving errors due to function file_close in file.c.
By adding the flag -std=gnu90, behavior from old GCC is restored and the project compiles succesfully.
Tested on Xubuntu 17.04, gcc (Ubuntu 6.3.0-12ubuntu2) 6.3.0 20170406.

Compilation using newest versions (5+) of GCC was giving an error while using GCC <=4 was working fine.
According to https://stackoverflow.com/a/14737642/1152602 , in GCC 5 the default was changed from -std=gnu90 to -std=gnu11.
Among other things this changes the behavior of the "inline" directive, which was giving errors due to function file_close in file.c.
By adding the flag -std=gnu90, behavior from old GCC is restored and the project compiles succesfully.
Tested on Xubuntu 17.04, gcc (Ubuntu 6.3.0-12ubuntu2) 6.3.0 20170406.
@pinturic
Copy link

I had problems compiling it and with this everything worked correctly thanks

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