Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 343923 - [pro-audio overlay] media-sound/shism-tracker (all) has excess and somewhat troublesome dependancies.
Summary: [pro-audio overlay] media-sound/shism-tracker (all) has excess and somewhat...
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Frieder Bürzele
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-11-03 00:14 UTC by Elliot Thomas
Modified: 2015-09-06 09:02 UTC (History)
0 users

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 Elliot Thomas 2010-11-03 00:14:50 UTC
media-sound/shismtracker "depends" on alsa-lib, however, shismtracker uses libsdl for audio output, and has no direct dependency on alsa-lib. This makes it impossible for users of systems without alsa-lib to emerge shismtracker. Manual compilation from the latest tarball from their website works perfectly, but is far from ideal.

Reproducible: Always

Steps to Reproduce:
1.Use a system without alsa-lib installed.
2.Try and emerge shismtracker.
3.Watch in horror as alsa-lib is forced upon you.

Actual Results:  
Working program with unwanted libraries.

Expected Results:  
Working program without unwanted libraries.

This "bug" is conceptually easy to fix. Simply removing alsa-lib from RDEPEND should remedy the situation, unless it was added there for a reason.
Comment 1 Samuli Suominen (RETIRED) gentoo-dev 2010-11-03 00:22:55 UTC
headers used:

schism/main.c:#include <alsa/pcm.h>
sys/alsa/midi-alsa.c:#include <alsa/asoundlib.h>
sys/alsa/midi-alsa.c:#include <alsa/control.h>
sys/alsa/midi-alsa.c:#include <alsa/seq.h>
sys/alsa/mixer-alsa.c:#include <alsa/asoundlib.h>
sys/alsa/mixer-alsa.c:#include <alsa/mixer.h>
sys/alsa/mixer-alsa.c:#include <alsa/mixer.h>

library checked from configure:

saved_libs=$LIBS
alsa=no
AC_CHECK_LIB(asound, snd_seq_open,[alsa=yes])
alsadltrick=no
if test "$alsa" = "yes"; then
        if test "$ac_cv_header_sys_soundcard_h" = "yes"; then
                alsadltrick=yes
                LIBS=$saved_libs
        fi
fi
AM_CONDITIONAL([USE_ALSA], [test "$alsa" = yes])
AM_CONDITIONAL([USE_ALSA_DLTRICK], [test "$alsadltrick" = yes])

library loaded at runtime by dlopen:

schism/main.c:        _dltrick_handle = dlopen("libasound.so.2", RTLD_NOW);
schism/main.c:                _dltrick_handle = dlopen("libasound.so", RTLD_NOW);

so I'd say alsa is pretty heavily used by schismtracker directly
Comment 2 Elliot Thomas 2010-11-03 17:31:17 UTC
I stand corrected. However, alsa-lib (or, more correctly, ALSA support) is optional, and the program compiles and runs fine without it. Perhaps there should be an alsa USE flag that depends on alsa-lib instead?
Comment 3 Frieder Bürzele 2015-09-06 09:02:39 UTC
ebuild is no longer in the repository