On 6 September 2018 at 12:02, Eric Engestrom <eric.engestrom@xxxxxxxxx> wrote: > On Thursday, 2018-09-06 11:01:17 +0100, Daniel Stone wrote: >> GitLab CI already captures all the stdout/stderr output from the build >> process as the log. However, some other important information is hidden >> in other log files. >> >> Taken from Wayland, capture logs from the configuration process as well >> as from every check. >> >> Signed-off-by: Daniel Stone <daniels@xxxxxxxxxxxxx> >> Cc: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx> >> Cc: Lucas De Marchi <lucas.de.marchi@xxxxxxxxx> >> Cc: Eric Engeström <eric.engestrom@xxxxxxxxx> >> Cc: Daniel Vetter <daniel.vetter@xxxxxxxx> >> --- >> .gitlab-ci.yml | 26 ++++++++++++++++++++++++++ >> 1 file changed, 26 insertions(+) >> >> diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml >> index eee6abfc..50ec8527 100644 >> --- a/.gitlab-ci.yml >> +++ b/.gitlab-ci.yml >> @@ -1,6 +1,29 @@ >> +.artifacts-meson: &artifacts-meson > > Ooooh, variables? I might try to dedup the 'meson build' and > 'autotools build' code with those :) > >> + when: always >> + paths: >> + - _build/meson-logs >> + >> +.artifacts-autotools: &artifacts-autotools >> + when: always >> + paths: >> + - _build/*.log >> + - _build/amdgpu/*.log >> + - _build/etnaviv/*.log >> + - _build/exynos/*.log >> + - _build/freedreno/*.log >> + - _build/intel/*.log >> + - _build/libkms/*.log >> + - _build/nouveau/*.log >> + - _build/omap/*.log >> + - _build/radeon/*.log >> + - _build/tegra/*.log > > All of the above can be simplified to: > - _build/*/*.log > > (vc4 is missing btw) > > With vc4 added, or /*/ used: > Acked-by: Eric Engestrom <eric.engestrom@xxxxxxxxx> > I would fold everything in these tree lines. - _build/*.log - _build/*/*.log - _build/*/*/*.log It covers everything existing, plus catches future ones ;-) With that Reviewed-by: Emil Velikov <emil.velikov@xxxxxxxxxxxxx> -Emil _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
