By default, Bugzilla does not search the list of RESOLVED bugs.
You can force it to do so by putting the upper-case word ALL in front of your search query, e.g.: ALL tdelibs
We recommend searching for bugs this way, as you may discover that your bug has already been resolved and fixed in a later release.

Bug 2160

Summary:	Build issue: artsc-config script returns invalid header directory
Product:	TDE	Reporter:	Francois Andriot <albator78>
Component:	arts	Assignee:	Francois Andriot <albator78>
Status:	RESOLVED FIXED
Severity:	normal	CC:	albator78, bugwatch, slavek.banko
Priority:	P5
Version:	R14.0.0 [Trinity]
Hardware:	Other
OS:	Linux
Compiler Version:		TDE Version String:
Application Version:		Application Name:
Attachments:	arts: fix artsc-config script to return valid include directory arts: fix artsc-config script to return valid include directory (2) arts: fix artsc-config script to return valid include directory (3) arts: fix artsc-config script to return valid include directory (4)

Description Francois Andriot 2014-10-12 12:17:23 CDT

Currently, ARTS install headers under:
  /opt/trinity/include/tde/arts
Except for the "artsc" stuf, which installs outside the tde folder:
  /opt/trinity/include/artsc

I don't know if this is on purpose or not, but anyway, the "artsc-config --cflags" returns "-I/opt/trinity/include/tde/artsc" , which is invalid.

Comment 1 Francois Andriot 2014-10-12 12:19:06 CDT

Created attachment 2308 [details]
arts: fix artsc-config script to return valid include directory

In case the current "artsc" directory is correct, the attached patch fixes the "artsc-config" script to return the correct value.

Comment 2 Francois Andriot 2014-10-12 12:31:05 CDT

Created attachment 2310 [details]
arts: fix artsc-config script to return valid include directory (2)

Also, the "$libdl" variable is evaluated "-l" when DL_LIBRARIES is empty, causing invalid LDFLAGS at the end.

Comment 3 Francois Andriot 2014-10-16 16:08:14 CDT

Created attachment 2328 [details]
arts: fix artsc-config script to return valid include directory (3)

Fixed for openbsd and linux.

Comment 4 Francois Andriot 2014-10-16 16:23:16 CDT

Created attachment 2329 [details]
arts: fix artsc-config script to return valid include directory (4)

Forgot second half of the patch. Time to go to bed.

Comment 5 Slávek Banko 2014-11-26 02:15:52 CST

Comment on attachment 2329 [details]
arts: fix artsc-config script to return valid include directory (4)

Added to includedir was also used to install headers.
Pushed to GIT in hash 775e3e46.