Skip to content

Conversation

@thaJeztah
Copy link
Member

@thaJeztah thaJeztah commented Mar 20, 2020

follow-up to #416

  • Enable PowerTools repo
  • Disable building btrfs, which is no longer supported on
    CentOS/RHEL

- Enable PowerTools repo
- Disable building btrfs, which is no longer supported on
  CentOS/RHEL

Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah
Copy link
Member Author

ping @arkodg @tiborvass @silvin-lubecki PTAL

Copy link
Contributor

@silvin-lubecki silvin-lubecki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

ln -s /root/rpmbuild/BUILD/src/engine /go/src/github.com/docker/docker

pushd engine
pushd /root/rpmbuild/BUILD/src/engine
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: can't we use an RPM variable here, like $RPM_BUILD_ROOT ?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll look at some cleaning up in a follow-up 👍

@thaJeztah thaJeztah merged commit c92ef9a into docker:master Mar 23, 2020
@thaJeztah thaJeztah deleted the fix_centos8 branch March 23, 2020 10:00
@mattvw
Copy link

mattvw commented Mar 23, 2020

Just curious: How long do you expect it will take to start seeing the CentOS 8 RPMs in the docker-de repo (e.g. under https://download.docker.com/linux/centos/)? Will we need to wait until the next docker-ce release? Thanks! :)

@thaJeztah
Copy link
Member Author

Yes, the plan is to add CentOS 8 for the next release, not in a 19.03.x patch release

@thaJeztah thaJeztah changed the title Fix CentOS 8 build [master] Fix CentOS 8 build May 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants