> Plain awk is commonly available, but it's quite possible to have eg a minimal
> image where it's not installed. So I think we should check for it explicitly
> and fail if not found.
OK, I admit I was just being a bit lazy here :sweat_smile: The point of (cmake)
configuration is not to *assume* but to *verify*.
In that case, though, there's at least one other direct call to awk in our
build process (in the test-suite) which we should also treat the same way then
(so I'll handle that as part of this PR).
> ... actually we _are_ already looking for awk, we just don't check the
> outcome:
>
> ```
> findutil(__AWK awk)
> ```
>
> We _could_ also use the tool discovered here for this particular task I guess
> 😄
Yep, although `findutil()` doesn't fail, like you said, so we either need to
have it always fail, or based on a boolean argument. Which leads me to thinking
- shouldn't we always fail in `findutil()`?
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/2960#issuecomment-1991212427
You are receiving this because you are subscribed to this thread.
Message ID: <rpm-software-management/rpm/pull/2960/c1991212...@github.com>
_______________________________________________
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint