Skip to content

Makefile cleanup#8354

Merged
alexey-tikhonov merged 6 commits into
SSSD:masterfrom
alexey-tikhonov:makefile
Jan 20, 2026
Merged

Makefile cleanup#8354
alexey-tikhonov merged 6 commits into
SSSD:masterfrom
alexey-tikhonov:makefile

Conversation

@alexey-tikhonov

@alexey-tikhonov alexey-tikhonov commented Jan 14, 2026

Copy link
Copy Markdown
Member

and removal of unused code.

@alexey-tikhonov alexey-tikhonov added the no-backport This should go to target branch only. label Jan 14, 2026
gemini-code-assist[bot]

This comment was marked as outdated.

@alexey-tikhonov
alexey-tikhonov force-pushed the makefile branch 6 times, most recently from 2cbf402 to 697ddb0 Compare January 19, 2026 10:58
@alexey-tikhonov
alexey-tikhonov marked this pull request as ready for review January 19, 2026 13:48
@justin-stephenson justin-stephenson self-assigned this Jan 19, 2026
Comment thread Makefile.am

@thalman thalman 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.

CI errors are not connected with the change.

'child_common.c' was renamed to 'child_handlers.c' in 8bddb6a

Reviewed-by: Justin Stephenson <[email protected]>
Reviewed-by: Tomáš Halman <[email protected]>
where it's not needed

Reviewed-by: Justin Stephenson <[email protected]>
Reviewed-by: Tomáš Halman <[email protected]>
It's not used since 15a2213

Reviewed-by: Justin Stephenson <[email protected]>
Reviewed-by: Tomáš Halman <[email protected]>
Reviewed-by: Justin Stephenson <[email protected]>
Reviewed-by: Tomáš Halman <[email protected]>
`SSS_CRYPT_*` vars also do not make much sense.

Reviewed-by: Justin Stephenson <[email protected]>
Reviewed-by: Tomáš Halman <[email protected]>
@sssd-bot

Copy link
Copy Markdown
Contributor

The pull request was accepted by @thalman with the following PR CI status:


🟢 CodeFactor (success)
🟢 CodeQL (success)
🟢 osh-diff-scan:fedora-rawhide-x86_64:upstream (success)
🟢 rpm-build:centos-stream-10-x86_64:upstream (success)
🟢 rpm-build:fedora-42-x86_64:upstream (success)
🟢 rpm-build:fedora-43-x86_64:upstream (success)
🟢 rpm-build:fedora-rawhide-x86_64:upstream (success)
🟢 Analyze (target) / cppcheck (success)
🟢 Build / freebsd (success)
🟢 Build / make-distcheck (success)
🟢 ci / intgcheck (centos-10) (success)
🟢 ci / intgcheck (fedora-42) (success)
🟢 ci / intgcheck (fedora-43) (success)
🟢 ci / intgcheck (fedora-44) (success)
🟢 ci / prepare (success)
🟢 ci / system (centos-10) (success)
🟢 ci / system (fedora-42) (success)
🟢 ci / system (fedora-43) (success)
🔴 ci / system (fedora-44) (failure)
➖ Coverity scan / coverity (skipped)
🟢 Static code analysis / codeql (success)
🟢 Static code analysis / pre-commit (success)
🔴 Static code analysis / python-system-tests (failure)


There are unsuccessful or unfinished checks. Make sure that the failures are not related to this pull request before merging.

@alexey-tikhonov
alexey-tikhonov merged commit 8d376e8 into SSSD:master Jan 20, 2026
16 of 17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Accepted no-backport This should go to target branch only.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants