On Fri, Nov 10, 2023 at 12:58:14PM +0100, Caspar Schutijser wrote:
> Alright, thanks. Diff for that below (should be obvious). I also tweaked
> the manual page a little bit; at the bottom of the diff, I mention that
> USE_SCCACHE=Yes sets it up specifically for Rust (otherwise, people
> may expect it to cache results for C and C++ too, which is not true).

Thanks, forgot about the manual.
OK kn, suggestion inline.

> Index: bsd.port.mk.5
> ===================================================================
> RCS file: /cvs/src/share/man/man5/bsd.port.mk.5,v
> retrieving revision 1.638
> diff -u -p -r1.638 bsd.port.mk.5
> --- bsd.port.mk.5     2 Nov 2023 16:14:46 -0000       1.638
> +++ bsd.port.mk.5     10 Nov 2023 11:55:20 -0000
> @@ -2523,6 +2523,13 @@ See also
>  .Ev IGNORE
>  and
>  .Ev TRY_BROKEN .
> +.It Ev NO_SCCACHE
> +Set to
> +.Sq Yes
> +to prevent sccache from being used when building a certain port,
> +even when
> +.Ev USE_SCCACHE
> +is set.
>  .It Ev NO_TEST
>  Port does not have any regression tests.
>  Only set to
> @@ -3171,6 +3178,15 @@ The corresponding packages will be built
>  stage, and
>  .Xr pkg_add 1
>  will take care of installing them.
> +.It Ev SCCACHE_DIR
> +Sets the cache directory used when
> +.Ev USE_SCCACHE
> +is set to yes.
> +Defaults to ${WRKOBJDIR}/.sccache.
> +.It Ev SCCACHE_ENV
> +Sets additional environment variables when
> +.Ev USE_SCCACHE
> +is set to yes.
>  .It Ev SEPARATE_BUILD
>  Many GNU configure ports can be built in a directory distinct from the
>  place they were unpacked.
> @@ -3656,6 +3672,12 @@ adding
>  .Fl -no-execute-only .
>  Use when a port does not work with execute-only (unreadable) code sections
>  which are used by default by the linker on some architectures.
> +.It Ev USE_SCCACHE
> +User settings.
> +Set to
> +.Sq Yes
> +to use sccache when building ports.
> +Sets up the build environment so that it is used for Rust code.

This save a sentence and better distinguishes from the ccache wording:
+to usse sccache when buildding Rust ports.

>  .It Ev USE_WXNEEDED
>  If set to
>  .Sq Yes ,
> 

Reply via email to