-
Notifications
You must be signed in to change notification settings - Fork 8
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
Enable copr builds and add packit config #10
base: main
Are you sure you want to change the base?
Conversation
/packit copr-build |
There was an error while running a copr build:
You can re-trigger copr build by adding a comment ( |
^ that error is actually coming from kubernetes and the original cause is something like |
/packit build |
Congratulations! The build has finished successfully. 🍾 You can install the built RPMs by following these steps:
Please note that the RPMs should be used only in a testing environment. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
\o/
There was an error while running a copr build:
You can re-trigger copr build by adding a comment ( |
128453e
to
3fc49c9
Compare
There was an error while running a copr build:
You can re-trigger copr build by adding a comment ( |
Pushed further simplification which we've agreed on today's meeting. @lachmanfrantisek, please review if everything's as expected and let me know when the updated code is in production. Thanks. |
0c9961d
to
e4dc0d0
Compare
There was an error while creating a SRPM. You can re-trigger copr build by adding a comment ( Output:
Return code: 2 |
There was an error while running a copr build:
You can re-trigger copr build by adding a comment ( |
|
||
actions: | ||
create-archive: | ||
- make tarball |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works for me locally on the packit master
. Let's wait for the redeploy.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice! Looking forward :)
/packit build |
:/ |
I'll take a look on the paths for sandcastle (our sandboxing system) -- it's probably different than the local build. |
A little out-of-the topic info:
|
/packit build |
@@ -28,6 +28,7 @@ build: | |||
|
|||
tarball: build | |||
cd $(TMP) && tar cfj SOURCES/$(PACKAGE).tar.bz2 $(PACKAGE) | |||
@printf "$(shell realpath $(TMP)/SOURCES/$(PACKAGE).tar.bz2)\n" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the problem here is that the path is absolute - it should ideally be relative within the project
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess this the same issue as teemtee/tmt/pull/249. I've tried relative path there as well but it did not help.
Let us introduce packit service to you - the automation to integrate upstream open source projects into Fedora operating system.
After merging this PR, you are just a few steps away from RPM builds being automatically triggered on your PR's.
It means, that you'll be able to try and play with your change, packaged as an RPM.
But there is more. By using packit, you can for example enable adding new releases into Fedora Rawhide.
What are the next steps?
For more info, please: