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

Failed to compile vorbis-tools #2295

Closed
realaboo opened this issue Sep 27, 2022 · 5 comments
Closed

Failed to compile vorbis-tools #2295

realaboo opened this issue Sep 27, 2022 · 5 comments

Comments

@realaboo
Copy link

What I did:
Just run media-autobuild_suite.bat in powershell on a Windows 10 machine, select some build options and then it failed in a mintty window.

make[3]: Entering directory '/build/vorbis-tools-git/build-64bit/oggenc'
  CC       oggenc.o
../../oggenc/oggenc.c:735:30: error: incompatible integer to pointer conversion passing 'int' to parameter of type 'int (*)'; take the address with & [-Wint-conversion]
    while((ret = getopt_long(argc, argv, "a:b:B:c:C:d:G:hkl:L:m:M:n:N:o:P:q:QrR:s:t:VX:Y:",
                             ^~~~
                             &
../../include/getopt.h:140:29: note: passing argument to parameter '__imp___argc' here
extern int getopt_long (int __argc, char *const *__argv, const char *__shortopts,
                            ^
D:/code/mabs/msys64/mingw64/include/stdlib.h:283:19: note: expanded from macro '__argc'
#define __argc (* __MINGW_IMP_SYMBOL(__argc))
                  ^
D:/code/mabs/msys64/mingw64/include/_mingw_mac.h:119:35: note: expanded from macro '__MINGW_IMP_SYMBOL'
#  define __MINGW_IMP_SYMBOL(sym) __imp_##sym
                                  ^
<scratch space>:41:1: note: expanded from here
__imp___argc
^
../../oggenc/oggenc.c:735:36: warning: incompatible pointer types passing 'char **' to parameter of type 'char *const *(*(*)())' [-Wincompatible-pointer-types]
    while((ret = getopt_long(argc, argv, "a:b:B:c:C:d:G:hkl:L:m:M:n:N:o:P:q:QrR:s:t:VX:Y:",
                                   ^~~~
../../include/getopt.h:140:50: note: passing argument to parameter '__p___argv' here
extern int getopt_long (int __argc, char *const *__argv, const char *__shortopts,
                                                 ^
D:/code/mabs/msys64/mingw64/include/stdlib.h:287:19: note: expanded from macro '__argv'
#define __argv  (* __p___argv())
                   ^
1 warning and 1 error generated.

Full log:
oWFK.zip

@Barough
Copy link

Barough commented Sep 27, 2022

Just did a test and it compiles just fine here

Running git clone for vorbis-tools...
Running git update for vorbis-tools...
┌ vorbis-tools git ................................... [Recently updated]
├ Running autoreconf...
├ Running configure...
├ Running make...
├ Stripping...
└ vorbis-tools-git ............................................ [Updated]

It seems like your issue is due to using Clang. No issue for me when using GCC

[EDIT]
xiph/vorbis-tools#38

@realaboo
Copy link
Author

Just did a test and it compiles just fine here

Running git clone for vorbis-tools... Running git update for vorbis-tools... ┌ vorbis-tools git ................................... [Recently updated] ├ Running autoreconf... ├ Running configure... ├ Running make... ├ Stripping... └ vorbis-tools-git ............................................ [Updated]

It seems like your issue is due to using Clang. No issue for me when using GCC

[EDIT] xiph/vorbis-tools#38

Good point! Will try on gcc. Thanks.

@realaboo
Copy link
Author

gcc works. close this issue.

@snoopchallenge
Copy link

might wanna reopen this so someone can fix this in their spare time

@1480c1
Copy link
Member

1480c1 commented Oct 3, 2022

Fixed with 4576d38

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

4 participants