Opened 14 years ago

Closed 14 years ago

Last modified 14 years ago

#1310 closed patch (invalid)

aRts detection problem - don't try to be smarter than library author is

Reported by: Cougar <cougar@…> 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)

configure-9014.patch (534 bytes) - added by Cougar <cougar@…> 14 years ago.

Download all attachments as: .zip

Change History (4)

Changed 14 years ago by Cougar <cougar@…>

Attachment: configure-9014.patch added

comment:1 Changed 14 years ago by bjm

Resolution: invalid
Status: newclosed

Post question and opinions to the mailing lists.

comment:2 Changed 14 years ago by Cougar <cougar@…>

Type: defectpatch

comment:3 Changed 14 years ago by Elan Ruusamäe <glen@…>

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.

Note: See TracTickets for help on using tickets.