litex: init at 2023.04 #237658
Draft
litex: init at 2023.04 #237658
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/2023-summer-of-nix-program-goals/30376/3 |
Comment on lines
+6246
to
+6248
| pythondata-cpu-vexriscv = callPackage ../development/python-modules/pythondata-cpu-vexriscv { }; | ||
| pythondata-software-picolibc = callPackage ../development/python-modules/pythondata-software-picolibc { }; | ||
| pythondata-software-compiler_rt = callPackage ../development/python-modules/pythondata-software-compiler_rt { }; |
There was a problem hiding this comment.
Suggested change
| pythondata-cpu-vexriscv = callPackage ../development/python-modules/pythondata-cpu-vexriscv { }; | |
| pythondata-software-picolibc = callPackage ../development/python-modules/pythondata-software-picolibc { }; | |
| pythondata-software-compiler_rt = callPackage ../development/python-modules/pythondata-software-compiler_rt { }; | |
| pythondata-cpu-vexriscv = callPackage ../development/python-modules/pythondata-cpu-vexriscv { }; | |
| pythondata-software-picolibc = callPackage ../development/python-modules/pythondata-software-picolibc { }; | |
| pythondata-software-compiler-rt = callPackage ../development/python-modules/pythondata-software-compiler-rt { }; |
Per python ecosystem docs we always normalise attrs and pnames and replace special characters like underscore with a dash and make it all lower cased.
| meta = with lib; { | ||
| description = "Python module containing data files for compiler_rt software (for use with LiteX"; | ||
| homepage = "https://github.com/litex-hub/pythondata-software-compiler_rt"; | ||
| license = with licenses; [ ]; |
There was a problem hiding this comment.
Suggested change
| license = with licenses; [ ]; | |
| license = licenses.asl20; |
see https://github.com/litex-hub/pythondata-software-compiler_rt/blob/master/LICENSE
|
@SuperSandro2000 FYI, this PR is not for review and you are reviewing code that will completely change. |
wegank
added
the
2.status: stale
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Description of changes
LiteX is a set of tools to work with FPGAs, I package the bare minimum to compile a VexRISCV for iCEBreaker iCE40 FPGA board.
This was tested with:
producing a final Gateware for the iCEBreaker which was not tested on the hardware yet :).
TODO : I disabled tests everywhere because for some reason, it's complaining about migen not being there, etc. I am debugging this and this will be fixed or commented properly before merge.
Things done
sandbox = trueset innix.conf? (See Nix manual)nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/)