Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[master] Fix CentOS 8 build #444

Merged
merged 1 commit into from
Mar 23, 2020
Merged

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 <github@gone.nl>
@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

@@ -79,7 +81,7 @@ export DOCKER_GITCOMMIT=%{_gitcommit}
mkdir -p /go/src/github.com/docker
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