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

make error when use docker #47

Closed
silveric10 opened this issue May 28, 2019 · 5 comments
Closed

make error when use docker #47

silveric10 opened this issue May 28, 2019 · 5 comments
Assignees
Labels
bug docker Issues related to our docker builds

Comments

@silveric10
Copy link

Please use this template for reporting suspected bugs or requests for help.

Issue Description

Scanning dependencies of target OpenrCtrl-cpp2-target
[ 32%] Generating OpenrCtrl files. Output: /home/openr/build/thrift-out/openr/if
[ 32%] Built target OpenrCtrl-cpp2-target
Scanning dependencies of target OpenrCtrl-cpp2-obj
[ 32%] Building CXX object CMakeFiles/OpenrCtrl-cpp2-obj.dir/thrift-out/openr/if/gen-cpp2/OpenrCtrl_constants.cpp.o
In file included from /home/openr/build/thrift-out/openr/if/gen-cpp2/OpenrCtrl_constants.h:11:0,
from /home/openr/build/thrift-out/openr/if/gen-cpp2/OpenrCtrl_constants.cpp:8:
/home/openr/build/thrift-out/openr/if/gen-cpp2/OpenrCtrl_types.h:16:46: fatal error: openr/if/gen-cpp2/Decision_types.h: No such file or directory
compilation terminated.
make[2]: *** [CMakeFiles/OpenrCtrl-cpp2-obj.dir/thrift-out/openr/if/gen-cpp2/OpenrCtrl_constants.cpp.o] Error 1

Environment

ubuntu 18.04 gcc 7.4

Minimal test code / Steps to reproduce the issue

What's the actual result?

What's the expected result?

build success

@silveric10
Copy link
Author

This error also happens when I make without docker.
[ 23%] Building CXX object CMakeFiles/OpenrCtrl-cpp2-obj.dir/thrift-out/openr/if/gen-cpp2/OpenrCtrl_constants.cpp.o
In file included from /home/yy/openr/build/thrift-out/openr/if/gen-cpp2/OpenrCtrl_constants.h:11:0,
from /home/yy/openr/build/thrift-out/openr/if/gen-cpp2/OpenrCtrl_constants.cpp:8:
/home/yy/openr/build/thrift-out/openr/if/gen-cpp2/OpenrCtrl_types.h:17:10: fatal error: openr/if/gen-cpp2/Dual_types.h: No such file or directory
#include "openr/if/gen-cpp2/Dual_types.h"
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.
CMakeFiles/OpenrCtrl-cpp2-obj.dir/build.make:102: recipe for target 'CMakeFiles/OpenrCtrl-cpp2-obj.dir/thrift-out/openr/if/gen-cpp2/OpenrCtrl_constants.cpp.o' failed

@saifhhasan
Copy link
Contributor

Hello @silveric10 - can you try build again with latest code, we had some changes and fixes in cmake dependencies that will resolve your issue.

@silveric10
Copy link
Author

Hello @silveric10 - can you try build again with latest code, we had some changes and fixes in cmake dependencies that will resolve your issue.

Hi @saifhhasan , your update still doesn't work, and I want to tell you that the build script "build_openr.sh" also has a bug.

install_openr() {
pushd .
cd "$BUILD_DIR"
cmake
-DBUILD_SHARED_LIBS=OFF
-DBUILD_TESTS=ON
-DADD_ROOT_TESTS=ON
-DCMAKE_CXX_FLAGS="-Wno-unused-parameter -fPIC"
../openr/ //There is no "CMakeLists.txt" in this directory, I think it should be “..” not "../openr"

By the way, the compile error is the same.
[ 13%] Built target OpenrCtrl-cpp2-target
Scanning dependencies of target OpenrCtrl-cpp2-obj
[ 13%] Building CXX object CMakeFiles/OpenrCtrl-cpp2-obj.dir/thrift-out/openr/if/gen-cpp2/OpenrCtrl_constants.cpp.o
In file included from /home/yy/openr/thrift-out/openr/if/gen-cpp2/OpenrCtrl_constants.h:11:0,
from /home/yy/openr/thrift-out/openr/if/gen-cpp2/OpenrCtrl_constants.cpp:8:
/home/yy/openr/thrift-out/openr/if/gen-cpp2/OpenrCtrl_types.h:17:10: fatal error: openr/if/gen-cpp2/Decision_types.h: No such file or directory
#include "openr/if/gen-cpp2/Decision_types.h"
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.

@cooperlees cooperlees self-assigned this Jul 17, 2020
@cooperlees cooperlees added bug docker Issues related to our docker builds labels Jul 17, 2020
@cooperlees
Copy link
Contributor

Hi, once I close #56 I'll try make the docker builds all work again and will update when I feel they are working.

facebook-github-bot pushed a commit that referenced this issue Aug 14, 2020
Summary:
Title: Add a Dockerfile for OpenR

Description:
- Add a Dockerfile that uses `getdeps.py` to build OpenR and deps in an Ubuntu 20.04 container

breeze is not working at the moment. Will follow up a diff/PR for that.

Pull Request resolved: #61

Test Plan:
- `docker exec -it HASH bash`
- Fix env and see help output: https://pastebin.com/ycT9pjhe

Fixes #55 #47

Differential Revision: D23114897

Pulled By: cooperlees

fbshipit-source-id: d190fb56efe95d6f5e2384c379ca3eb0ca3fc46a
@cooperlees
Copy link
Contributor

Please try the new Dockerfile

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug docker Issues related to our docker builds
Projects
None yet
Development

No branches or pull requests

3 participants