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 sudo make docker-test work, too. #46

Closed
filmil opened this issue Jan 14, 2020 · 1 comment · Fixed by #47
Closed

Make sudo make docker-test work, too. #46

filmil opened this issue Jan 14, 2020 · 1 comment · Fixed by #47
Assignees

Comments

@filmil
Copy link
Member

filmil commented Jan 14, 2020

From @kpozin

$ sudo make docker-test
mkdir -p /tmp/rust_icu-root-target
docker run --tty --interactive \
                --user=0:0 \
                --volume=:/src/rust_icu \
                --volume=/tmp/rust_icu-root-target:/build/cargo \
                --volume=/root/.cargo:/usr/local/cargo \
                --env="CARGO_TEST_ARGS=" \
                filipfilmar/rust_icu_testenv-64:0.0.4
+ env
ICU_SOURCE_DIR=/src/icu
HOSTNAME=0b7d0ac807af
CARGO_TEST_ARGS=
RUST_ICU_SOURCE_DIR=/src/rust_icu
PWD=/src
ICU4C_BUILD_DIR=/build/icu4c-build
HOME=/root
CARGO_HOME=/usr/local/cargo
TERM=xterm
RUSTUP_HOME=/usr/local/rustup
SHLVL=1
RUST_VERSION=1.40.0
PATH=/usr/local/cargo/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin
CARGO_BUILD_DIR=/build/cargo
_=/usr/bin/env
+ cd /src/rust_icu
+ cargo install bindgen rustfmt
/entrypoint.sh: line 5: cargo: command not found
+ cd rust_icu_sys
/entrypoint.sh: line 7: cd: rust_icu_sys: No such file or directory
+ env LD_LIBRARY_PATH=/usr/local/lib cargo test
env: 'cargo': No such file or directory
+ cd rust_icu_common
/entrypoint.sh: line 11: cd: rust_icu_common: No such file or directory
+ env LD_LIBRARY_PATH=/usr/local/lib cargo test
env: 'cargo': No such file or directory
+ env LD_LIBRARY_PATH=/usr/local/lib cargo test
env: 'cargo': No such file or directory
make: *** [Makefile:38: docker-test] Error 127
@filmil
Copy link
Member Author

filmil commented Jan 14, 2020

@kpozin sorry about that. The PR above should fix the issue.

Tested with both sudo make docker-test and make docker-test and they both seem to work.

@filmil filmil self-assigned this Jan 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant