-
Notifications
You must be signed in to change notification settings - Fork 2
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
Add 2023.2 and 2024.1 #6
base: main
Are you sure you want to change the base?
Conversation
@@ -37,8 +37,16 @@ | |||
xilinx-unified-unwraped = pkgs.xilinx-unified-unwrapped; | |||
xilinx-unified = pkgs.xilinx-unified; | |||
|
|||
xilinx-unified-2023-1-unwraped = pkgs.xilinx-unified-unwrapped; |
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.
lets not assign an unspecific version to a specific one.
xilinx-unified = final.callPackage ./pkgs/wrap-xilinx.nix { | ||
inputDerivation = final.xilinx-unified-unwrapped; | ||
}; | ||
|
||
xilinx-unified-2023-1-unwrapped = final.callPackage ./pkgs/xilinx-unified.nix { }; |
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.
lets define all versions here and do not rely on some defaults in the xilinx-unified derivation. So we dont have to change/add future versions in multiple files.
and cleaned some things up.