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

Add libpulse dependency for sdl #36087

Closed
wants to merge 1 commit into from
Closed

Conversation

kevingranade
Copy link
Member

Summary

SUMMARY: None

Purpose of change

Build fix for clang-tidy build, see e.g. https://travis-ci.org/CleverRaven/Cataclysm-DDA/jobs/624829027

Testing

Just needs to build cleanly.

Additional context

Some other build stages may need the same treatment.

@kevingranade
Copy link
Member Author

Still broken, I'm not clear how to resolve this, will need to investigate some more:

The following packages have unmet dependencies:

 libpulse-dev : Depends: libpulse0 (= 1:8.0-0ubuntu3.10) but 1:8.0-0ubuntu3.11 is to be installed

                Depends: libpulse-mainloop-glib0 (= 1:8.0-0ubuntu3.10) but it is not going to be installed

E: Unable to correct problems, you have held broken packages.
apt-get.diagnostics

apt-get install failed

The command "sudo -E apt-get -yq --no-install-suggests --no-install-recommends $(travis_apt_get_options) install clang-8 libclang-8-dev llvm-8-dev llvm-8-tools libsdl2-dev libsdl2-ttf-dev libsdl2-image-dev libsdl2-mixer-dev libpulse-dev" failed and exited with 100 during .

@jbytheway
Copy link
Contributor

I believe #36131 is a working version of this. Looks like Travis has provided an updated libpulse but not libpulse-dev in their tweaks to the standard Xenial, so you need to explicitly specify a lower version of libpulse to trigger a downgrade.

@ZhilkinSerg
Copy link
Contributor

Closing in favor of #36131.

@kevingranade kevingranade deleted the kevingranade-libpulse-fix branch December 16, 2019 22:15
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.

3 participants