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
python3Packages.sslyze: 3.1.0 -> 4.1.0 #122175
Conversation
Result of 6 packages built:
|
Result of 6 packages built successfully:
2 suggestions:
Result of 4 packages failed to build:
2 packages built successfully:
2 suggestions:
Note that build failures may predate this PR, and could be nondeterministic or hardware dependent. |
postPatch = | ||
let | ||
legacyOpenSSLVersion = lib.replaceStrings [ "." ] [ "_" ] opensslLegacyStatic.version; | ||
modernOpenSSLVersion = lib.replaceStrings [ "." ] [ "_" ] opensslStatic.version; | ||
zlibVersion = zlibStatic.version; | ||
in | ||
'' | ||
mkdir -p deps/openssl-OpenSSL_${legacyOpenSSLVersion}/ | ||
cp ${opensslLegacyStatic.out}/lib/libssl.a \ | ||
${opensslLegacyStatic.out}/lib/libcrypto.a \ | ||
deps/openssl-OpenSSL_${legacyOpenSSLVersion}/ | ||
ln -s ${opensslLegacyStatic.out.dev}/include deps/openssl-OpenSSL_${legacyOpenSSLVersion}/include | ||
ln -s ${opensslLegacyStatic.bin} deps/openssl-OpenSSL_${legacyOpenSSLVersion}/apps | ||
|
||
mkdir -p deps/openssl-OpenSSL_${modernOpenSSLVersion}/ | ||
cp ${opensslStatic.out}/lib/libssl.a \ | ||
${opensslStatic.out}/lib/libcrypto.a \ | ||
deps/openssl-OpenSSL_${modernOpenSSLVersion}/ | ||
ln -s ${opensslStatic.out.dev}/include deps/openssl-OpenSSL_${modernOpenSSLVersion}/include | ||
ln -s ${opensslStatic.bin}/bin deps/openssl-OpenSSL_${modernOpenSSLVersion}/apps | ||
mkdir -p deps/openssl-OpenSSL_${modernOpenSSLVersion}/ | ||
cp ${opensslStatic.out}/lib/libssl.a \ | ||
${opensslStatic.out}/lib/libcrypto.a \ | ||
deps/openssl-OpenSSL_${modernOpenSSLVersion}/ | ||
ln -s ${opensslStatic.out.dev}/include deps/openssl-OpenSSL_${modernOpenSSLVersion}/include | ||
ln -s ${opensslStatic.bin}/bin deps/openssl-OpenSSL_${modernOpenSSLVersion}/apps | ||
|
||
mkdir -p deps/zlib-${zlibVersion}/ | ||
cp ${zlibStatic.out}/lib/libz.a deps/zlib-${zlibVersion}/ | ||
''; | ||
mkdir -p deps/zlib-${zlibVersion}/ | ||
cp ${zlibStatic.out}/lib/libz.a deps/zlib-${zlibVersion}/ | ||
''; |
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 we should minimize whitespace changes.
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.
Reverted the formatting.
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.
otherwise LGTM
pytestFlagsArray = [ | ||
"-k" | ||
"'not Online'" | ||
]; |
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 is the more succinct way to express this, but essentially does the same thing.
pytestFlagsArray = [ | |
"-k" | |
"'not Online'" | |
]; | |
disabledTests = [ | |
"Online" | |
]; |
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.
Thanks for the suggestion! Changed it accordingly 🙂
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
Result of nixpkgs-review pr 122175 run on x86_64-linux 1
6 packages built:
- python38Packages.nassl
- sslyze (python38Packages.sslyze)
- python38Packages.tls-parser
- python39Packages.nassl
- python39Packages.sslyze
- python39Packages.tls-parser
Motivation for this change
ZHF: #122042
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)