Skip to content

adblock: bugfix 3.8.13 - #10747

Merged
dibdot merged 1 commit into
openwrt:masterfrom
dibdot:adblock
Dec 10, 2019
Merged

adblock: bugfix 3.8.13#10747
dibdot merged 1 commit into
openwrt:masterfrom
dibdot:adblock

Conversation

@dibdot

@dibdot dibdot commented Dec 10, 2019

Copy link
Copy Markdown
Contributor

Maintainer: me / @dibdot
Compile tested: -
Run tested: OpenWrt SNAPSHOT r11651-487e0631d0

Description:

  • remove 'ransomware' blocklist by abbuse.ch (discontinued)
    from default adblock config
  • fix/switch 'someonewhocares' config to https only
  • fix curl download parameters to follow redirects and
    suppress needless output
  • made the tmp directory of sort operations configurable,
    set 'adb_sorttmp' accordingly (only supported by 'coreutils-sort')

Signed-off-by: Dirk Brenken dev@brenken.org

* remove 'ransomware' blocklist by abbuse.ch (discontinued)
  from default adblock config
* fix/switch 'someonewhocares' config to https only
* fix curl download parameters to follow redirects and
  suppress needless output
* made the tmp directory of sort operations configurable,
  set 'adb_sorttmp' accordingly (only supported by 'coreutils-sort')

Signed-off-by: Dirk Brenken <dev@brenken.org>
@dibdot
dibdot merged commit 1ed53b3 into openwrt:master Dec 10, 2019
@dibdot
dibdot deleted the adblock branch December 10, 2019 19:21
@fantom-x

Copy link
Copy Markdown

Would you consider adding --compress-program=/bin/gzip to compress temporary files? I just tested this on ipq806x and ipq40x and there is no noticeable slowness. The adblock files are well compressed and this should free up a significant chunk of RAM.

To use the max compression: GZIP_OPT=-9 sort --compress-program=/bin/gzip. No sure if this is necessary, though.

      --compress-program=PROG  compress temporaries with PROG;
                              decompress them with PROG -d

@fantom-x

Copy link
Copy Markdown

This change would complement the new adb_sorttmp parameter.

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.

2 participants