[Tarantool-patches] [PATCH v1] gitlab-ci: add packaging for Fedora 32
Alexander V. Tikhonov
avtikhon at tarantool.org
Wed Dec 23 09:12:39 MSK 2020
Hi Alexander, thanks for the review. Please check my comments below.
On Sun, Jun 07, 2020 at 05:03:46PM +0300, Alexander Turenko wrote:
> On Sun, Jun 07, 2020 at 04:40:40PM +0300, Alexander V. Tikhonov wrote:
> > Added packaging jobs for Fedora 32.
> >
> > Closes #4966
> >
> > (cherry picked from commit 5613beedf83a679b725cea816a4176eabe3e1601)
>
> Are you added 'cherry picked from' by mistake?
>
Right the wrong branch was used, in real branch to be used is:
avtikhon/gh-4966-fedora-32-full-ci
> > Also added job to travis-ci.
> > ---
> >
> > Github: https://github.com/tarantool/tarantool/tree/avtikhon/gh-4966-fedora-32-old-full-ci
> > Issue: https://github.com/tarantool/tarantool/issues/4966
> >
> > The patch should be committed only after commit
> > bdd90686f82ec2e22d4ea7adfb9a6173bcc8af4a
> > will be pushed to be able to fix Fedoar 32 build.
>
> It should not be pushed until all warnings from [1] will be resolved or
> at least will be proven to be false-positive.
>
> [1]: https://github.com/tarantool/tarantool/issues/4966
>
Right, issue #4966 is fixed and closed for now.
> >
> > Also current fix is on for 1.10 branch only, for other branches
> > need to make changes in .travis.yml changes:
> > 2.4: 1.10 -> 2.4
> > 2.3: 1.10 -> 2.3
> > 2.2: 1.10 -> 2.2
> > For master branch .travis.yml doesn't need to be updated.
> > Use 5613beedf83a679b725cea816a4176eabe3e1601 for master branch.
>
> I would rather implement the patch on top of master and mention what
> should be done after cherry-picking for 2.4, 2.3 and 1.10.
Not needed for now, due to travis-ci is not used any more.
More information about the Tarantool-patches
mailing list