Page MenuHomeFreeBSD

Scripts/qa.sh: fine tune USES=ssl recommendation
ClosedPublic

Authored by fernape on Sep 25 2023, 12:05 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Nov 7, 12:26 PM
Unknown Object (File)
Wed, Nov 6, 11:27 PM
Unknown Object (File)
Oct 18 2024, 9:29 AM
Unknown Object (File)
Oct 17 2024, 8:44 AM
Unknown Object (File)
Oct 16 2024, 6:59 AM
Unknown Object (File)
Oct 15 2024, 6:00 PM
Unknown Object (File)
Oct 15 2024, 6:00 PM
Unknown Object (File)
Oct 14 2024, 8:13 AM
Subscribers

Details

Summary

Try to avoid the "you need USES=ssl" in ports that provide a libssl.so or
libcrypto.so libraries themselves as security/openssl or security/libressl.

Comes from PR270035

Test Plan

Apply patch and make openssl, libressl or other ssl ports.
If building other ports that don't provide those libraries by themselves, then
the warning still applies

Diff Detail

Repository
R11 FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 55494
Build 52383: arc lint + arc unit

Event Timeline

Mk/Scripts/qa.sh
140–146

As we don't need whatever list_stagedir_elfs outputs, maybe here tests its output directly, something like :

if list_stagedir_elfs ... | grep -q 'lib.*.so'; then
...
Mk/Scripts/qa.sh
118

This is not needed any more now.

140–146

I know I'm being pedantic but could you add an empty line between the EOF and the if ? (to ease reading)

Remove unused variable
Add empty line to improve readability

This revision is now accepted and ready to land.Jan 20 2024, 5:41 PM