Skip to content

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 <[email protected]> 1569873498 -0600
committer Dmytro Makovey <[email protected]> 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 <[email protected]>
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 <[email protected]>
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