On Thu, Aug 12, 2021 at 04:48:13PM +0200, Theo Buehler wrote:
> Now that a few Rust ports use crates.inc, I think we should teach
> portcheck about it. I didn't use @(|) for the pattern since the result
> seemed less readable.
>
> Two things:
>
> A few Rust ports use '.include <crates.inc>' I'll switch those to using
> '.include "crates.inc"' so the current directory is searched first.
I agree that "x" is better than <x>.
> For some reason, it was decided to do '.include "./modules.inc"' for Go
> ports. I doubt there is actually a benefit in adding ./ here, but I may
> be wrong. If there is none, I think we should leave it out since it's
> just noise.
Re-reading make(1) man page about .include :
If double quotes are used, the including makefile's directory and
any directories specified using the -I option are searched before
the system makefile directory.
From openbsd source trees (excepting modules.inc or crates.inc), "./x"
is never used. Only "${.CURDIR}/x" could be seen, but it should only
makes sense with obj/ usage (and so not with ports).
> Regardless of whether ./ is included or not, I think Go and
> Rust ports should use the same idiom. I kept the ./ in the check for
> now.
I agree with the fact to have the same idiom.
> Index: portcheck
> ===================================================================
> RCS file: /cvs/ports/infrastructure/bin/portcheck,v
> retrieving revision 1.138
> diff -u -p -r1.138 portcheck
> --- portcheck 2 Jun 2021 13:07:23 -0000 1.138
> +++ portcheck 12 Aug 2021 14:33:41 -0000
> @@ -612,9 +612,9 @@ check_port_dir() {
> portmk_exists=true
> ;;
>
> - modules.inc)
> + crates.inc|modules.inc)
> test -f "$F" || err "$F is not a file"
> - fgrep -qx '.include "./modules.inc"' "$dir"/Makefile ||
> + fgrep -qx ".include \"./$F\"" "$dir"/Makefile ||
> err "$F not included in Makefile"
> ((++non_portmk))
> ;;
>
--
Sebastien Marie
No comments:
Post a Comment