> 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

Reply via email to