Skip to content

[19.03] RHEL8 build fix#410

Merged
droopy4096 merged 2 commits intodocker:19.03from
droopy4096:19.03-iptables
Nov 28, 2019
Merged

[19.03] RHEL8 build fix#410
droopy4096 merged 2 commits intodocker:19.03from
droopy4096:19.03-iptables

Conversation

@droopy4096
Copy link
Contributor

fix SPEC file declaration for boolean dependency

Dmytro Makovey added 2 commits November 20, 2019 10:41
author Dmytro Makovey <dmytro.makovey@docker.com> 1569873498 -0600
committer Dmytro Makovey <dmytro.makovey@docker.com> 1574271589 -0700

add optional requirement for nftables on RHEL8+ derivatives
Copy link
Contributor

@dave-tucker dave-tucker left a comment

Choose a reason for hiding this comment

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

Fix LGTM @droopy4096.
Could you squash your commits please? Ta.

@dave-tucker
Copy link
Contributor

Ignore me @droopy4096. Not sure squashing is right at all.
I think the cherry pick of a674309 might have gone wrong.
Did you use git cherry-pick -x?

@droopy4096 droopy4096 merged commit cfc7f43 into docker:19.03 Nov 28, 2019
@droopy4096
Copy link
Contributor Author

squashed before merge :)

thaJeztah pushed a commit to thaJeztah/docker-ce-packaging that referenced this pull request May 1, 2020
add optional requirement for nftables on RHEL8+ derivatives

(cherry picked from commit cfc7f43)
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
tiborvass added a commit that referenced this pull request May 1, 2020
@thaJeztah thaJeztah changed the title RHEL8 build fix [19.03] RHEL8 build fix May 1, 2020
HeroCC pushed a commit to HeroCC/docker-ce-packaging that referenced this pull request Oct 6, 2021
add optional requirement for nftables on RHEL8+ derivatives

(cherry picked from commit cfc7f43)
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
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.

2 participants