[CRIU] [PATCH] build: Add dev-dep target to install build-required packages
Dmitry Safonov
0x7f454c46 at gmail.com
Tue Oct 31 18:30:55 MSK 2017
2017-10-31 14:25 GMT+00:00 Kirill Tkhai <ktkhai at virtuozzo.com>:
> This allows to obtain via a single command the requirements,
> need to build and install criu.
>
> Signed-off-by: Kirill Tkhai <ktkhai at virtuozzo.com>
There is this scripts/install-debian-pkgs.sh
Can we clean it a bit and make more generic way to install
dev packages for rh/debian/etc?
> ---
> Makefile | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/Makefile b/Makefile
> index 796cad874..61a6a89b7 100644
> --- a/Makefile
> +++ b/Makefile
> @@ -347,6 +347,11 @@ TAGS_FILES_REGEXP := . -name '*.[hcS]' ! -path './.*' \( ! -path './test/*' -o -
> docker run --rm -it --privileged criu-x86_64 ./test/zdtm.py run -a -x tcp6 -x tcpbuf6 -x static/rtc -x cgroup
> .PHONY: docker-test
>
> +dev-dep:
> + $(Q) test ! -f /etc/redhat-release || \
> + yum install gcc protobuf-devel protobuf-c-devel libnet-devel libnl3-devel libcap-devel asciidoc xmlto
> +.PHONY: dev-dep
It would be nice to have target name saying `install' or something.
`dep' on the other hand may mean make-file dependency, so
it's better to omit. Maybe `install-dev-pkgs'?
Also, add it to `help' target, please.
> +
> help:
> @echo ' Targets:'
> @echo ' all - Build all [*] targets'
>
--
Dmitry
More information about the CRIU
mailing list