Gets rid of librt, which doesn't exist on OpenBSD. The version of librtlsdr in the OpenBSD ports tree is extremely old (~2013), so this should help some users.
Tested against tag 0.6.0, but it should apply just fine to HEAD.
Thanks, Brian Waters
Looks like I mistakenly included the patch as an attachment, so it may not have gone through. Here it is:
diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt index 07d64ab..015fd48 100644 --- a/src/CMakeLists.txt +++ b/src/CMakeLists.txt @@ -125,7 +125,7 @@ if(UNIX) target_link_libraries(rtl_fm m) target_link_libraries(rtl_adsb m) target_link_libraries(rtl_power m) -if(APPLE) +if(APPLE OR CMAKE_SYSTEM MATCHES "OpenBSD") target_link_libraries(rtl_test m) else() target_link_libraries(rtl_test m rt)
Thanks, BW
is this applicable to the other BSDs too?
Ideally the rule would be "only link in rt if it's present", right?
-adrian
On Sat, 9 Nov 2019 at 03:08, Brian Waters brian@brianmwaters.net wrote:
Looks like I mistakenly included the patch as an attachment, so it may not have gone through. Here it is:
diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt index 07d64ab..015fd48 100644 --- a/src/CMakeLists.txt +++ b/src/CMakeLists.txt @@ -125,7 +125,7 @@ if(UNIX) target_link_libraries(rtl_fm m) target_link_libraries(rtl_adsb m) target_link_libraries(rtl_power m) -if(APPLE) +if(APPLE OR CMAKE_SYSTEM MATCHES "OpenBSD") target_link_libraries(rtl_test m) else() target_link_libraries(rtl_test m rt)
Thanks, BW
-- Brian M. Waters brian@brianmwaters.net
On Tue, Nov 5, 2019, at 5:05 PM, Brian Waters wrote:
Gets rid of librt, which doesn't exist on OpenBSD. The version of librtlsdr in the OpenBSD ports tree is extremely old (~2013), so this should help some users.
Tested against tag 0.6.0, but it should apply just fine to HEAD.
Thanks, Brian Waters
-- Brian M. Waters brian@brianmwaters.net Attachments:
- openbsd.patch
Hi Brian,
On Tue, Nov 05, 2019 at 05:05:47PM -0500, Brian Waters wrote:
Gets rid of librt, which doesn't exist on OpenBSD. The version of librtlsdr in the OpenBSD ports tree is extremely old (~2013), so this should help some users.
thanks, applied.