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

rpm: drop the --remote argument from git-archive call #7639

Closed
wants to merge 1 commit into from

Conversation

praiskup
Copy link
Contributor

@praiskup praiskup commented Oct 14, 2024

It seems that current (autumn 2024) git releases somehow dislike
the use of --remote=file:// when it applies the [safe] directory
checks. The option doesn't seem to be useful though, so let's drop
it to fix the Copr builds.

Relates: fedora-copr/copr#3421

It seems that current (autumn 2024) git releases somehow dislike
the use of `--remote=file://` when it applies the [safe] directory
checks.  The option doesn't seem to be useful though, so let's drop
it to fix the Copr builds.

Relates: fedora-copr/copr#3421
@pbrezina
Copy link
Member

Pushed PR: #7639

  • master
    • c1434c1 - rpm: drop the --remote argument from git-archive call
  • sssd-2-8
    • 8a165c6 - rpm: drop the --remote argument from git-archive call
  • sssd-2-9
    • 5a6cd13 - rpm: drop the --remote argument from git-archive call

@pbrezina pbrezina added Pushed and removed Accepted Ready to push Ready to push labels Oct 14, 2024
@pbrezina pbrezina closed this Oct 14, 2024
@pbrezina
Copy link
Member

Ok, it works locally, let's see if it works in copr.

@pbrezina
Copy link
Member

Works, thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants