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

conky: fix maintainer mode #53852

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

zlice
Copy link
Contributor

@zlice zlice commented Jan 5, 2025

Testing the changes

  • I tested the changes in this PR: YES

Local build testing

  • I built this PR locally for my native architecture, x86_64
  • I built this PR locally for these architectures (if supported. mark crossbuilds):
    • aarch64-musl

Comments

per last update - #52924

brndnmtthws/conky#2131

Void and nix both have MAINTAINER_MODE on when it should not be. It is for conky"s own CI and turns on BUILD_TESTS when enabled.

The new conky code does bomb with BUILD_TESTS on and MAINTAINER_MODE off. But Void disables checks for cross builds anyway, so it"s good enough to disable tests and maint-mode.

This doesn"t really need rebuilt, just a better fix than previous fix.

@Caellian
Copy link

Caellian commented Jan 5, 2025

I just finished and merged conky#2132, which renames BUILD_TESTS to BUILD_TESTING. Variable name should probably be updated in this PR.

Remove MAINTAINER_MODE completely, it"s OFF by default and always will be. Shouldn"t be used from packaging.

Lastly, I added support for USE_CCACHE and sccache/ccache detection which is enabled by default. I believe this is a good fit for Void (unlike Nix), but turn it off if you want to explicitly disable binary caching.

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