Bug 471106

Summary:	www-servers/nginx - add USE flag to allow the addition of the ngnx_pagespeed mod
Product:	Gentoo Linux	Reporter:	Simon Alman <haven>
Component:	Current packages	Assignee:	No maintainer - Look at https://wiki.gentoo.org/wiki/Project:Proxy_Maintainers if you want to take care of it <maintainer-needed>
Status:	CONFIRMED ---
Severity:	minor	CC:	bug, bugs, chrisyu.gm, dev-zero, hydrapolic, jens, kyle.leet, volker, yolabingo
Priority:	Normal
Version:	unspecified
Hardware:	All
OS:	Linux
Whiteboard:
Package list:		Runtime testing required:	---
Attachments:	Patch to add the use flag ngx_pagespeed to nginx to support the pagespeed module Patch against the pagespeed module to fix an conflict with syslog variable declerations in Nginx patch nginx-1.15.8-r1.ebuild to include ngx_pagespeed module

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 (RETIRED) 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.