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

ananicy_cpp-rules -> ananicy-rules-cachyos_git: move to gitOverride #909

Merged
merged 1 commit into from
Oct 31, 2024

Conversation

JohnRTitor
Copy link
Collaborator

🐟 What?

  1. It's packaged in nixpkgs, so we could use an override.

🎣 Why?

  • (1) This reduces the tarball size?

🐳 Extras

  • I like chocolate cake.

Questions:

Are automatic update scripts affected by this? Let me know if I have to change something.

@JohnRTitor JohnRTitor requested a review from PedroHLC October 31, 2024 19:56
@JohnRTitor JohnRTitor requested a review from a team as a code owner October 31, 2024 19:56
@JohnRTitor
Copy link
Collaborator Author

Should be ready to merge, I tried to move smaller packages to gitOverride before trying the big giant :)

@JohnRTitor JohnRTitor added the needs-cache Needs to be pushed to Cachix label Oct 31, 2024
Copy link
Contributor

Oops! Something went wrong while caching this PR! Check the logs for more information.

@github-actions github-actions bot removed the needs-cache Needs to be pushed to Cachix label Oct 31, 2024
@@ -83,7 +83,7 @@ in

alacritty_git = callOverride ../pkgs/alacritty-git { };

ananicy-cpp-rules = final.callPackage ../pkgs/ananicy-cpp-rules { };
ananicy-cpp-rules = callOverride ../pkgs/ananicy-cpp-rules { };
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suffix it with _git here, and to be more explicit, it should be named ananicy-rules-cachyos_git. (nyxKey must be updated to match, no need to change the directory's name)

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure, I just wanted to make backwards-compatibility.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done, do you want me to maintain an alias?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nope

It's packaged in nixpkgs, so we could just use an override.
This is a breaking change.
@JohnRTitor JohnRTitor changed the title ananicy_cpp-rules: move to gitOverride ananicy_cpp-rules -> ananicy-rules-cachyos_git: move to gitOverride Oct 31, 2024
@PedroHLC PedroHLC merged commit b97628e into chaotic-cx:main Oct 31, 2024
2 checks passed
@PedroHLC PedroHLC deleted the ananicy-cpp branch October 31, 2024 21:03
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