This is a summary of discussions relative to the merge request created by
pietro (pietro) <[email protected]> titled
Check generated files on the forge
since its creation.
Description: Use a container image with all dependencies pre-installed. Copy
autoregen.py from the builder repository. Fail if any files change.
ChangeLog:
* .forgejo/workflows/sanity-checks.yaml (container): Use
ghcr.io/pietro/autoregen:main
(gcc_autoregen): New step.
contrib/ChangeLog:
* autoregen.py: New file.
Signed-off-by: Pietro Monteiro <[email protected]>
The full and up to date discussion can be found at
https://forge.sourceware.org/gcc/gcc-TEST/pulls/116
The merge request has been closed without being merged directly on the forge
repository.
On 2025-10-24 18:16:44+00:00, pietro (pietro) <[email protected]> requested
that Claudio Bantaloukas (rdfm) <[email protected]> review the code:
On 2025-10-24 18:16:44+00:00, pietro (pietro) <[email protected]> requested
that Christophe Lyon (clyon) <[email protected]> review the code:
On 2025-11-17 16:39:15+00:00, Christophe Lyon (clyon) wrote:
maybe @mark or @rdfm can comment on the image?
On 2025-11-19 11:20:52+00:00, Mark J. Wielaard (mjw) wrote:
Could we simply reuse
https://sourceware.org/cgit/builder/tree/builder/containers/Containerfile-autotools
?
On 2025-11-19 14:14:53+00:00, Christophe Lyon (clyon) wrote:
I think that container would lack `nodejs python3-git python3-termcolor
python3-unidiff`
which are currently installed by `sanity-checks.yaml`
On 2025-11-19 14:33:19+00:00, pietro (pietro) wrote:
@mjw wrote in
https://forge.sourceware.org/gcc/gcc-TEST/pulls/116#issuecomment-4107:
> Could we simply reuse
> https://sourceware.org/cgit/builder/tree/builder/containers/Containerfile-autotools
> ?
I based my image on that one. As @clyon said we need to install nodejs and some
python deps for the scripts, so I added them to the image instead of installing
them on each run. I also used the debian slim image as base and I removed the
build deps to make the image smaller too.
On 2025-12-18 17:32:50+00:00, Claudio Bantaloukas (rdfm) wrote:
Hi @pietro, could you create a PR, adding the contents of the Dockerfile you're
using as contrib/ci-container/essential/Containerfile please? Thanks!
On 2025-12-18 23:16:12+00:00, pietro (pietro) wrote:
@rdfm wrote in
https://forge.sourceware.org/gcc/gcc-TEST/pulls/116#issuecomment-4545:
> Hi @pietro, could you create a PR, adding the contents of the Dockerfile
> you're using as contrib/ci-container/essential/Containerfile please? Thanks!
[#132](https://forge.sourceware.org/gcc/gcc-TEST/pulls/132)
