Skip to content

csmock: use --installdeps for legacy chroots#195

Merged
kdudka merged 3 commits intocsutils:mainfrom
kdudka:el7-ipa-fix
Feb 18, 2025
Merged

csmock: use --installdeps for legacy chroots#195
kdudka merged 3 commits intocsutils:mainfrom
kdudka:el7-ipa-fix

Conversation

@kdudka
Copy link
Member

@kdudka kdudka commented Feb 17, 2025

RHEL-7 packages do not use dynamic build dependencies and the new way to install build dependencies breaks build/scan in some cases, such as with ipa-4.6.5-11.el7_7.4.

Fixes: commit 50b1c8c
Related: #188

@kdudka kdudka self-assigned this Feb 17, 2025
kdudka added a commit to kdudka/csmock that referenced this pull request Feb 17, 2025
RHEL-7 packages do not use dynamic build dependencies and the new way
to install build dependencies breaks build/scan in some cases, such as
with `ipa-4.6.5-11.el7_7.4`.

Fixes: commit 50b1c8c
Related: csutils#188
Closes: csutils#195
kdudka added a commit to kdudka/csmock that referenced this pull request Feb 17, 2025
RHEL-7 packages do not use dynamic build dependencies and the new way
to install build dependencies breaks build/scan in some cases, such as
with `ipa-4.6.5-11.el7_7.4`.

Fixes: commit 50b1c8c
Related: csutils#188
Closes: csutils#195
kdudka added a commit to kdudka/csmock that referenced this pull request Feb 17, 2025
RHEL-7 packages do not use dynamic build dependencies and the new way
to install build dependencies breaks build/scan in some cases, such as
with `ipa-4.6.5-11.el7_7.4`.

Fixes: commit 50b1c8c
Related: csutils#188
Resolves: https://issues.redhat.com/browse/PSSECAUT-989
Closes: csutils#195
Fedora 42 default minimal buildroot has non-working `/bin/su`.

Related: fedora-copr/copr#3631
kdudka added a commit to kdudka/csmock that referenced this pull request Feb 17, 2025
RHEL-7 packages do not use dynamic build dependencies and the new way
to install build dependencies breaks build/scan in some cases, such as
with `ipa-4.6.5-11.el7_7.4`.

Fixes: commit 50b1c8c
Related: csutils#188
Resolves: https://issues.redhat.com/browse/PSSECAUT-989
Closes: csutils#195
@jperezdealgaba
Copy link
Collaborator

The changes in csmock /LGTM. The tests seems to be failing

kdudka added a commit to kdudka/csmock that referenced this pull request Feb 17, 2025
RHEL-7 packages do not use dynamic build dependencies and the new way
to install build dependencies breaks build/scan in some cases, such as
with `ipa-4.6.5-11.el7_7.4`.

Fixes: commit 50b1c8c
Related: csutils#188
Resolves: https://issues.redhat.com/browse/PSSECAUT-989
Closes: csutils#195
@kdudka
Copy link
Member Author

kdudka commented Feb 17, 2025

@jperezdealgaba Thanks for review! The CI was broken by two independent issues:

kdudka added a commit to kdudka/csmock that referenced this pull request Feb 18, 2025
RHEL-7 packages do not use dynamic build dependencies and the new way
to install build dependencies breaks build/scan in some cases, such as
with `ipa-4.6.5-11.el7_7.4`.

Fixes: commit 50b1c8c
Related: csutils#188
Resolves: https://issues.redhat.com/browse/PSSECAUT-989
Closes: csutils#195
The latest build of the `units` RPM package cannot be rebuilt
with gcc-15 because its source code is not yet ready for C23.

Related: https://bugzilla.redhat.com/2341484
RHEL-7 packages do not use dynamic build dependencies and the new way
to install build dependencies breaks build/scan in some cases, such as
with `ipa-4.6.5-11.el7_7.4`.

Fixes: commit 50b1c8c
Related: csutils#188
Resolves: https://issues.redhat.com/browse/PSSECAUT-989
Closes: csutils#195
@kdudka kdudka closed this in 7f3d39d Feb 18, 2025
@kdudka kdudka merged commit 7f3d39d into csutils:main Feb 18, 2025
40 checks passed
@kdudka kdudka deleted the el7-ipa-fix branch February 18, 2025 08:28
kdudka added a commit to kdudka/csmock that referenced this pull request May 22, 2025
kdudka added a commit to kdudka/csmock that referenced this pull request May 22, 2025
kdudka added a commit to kdudka/csmock that referenced this pull request May 22, 2025
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.

2 participants