Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 626470 - games-strategy/0ad-0.0.22_alpha : missing new dependency media-libs/openal
Summary: games-strategy/0ad-0.0.22_alpha : missing new dependency media-libs/openal
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: AMD64 Linux
: Normal normal (vote)
Assignee: Gentoo Games
URL:
Whiteboard:
Keywords:
: 628532 (view as bug list)
Depends on:
Blocks:
 
Reported: 2017-07-28 17:17 UTC by Arve Barsnes
Modified: 2017-08-22 08:15 UTC (History)
4 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Arve Barsnes 2017-07-28 17:17:27 UTC
Compile of games-strategy/0ad-0.0.22_alpha fails with missing headers if media-libs/openal is not installed. Probably because of https://trac.wildfiregames.com/changeset/19877

When openal is installed, build instead fails on error detailed in #626398.

Reproducible: Always

Steps to Reproduce:
1. Compile new 0ad alpha22 without openal installed
2.
3.
Actual Results:  
Build fails with missing headers

Expected Results:  
Build should have the needed headers available.
Comment 1 augustin 2017-08-05 03:22:56 UTC
$ equery depgraph games-strategy/0ad
* dependency graph for games-strategy/0ad-0.0.22_alpha
   `--  net-libs/gloox-1.0.20  (>=net-libs/gloox-1.0.20) ~amd64 
   `--  media-libs/openal-1.15.1-r2  (media-libs/openal) amd64 
   ...

It appears that the bug was fixed today.
Yesterday, I couldn't compile 0ad either.
Today, emerge update had me add net-libs/gloox in package.accept_keywords and then 0ad just compiled successfully.

Can you sync your system and check again, and mark this as resolved if you can confirm the fix?
Comment 2 Arve Barsnes 2017-08-05 08:28:16 UTC
Have you not adjusted your USE flags? The lobby flag adds gloox, while the sound flag adds openal, so these two dependencies don't look to be related.

Looking at the changeset I linked earlier from 0ad, media-libs/openal can not be a conditional dependency anymore, it needs to be installed even if -sound is specified for a part of the compile. This might be an error on their part.

Personally, I moved to having the sound flag set. It should probably be the default?
Comment 3 augustin 2017-08-06 03:29:55 UTC
I apologize for confusing the issue. If no change were made to the package, I am not sure what happenned in my system, but it's no longer relevant. For the record, I indeed have 0ad +sound +lobby.

Back on topic of openal and sound:

- I am not competent to comment on the hard dependency on openal 
  even with -sound.

- However, I strongly support having the sound flag set by default. The package should probably follow the "Principle of Least Astonishment": https://en.wikipedia.org/wiki/Principle_of_least_astonishment 
Compiling 0ad, start playing it and wonder why there is no sound, and then read the (otherwise excellent) documentation and figure out that one needs to explicitely set +sound is not very user friendly.

Only a tiny minority would play 0ad without sound, which is such an integral part of the game. Even excluding the musical score (which cleverly fits the gameplay, with alternative martial music or peaceful music fitting the action in the game), the sound effects help a lot during the game. How many times do we have a first inkling of being attacked when we *hear* our female citizens screaming in agony as they die...??
Comment 4 Lars Wendler (Polynomial-C) (RETIRED) gentoo-dev 2017-08-22 07:15:12 UTC
commit 63ef41c718a62fa23050d4f8abc9086b0e02feea (HEAD -> master, origin/master, origin/HEAD)             
Author: Lars Wendler <polynomial-c@gentoo.org>      
Date:   Tue Aug 22 09:13:55 2017                    

    games-strategy/0ad: Removed "sound" USE flag.   
                                                    
    A game without sound does not really provide optimal game experience.                                
                                                    
    Package-Manager: Portage-2.3.8, Repoman-2.3.3
Comment 5 Arfrever Frehtes Taifersar Arahesis 2017-08-22 08:15:38 UTC
*** Bug 628532 has been marked as a duplicate of this bug. ***