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

libvorbis contains a main function #41

Open
9chu opened this issue Feb 26, 2019 · 3 comments
Open

libvorbis contains a main function #41

9chu opened this issue Feb 26, 2019 · 3 comments

Comments

@9chu
Copy link

9chu commented Feb 26, 2019

src/contrib/libvorbis/lib/barkmel.c contains an int main entry that should be removed.

@gurka
Copy link
Contributor

gurka commented Feb 26, 2019

Better report this upstream, perhaps at https://github.com/xiph/vorbis ?

However, I don't think that this is an error. libvorbis contains three utility / extra programs: barkmel, tone and psytune, see https://github.com/xiph/vorbis/blob/ea8b03fce93444cb3cf0131909e15b4f8856e863/lib/Makefile.am#L29 So the corresponding c-files should have an int main.

@9chu
Copy link
Author

9chu commented Feb 27, 2019

Better report this upstream, perhaps at https://github.com/xiph/vorbis ?

However, I don't think that this is an error. libvorbis contains three utility / extra programs: barkmel, tone and psytune, see https://github.com/xiph/vorbis/blob/ea8b03fce93444cb3cf0131909e15b4f8856e863/lib/Makefile.am#L29 So the corresponding c-files should have an int main.

Yes, it's an utility program. So the real question is that the CMakeLists.txt is not supposed to include it.

In src/engine/audio/CMakeLists.txt:

file (GLOB_RECURSE VORBIS_FILES "../../contrib/libvorbis/*.c")

That may lead to a link problem.

@gurka
Copy link
Contributor

gurka commented Feb 27, 2019

Aha, yes that is a problem. I wonder why they don't just add the libvorbis directory with add_subdirectory. libvorbis uses CMake as well, so it should be possible.

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

No branches or pull requests

2 participants