Opened 18 years ago
Closed 18 years ago
Last modified 18 years ago
#1310 closed patch (invalid)
aRts detection problem - don't try to be smarter than library author is
Reported by: | Owned by: | Isaac Richards | |
---|---|---|---|
Priority: | minor | Milestone: | unknown |
Component: | mythtv | Version: | head |
Severity: | medium | Keywords: | aRts configure compiler |
Cc: | Ticket locked: | no |
Description
Hello,
In configure script is some sort of hack that right now breaks aRts detection (SuSE 10.0). artsc-config outputs right compiler flags but this hack breaks one. In configure script is include path replacement code that should be removed IMHO.
audio_arts_tmp=`artsc-config --cflags` audio_arts_cflags=`echo $audio_arts_tmp | sed "s/include\/artsc/include/g"`
Also attached patch to fix it. If sombody really needs this I suggest to use both $audio_arts_tmp and $audio_arts_cflags flags together.
Attachments (1)
Change History (4)
Changed 18 years ago by
Attachment: | configure-9014.patch added |
---|
comment:1 Changed 18 years ago by
Resolution: | → invalid |
---|---|
Status: | new → closed |
comment:2 Changed 18 years ago by
Type: | defect → patch |
---|
comment:3 Changed 18 years ago by
Can confirm, in PLD Linux the patch fixed arts compiling problem. So vote +1 for removing for that hack :)
and one should fix arts-config to output correct include dir not to include hack in mythtv. for comparison ticket #946 was marked invalid as complying to API changes in mysqlclient library was not considered mythtv concern.
Post question and opinions to the mailing lists.