Skip to content

Conversation

@pcolberg
Copy link
Contributor

@pcolberg pcolberg commented Jan 23, 2025

The RPMs before and after this PR were compared for regressions with

  • rpm -qpl, for package files,
  • rpm -qpi, for package description, and
  • rpm -qpR, for package dependencies.

@pcolberg pcolberg self-assigned this Jan 23, 2025
@coveralls
Copy link

coveralls commented Jan 23, 2025

Pull Request Test Coverage Report for Build 13167589161

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 64.101%

Totals Coverage Status
Change from base Build 13121208148: 0.0%
Covered Lines: 15826
Relevant Lines: 24689

💛 - Coveralls

@pcolberg pcolberg force-pushed the pcolberg/fedora41 branch 3 times, most recently from 946a5e2 to 3c096ea Compare February 5, 2025 22:14
pcolberg and others added 3 commits February 5, 2025 17:14
Manual installation of more recent Python dependencies is no longer needed.

Signed-off-by: Peter Colberg <[email protected]>
As a prerequisite for building on Fedora 41, omit manual `pip install`
in favour of letting the build automatically install Python modules.
In docker containers, add new build dependency on python3-wheel.

Remove manual package dependencies in favour of letting the package
build automatically determine the correct package dependencies.

Link: https://src.fedoraproject.org/rpms/opae/blob/rawhide/f/opae.spec
Signed-off-by: Peter Colberg <[email protected]>
@pcolberg pcolberg marked this pull request as ready for review February 5, 2025 23:46
@pcolberg pcolberg requested review from a team as code owners February 5, 2025 23:46
@pcolberg pcolberg merged commit 6bfdcc5 into master Feb 5, 2025
29 checks passed
@pcolberg pcolberg deleted the pcolberg/fedora41 branch February 5, 2025 23:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants