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

s4/tests: fix build failure against gcc-10 #1

Merged
merged 1 commit into from Oct 16, 2020
Merged

s4/tests: fix build failure against gcc-10 #1

merged 1 commit into from Oct 16, 2020

Conversation

trofi
Copy link
Contributor

@trofi trofi commented Jan 28, 2020

On gcc-10 (and gcc-9 -fno-common) build fails as:

ld: tests/s4/t_transactions.c.2.o:/tests/s4/t_transactions.c:22:
  multiple definition of `s4'; tests/s4/t_s4.c.2.o:tests/s4/t_s4.c:31: first defined here
collect2: error: ld returned 1 exit status
Waf: Leaving directory `/home/slyfox/dev/git/s4/_build_'
Build failed
 -> task in 'test_s4' failed (exit status 1):

gcc-10 will change the default from -fcommon to fno-common:
https://gcc.gnu.org/PR85678.

The error also happens if CFLAGS=-fno-common passed explicitly.

Reported-by: Toralf Förster
Bug: https://bugs.gentoo.org/706940

On gcc-10 (and gcc-9 -fno-common) build fails as:

```
ld: tests/s4/t_transactions.c.2.o:/tests/s4/t_transactions.c:22:
  multiple definition of `s4'; tests/s4/t_s4.c.2.o:tests/s4/t_s4.c:31: first defined here
collect2: error: ld returned 1 exit status
Waf: Leaving directory `/home/slyfox/dev/git/s4/_build_'
Build failed
 -> task in 'test_s4' failed (exit status 1):
```

gcc-10 will change the default from -fcommon to fno-common:
https://gcc.gnu.org/PR85678.

The error also happens if CFLAGS=-fno-common passed explicitly.

Reported-by: Toralf Förster
Bug: https://bugs.gentoo.org/706940
Signed-off-by: Sergei Trofimovich <slyfox@gentoo.org>
@dsvensson
Copy link
Member

amazeballs! 👏

@dsvensson dsvensson merged commit 652ac58 into xmms2:master Oct 16, 2020
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.

None yet

2 participants