Skip to content

adblock-fast: fix unbound out filter name#28473

Closed
tobiaswaldvogel wants to merge 1 commit intoopenwrt:masterfrom
tobiaswaldvogel:adblock-fast-fix-unbound
Closed

adblock-fast: fix unbound out filter name#28473
tobiaswaldvogel wants to merge 1 commit intoopenwrt:masterfrom
tobiaswaldvogel:adblock-fast-fix-unbound

Conversation

@tobiaswaldvogel
Copy link
Copy Markdown
Contributor

📦 Package Details

Maintainer: @stangri

Description:
After renaming the the output filter, the unbound filter name was not updated everywhere, which broke the unbound integration

🧪 Run Testing Details

  • OpenWrt Version: trunk
  • OpenWrt Target/Subtarget: Rockchip/RK33xx/RK35xx boards (64 bit)
  • OpenWrt Device: FriendlyARM NanoPi R4S 4GB LPDDR4

✅ Formalities

  • I have reviewed the CONTRIBUTING.md file for detailed contributing guidelines.

If your PR contains a patch:

  • It can be applied using git am
  • It has been refreshed to avoid offsets, fuzzes, etc., using
    make package/<your-package>/refresh V=s
  • It is structured in a way that it is potentially upstreamable
    (e.g., subject line, commit description, etc.)
    We must try to upstream patches to reduce maintenance burden.

Signed-off-by: Tobias Waldvogel <tobias.waldvogel@gmail.com>

The last commit before changed the name of the output filter
$unboundFilter to $unboundOutputFormatFilter. As this was not
adjusted everywhere this broke the unbound integration.
@tobiaswaldvogel tobiaswaldvogel force-pushed the adblock-fast-fix-unbound branch from 73c89ce to e6e8382 Compare February 1, 2026 22:26
@GeorgeSapkin
Copy link
Copy Markdown
Member

GeorgeSapkin commented Feb 1, 2026

You should bump the PKG_RELEASE in the Makefile.

And the sign-off should be at the end of the commit message.

@stangri
Copy link
Copy Markdown
Member

stangri commented Apr 27, 2026

Closing as the main codebase moved to ucode and this is no longer relevant.

@stangri stangri closed this Apr 27, 2026
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