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

Update nix.md #636

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

Update nix.md #636

wants to merge 1 commit into from

Conversation

Ryoschin
Copy link

@Ryoschin Ryoschin commented Nov 22, 2024

  • Separate the additional packages and additional libraries declaration in the home.nix section by using a with expression (which also removes the need to write inputs.ags.packages.${pkgs.system} every new library the user would want to add)

  • Add an example for a library containing two uppercase letters in it's name (WirePlumber becomes wireplumber, not wirePlumber, or such)

- Separate the additional packages and additional libraries declaration in the home.nix section by using a "with" expression (which also removes the need to write "inputs.ags.packages.${pkgs.system}" for every new library the user would want to add)

- Add an example for a library containing two uppercase letters in it's name (WirePlumber becomes wireplumber, not wirePlumber, or such)
@Aylur
Copy link
Owner

Aylur commented Dec 28, 2024

We can replace battery for wireplumber to make an example, but I don't like separating the two as it just adds unnecessary noise. We are showing how to work with ags and not teaching nix. Users can separate them themselves if they like.

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