Skip to content

libdvbcsa: remove BUILD_PATENTED - #30500

Open
neheb wants to merge 3 commits into
openwrt:masterfrom
neheb:l
Open

neheb wants to merge 3 commits into
openwrt:masterfrom
neheb:l

Conversation

@neheb

@neheb neheb commented Sep 11, 2026

Copy link
Copy Markdown
Contributor

All patents seem to have expired between 2014-2020.

📦 Package Details

Maintainer: @Rafostar

All patents seem to have expired between 2014-2020.

Signed-off-by: Rosen Penev <rosenp@gmail.com>

@openwrt-ai openwrt-ai left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed 1 new commit.


Generated by Claude Code

Comment thread libs/libdvbcsa/Makefile
CATEGORY:=Libraries
TITLE:=DVB Common Scrambling Algorithm Library
URL:=https://www.videolan.org/developers/libdvbcsa.html
DEPENDS:=@BUILD_PATENTED

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: the two in-tree consumers still gate libdvbcsa on the same symbol — +BUILD_PATENTED&&TVHEADEND_CSA:libdvbcsa with TVHEADEND_CSA itself depends on BUILD_PATENTED, and +BUILD_PATENTED:libdvbcsa plus its --enable-dvbcsa toggle. So CSA descrambling stays unreachable unless BUILD_PATENTED=y; is dropping those gates intended as a follow-up, or should it be part of this series?


Generated by Claude Code

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed, thanks


Generated by Claude Code

@Rafostar

Copy link
Copy Markdown
Contributor

LGTM. Although, as bot noticed, other packages that conditionally add libdvbcsa behind BUILD_PATENTED flag also need to be updated.

libdvbcsa no longer requires BUILD_PATENTED, so drop the gate from the
libdvbcsa dependency and the configure args. Software descrambling is
now enabled via TVHEADEND_CSA alone.

Signed-off-by: Rosen Penev <rosenp@gmail.com>
libdvbcsa no longer requires BUILD_PATENTED, so enable dvbcsa
unconditionally and drop the BUILD_PATENTED dependency and config
toggle.

Signed-off-by: Rosen Penev <rosenp@gmail.com>

@openwrt-ai openwrt-ai left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed 2 new commits.


Generated by Claude Code

SECTION:=multimedia
CATEGORY:=Multimedia
DEPENDS:=+MINISATIP_AES:libopenssl +BUILD_PATENTED:libdvbcsa
DEPENDS:=+MINISATIP_AES:libopenssl +libdvbcsa

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: libdvbcsa becomes a hard dependency for every minisatip install, while the sibling commit keeps it optional in tvheadend behind +TVHEADEND_CSA:libdvbcsa. The other two optional features here still have toggles in Package/minisatip/config; was dropping the opt-out for dvbcsa rather than converting it to a MINISATIP_DVBCSA symbol intentional?


Generated by Claude Code

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants