Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Flake support, project reorganization, asahi overlay #47
Flake support, project reorganization, asahi overlay #47
Changes from 18 commits
d109ab6
cbd3029
66b3dc4
49e5c97
df2a90e
feb4490
08f27ae
52b0fa9
14a4ef5
4fe6325
5c91fea
4066a94
96f70c8
7d984c2
8361211
32ebdd7
74112e2
e90b07b
677aacf
516c0aa
27ff070
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This has definitely broken
hardware.asahi.buildPkgs
. I am pretty sure the above comment is wrong. The real issue is that theconfig.nix
file does not make its way to the ISO. Thus, when the configuration is evaluated, therunCommand
needs to be executed to get the complete source code, butrunCommand
expects to run on ahardware.asahi.buildPkgs
system and fails.I want to keep the
localPkgs
hack for now (we don't needlib = localPkgs.lib
though) and fix it later. That option will still be broken as it was for flakes users before, but it will be fine for the install scenario.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the solution is to replace the
"config.nix"
IFD with a pure nix regex parser. It seems like it'll be pretty straightforward, and it can be upstreamed to nixpkgs since nixpkgs also uses an IFD currently.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
opened #51
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've added back the
localPkgs
workaround for now.This file was deleted.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.