Skip to content

ci: test local package building - #518

Open
7flying wants to merge 3 commits into
fdo-rs:mainfrom
7flying:ci-rpm-build
Open

ci: test local package building#518
7flying wants to merge 3 commits into
fdo-rs:mainfrom
7flying:ci-rpm-build

Conversation

@7flying

@7flying 7flying commented Jun 22, 2023

Copy link
Copy Markdown
Contributor

Adds a new CI test section to check that the local package building target (make rpm) works every time.

This will help testing upcoming PR to tackle #516

@7flying
7flying force-pushed the ci-rpm-build branch 6 times, most recently from 97f1229 to 17eadd6 Compare June 22, 2023 13:18
@7flying

7flying commented Jun 22, 2023

Copy link
Copy Markdown
Contributor Author

Currently this new CI test fails (as it should) and reproduces the behaviour of #516

@puiterwijk puiterwijk left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a general suggestion, otherwise LGTM.

Comment thread .github/workflows/ci.yml Outdated
@7flying
7flying force-pushed the ci-rpm-build branch 2 times, most recently from bb357e9 to 8774ad9 Compare June 28, 2023 06:49
Comment thread .github/workflows/ci.yml
@7flying
7flying force-pushed the ci-rpm-build branch 2 times, most recently from 8651b30 to f853d52 Compare July 5, 2023 12:52
@7flying
7flying force-pushed the ci-rpm-build branch 2 times, most recently from 01cef09 to 8eaa633 Compare July 27, 2023 07:13
Adds a new CI test section to check that the local package building
target (`make rpm`) works every time, and that the RPMs can be installed.

Signed-off-by: Irene Diez <idiez@redhat.com>
7flying added 2 commits August 1, 2023 17:27
Signed-off-by: Irene Diez <idiez@redhat.com>
We are setting a .tar.xz extension so we should build a .tar.xz
instead of a bzip2. This sets the appropiate option for that in
make-vendored-tarfile.sh

Signed-off-by: Irene Diez <idiez@redhat.com>
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.

3 participants