Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

gtsam: unvendor metis, correct option defaults, drop deprecated option #22323

Merged
merged 20 commits into from
Apr 10, 2024

Conversation

valgur
Copy link
Contributor

@valgur valgur commented Jan 14, 2024

The default values of many of the options did not match the project defaults at https://github.com/borglab/gtsam/blob/4.2.0/cmake/HandleGeneralOptions.cmake#L17-L31

That's not good, as these can cause unexpected subtle differences in behavior.

I also think the application of CMakeToolchain variables were probably been broken altogether due to CMake version being set to < 3.13 and CMP0077 not being applied.

@conan-center-bot

This comment has been minimized.

@conan-center-bot conan-center-bot removed the Missing dependencies Build failed due missing dependencies in Conan Center label Jan 14, 2024
@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@valgur
Copy link
Contributor Author

valgur commented Jan 18, 2024

@RubenRBS Can you maybe somehow check why the Conan v2 build appears to be failing in C3I?

@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@danimtb
Copy link
Member

danimtb commented Feb 27, 2024

@valgur sorry for the late reply.

It seems so be an issue in the test_package at link time:

  test_package.cpp
     Creating library C:/J2/w/prod-v2/bsr/cci-c2efb684/recipes/gtsam/all/test_package/build/msvc-192-x86_64-14-release/Release/test_package.lib and object C:/J2/w/prod-v2/bsr/cci-c2efb684/recipes/gtsam/all/test_package/build/msvc-192-x86_64-14-release/Release/test_package.exp
libgtsam.lib(Ordering.obj) : error LNK2019: unresolved external symbol METIS_NodeND referenced in function "public: static class gtsam::Ordering __cdecl gtsam::Ordering::Metis(class gtsam::MetisIndex const &)" (?Metis@Ordering@gtsam@@SA?AV12@AEBVMetisIndex@2@@Z) [C:\J2\w\prod-v2\bsr\cci-c2efb684\recipes\gtsam\all\test_package\build\msvc-192-x86_64-14-release\test_package.vcxproj]
C:\J2\w\prod-v2\bsr\cci-c2efb684\recipes\gtsam\all\test_package\build\msvc-192-x86_64-14-release\Release\test_package.exe : fatal error LNK1120: 1 unresolved externals [C:\J2\w\prod-v2\bsr\cci-c2efb684\recipes\gtsam\all\test_package\build\msvc-192-x86_64-14-release\test_package.vcxproj]

https://c3i.jfrog.io/c3i/misc-v2/summary.html?json=https://c3i.jfrog.io/c3i/misc-v2/logs/pr/22323/13-windows-msvc/gtsam/4.0.3//summary.json

@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

  add_library cannot create ALIAS target "metis-gtsam-if" because target
  "metis::metis" is imported but not globally visible.
@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@conan-center-bot

This comment has been minimized.

@conan-center-bot
Copy link
Collaborator

Conan v1 pipeline ✔️

All green in build 5 (cdcd70f580d1d06074ba47195e27d4607bfbc614):

  • gtsam/4.2:
    All packages built successfully! (All logs)

  • gtsam/4.1.1:
    All packages built successfully! (All logs)

  • gtsam/4.0.3:
    All packages built successfully! (All logs)


Conan v2 pipeline ✔️

Note: Conan v2 builds are now mandatory. Please read our discussion about it.

All green in build 3 (cdcd70f580d1d06074ba47195e27d4607bfbc614):

  • gtsam/4.2:
    All packages built successfully! (All logs)

  • gtsam/4.1.1:
    All packages built successfully! (All logs)

  • gtsam/4.0.3:
    All packages built successfully! (All logs)

@conan-center-bot conan-center-bot merged commit 28686f4 into conan-io:master Apr 10, 2024
23 checks passed
yhsng pushed a commit to yhsng/conan-center-index that referenced this pull request Apr 12, 2024
… deprecated option

* gtsam: bump deps

* gtsam: drop deprecated option

* gtsam: fix default_options not matching project defaults

https://github.com/borglab/gtsam/blob/4.2.0/cmake/HandleGeneralOptions.cmake#L17-L31

* gtsam: require newer CMake standard

* gtsam: tidy

* gtsam: unvendor Metis

* gtsam: add topics

* gtsam: with_vendored_metis cannot be disabled on C3I due to non-default option

* gtsam: fix v4.1.1 patch

* gtsam: fix v4.0.3 patch

* gtsam: fix v4.0.3 build

* gtsam: fix TBB handling

* gtsam: support_nested_dissection is broken on MSVC v4.0.3

* gtsam: with_vendored_metis is not compatible with MSVC

* gtsam: always use METIS from Conan

* gtsam: bump deps

* gtsam: fix CMake error on METIS from Conan

  add_library cannot create ALIAS target "metis-gtsam-if" because target
  "metis::metis" is imported but not globally visible.
ericLemanissier pushed a commit to ericLemanissier/conan-center-index that referenced this pull request Apr 16, 2024
… deprecated option

* gtsam: bump deps

* gtsam: drop deprecated option

* gtsam: fix default_options not matching project defaults

https://github.com/borglab/gtsam/blob/4.2.0/cmake/HandleGeneralOptions.cmake#L17-L31

* gtsam: require newer CMake standard

* gtsam: tidy

* gtsam: unvendor Metis

* gtsam: add topics

* gtsam: with_vendored_metis cannot be disabled on C3I due to non-default option

* gtsam: fix v4.1.1 patch

* gtsam: fix v4.0.3 patch

* gtsam: fix v4.0.3 build

* gtsam: fix TBB handling

* gtsam: support_nested_dissection is broken on MSVC v4.0.3

* gtsam: with_vendored_metis is not compatible with MSVC

* gtsam: always use METIS from Conan

* gtsam: bump deps

* gtsam: fix CMake error on METIS from Conan

  add_library cannot create ALIAS target "metis-gtsam-if" because target
  "metis::metis" is imported but not globally visible.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants