Skip to content

Gaudi: separate variant for building documentation#16515

Merged
adamjstewart merged 2 commits intospack:developfrom
iarspider:patch-6
May 8, 2020
Merged

Gaudi: separate variant for building documentation#16515
adamjstewart merged 2 commits intospack:developfrom
iarspider:patch-6

Conversation

@iarspider
Copy link
Copy Markdown
Contributor

@iarspider iarspider commented May 7, 2020

There seems to be a bug in how Gaudi (or CMake) runs doxygen, namely - it ignores LD_LIBRARY_PATH, which is required in my case (external compiler without relocation information):

-- Adding directory GaudiRelease/doc/doxygen (22/25)
/workspace/spack_97/spack/opt/spack/linux-centos7-haswell/gcc-8.3.0/doxygen-1.8.15-apcnevjpd3wrjlruqzn4a2g6fvusbiu7/bin/doxygen: /usr/lib64/libstdc++.so.6: version `CXXABI_1.3.8' not found (required by /workspace/spack_97/spack/opt/spack/linux-centos7-haswell/gcc-8.3.0/doxygen-1.8.15-apcnevjpd3wrjlruqzn4a2g6fvusbiu7/bin/doxygen)
/workspace/spack_97/spack/opt/spack/linux-centos7-haswell/gcc-8.3.0/doxygen-1.8.15-apcnevjpd3wrjlruqzn4a2g6fvusbiu7/bin/doxygen: /usr/lib64/libstdc++.so.6: version `CXXABI_1.3.9' not found (required by /workspace/spack_97/spack/opt/spack/linux-centos7-haswell/gcc-8.3.0/doxygen-1.8.15-apcnevjpd3wrjlruqzn4a2g6fvusbiu7/bin/doxygen)
CMake Warning at /workspace/spack_97/spack/opt/spack/linux-centos7-haswell/gcc-8.3.0/cmake-3.14.3-bet5h64ciwfhkxi5gdrpui4vfsgblwru/share/cmake-3.14/Modules/FindDoxygen.cmake:434 (message):
  Unable to determine doxygen version: 1
Call Stack (most recent call first):
  /workspace/spack_97/spack/opt/spack/linux-centos7-haswell/gcc-8.3.0/cmake-3.14.3-bet5h64ciwfhkxi5gdrpui4vfsgblwru/share/cmake-3.14/Modules/FindDoxygen.cmake:597 (_Doxygen_find_doxygen)
  GaudiRelease/doc/doxygen/CMakeLists.txt:18 (find_package)


-- Found Doxygen: /workspace/spack_97/spack/opt/spack/linux-centos7-haswell/gcc-8.3.0/doxygen-1.8.15-apcnevjpd3wrjlruqzn4a2g6fvusbiu7/bin/doxygen (found version "") found components:  doxygen dot 
/workspace/spack_97/spack/opt/spack/linux-centos7-haswell/gcc-8.3.0/doxygen-1.8.15-apcnevjpd3wrjlruqzn4a2g6fvusbiu7/bin/doxygen: /usr/lib64/libstdc++.so.6: version `CXXABI_1.3.8' not found (required by /workspace/spack_97/spack/opt/spack/linux-centos7-haswell/gcc-8.3.0/doxygen-1.8.15-apcnevjpd3wrjlruqzn4a2g6fvusbiu7/bin/doxygen)
/workspace/spack_97/spack/opt/spack/linux-centos7-haswell/gcc-8.3.0/doxygen-1.8.15-apcnevjpd3wrjlruqzn4a2g6fvusbiu7/bin/doxygen: /usr/lib64/libstdc++.so.6: version `CXXABI_1.3.9' not found (required by /workspace/spack_97/spack/opt/spack/linux-centos7-haswell/gcc-8.3.0/doxygen-1.8.15-apcnevjpd3wrjlruqzn4a2g6fvusbiu7/bin/doxygen)
CMake Error at /workspace/spack_97/spack/opt/spack/linux-centos7-haswell/gcc-8.3.0/cmake-3.14.3-bet5h64ciwfhkxi5gdrpui4vfsgblwru/share/cmake-3.14/Modules/FindDoxygen.cmake:676 (message):
  Unable to generate Doxyfile template: 1
Call Stack (most recent call first):
  GaudiRelease/doc/doxygen/CMakeLists.txt:18 (find_package)


-- Configuring incomplete, errors occurred!

(for the record: spack-build-out, spack-build-env ).

@vvolkl
Copy link
Copy Markdown
Contributor

vvolkl commented May 8, 2020 •
edited
Loading

You should then also change

 depends_on('doxygen +graphviz', when='+optional')

to

 depends_on('doxygen +graphviz', when='+docs')

otherwise, looks good to me.

@iarspider
Copy link
Copy Markdown
Contributor Author

iarspider commented May 8, 2020

@vvolkl Nice catch!

@adamjstewart
Copy link
Copy Markdown
Member

adamjstewart commented May 8, 2020

There seems to be a bug in how Gaudi (or CMake) runs doxygen, namely - it ignores LD_LIBRARY_PATH, which is required in my case (external compiler without relocation information):

How are you setting LD_LIBRARY_PATH? Spack unsets all *_LIBRARY_PATH environment variables before building anything. You'll have to set it via compilers.yaml.

@adamjstewart adamjstewart merged commit 08528a7 into spack:develop May 8, 2020
@iarspider
Copy link
Copy Markdown
Contributor Author

iarspider commented May 8, 2020

@adamjstewart via compilers.yaml: https://gist.github.com/iarspider/7a9a7685e9c93fb214b608bb85bf9f19#file-compilers-yaml

@iarspider iarspider deleted the patch-6 branch May 8, 2020 18:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@adamjstewart adamjstewart adamjstewart approved these changes

Assignees

No one assigned

Labels

new-variant

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@iarspider @vvolkl @adamjstewart