mirror of
https://git.proxmox.com/git/mirror_zfs.git
synced 2024-11-17 01:51:00 +03:00
5ca349f95d
The test added in commit
4313a5b4c5 ("Detect if sed supports --in-place")
doesn't work at least on my system (autoconfig-2.69).
The issue is that SED has already been found and cached before this
function is evaluated, with the result that the test is completely
skipped.
...
checking for a sed that does not truncate output... /usr/bin/sed
...
checking for sed --in-place... (cached) /usr/bin/sed
The first test is executed by libtool.m4. This looks to have been around
in libtool for at least 15 years or so, not sure why this was not
encountered at the time of the original commit.
Fix this by caching the value of the ac_inplace flag rather than the
path to SED. Also use $SED and add AC_REQUIRE to ensure that we use the
sed that was located by the standard configure test.
Reviewed-by: Ryan Moeller <ryan@iXsystems.com>
Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
Signed-off-by: Arvind Sankar <nivedita@alum.mit.edu>
Closes #10493
17 lines
536 B
Plaintext
17 lines
536 B
Plaintext
dnl #
|
|
dnl # Set the flags used for sed in-place edits.
|
|
dnl #
|
|
AC_DEFUN([ZFS_AC_CONFIG_ALWAYS_SED], [
|
|
AC_REQUIRE([AC_PROG_SED])dnl
|
|
AC_CACHE_CHECK([for sed --in-place], [ac_cv_inplace], [
|
|
tmpfile=$(mktemp conftest.XXX)
|
|
echo foo >$tmpfile
|
|
AS_IF([$SED --in-place 's#foo#bar#' $tmpfile 2>/dev/null],
|
|
[ac_cv_inplace="--in-place"],
|
|
[$SED -i '' 's#foo#bar#' $tmpfile 2>/dev/null],
|
|
[ac_cv_inplace="-i ''"],
|
|
[AC_MSG_ERROR([$SED does not support in-place])])
|
|
])
|
|
AC_SUBST([ac_inplace], [$ac_cv_inplace])
|
|
])
|