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 2108 - Build issue: Arts on openbsd
Summary: Build issue: Arts on openbsd
Status: RESOLVED FIXED
Alias: None
Product: TDE
Classification: Unclassified
Component: non-core programs (show other bugs)
Version: R14.0.0 [Trinity]
Hardware: All Other
: P5 normal
Assignee: Francois Andriot
URL:
Depends on:
Blocks:
 
Reported: 2014-09-06 09:06 CDT by Francois Andriot
Modified: 2014-09-15 14:50 CDT (History)
3 users (show)

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

Attachments
Arts: fix hardcoded link flag to "dl" library (294 bytes, patch)
2014-09-06 09:08 CDT, Francois Andriot 		Details | Diff
cmake: fix openbsd library naming convention (1.53 KB, patch)
2014-09-06 09:10 CDT, Francois Andriot 		Details | Diff
Arts: add sndio support (part 1) + fix "dl" library detection (1.37 KB, patch)
2014-09-06 09:11 CDT, Francois Andriot 		Details | Diff
Arts: add sndio support (part 2) (211 bytes, patch)
2014-09-06 09:11 CDT, Francois Andriot 		Details | Diff
Arts: add sndio support (part 3) (6.42 KB, patch)
2014-09-06 09:11 CDT, Francois Andriot 		Details | Diff
Arts: add sndio support (part 4) (967 bytes, patch)
2014-09-06 09:12 CDT, Francois Andriot 		Details | Diff
Show Obsolete (6) 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-09-06 09:06:47 CDT 
This bug report is intented to post all patches required to build Arts on openbsd.
Comment 1 Francois Andriot 2014-09-06 09:08:33 CDT 
Created attachment 2150 [details]
Arts: fix hardcoded link flag to "dl" library
Comment 2 Francois Andriot 2014-09-06 09:10:05 CDT 
Created attachment 2151 [details]
cmake: fix openbsd library naming convention

This patch is not specific to Arts. It is common to all TDE packages.
Comment 3 Francois Andriot 2014-09-06 09:11:05 CDT 
Created attachment 2152 [details]
Arts: add sndio support (part 1) + fix "dl" library detection
Comment 4 Francois Andriot 2014-09-06 09:11:23 CDT 
Created attachment 2153 [details]
Arts: add sndio support (part 2)
Comment 5 Francois Andriot 2014-09-06 09:11:44 CDT 
Created attachment 2154 [details]
Arts: add sndio support (part 3)
Comment 6 Francois Andriot 2014-09-06 09:12:28 CDT 
Created attachment 2155 [details]
Arts: add sndio support (part 4)

The "sndio" is the audio subsystem for openbsd.
Comment 7 Slávek Banko 2014-09-06 18:57:35 CDT 
Patches look good. For me no objections.

François, you're ready to have any additional patches for admin and cmake common modules?
Comment 8 Francois Andriot 2014-09-07 02:49:03 CDT 
(In reply to Slávek Banko from comment #7)
> Patches look good. For me no objections.
> 
> François, you're ready to have any additional patches for admin and cmake
> common modules?

No, this is the only cmake/admin patch that I have now.
Comment 9 Slávek Banko 2014-09-14 12:50:37 CDT 
Comment on attachment 2151 [details]
cmake: fix openbsd library naming convention

Pushed to GIT in hash 38a8a335.
Comment 10 Slávek Banko 2014-09-14 13:35:46 CDT 
Comment on attachment 2150 [details]
Arts: fix hardcoded link flag to "dl" library

Pushed to GIT in hash c73ca85a.
Comment 11 Slávek Banko 2014-09-14 13:47:06 CDT 
Openbsd sndio support pushed in GIT hash d6fd5da5.
Comment 12 Francois Andriot 2014-09-15 13:11:26 CDT 
Slavek, currently the cmake step fails in openbsd because the check_function_exists( dlopen HAVE_DLOPEN ) was put BEFORE the include( CheckFunctionExists )

Either move the check_function_exits() down or move the include up...
Comment 13 Slávek Banko 2014-09-15 14:50:39 CDT 
(In reply to Francois Andriot from comment #12)
> Slavek, currently the cmake step fails in openbsd because the
> check_function_exists( dlopen HAVE_DLOPEN ) was put BEFORE the include(
> CheckFunctionExists )
> 
> Either move the check_function_exits() down or move the include up...

Thank you for heads up. All includes was moved to top of the CMakeLists.txt in GIT hash e39b3d2d.