Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 628898 - www-servers/nginx: add ngx_brotli module
Summary: www-servers/nginx: add ngx_brotli module
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Thomas Deutschmann (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on: 643182
Blocks:
  Show dependency tree
 
Reported: 2017-08-25 10:34 UTC by opensource
Modified: 2018-02-16 23:02 UTC (History)
4 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
ngx_brotli patch against nginx-1.13.4.ebuild (nginx-brotli.patch,2.71 KB, patch)
2017-08-25 10:34 UTC, opensource
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description opensource 2017-08-25 10:34:23 UTC
Created attachment 490526 [details, diff]
ngx_brotli patch against nginx-1.13.4.ebuild

Google shows seemingly no intention of getting this module upstreamed, so here's a patch to add brotli support to nginx gentoo-side.

The build system is pretty insane, in that it expects to find a copy of the brotli source code in deps/brotli, with which it very manually builds a static version. This apparently broke recently, which is why the git submodule in the repo is some old version.

The sed monstrosity here makes it just use app-arch/brotli, which appears to work just fine.
Comment 1 Steffen Weber 2017-12-14 10:06:49 UTC
A new version of this module that uses Brotli 1.0 is available at: https://github.com/eustas/ngx_brotli
Comment 2 Larry the Git Cow gentoo-dev 2018-02-16 23:02:41 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=1d52592200d9861a608bc0096550eb5bb311dd13

commit 1d52592200d9861a608bc0096550eb5bb311dd13
Author:     Thomas Deutschmann <whissi@gentoo.org>
AuthorDate: 2018-02-16 23:01:21 +0000
Commit:     Thomas Deutschmann <whissi@gentoo.org>
CommitDate: 2018-02-16 23:02:32 +0000

    www-servers/nginx: Rev bump of "mainline" slot to add/update modules
    
    Ebuild changes:
    ===============
    - LDAP auth module bumped to commit 42d195d7a7575ebab1c369ad3fc5d78dc2c2669c
      to add OpenSSL 1.1.x support and other bugfixes.
    
    - HTTP upstream check module bumped to commit 9aecf15ec379fe98f62355c57b60c0bc83296f04
      to fix possible segfault when reloading configuration.
    
    - Virtual host traffic status module added.
    
    - nginScript module added. [PR 3200]
    
    - Brotli module added. [Bug 628898]
    
    See: https://github.com/gentoo/gentoo/pull/3200
    Closes: https://bugs.gentoo.org/628898
    Package-Manager: Portage-2.3.24, Repoman-2.3.6

 profiles/desc/nginx_modules_http.desc              |    3 +
 profiles/desc/nginx_modules_stream.desc            |    1 +
 www-servers/nginx/Manifest                         |    5 +
 .../nginx/files/http_brotli-detect-brotli.patch    |  111 ++
 www-servers/nginx/nginx-1.13.8-r1.ebuild           | 1059 ++++++++++++++++++++
 5 files changed, 1179 insertions(+)