Skip to content
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

[Backport][ipa-4-9] configure: Make rpmlint optional #5676

Closed
wants to merge 2 commits into from

Conversation

abbra
Copy link
Contributor

@abbra abbra commented Mar 29, 2021

This PR was opened automatically because PR #5658 was pushed to master and backport to ipa-4-9 is required.

Distributions may want to run comprehensive fastcheck or lint tasks,
but rpmlint tool is mandatory for these targets while some platforms
don't have it at all.

With this change the rpmlint becomes optional for fastcheck, devcheck
and lint make targets.

Note: rpmlint option is disabled by default.
To enable: ./configure --enable-rpmlint
To explicitly disable: ./configure --disable-rpmlint

Fixes: https://pagure.io/freeipa/issue/8768
Signed-off-by: Stanislav Levin <slev@altlinux.org>
Template the autoconf phase.

Fixes: https://pagure.io/freeipa/issue/8768
Signed-off-by: Stanislav Levin <slev@altlinux.org>
@abbra
Copy link
Contributor Author

abbra commented Mar 29, 2021

PR was ACKed automatically because this is backport of PR #5658. Wait for CI to finish before pushing. In case of questions or problems contact @stanislavlevin who is author of the original PR.

@abbra abbra added ack Pull Request approved, can be merged pushed Pull Request has already been pushed labels Mar 29, 2021
@abbra
Copy link
Contributor Author

abbra commented Mar 29, 2021

ipa-4-9:

  • c441397 configure: Make rpmlint optional
  • 8c38c57 Azure: Run rpmlint on Fedora

@abbra abbra closed this Mar 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ack Pull Request approved, can be merged pushed Pull Request has already been pushed
Projects
None yet
2 participants