On Mon, 10 Mar 2025, Andrew Cooper wrote: > On 10/03/2025 6:38 pm, Roger Pau Monné wrote: > > On Mon, Mar 10, 2025 at 06:30:15PM +0000, Andrew Cooper wrote: > >> On 10/03/2025 6:16 pm, Roger Pau Monne wrote: > >>> Always store xen/.config as an artifact, renamed to xen-config to match > >>> the naming used in the Gitlab CI tests. > >>> > >>> Reported-by: Andrew Cooper <andrew.coop...@citrix.com> > >>> Signed-off-by: Roger Pau Monné <roger....@citrix.com> > >> Looking at this, I suspect my failure was caused by trying to capture > >> ".config" directly. > >> > >>> --- > >>> .cirrus.yml | 12 ++++++++++++ > >>> 1 file changed, 12 insertions(+) > >>> > >>> diff --git a/.cirrus.yml b/.cirrus.yml > >>> index e2949d99d73a..1a39f5026f9a 100644 > >>> --- a/.cirrus.yml > >>> +++ b/.cirrus.yml > >>> @@ -15,6 +15,14 @@ freebsd_template: &FREEBSD_ENV > >>> APPEND_INCLUDES: /usr/local/include > >>> CIRRUS_CLONE_DEPTH: 1 > >>> > >>> +freebsd_artifacts: &FREEBSD_ARTIFACTS > >>> + always: > >>> + rename_script: > >>> + - cp xen/.config xen-config > >>> + config_artifacts: > >>> + path: xen-config > >>> + type: text/plain > >> Can't this be part of freebsd_template directly? > >> > >> Or is there an ordering problem with the regular build_script ? > > Exactly, that was my first attempt (placing it in freebsd_template), > > but then the collection would be done before the build, as > > freebsd_template sets the env variables ahead of the build, see: > > > > https://cirrus-ci.com/task/5086615544004608 > > > > Thanks, Roger. > > Ok. > > Reviewed-by: Andrew Cooper <andrew.coop...@citrix.com>
Acked-by: Stefano Stabellini <sstabell...@kernel.org>
