astro-informatics / ssht

Fast and exact spin spherical harmonic transforms
http://astro-informatics.github.io/ssht/
GNU General Public License v3.0
11 stars 2 forks source link

Installation failed. Error while parsing conanfile.txt #72

Closed klb2 closed 1 year ago

klb2 commented 1 year ago

Describe the bug When trying to install the package, an error related to parsing the conanfile.txt file is thrown. This occurs for both the Python installation (pip install pyssht) and C installation (cmake -DCMAKE_BUILD_TYPE=Release -DCMAKE_INSTALL_PREFIX=/usr/local -Dconan_deps=True ..).

To Reproduce Steps to reproduce the behavior:

  1. Run the pip installation command pip install pyssht

Expected behavior The package installs without errors.

Desktop/Versions

Error message The following error message is shown (shortened):

      -- Trying 'Ninja' generator
      --------------------------------
      Not searching for unused variables given on the command line.
      -- The C compiler identification is GNU 12.2.1
      -- Detecting C compiler ABI info
      -- Detecting C compiler ABI info - done
      -- Check for working C compiler: /usr/bin/cc - skipped
      -- Detecting C compile features
      -- Detecting C compile features - done
      -- Configuring done
      -- Generating done
      -- Build files have been written to: /tmp/pip-install-cz0asrss/pyssht_e462555ff39346da8ccf502b3882581e/_cmake_test_compile/build
      --
      --------------------------------
      -- Trying 'Ninja' generator - success
      --------------------------------------------------------------------------------
-- The C compiler identification is GNU 12.2.1
      -- Detecting C compiler ABI info
      -- Detecting C compiler ABI info - done
      -- Check for working C compiler: /usr/bin/cc - skipped
      -- Detecting C compile features
      -- Detecting C compile features - done
      -- Downloading conan.cmake from https://github.com/conan-io/cmake-conan
      -- Conan: checking conan executable
      -- Conan: Found program /tmp/pip-build-env-fnsjarpm/overlay/bin/conan
      -- Conan: Version found Conan version 2.0.1

      -- Conan executing: /tmp/pip-build-env-fnsjarpm/overlay/bin/conan install /tmp/pip-install-cz0asrss/pyssht_e462555ff39346da8ccf502b3882581e/conanfile.txt --remote conancenter --generator cmake_find_package --generator cmake_paths --build missing --settings build_type=Release --settings compiler=gcc --settings compiler.version=12

      ======== Input profiles ========
      Profile host:
      [settings]
      arch=x86_64
      build_type=Release
      compiler=gcc
      compiler.cppstd=gnu17
      compiler.libcxx=libstdc++11
      compiler.version=12
      os=Linux

      Profile build:
      [settings]
      arch=x86_64
      build_type=Release
      compiler=gcc
      compiler.cppstd=gnu17
      compiler.libcxx=libstdc++11
      compiler.version=12
      os=Linux

      ERROR: Error while parsing [options] in conanfile.txt
      Options should be specified as 'pkg/*:option=value'
      CMake Error at _skbuild/linux-x86_64-3.10/cmake-build/conan.cmake:631 (message):
        Conan install failed='1'
      Call Stack (most recent call first):
        cmake/conan_dependencies.cmake:12 (conan_cmake_install)
        CMakeLists.txt:17 (include)

      -- Configuring incomplete, errors occurred!
jasonmcewen commented 1 year ago

Thanks for letting us know. Conan was updated recently, which broke the installation. I think this is fixed now (@mdavezac, can you confirm and if so can we close this?). Hopefully you should be able to install now but let us know if you are still having problems?

mdavezac commented 1 year ago

It was corrected yesterday in version 1.5.2. Could you give it another go?

klb2 commented 1 year ago

Thank you for fixing it. The installation using pip worked now. However, the same issue is also present in the s2let package. (I now opened an issue there too https://github.com/astro-informatics/s2let/issues/47)

mdavezac commented 1 year ago

Thanks @klb2, yes the issue should be present on so3 and s2let. It should be all fixed by Monday.