Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 471106 - www-servers/nginx - add USE flag to allow the addition of the ngnx_pagespeed mod
Summary: www-servers/nginx - add USE flag to allow the addition of the ngnx_pagespeed mod
Status: CONFIRMED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal minor (vote)
Assignee: Thomas Deutschmann
URL:
Whiteboard:
Keywords:
: 477706 (view as bug list)
Depends on:
Blocks:
 
Reported: 2013-05-23 19:46 UTC by Simon Alman
Modified: 2019-01-18 02:59 UTC (History)
9 users (show)

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


Attachments
Patch to add the use flag ngx_pagespeed to nginx to support the pagespeed module (nginx-pagespeed.patch,2.64 KB, patch)
2013-05-23 19:48 UTC, Simon Alman
Details | Diff
Patch against the pagespeed module to fix an conflict with syslog variable declerations in Nginx (nginx-1.4.1-fix-syslog-pagespeed.patch,621 bytes, patch)
2013-05-23 19:51 UTC, Simon Alman
Details | Diff
patch nginx-1.15.8-r1.ebuild to include ngx_pagespeed module (ebuild-1.15.8-r1.patch,2.73 KB, patch)
2019-01-18 02:59 UTC, Chris Yu
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Simon Alman 2013-05-23 19:46:40 UTC
The pagespeed mod exists for Nginx but it not supporting in the gentoo ebuild for nginx. I have written a patch to add it to the existing ebuild that I am using on my own systems and that I feel may benefit others. Please could you review and let me know whether this will be possible.

Reproducible: Always
Comment 1 Simon Alman 2013-05-23 19:48:37 UTC
Created attachment 349022 [details, diff]
Patch to add the use flag ngx_pagespeed to nginx to support the pagespeed module

Apply the patch against the latest nginx ebuild (nginx-1.4.1-r4.ebuild at the time of writing)
Comment 2 Simon Alman 2013-05-23 19:51:25 UTC
Created attachment 349024 [details, diff]
Patch against the pagespeed module to fix an conflict with syslog variable declerations in Nginx

See: https://github.com/pagespeed/ngx_pagespeed/issues/197#issuecomment-15655467 for details of the issue (where this patch was taken from).

Place in: nginx/files/nginx-1.4.1-fix-syslog-pagespeed.patch
Comment 3 Tiziano Müller gentoo-dev 2013-05-23 20:48:59 UTC
Needs PSOL package first, see https://github.com/pagespeed/ngx_pagespeed/wiki/Building-PSOL-From-Source
... until then a no-go since it bundles arbitrary libs.
Comment 4 Johan Bergström 2013-05-26 08:16:38 UTC
This is unfortunately a pretty big undertaking. Good luck to whoever gives it a go :)
Comment 5 Johan Bergström 2013-08-07 04:01:13 UTC
*** Bug 477706 has been marked as a duplicate of this bug. ***
Comment 6 Chris Yu 2019-01-18 02:59:53 UTC
Created attachment 561910 [details, diff]
patch nginx-1.15.8-r1.ebuild to include ngx_pagespeed module

I made a patch against nginx-1.15.8-r1.ebuild and incubator-pagespeed-ngx v1.13.35.2-stable.

It compiled successfully on amd64 and x86, pagespeed version 1.13.35.2-stable and 1.13.35.2-beta.
I tested on my server (amd64/nginx-1.15.8-r1/pagespeed 1.13.35.2-stable) for several days, seems no issue.

Patch mainly copied from mva overlay's pagespeed package, and thanks the maintainer.