[Buildroot] [PATCH v2 1/1] igt-gpu-tools: add igt-gpu-tools package

Bernd Kuhls bernd at kuhls.net
Thu Oct 19 17:00:44 UTC 2023


Hi Andy,

just some small quirks.

There are warnings by check-package

$ utils/check-package package/igt-gpu-tools/*
package/igt-gpu-tools/Config.in:28: should be indented with one tab 
(http://nightly.buildroot.org/#_config_files)
package/igt-gpu-tools/igt-gpu-tools.hash:2: separation does not match 
expectation (http://nightly.buildroot.org/#adding-packages-hash)
package/igt-gpu-tools/igt-gpu-tools.hash:3: separation does not match 
expectation (http://nightly.buildroot.org/#adding-packages-hash)
package/igt-gpu-tools/igt-gpu-tools.hash:6: separation does not match 
expectation (http://nightly.buildroot.org/#adding-packages-hash)
package/igt-gpu-tools/igt-gpu-tools.mk:3: line contains trailing 
whitespace
50 lines processed
5 warnings generated

> +++ b/package/igt-gpu-tools/igt-gpu-tools.hash
> @@ -0,0 +1,6 @@
> +# Locally calculated from download
> +sha256 
ba15b321c5a90da629baf326912c9c0406a903719040eb4959e0af1197935133  igt-gpu-
tools-v1.28.tar.gz

Also the hash for the gz tarball is not needed.

+IGT_GPU_TOOLS_VERSION = v1.28

Please move the "v" from here to this line

+IGT_GPU_TOOLS_SITE = https://gitlab.freedesktop.org/drm/igt-gpu-tools/-/
archive/$(IGT_GPU_TOOLS_VERSION)

so it looks like this

+IGT_GPU_TOOLS_SITE = https://gitlab.freedesktop.org/drm/igt-gpu-tools/-/
archive/v$(IGT_GPU_TOOLS_VERSION)
        ^

This will help using the release monitoring site:
https://release-monitoring.org/project/323454/

Regards, Bernd




More information about the buildroot mailing list