From: <ch...@su...> - 2013-08-07 12:55:25
|
Hi! > > 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(). > I think that casting mod_name to (char *) is easier than casting array > before it gets to execvp(). And it doesn't require > any changes of tst_run_cmd interfaces (ie they accepts argv the same way > as execvp does)... > > Maybe leave everything as is and just cast mod_name to (char *)? That is easier for one testcase, but will require the cast in each testcase that will use the interface. From this point of view cast in the library is better. Not elegant solution though. :( But in the end any of the solutions will work... -- Cyril Hrubis ch...@su... |