Page MenuHomeFreeBSD

multimedia/librtmp: Fix build with libressl 3.8
Needs ReviewPublic

Authored by zirias on Nov 21 2022, 2:27 PM.
Tags
None
Referenced Files
Unknown Object (File)
Dec 26 2023, 8:48 AM
Unknown Object (File)
Nov 9 2023, 10:55 AM
Unknown Object (File)
Apr 4 2023, 10:55 PM
Unknown Object (File)
Mar 5 2023, 6:08 PM
Unknown Object (File)
Feb 28 2023, 6:00 PM
Unknown Object (File)
Feb 18 2023, 7:20 AM
Unknown Object (File)
Jan 10 2023, 8:10 PM
Unknown Object (File)
Jan 7 2023, 11:52 AM
Subscribers

Details

Reviewers
hrs
Summary

Remove all special-casing for libressl from patches. They caused
directly accessing structs that are now opaqued in libressl as well.

Diff Detail

Repository
R11 FreeBSD ports repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 56358
Build 53246: arc lint + arc unit

Event Timeline

This revision is now accepted and ready to land.Nov 21 2022, 3:00 PM

Can we please stop adding sporadic libressl patches to the tree? Upstream and backport.

Can we please stop adding sporadic libressl patches to the tree? Upstream and backport.

This *removes* libressl-checks from *local* patches.

@zirias
This specific patch is a positive change so thank you however it would be nicer if we could avoid having local patches in the first place since you're one of a handful of people that works on libressl support.

@diizzy "upstreaming" patches is a good thing in general (and I did it quite often in the past). With libressl specifically, it won't do any good in most cases. Upstreams that care about libressl are a minority, for them, it makes sense. Most upstreams will either reluctantly or indifferently add some patch (in which case all you get from "usptreaming" is actually MORE work) or outright refuse the patches. IMHO, supporting libressl is only possible by maintaining local patches in the end. Whether FreeBSD ports want to do that or not is something that should be decided.

Back to *this* review, I'm not sure whether we need the patches *at all* now that they don't contain special handling for libressl any more. @hrs can you tell?

zirias retitled this revision from multimedia/librtmp: Fix build with libressl 3.5 to multimedia/librtmp: Fix build with libressl 3.8.Mar 1 2024, 8:01 AM
zirias edited the summary of this revision. (Show Details)
zirias removed a reviewer: tcberner.
This revision now requires review to proceed.Mar 1 2024, 8:01 AM
zirias edited the summary of this revision. (Show Details)

I totally forgot I was still waiting for maintainer approval on this one. Now it finally popped up in my builds again, so updated and rebased.

I had a closer look btw, the patches will need to stay as upstream code is only compatible with the old/legacy OpenSSL API.