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.

Sign up for GitHub

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

Jump to bottom

Explicit -DADIOS2_USE_Python=ON fails #193

Closed
williamfgc opened this issue Jun 27, 2017 · 6 comments
Closed

Explicit -DADIOS2_USE_Python=ON fails #193

williamfgc opened this issue Jun 27, 2017 · 6 comments
Assignees
@chuckatkins
Labels
bug

Comments

@williamfgc
Copy link
Contributor

-- Found PythonModule_mpi4py: /usr/lib/python2.7/dist-packages/mpi4py
CMake Error at bindings/python/CMakeLists.txt:37 (install):
install TARGETS given no LIBRARY DESTINATION for module target "adios2py".
@williamfgc williamfgc mentioned this issue Jun 27, 2017
Closed
@chuckatkins
Copy link
Contributor

Does it work for ADIOS2_USE_Python=AUTO?

@williamfgc
Copy link
Contributor Author

williamfgc commented Jun 27, 2017 •
edited
Loading

AUTO, OFF and no passing anything work as intended. It only fails with explicit ON. Can you reproduce the error? I am using Ubuntu 16.04 LTS

@chuckatkins
Copy link
Contributor

Yep, I can reproduce it.

chuckatkins pushed a commit to chuckatkins/ADIOS2 that referenced this issue Jun 27, 2017
Always check for python dev libs when ADIOS2_HAVE_PYTHON is TRUE
5c2761f 
Fixes ornladios#193
@chuckatkins chuckatkins mentioned this issue Jun 27, 2017
Merged
@williamfgc
Copy link
Contributor Author

Great, thanks!

@chuckatkins
Copy link
Contributor

Does it work for you now?

@williamfgc
Copy link
Contributor Author

williamfgc commented Jun 27, 2017 •
edited
Loading

It works, but I found another issue. cmake test fails since I compile in a different location from the one before. This new location for adios2.so is not in $PYTHONPATH
Start 24: PythonBPWrite
24/25 Test #24: PythonBPWrite ....................................***Failed 0.12 sec
I will merge this and open a new issue

chuckatkins pushed a commit to chuckatkins/ADIOS2 that referenced this issue Jan 7, 2019
pugixml 2019-01-01 (9fcae40b)
4376f6b 
Code extracted from:

    https://github.com/zeux/pugixml.git

at commit 9fcae40bb5577381a37c493a34633f7948fab208 (master).

Upstream Shortlog
-----------------

Alexander Straub (1):
      4c791586 Suffixes for different build types (ornladios#191)

Arseny Kapoulkine (38):
      951f1ed4 Add a comment to clarify subtle branch in node_copy_tree
      cb0e8937 docs: Update v1.9 release date to 04.04
      4f9af798 Work around gcc-8 warning
      cff1933e docs: Update changelog
      a3258528 tests: Fix PUGIXML_NO_STL build
      0c74e117 Update version to 1.9
      be260a09 docs: Regenerate HTML documentation
      62728d9c tests: Fix PUGIXML_NO_EXCEPTIONS tests
      5f4afe3b tests: Fix PUGIXML_NO_XPATH build
      474a4a3f tests: Fix PUGIXML_NO_XPATH,PUGIXML_COMPACT build
      8436f2a6 tests: Fix PUGIXML_COMPACT build on some platforms
      341cea5a docs: Fix changelog formatting
      55277326 Add brief license information back to README.md
      e6dde9cf scripts: Use LICENSE file in pugixml.podspec
      24a7064e Update README.md
      43b0a6a0 Clarify comment in get_strconv_attribute
      c53fdab9 Clarify comment in get_strconv_pcdata as well
      51322cff Move CMake build postfix setup behind an off-by-default USE_POSTFIX
      e584ea33 docs: Mention that node is a container of children in ranged for section
      699143c5 scripts: Set file permissions when building .zip archive
      f3139f4c Add .gitattributes file
      514478d1 Fix version comment; 190 = 1.9.0 (major.minor.patch)
      6b9c07e6 tests: Allow document_load_file_special_folder to load empty document
      cb4a74d5 Minor CMakeLists.txt cleanup
      1a96777b tests: Fix XPath denorm tests on Intel compiler
      e3b5e9ce XPath: Refactor xpath_node_set short buffer optimization
      81c82588 Work around clang --analyze warnings
      d9fadc74 XPath: Workaround Coverity false positive
      fa686f00 Update all URLs to pugixml.org to https://
      aac75cd2 Escape TAB character in attribute values with 	
      f9a2a7d1 Fix Wdouble-promotion warnings
      1a9c3f66 Enable config=sanitize in Travis CI
      7664bbf9 tests: Only use load_file_special_folder test on macOS
      4b57d098 Add support for MinGW builds on AppVeyor
      7d0d8ee7 Update .travis.yml
      2da15249 Add PUGIXML_WCHAR_MODE configuration to MinGW tests
      ba84465d Move unreachable line handling to Makefile
      9fcae40b Happy New Year!

Bernd Amend (1):
      12139f58 fix cmake warning "Policy CMP0048 is not set" (ornladios#214)

Bruno Pagani (1):
      61f2180d Split LICENSE to a separate file for easier distribution

Dan Lipsa (1):
      273fa0ab Remove warning in Visual Studio (ornladios#235)

Eli Schwartz (1):
      daeb8013 cmake: always install the pkg-config file (ornladios#193)

Millian Poquet (1):
      b3db08ff pkg-config: Use CMake GnuInstallDirs FULL vars

Vyacheslav Egorov (1):
      c51214f2 Visual Studio Natvis visualization (ornladios#227)

Wolfgang Stöggl (2):
      c12889f7 Use CMAKE_INSTALL_LIBDIR for pugixml.pc (ornladios#215)
      d3dd79d5 Update URL to https in pugixml.pc.in

cecilios (1):
      7d2436ec Add coverity annotations
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@chuckatkins chuckatkins

Labels
bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@chuckatkins @williamfgc