Re: [LTP] [PATCH] tst_run_cmd: const correctness

[<ch...@su...>]

Hi!
>  void tst_run_cmd_fds(void (cleanup_fn)(void),
> -			char *const argv[],
> +			const char *const argv[],
>  			int stdout_fd,
>  			int stderr_fd);
>  
> @@ -236,7 +236,7 @@ void tst_run_cmd_fds(void (cleanup_fn)(void),
>   * not needed.
>   */
>  void tst_run_cmd(void (cleanup_fn)(void),
> -		char *const argv[],
> +		const char *const argv[],
>  		const char *stdout_path,
>  		const char *stderr_path);

Gosh, that fixed the warning in the test but I've overlooked that
execvp() takes char *const argv[] argument so it introduced warning in
the lib. Now I wonder why execvp() has such strange parameter type...

I'm trying to figure out how to fix it 'right' but it gets unnecessary
complicated. I guess that we can fix this patch by casting the array in
the lib before it gets to execvp().

-- 
Cyril Hrubis
ch...@su...