Subject: Re: [PATCH 3/5] ci: configure GitHub Actions for CI/PR

From: Danh Doan <[email-masked]>
Date: Sun, 5 Apr 2020 06:55:28 +0700

On 2020-04-04 19:59:37+0200, Johannes Schindelin <[email-masked]> wrote:
> > > +  linux-clang:
> > > +    runs-on: ubuntu-latest
> > > +    steps:
> > > +    - uses: actions/checkout@v1
> > > +    - name: install dependencies
> > > +      env:
> > > +        CC: clang
> > > +      run: |
> > > +        sudo apt-get update &&
> > > +        sudo apt-get -y install git gcc make libssl-dev libcurl4-openssl-dev libexpat-dev tcl tk gettext git-email zlib1g-dev apache2-bin &&
> >
> > Why install all these dependencies here, when the very next command is
> > 'ci/install-dependencies.sh', whose purpose is to install the
> > dependencies?
> 
> I copy/pasted these from `azure-pipelines.yml`, and agree that they should
> be handled via `ci/install-dependencies.sh` instead.
> 
> Danh (is this the correct way to address you?), if you want to have a look
> at that, go for it. Otherwise I'll try to find some time to do that
> myself.

I've managed to fix this part in the v4 series (only send privately to
Dscho, because I'm waiting for other conversations to settle).

-- 
Danh