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

Summary: Build issue: Arts on openbsd
Product: TDE Reporter: Francois Andriot <albator78>
Component: non-core programsAssignee: Francois Andriot <albator78>
Status: RESOLVED FIXED    
Severity: normal CC: albator78, bugwatch, slavek.banko
Priority: P5    
Version: R14.0.0 [Trinity]   
Hardware: All   
OS: Other   
Compiler Version: TDE Version String:
Application Version: Application Name:
Attachments: Arts: fix hardcoded link flag to "dl" library
cmake: fix openbsd library naming convention
Arts: add sndio support (part 1) + fix "dl" library detection
Arts: add sndio support (part 2)
Arts: add sndio support (part 3)
Arts: add sndio support (part 4)

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.