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

Docker build error #917

Closed
Itay-max opened this issue Oct 30, 2023 · 3 comments · Fixed by #913
Closed

Docker build error #917

Itay-max opened this issue Oct 30, 2023 · 3 comments · Fixed by #913

Comments

@Itay-max
Copy link

Describe the Bug

I am currently using the latest version of DAGMC and I was trying to build the docker image inside the CI directory. Alsmost all commands in the dockerfile were running great except for "RUN mkdir -p ${dagmc_build_dir}/build && ..." :
=> ERROR [dagmc 4/4] RUN mkdir -p /root/build_dir/dagmc/build && ...

Looked at the log and found the following CMAKE error:

CMake Error at cmake/DAGMC_macros.cmake:228 (add_library):
2.944 Cannot find source file:
2.944
2.944 /root/build_dir/dagmc/src/pyne/pyne.cpp

To Reproduce

build from repository root directory with command: docker build -t dagmc -f CI/Dockerfile .

Expected Behavior

Building the Image without errors

@gonuke
Copy link
Member

gonuke commented Oct 31, 2023

Thanks for the question. You'll need to make sure you have checked out the submodule in the src/pyne/pyne directory.

You can do this from the root directory of the DAGMC repository with:
git submodule update --init

I hope that helps.

@Itay-max
Copy link
Author

Thank you for your reply,
the command you specified is part of the docker commands and it was running successfully:
image

However the CMAKE error still exists.

@gonuke
Copy link
Member

gonuke commented Nov 24, 2023

Sorry for the delay - are you using the latest release version or the most up-to-date code in the develop branch?

We're working on a release and the most up-to-date code is working ok.

@ahnaf-tahmid-chowdhury ahnaf-tahmid-chowdhury linked a pull request Dec 18, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants