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
powershell: 6.2.3 -> 7.0.0 #84454
powershell: 6.2.3 -> 7.0.0 #84454
Conversation
Package and openssl dependency version bump and cascading changes.
@jonringer @yrashk Please review if you have time. |
@GrahamcOfBorg build powershell |
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.
LGTM
able to start interpreter
[3 built, 3 copied (2.5 MiB), 0.5 MiB DL]
https://github.com/NixOS/nixpkgs/pull/84454
1 package built:
powershell
rm $pslibs/libcrypto.so.1.0.0 | ||
rm $pslibs/libssl.so.1.0.0 | ||
|
||
patchelf --replace-needed libcrypto.so.1.0.0 libcrypto.so.1.1 $pslibs/libmi.so | ||
patchelf --replace-needed libssl.so.1.0.0 libssl.so.1.1 $pslibs/libmi.so |
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.
do you mind making this darwin compatible?
rm: cannot remove '/nix/store/kca66mmwd5sakpa836hmcdz1dqj260ms-powershell-7.0.0/share/powershell/libcrypto.so.1.0.0': No such file or directory
builder for '/nix/store/a3yv0rvwf11jms0i3h5466ph7kmcwfwg-powershell-7.0.0.drv' failed with exit code 1
error: build of '/nix/store/a3yv0rvwf11jms0i3h5466ph7kmcwfwg-powershell-7.0.0.drv' failed
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 thought autoPatchelfHook
does not exists on macOS
at all.
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.
Nevertheless the patchelf code needs to be hidden behind a optionalString !stdenv.isDarwin
flag.
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.
see #86423
Package and openssl dependency version bump and cascading changes.
Motivation for this change
Version bump
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)