[Bug 1074515] Review Request: passenger - Passenger Ruby web application server

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=1074515



--- Comment #8 from Jan Kaluža <jkaluza@xxxxxxxxxx> ---
(In reply to Vít Ondruch from comment #5)
> Isn't the description outdated a bit? I thought that the rename is mainly
> because passenger is now far more then Ruby thing.

Yes, I'm going to fix that. Upstream proposed better description text and I
have it already in my local passenger.spec which will be in the new srpm I will
upload to this review.

(In reply to Vít Ondruch from comment #6)
> And I would appreciate if you can fix bug 1064822.

Will do that.

(In reply to Vít Ondruch from comment #6)
> And I would appreciate if you can fix bug 1064822.

(In reply to Vít Ondruch from comment #6)
> Moreover, I dont udnerstand why you completely dropped the rubygem- and
> install stuff into %{ruby_vendor{lib,arch}dir}. This is discouraged practice.

You mean why I dropped the "rubygem-" prefix and want the passenger to rename?
There are following reasons pointed out by upstream:

> It's currently called "rubygem-passenger", but
> Passenger is no longer about Ruby only. Nowadays Passenger is a true
> polyglot server, with support for Python, Node.js and Meteor as well.
> Furthermore, Passenger isn't a gem in the usual sense; it's
> distributed over a wide variety of channels such as tarball, deb,
> HomeBrew, etc. RubyGems is just another distribution channel for us.

Maintainer and other co-maintainers of rubygem-passenger agreed with this
change but this decision has not resulted in any action, so I started this
process.

I treated Passenger as "Non-Gem Packages" category in the Ruby Packaging
Guidelines and got the feeling that I should use ruby_vendor* macros. However,
if you really think it should be installed into different directories, I can
change that, since it's clear you have much more experience with Ruby packages
than me.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review





[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]