"Johannes Schindelin via GitGitGadget" <gitgitgadget@xxxxxxxxx>
writes:
> ...
> In the meantime, the current Coverity documentation describes a very
> different way to install the analysis tool, recommending to add the
> `bin/` directory to the _end_ of `PATH` (when originally, IIRC, it was
> recommended to add it to the _beginning_ of the `PATH`).
> ..., and finding the "wrong" ones first on the
> `PATH` misleads that logic.
>
> Let's fix this problem by following Coverity's current recommendation
> and append the `bin/` directory in which `cov-int` can be found to the
> _end_ of `PATH`.
Wow, that is a very well described change.
> Signed-off-by: Johannes Schindelin <johannes.schindelin@xxxxxx>
> ---
> .github/workflows/coverity.yml | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/.github/workflows/coverity.yml b/.github/workflows/coverity.yml
> index 124301dbbe2f..a5d99e59d4eb 100644
> --- a/.github/workflows/coverity.yml
> +++ b/.github/workflows/coverity.yml
> @@ -147,7 +147,7 @@ jobs:
> key: cov-build-${{ env.COVERITY_LANGUAGE }}-${{ env.COVERITY_PLATFORM }}-${{ steps.lookup.outputs.hash }}
> - name: build with cov-build
> run: |
> - export PATH="$RUNNER_TEMP/cov-analysis/bin:$PATH" &&
> + export PATH="$PATH:$(cygpath -au "$RUNNER_TEMP")/cov-analysis/bin" &&
Additionally two things are lacking explanation in the proposed log
message, though, or an uninitiated will still be left scratching his
head:
- Why didn't the original need "cygpath -au"?
- Even though many steps in this job deals with different
env.COVERITY_PLATFORM, this part does not seem to be conditional.
Why is $(cygpath -au ...) safe outside Windows environment?
Other than that, nicely done and very nicely explained.
> cov-configure --gcc &&
> cov-build --dir cov-int make
> - name: package the build
