Merge pull request #75 from rahil-makadia/dev

update build binary file handling

.github/workflows/docs.yml

@@ -26,6 +26,9 @@ jobs:
           python3 -m pip install --upgrade pip
           sudo apt-get install pandoc doxygen
           pip3 install "pybind11[global]>=2.10.0"
+          cd extern
+          python3 get_cspice.py
+          cd ..
           pip3 install .
       - name: Build docs
         run: |

CMakeLists.txt

@@ -38,6 +38,10 @@ find_package(pybind11 REQUIRED)
 pybind11_add_module(libgrss src/${PROJECT_NAME}.cpp)
 target_link_libraries(libgrss PRIVATE ${PROJECT_NAME})
 
+# copy the libgrss* files to the python package directory
+file(GLOB LIBGRSS_FILES ${CMAKE_SOURCE_DIR}/build/libgrss.*)
+file(COPY ${LIBGRSS_FILES} DESTINATION ${CMAKE_SOURCE_DIR}/grss)
+
 # compile tests
 add_subdirectory(tests/cpp/prop)
 

grss/version.txt

@@ -1 +1 @@
-4.1.2
+4.1.3

setup.py

@@ -37,14 +37,9 @@ def build_extension(self, ext: CMakeExtension) -> None:
         if not binary_created:
             raise FileNotFoundError("libgrss binary for C++ source code not found "
                                     "in cmake build directory")
-        os.system(f"cp {ext.sourcedir}/build/libgrss* {self.build_lib}/grss/")
+        os.system(f"cp {ext.sourcedir}/build/libgrss.* {self.build_lib}/grss/")
         return
 
-# run get_cspice in the extern folder if installing from source
-if (not os.path.exists("./extern/cspice/lib/cspice.a") or
-    not os.path.exists("./extern/cspice/include/SpiceUsr.h") ):
-    subprocess.run(["python", "get_cspice.py"], cwd="./extern", check=True)
-
 setup(
     ext_modules=[CMakeExtension("libgrss")],
     cmdclass={"build_ext": CMakeBuild}