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

Introduce packit/source-git support #139

Closed
wants to merge 11 commits into from

Conversation

matejak
Copy link
Contributor

@matejak matejak commented Mar 2, 2021

No description provided.

@matejak matejak added this to the 1.2 milestone Mar 2, 2021
TomasTomecek and others added 2 commits March 3, 2021 15:49
Since you don't have linear history, packit is making it linear before
generating patches using `git filter-branch`. The command requires the
repo to be clean, hence the "weird" error in the first build.

Packit will tell you in future once this happens how to proceed:
packit/packit#1137
packit: commit spec changes once fetched
@TomasTomecek
Copy link

we don't invoke actions in shell by default so you need to wrap commands with bash -c "..." if you need it

Absence of those broke the commit.
TomasTomecek and others added 6 commits March 11, 2021 17:24
Signed-off-by: Tomas Tomecek <ttomecek@redhat.com>
for some reason, patch is not able to apply patches generated from
git-format-patch - this is a safe bet, generate and apply patches with
git

Signed-off-by: Tomas Tomecek <ttomecek@redhat.com>
apply patches with git instead of patch
Signed-off-by: Tomas Tomecek <ttomecek@redhat.com>
packit: ignore changes to files not in tarball
@scrutinizer-notifier
Copy link

The inspection completed: No new issues

@matejak matejak modified the milestones: 1.2, 1.3.0 Jun 21, 2021
@matejak
Copy link
Contributor Author

matejak commented Apr 28, 2022

Obsoleted by likes of #200

@matejak matejak closed this Apr 28, 2022
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.

None yet

3 participants