-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
spec: Use Requires: git-core if distribution is new enough #18538
Conversation
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: cgwalters Assign the PR to them by writing The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these OWNERS Files:
You can indicate your approval by writing |
I am planning to propose we include `origin-clients` in Fedora Atomic Workstation by default. It was pointed out that that pulls in `git` which pulls in Perl. A while ago we added a `git-core` which is really all we need: https://bugzilla.redhat.com/show_bug.cgi?id=1161251
4dfc967
to
cb73679
Compare
Can someone take a look at this? It seems minor, though it'd make |
/test cross |
/retest |
@cgwalters: The following tests failed, say
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
Issues go stale after 90d of inactivity. Mark the issue as fresh by commenting If this issue is safe to close now please do so with /lifecycle stale |
Looks like we can close this? The dep was completely dropped in #19509. |
I am planning to propose we include
origin-clients
in Fedora AtomicWorkstation by default.
It was pointed out that that pulls in
git
which pulls in Perl. Awhile ago we added a
git-core
which is really all we need:https://bugzilla.redhat.com/show_bug.cgi?id=1161251