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)
Sat, May 25, 6:12 AM
Unknown Object (File)
Sun, May 19, 8:15 AM
Unknown Object (File)
Apr 26 2024, 2:59 AM
Unknown Object (File)
Apr 26 2024, 2:59 AM
Unknown Object (File)
Apr 26 2024, 2:59 AM
Unknown Object (File)
Apr 26 2024, 2:57 AM
Unknown Object (File)
Apr 26 2024, 2:57 AM
Unknown Object (File)
Apr 26 2024, 1:50 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
rP FreeBSD ports repository
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 53712
Build 50603: arc lint + arc unit

Event Timeline

Mk/Scripts/qa.sh
142

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.

142–147

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