Differential D13889
Add www/rubygem-passenger-apache & www/rubygem-passenger-nginxRequired so we can have both Apache and Nginx options in the same repo.Flavors would be great for this, but they aren't ready yet.While here, make www/rubygem-passenger compile... Authored by dvl on Jan 12 2018, 7:31 PM. Tags None Referenced Files
Details
...without: > Running Q/A tests (stage-qa)Error: '/bin/bash' is an invalid shebang you need USES=shebangfix for 'lib/ruby/gems/2.4/gems/passenger-5.1.12/dev/ci/tests/debian/run'
Diff Detail
Event Timeline
Comment Actions Hi, no new ports/meta-ports required to support both technologies (apache and nginx), I believe it's enough to patch existing www/rubygem-passenger. Comment Actions I tried FLAVORS I am not convinced it is correct. The resulting package is 1.3G for Apache, and 11,000 files. I did locate the passenger module for Apache, but I didn't find it for pkg info -l rubygem-passenger-apache | grep mod_passenger.so /usr/local/lib/ruby/gems/2.4/gems/passenger-5.1.12/buildout/apache2/mod_passenger.so Comment Actions Looking at the Makefile, I now think rubygem-passenger-nginx-5.1.12 has the right modules: /usr/local/lib/ruby/gems/2.4/gems/passenger-5.1.12/buildout/ruby/ruby-2.4.3-x86_64-freebsd/Makefile /usr/local/lib/ruby/gems/2.4/gems/passenger-5.1.12/buildout/ruby/ruby-2.4.3-x86_64-freebsd/mkmf.log /usr/local/lib/ruby/gems/2.4/gems/passenger-5.1.12/buildout/ruby/ruby-2.4.3-x86_64-freebsd/passenger_native_support.o /usr/local/lib/ruby/gems/2.4/gems/passenger-5.1.12/buildout/ruby/ruby-2.4.3-x86_64-freebsd/passenger_native_support.so Comment Actions Oh, I should remove the OPTIONS and just use FLAVOR I do not yet know how to make a default FLAVOR Comment Actions The first flavor mentioned in 1 is always the default and doesn't need to be named as such. Example: FLAVORS= nginx apache In this case nginx would be the default flavor. Hope this helps. [1] https://www.freebsd.org/doc/en/books/porters-handbook/flavors-using.html Comment Actions
Thank you. I'd been there and overlooked it. Cheers. Next goal: get this to compile. @nginx compiles, but has 8000 more files and than the original port. apache does not compile yet.
Comment Actions This version builds correctly with no issues for both apache and nginx flavors I also included a symlink to mod_passenger.so so users do not have to update their Comment Actions This builds for me with without a flavor, with @apache, and with @nginx. All pass testport. I have some further tests to run tomorrow morning. Comment Actions Tests so far this morning are going well. If this continues, I plan to commit later today. Comment Actions Hi Dan, I've just done with my builds of both flavors on my local machine, all looks good, good job! Comment Actions I understand portmgr@ approval is required for commiting new FLAVORS. I have emailed them for such approval before proceeding.
| |||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||