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

openmpi - more requirements #11832

Closed

Conversation

paulharris
Copy link
Contributor

I found these were required while building cgns like so:

conan create . 4.3.0@cci/test -o cgns:shared=True -o cgns:with_hdf5=True -o cgns:parallel=False -o hdf5:parallel=True -o hdf5:enable_cxx=False --build=missing

I needed libudev and libpciaccess,
otherwise it complained about unresolved symbols including:

/usr/bin/ld: /build/conandata/conan-data/cgns/4.3.0/ccitest/shared/package/d119f4ebdc23c0fcf69efad1aef71967dd875de6/lib/libcgns.so: undefined reference to `pci_device_probe'
/usr/bin/ld: /build/conandata/conan-data/cgns/4.3.0/ccitest/shared/package/d119f4ebdc23c0fcf69efad1aef71967dd875de6/lib/libcgns.so: undefined reference to `pci_device_get_vendor_name'
/usr/bin/ld: /build/conandata/conan-data/cgns/4.3.0/ccitest/shared/package/d119f4ebdc23c0fcf69efad1aef71967dd875de6/lib/libcgns.so: undefined reference to `udev_device_get_property_value'
/usr/bin/ld: /build/conandata/conan-data/cgns/4.3.0/ccitest/shared/package/d119f4ebdc23c0fcf69efad1aef71967dd875de6/lib/libcgns.so: undefined reference to `pci_slot_match_iterator_create'
/usr/bin/ld: /build/conandata/conan-data/cgns/4.3.0/ccitest/shared/package/d119f4ebdc23c0fcf69efad1aef71967dd875de6/lib/libcgns.so: undefined reference to `pci_iterator_destroy'
/usr/bin/ld: /build/conandata/conan-data/cgns/4.3.0/ccitest/shared/package/d119f4ebdc23c0fcf69efad1aef71967dd875de6/lib/libcgns.so: undefined reference to `pci_device_next'
/usr/bin/ld: /build/conandata/conan-data/cgns/4.3.0/ccitest/shared/package/d119f4ebdc23c0fcf69efad1aef71967dd875de6/lib/libcgns.so: undefined reference to `pci_system_init'
/usr/bin/ld: /build/conandata/conan-data/cgns/4.3.0/ccitest/shared/package/d119f4ebdc23c0fcf69efad1aef71967dd875de6/lib/libcgns.so: undefined reference to `udev_device_unref'
/usr/bin/ld: /build/conandata/conan-data/cgns/4.3.0/ccitest/shared/package/d119f4ebdc23c0fcf69efad1aef71967dd875de6/lib/libcgns.so: undefined reference to `udev_device_new_from_subsystem_sysname'
/usr/bin/ld: /build/conandata/conan-data/cgns/4.3.0/ccitest/shared/package/d119f4ebdc23c0fcf69efad1aef71967dd875de6/lib/libcgns.so: undefined reference to `pci_system_cleanup'
/usr/bin/ld: /build/conandata/conan-data/cgns/4.3.0/ccitest/shared/package/d119f4ebdc23c0fcf69efad1aef71967dd875de6/lib/libcgns.so: undefined reference to `pci_device_cfg_read'
/usr/bin/ld: /build/conandata/conan-data/cgns/4.3.0/ccitest/shared/package/d119f4ebdc23c0fcf69efad1aef71967dd875de6/lib/libcgns.so: undefined reference to `pci_device_get_device_name'
/usr/bin/ld: /build/conandata/conan-data/cgns/4.3.0/ccitest/shared/package/d119f4ebdc23c0fcf69efad1aef71967dd875de6/lib/libcgns.so: undefined reference to `udev_new'
/usr/bin/ld: /build/conandata/conan-data/cgns/4.3.0/ccitest/shared/package/d119f4ebdc23c0fcf69efad1aef71967dd875de6/lib/libcgns.so: undefined reference to `udev_unref'
collect2: error: ld returned 1 exit status
ninja: build stopped: subcommand failed.

  • I've read the guidelines for contributing.
  • I've followed the PEP8 style guides for Python code in the recipes.
  • I've used the latest Conan client version.
  • I've tried at least one configuration locally with the conan-center hook activated.

I found these were required while building cgns with parallel enabled,
in shared mode.
@ghost
Copy link

ghost commented Jul 21, 2022

I detected other pull requests that are modifying openmpi/all recipe:

This message is automatically generated by https://github.com/ericLemanissier/conan-center-conflicting-prs so don't hesitate to report issues/improvements there.

@conan-center-bot

This comment has been minimized.

@paulharris
Copy link
Contributor Author

paulharris commented Jul 21, 2022

I'm also going to add 'm', thanks to these warnings.

[HOOK - conan-center.py] post_package(): WARN: [MISSING SYSTEM LIBS (KB-H043)] Library './lib/libmca_common_monitoring.so' links to system library 'm' but it is not in cpp_info.system_libs.
[HOOK - conan-center.py] post_package(): WARN: [MISSING SYSTEM LIBS (KB-H043)] Library './lib/libmpi.so' links to system library 'm' but it is not in cpp_info.system_libs.
[HOOK - conan-center.py] post_package(): WARN: [MISSING SYSTEM LIBS (KB-H043)] Library './lib/libmca_common_ompio.so' links to system library 'm' but it is not in cpp_info.system_libs.
[HOOK - conan-center.py] post_package(): WARN: [MISSING SYSTEM LIBS (KB-H043)] Library './lib/libopen-pal.so' links to system library 'm' but it is not in cpp_info.system_libs.
[HOOK - conan-center.py] post_package(): WARN: [MISSING SYSTEM LIBS (KB-H043)] Library './lib/openmpi/mca_coll_libnbc.so' links to system library 'm' but it is not in cpp_info.system_libs.
[HOOK - conan-center.py] post_package(): WARN: [MISSING SYSTEM LIBS (KB-H043)] Library './lib/openmpi/mca_topo_treematch.so' links to system library 'm' but it is not in cpp_info.system_libs.

I'll double-check those other warnings.

@paulharris paulharris changed the title openmpi - add more system lib requirements openmpi - more requirements Jul 21, 2022
@conan-center-bot

This comment has been minimized.

@conan-center-bot
Copy link
Collaborator

Failure in build 3 (763d1e6ad7d1b8ffa4b9547a03f9a8afec20d075):

  • openmpi/4.1.0@:
    Error running command conan export recipes/openmpi/all/conanfile.py openmpi/4.1.0@:
    ERROR: Error loading conanfile at '/home/conan/w/prod_cci_PR-11832/recipes/openmpi/all/conanfile.py': Unable to load conanfile in /home/conan/w/prod_cci_PR-11832/recipes/openmpi/all/conanfile.py
      File "/opt/pyenv/versions/3.7.13/lib/python3.7/imp.py", line 171, in load_source
        module = _load(spec)
      File "<frozen importlib._bootstrap>", line 696, in _load
      File "<frozen importlib._bootstrap>", line 677, in _load_unlocked
      File "<frozen importlib._bootstrap_external>", line 728, in exec_module
      File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
      File "/home/conan/w/prod_cci_PR-11832/recipes/openmpi/all/conanfile.py", line 3, in <module>
        from conan.tools.files import get, chdir, rm, rmdir
    ImportError: cannot import name 'rm' from 'conan.tools.files' (/opt/pyenv/versions/3.7.13/lib/python3.7/site-packages/conan/tools/files/__init__.py)
    

Note: To save resources, CI tries to finish as soon as an error is found. For this reason you might find that not all the references have been launched or not all the configurations for a given reference. Also, take into account that we cannot guarantee the order of execution as it depends on CI workload and workers availability.

@stale
Copy link

stale bot commented Sep 4, 2022

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Sep 4, 2022
@stale
Copy link

stale bot commented Oct 8, 2022

This pull request has been automatically closed because it has not had recent activity. Thank you for your contributions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants