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

nixos/hardware: adjust readme to the new location #91166

Open
wants to merge 1 commit into
base: master
from

Conversation

@Profpatsch
Copy link
Member

Profpatsch commented Jun 20, 2020

I tried to use "${pkgs.path}/nixos/hardware/…" first, but that cause
the module system to go into an infinite recursion.

Ideally we’d have something akin to modulesPath (hardwarePath?) or
something more general so that we don’t have to depend on
NIX_PATH (which is wrong in some circumstances).

I tried to use `"${pkgs.path}/nixos/hardware/…"` first, but that cause
the module system to go into an infinite recursion.

Ideally we’d have something akin to `modulesPath` (`hardwarePath`?) or
something more general so that we don’t have to depend on
`NIX_PATH` (which is wrong in some circumstances).
@Profpatsch Profpatsch requested a review from Mic92 Jun 20, 2020
@grahamc
Copy link
Member

grahamc commented Jun 20, 2020

I'm actually not sure how you made this PR :o the directory you're PR'ing to doesn't exist

@Profpatsch
Copy link
Member Author

Profpatsch commented Jun 20, 2020

I'm actually not sure how you made this PR :o the directory you're PR'ing to doesn't exist

It’s on the reverted branch.

@Mic92
Copy link
Contributor

Mic92 commented Jun 20, 2020

Maybe move to #91167?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.