Re: [PATCH] utilities/gitlab-ci: Fully update system before building

Leonardo Brás <[email protected]>
Newsgroups org.kernel.vger.perfbook
Message-ID <[email protected]>
On Fri, 2023-08-04 at 11:05 -0700, Paul E. McKenney wrote:
> On Fri, Aug 04, 2023 at 02:53:04PM -0300, Leonardo Brás wrote:
> > On Fri, 2023-08-04 at 14:46 -0300, Leonardo Bras wrote:
> > > Some recent CI errors come from the incompatibility between packages
> > > pre-installed in the docker image and the ones installed via pacman.
> > > 
> > > To avoid this, make a full system upgrade when installing the needed
> > > packages.
> > > 
> > > Signed-off-by: Leonardo Bras <[email protected]>
> > > ---
> > >  utilities/gitlab-ci.yml | 4 ++--
> > >  1 file changed, 2 insertions(+), 2 deletions(-)
> > > 
> > > diff --git a/utilities/gitlab-ci.yml b/utilities/gitlab-ci.yml
> > > index 544b9373..bcda8712 100644
> > > --- a/utilities/gitlab-ci.yml
> > > +++ b/utilities/gitlab-ci.yml
> > > @@ -9,7 +9,7 @@ test_codesamples:
> > >      stage: test
> > >  
> > >      before_script:
> > > -        - pacman --noconfirm -Sy make gcc liburcu
> > > +        - pacman --noconfirm -Syu make gcc liburcu
> > >  
> > >      script:
> > >          - make -C CodeSamples/
> > > @@ -19,7 +19,7 @@ generate_pdfs:
> > >      stage: build
> > >  
> > >      before_script:
> > > -        - pacman --noconfirm -Sy texlive-meta fig2dev ghostscript graphviz inkscape wget make which fontconfig git ttf-dejavu gnu-free-fonts ttf-liberation
> > > +        - pacman --noconfirm -Syu texlive-meta fig2dev ghostscript graphviz inkscape wget make which fontconfig git ttf-dejavu gnu-free-fonts ttf-liberation
> > >          - mkdir -p ~/.local/share/fonts && cp fonts/steel-city-comic.regular.ttf ~/.local/share/fonts/ && fc-cache
> > >  
> > >      script:
> > 
> > Sorry about resending. 
> > I had an issue with credentials, please ignore the ones sent from my employer
> > account: the gmail is the correct one.
> 
> Ah, I queued the wrong one.  I updated it and pushed it out, could you
> please double check?

Yeap, it's correct. Thanks!

(Sorry about the overhead)


Leo

> 
> 							Thanx, Paul
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.