[CRIU] [PATCH] zdtm: Force link inotify_system_nodel.c

Andrew Vagin avagin at parallels.com
Mon Dec 17 14:50:01 EST 2012


On Mon, Dec 10, 2012 at 06:28:12PM +0400, Cyrill Gorcunov wrote:
> From time to time I see
> 
>  | gcc -g -O2 -Wall -Werror -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=0 -I../../lib   inotify_system.c ../../lib/libzdtmtst.a   -o inotify_system
>  | ln -s inotify_system.c inotify_system_nodel.c
>  | ln: creating symbolic link `inotify_system_nodel.c': File exists
>  | make[3]: *** [inotify_system_nodel.c] Error 1
> 
> on make test command. So do a forced link here.
>

I don't like this patch without steps to reproduce...
I have never seen this one.

> Signed-off-by: Cyrill Gorcunov <gorcunov at openvz.org>
> ---
>  test/zdtm/live/static/Makefile | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/test/zdtm/live/static/Makefile b/test/zdtm/live/static/Makefile
> index 551102d..5343801 100644
> --- a/test/zdtm/live/static/Makefile
> +++ b/test/zdtm/live/static/Makefile
> @@ -146,7 +146,7 @@ all:	$(TST)
>  install: all
>  
>  inotify_system_nodel.c: inotify_system.c
> -	ln -s inotify_system.c inotify_system_nodel.c
> +	ln -f -s inotify_system.c inotify_system_nodel.c
>  
>  $(TST_NOFILE:%=%.pid):	%.pid:	%
>  	$(<D)/$(<F) --pidfile=$@ --outfile=$<.out
> -- 
> 1.8.0.1
> 


More information about the CRIU mailing list