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 - Build issue: artsc-config script returns invalid header directory
Summary: Build issue: artsc-config script returns invalid header directory
Status: RESOLVED FIXED
Alias: None
Product: TDE
Classification: Unclassified
Component: arts (show other bugs)
Version: R14.0.0 [Trinity]
Hardware: Other Linux
: P5 normal
Assignee: Francois Andriot
URL:
Depends on:
Blocks:
 
Reported: 2014-10-12 12:17 CDT by Francois Andriot
Modified: 2014-11-26 02:16 CST (History)
3 users (show)

See Also:
Compiler Version:
TDE Version String:
Application Version:
Application Name:

Attachments
arts: fix artsc-config script to return valid include directory (799 bytes, patch)
2014-10-12 12:19 CDT, Francois Andriot 		Details | Diff
arts: fix artsc-config script to return valid include directory (2) (916 bytes, patch)
2014-10-12 12:31 CDT, Francois Andriot 		Details | Diff
arts: fix artsc-config script to return valid include directory (3) (544 bytes, patch)
2014-10-16 16:08 CDT, Francois Andriot 		Details | Diff
arts: fix artsc-config script to return valid include directory (4) (808 bytes, patch)
2014-10-16 16:23 CDT, Francois Andriot 		Details | Diff
Show Obsolete (4) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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.