363517 – www-servers/nginx could use existing passenger gem rather than insisting on standalone

Bug 363517 - www-servers/nginx could use existing passenger gem rather than insisting on standalone

Summary: www-servers/nginx could use existing passenger gem rather than insisting on s...

Status:	RESOLVED WONTFIX

Alias:	None

Product:	Gentoo Linux
Classification:	Unclassified
Component:	New packages (show other bugs)
Hardware:	All Linux

Importance:	Normal normal
Assignee:	Benedikt Böhm (RETIRED)

URL:
Whiteboard:
Keywords:

Depends on:
Blocks:

Reported:	2011-04-13 22:45 UTC by Dennis Krupenik
Modified:	2011-04-14 14:06 UTC (History)
CC List:	1 user (show)

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

Attachments
adds support for passenger if gem is installed (nginx-passenger.diff,1.44 KB, patch) 2011-04-13 22:47 UTC, Dennis Krupenik	Details \| Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Dennis Krupenik 2011-04-13 22:45:27 UTC

Instead of dropping support of passenger in nginx, we can use installed passenger gem. That way we can still build nginx with passenger support while not clashing with www-apache/passenger (why in the world it is in www-apache anyway?). I'm attaching a patch for nginx ebuild, which I use for a long time now. 

Reproducible: Always

Comment 1 Dennis Krupenik 2011-04-13 22:47:29 UTC

Created attachment 269875 [details, diff]
adds support for passenger if gem is installed

Comment 2 Benedikt Böhm (RETIRED) gentoo-dev

2011-04-14 14:06:49 UTC

i consider passenger obsoleted by unicorn, and i don't want to maintain it's crappy build system in nginx. i'm also against using manually installed gems for build time, since it will most likely brake binary packages. if necessary one can still use NGINX_ADD_MODULES in make.conf to add passenger support.