Last active
June 6, 2020 23:42
-
-
Save EvanKrall/bd9a6088ce37b2d67b17e80716c04865 to your computer and use it in GitHub Desktop.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
diff --git a/cmake/Modules/GrSwig.cmake b/cmake/Modules/GrSwig.cmake | |
index 9849447..10d32ec 100644 | |
--- a/cmake/Modules/GrSwig.cmake | |
+++ b/cmake/Modules/GrSwig.cmake | |
@@ -114,7 +114,7 @@ macro(GR_SWIG_MAKE name) | |
endif() | |
#append additional include directories | |
- find_package(PythonLibs 2) | |
+ find_package(PythonLibs 3) | |
list(APPEND GR_SWIG_INCLUDE_DIRS ${PYTHON_INCLUDE_PATH}) #deprecated name (now dirs) | |
list(APPEND GR_SWIG_INCLUDE_DIRS ${PYTHON_INCLUDE_DIRS}) | |
list(APPEND GR_SWIG_INCLUDE_DIRS ${CMAKE_CURRENT_SOURCE_DIR}) | |
diff --git a/op25/gr-op25/CMakeLists.txt b/op25/gr-op25/CMakeLists.txt | |
index 3ab2a79..3c79adf 100644 | |
--- a/op25/gr-op25/CMakeLists.txt | |
+++ b/op25/gr-op25/CMakeLists.txt | |
@@ -94,11 +94,12 @@ find_package(CppUnit) | |
# set(GR_REQUIRED_COMPONENTS RUNTIME BLOCKS FILTER ...) | |
# find_package(Gnuradio "version") | |
set(GR_REQUIRED_COMPONENTS RUNTIME BLOCKS FILTER PMT) | |
-find_package(Gnuradio) | |
- | |
-if(NOT GNURADIO_RUNTIME_FOUND) | |
- message(FATAL_ERROR "GnuRadio Runtime required to compile op25") | |
+SET(MIN_GR_VERSION "3.8.0") | |
+find_package(Gnuradio REQUIRED) | |
+if("${Gnuradio_VERSION}" VERSION_LESS MIN_GR_VERSION) | |
+ MESSAGE(FATAL_ERROR "GnuRadio version required: >=\"" ${MIN_GR_VERSION} "\" found: \"" ${Gnuradio_VERSION} "\"") | |
endif() | |
+ | |
if(NOT CPPUNIT_FOUND) | |
message(FATAL_ERROR "CppUnit required to compile op25") | |
endif() | |
diff --git a/op25/gr-op25/lib/CMakeLists.txt b/op25/gr-op25/lib/CMakeLists.txt | |
index 255ef93..51b4154 100644 | |
--- a/op25/gr-op25/lib/CMakeLists.txt | |
+++ b/op25/gr-op25/lib/CMakeLists.txt | |
@@ -56,7 +56,7 @@ list(APPEND op25_sources | |
) | |
add_library(gnuradio-op25 SHARED ${op25_sources}) | |
-target_link_libraries(gnuradio-op25 ${Boost_LIBRARIES} ${GNURADIO_RUNTIME_LIBRARIES} itpp pcap) | |
+target_link_libraries(gnuradio-op25 ${Boost_LIBRARIES} gnuradio::gnuradio-runtime itpp pcap) | |
set_target_properties(gnuradio-op25 PROPERTIES DEFINE_SYMBOL "gnuradio_op25_EXPORTS") | |
######################################################################## | |
diff --git a/op25/gr-op25/python/__init__.py b/op25/gr-op25/python/__init__.py | |
index 428279d..7338b06 100644 | |
--- a/op25/gr-op25/python/__init__.py | |
+++ b/op25/gr-op25/python/__init__.py | |
@@ -42,7 +42,7 @@ if _RTLD_GLOBAL != 0: | |
# import swig generated symbols into the op25 namespace | |
-from op25_swig import * | |
+from .op25_swig import * | |
# import any pure python here | |
# | |
diff --git a/op25/gr-op25/swig/CMakeLists.txt b/op25/gr-op25/swig/CMakeLists.txt | |
index e99226f..2530c45 100644 | |
--- a/op25/gr-op25/swig/CMakeLists.txt | |
+++ b/op25/gr-op25/swig/CMakeLists.txt | |
@@ -21,7 +21,7 @@ | |
# Include swig generation macros | |
######################################################################## | |
find_package(SWIG) | |
-find_package(PythonLibs 2) | |
+find_package(PythonLibs 3) | |
if(NOT SWIG_FOUND OR NOT PYTHONLIBS_FOUND) | |
return() | |
endif() | |
@@ -31,9 +31,7 @@ include(GrPython) | |
######################################################################## | |
# Setup swig generation | |
######################################################################## | |
-foreach(incdir ${GNURADIO_RUNTIME_INCLUDE_DIRS}) | |
- list(APPEND GR_SWIG_INCLUDE_DIRS ${incdir}/gnuradio/swig) | |
-endforeach(incdir) | |
+set(GR_SWIG_INCLUDE_DIRS $<TARGET_PROPERTY:gnuradio::runtime_swig,INTERFACE_INCLUDE_DIRECTORIES>) | |
set(GR_SWIG_LIBRARIES gnuradio-op25) | |
set(GR_SWIG_DOC_FILE ${CMAKE_CURRENT_BINARY_DIR}/op25_swig_doc.i) | |
diff --git a/op25/gr-op25_repeater/CMakeLists.txt b/op25/gr-op25_repeater/CMakeLists.txt | |
index 68d51e8..6a1ab48 100644 | |
--- a/op25/gr-op25_repeater/CMakeLists.txt | |
+++ b/op25/gr-op25_repeater/CMakeLists.txt | |
@@ -91,14 +91,14 @@ find_package(CppUnit) | |
# of GR_REQUIRED_COMPONENTS (in all caps) and change "version" to the | |
# minimum API compatible version required. | |
# | |
-# set(GR_REQUIRED_COMPONENTS RUNTIME BLOCKS FILTER ...) | |
-# find_package(Gnuradio "version") | |
set(GR_REQUIRED_COMPONENTS RUNTIME BLOCKS FILTER PMT) | |
-find_package(Gnuradio) | |
- | |
-if(NOT GNURADIO_RUNTIME_FOUND) | |
- message(FATAL_ERROR "GnuRadio Runtime required to compile op25_repeater") | |
+# find_package(Gnuradio "version") | |
+set(MIN_GR_VERSION "3.8.0") | |
+find_package(Gnuradio REQUIRED) | |
+if("${Gnuradio_VERSION}" VERSION_LESS MIN_GR_VERSION) | |
+ MESSAGE(FATAL_ERROR "GnuRadio version required: >=\"" ${MIN_GR_VERSION} "\" found: \"" ${Gnuradio_VERSION} "\"") | |
endif() | |
+ | |
if(NOT CPPUNIT_FOUND) | |
message(FATAL_ERROR "CppUnit required to compile op25_repeater") | |
endif() | |
diff --git a/op25/gr-op25_repeater/apps/rx.py b/op25/gr-op25_repeater/apps/rx.py | |
index 48834e5..ee95b45 100755 | |
--- a/op25/gr-op25_repeater/apps/rx.py | |
+++ b/op25/gr-op25_repeater/apps/rx.py | |
@@ -848,7 +848,7 @@ class p25_rx_block (gr.top_block): | |
def process_qmsg(self, msg): | |
# return true = end top block | |
- RX_COMMANDS = 'skip lockout hold whitelist reload' | |
+ RX_COMMANDS = b'skip lockout hold whitelist reload' | |
s = msg.to_string() | |
if s == 'quit': return True | |
elif s == 'update': | |
diff --git a/op25/gr-op25_repeater/lib/CMakeLists.txt b/op25/gr-op25_repeater/lib/CMakeLists.txt | |
index 944563d..999cb61 100755 | |
--- a/op25/gr-op25_repeater/lib/CMakeLists.txt | |
+++ b/op25/gr-op25_repeater/lib/CMakeLists.txt | |
@@ -84,7 +84,7 @@ else(CMAKE_SYSTEM_NAME STREQUAL "Darwin") | |
find_library(GR_FILTER_LIBRARY libgnuradio-filter.so ) | |
endif(CMAKE_SYSTEM_NAME STREQUAL "Darwin") | |
set(GR_FILTER_LIBRARIES ${GR_FILTER_LIBRARY}) | |
-target_link_libraries(gnuradio-op25_repeater ${Boost_LIBRARIES} ${GNURADIO_RUNTIME_LIBRARIES} ${GR_FILTER_LIBRARIES} imbe_vocoder) | |
+target_link_libraries(gnuradio-op25_repeater ${Boost_LIBRARIES} gnuradio::gnuradio-runtime ${GR_FILTER_LIBRARIES} imbe_vocoder) | |
set_target_properties(gnuradio-op25_repeater PROPERTIES DEFINE_SYMBOL "gnuradio_op25_repeater_EXPORTS") | |
######################################################################## | |
diff --git a/op25/gr-op25_repeater/python/__init__.py b/op25/gr-op25_repeater/python/__init__.py | |
index 11bf534..bbbc45a 100644 | |
--- a/op25/gr-op25_repeater/python/__init__.py | |
+++ b/op25/gr-op25_repeater/python/__init__.py | |
@@ -42,7 +42,7 @@ if _RTLD_GLOBAL != 0: | |
# import swig generated symbols into the op25_repeater namespace | |
-from op25_repeater_swig import * | |
+from .op25_repeater_swig import * | |
# import any pure python here | |
# | |
diff --git a/op25/gr-op25_repeater/swig/CMakeLists.txt b/op25/gr-op25_repeater/swig/CMakeLists.txt | |
index 1d88bbd..72d7335 100644 | |
--- a/op25/gr-op25_repeater/swig/CMakeLists.txt | |
+++ b/op25/gr-op25_repeater/swig/CMakeLists.txt | |
@@ -21,7 +21,7 @@ | |
# Include swig generation macros | |
######################################################################## | |
find_package(SWIG) | |
-find_package(PythonLibs 2) | |
+find_package(PythonLibs 3) | |
if(NOT SWIG_FOUND OR NOT PYTHONLIBS_FOUND) | |
return() | |
endif() | |
@@ -31,9 +31,7 @@ include(GrPython) | |
######################################################################## | |
# Setup swig generation | |
######################################################################## | |
-foreach(incdir ${GNURADIO_RUNTIME_INCLUDE_DIRS}) | |
- list(APPEND GR_SWIG_INCLUDE_DIRS ${incdir}/gnuradio/swig) | |
-endforeach(incdir) | |
+set(GR_SWIG_INCLUDE_DIRS $<TARGET_PROPERTY:gnuradio::runtime_swig,INTERFACE_INCLUDE_DIRECTORIES>) | |
set(GR_SWIG_LIBRARIES gnuradio-op25_repeater) | |
set(GR_SWIG_DOC_FILE ${CMAKE_CURRENT_BINARY_DIR}/op25_repeater_swig_doc.i) |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment