env: add FIXME comment about execvp()

Ecordonnier · Ecordonnier · commit 73a0844cdcfe · 2025-12-17T16:20:26.000+01:00
Signed-off-by: Etienne Cordonnier &lt;ecordonnier@snap.com&gt;
diff --git a/src/uu/env/src/env.rs b/src/uu/env/src/env.rs
@@ -628,6 +628,9 @@ impl EnvAppData {
                 argv.push(arg_cstring);
             }
 
+            // FIXME: libc::execvp() can be replaced with Command::exec() once https://github.com/rust-lang/rust/issues/97889#issuecomment-2007391597
+            // is merged into rust stable.
+
             // Execute the program using execvp. this replaces the current
             // process. The execvp function takes care of appending a NULL
             // argument to the argument list so that we don't have to.

Original file line number	Diff line number	Diff line change
`@@ -628,6 +628,9 @@ impl EnvAppData {`
`628`	`628`	`argv.push(arg_cstring);`
`629`	`629`	`}`
`630`	`630`
	`631`	`+ // FIXME: libc::execvp() can be replaced with Command::exec() once https://github.com/rust-lang/rust/issues/97889#issuecomment-2007391597`
	`632`	`+ // is merged into rust stable.`
	`633`	`+`
`631`	`634`	`// Execute the program using execvp. this replaces the current`
`632`	`635`	`// process. The execvp function takes care of appending a NULL`
`633`	`636`	`// argument to the argument list so that we don't have to.`