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

Fix building with GCC 10 / CFLAGS=-fno-common #12

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

Fix building with GCC 10 / CFLAGS=-fno-common #12

wants to merge 1 commit into from

Conversation

jer-gentoo
Copy link

No description provided.

@nyxdis
Copy link
Owner

nyxdis commented Sep 10, 2020

Hi! Thanks for these changes. Unfortunately it seems you've forgot to add the extern keyword to the buffer variable in audioscrobbler.h (I guess?). The current version does not compile with gcc10 here.
If you could fix that and test it (I cannot test it currently), I'd be glad to merge your pull request.

@jer-gentoo
Copy link
Author

I'm not sure why that would fail with gcc10 for you but not for me. The declaration of struct as_conn_t in src/audioscrobbler.h is used exclusively in src/audioscrobbler.c, where its definition as as_conn is declared static for that reason.

@nyxdis
Copy link
Owner

nyxdis commented Sep 11, 2020

It doesn't fail because of as_conn, it fails because of buffer that's declared both in audioscrobbler.c and audioscrobbler.h.

@jer-gentoo
Copy link
Author

Oh I see. The changes committed here differ from those in the patch I applied here at [0]. Not sure how some of those changes were lost.

[0] https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=c9f42ac8feb314f3f2f6ec7cfd8c535ff33fe735

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