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

Something's not right in ao_rsound.c #3501

Closed
OrdinaryMagician opened this issue Sep 7, 2016 · 7 comments
Closed

Something's not right in ao_rsound.c #3501

OrdinaryMagician opened this issue Sep 7, 2016 · 7 comments

Comments

@OrdinaryMagician
Copy link

latest git master, x86_64 arch linux

Reproduction steps

Compile.

Expected behavior

It compiles.

Actual behavior

It doesn't compile.

In file included from ../audio/out/ao_rsound.c:31:0:
../audio/out/ao_rsound.c:162:13: error: request for member 'deprecation_message' in something not a structure or union
             .deprecation_message = "request --audio-device support on issue tracker"),
             ^
/home/marisa/.cache/pacaur/mpv-build-git/src/mpv/options/m_option.h:551:32: note: in definition of macro 'OPT_GENERAL'
     {.name = optname, .flags = flagv,                                   \
                                ^~~~~
../audio/out/ao_rsound.c:161:9: note: in expansion of macro 'OPT_STRING'
         OPT_STRING("port", port, 0
         ^~~~~~~~~~

Log files

N/A

Sample files

N/A

@ghost ghost closed this as completed in 0624233 Sep 7, 2016
@ghost
Copy link

ghost commented Sep 7, 2016

This probably fixes it, though I didn't try it myself.

@OrdinaryMagician
Copy link
Author

It did.

@ghost
Copy link

ghost commented Sep 7, 2016

Thanks. Are you actually using rsound, or was it just automatically enabled because it was on your system?

@OrdinaryMagician
Copy link
Author

Possibly the latter. I haven't checked what the AUR package enables.

@qmega
Copy link
Contributor

qmega commented Sep 12, 2016

If you're using the mpv-git AUR package, it doesn't explicitly enable anything other than the shared library and zsh completion. mpv's build enables whatever it finds automatically and the package picks up whatever dependencies are linked against. If rsound were to be removed or default-disabled in mpv, the package would stop using it next time you built it. I think you'd have to use --ao=rsound to actually use it, so if you don't know you are then you're probably not.

On Sep 12, 2016, at 07:26, Marisa Kirisame notifications@github.com wrote:

Possibly the latter. I haven't checked what the AUR package enables.


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@OrdinaryMagician
Copy link
Author

Oh well, I do have rsound installed and it's listed as a dependency of mpv-build-git so...

@ghost
Copy link

ghost commented Sep 14, 2016

I've only asked because I never met anyone who actually used it. (As in, really used it, not just built it.)

This issue was closed.
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