Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 421299 - www-servers/nginx: add spdy support
Summary: www-servers/nginx: add spdy support
Status: RESOLVED DUPLICATE of bug 456224
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Benedikt Böhm (RETIRED)
URL: http://nginx.org/patches/spdy/
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-06-15 23:33 UTC by James Cloos
Modified: 2013-03-26 20:59 UTC (History)
6 users (show)

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


Attachments
add USE spdy (file_421299.txt,1.29 KB, text/plain)
2012-09-08 09:15 UTC, Qing Lei
Details
nginx-use-spdy.patch (nginx-spdy.patch,2.28 KB, patch)
2012-09-10 05:50 UTC, Qing Lei
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description James Cloos 2012-06-15 23:33:57 UTC
The current (versioned) patch is:

    http://nginx.org/patches/spdy/patch.spdy-36.txt

If added, it should be controlled via a ‘spdy’ USE flag.

It is a patch, not an nginx module, so the modules code in nginx-1.3.1.ebuild, unfortunately, is unlikely to be the right way to add this....
Comment 1 James Cloos 2012-08-15 21:44:33 UTC
For reference, the current version (2012-Aug-13) of the patch is:

http://nginx.org/patches/spdy/patch.spdy-52.txt

and http://nginx.org/patches/spdy/ works to show the list of such patches.
Comment 2 Benedikt Böhm (RETIRED) gentoo-dev 2012-08-16 12:33:56 UTC
i'd really like to wait for upstream to merge this into nginx. if that is not going to happen in the nearer future, than we can redecide. i'll leave the bug open for now ...
Comment 3 Qing Lei 2012-09-08 09:15:26 UTC
Created attachment 323194 [details]
add USE spdy

wget http://nginx.org/patches/spdy/patch.spdy-52.txt to ${FILESDIR} at first.

stolen from https://raw.github.com/olemarkus/olemarkus-overlay/master/www-servers /nginx/nginx-1.3.1.ebuild
Comment 4 James Cloos 2012-09-08 20:25:59 UTC
I also have a working ebuild in my overlay:

  git://people.freedesktop.org/~cloos/overlay.git

Cf: 

http://cgit.freedesktop.org/~cloos/overlay/commit/?id=b86d76ebe4eae25786f9e579

I tried to follow the pattern of the existing additions.
Comment 5 Qing Lei 2012-09-10 05:50:26 UTC
Created attachment 323374 [details, diff]
nginx-use-spdy.patch

make a patch from Cloos's ebuild, and apply patch in src_prepare() instead of src_unpack().
Comment 6 Fabio Bonfante 2013-03-11 16:24:49 UTC
Looking at http://nginx.org/patches/spdy/README.txt seems that now the SPDY support is provided (rightly) through a nginx_module.
So giving that:
* the patch is done/supported by the official team of nginx
* activating this USE flag is a consciously user choice
* we are talking about a development release (so nobody can complain if it's not work as expected)
* as Benedict rightly stated: "if that is not going to happen in the nearer future" as it's seem it to me now

I think we can put in portage. What you think guys?
Comment 7 Johan Bergström 2013-03-26 20:59:28 UTC
Nginx 1.3.15 now incorporates spdy support. Lets work on getting that version in instead.

*** This bug has been marked as a duplicate of bug 456224 ***