https://bugzilla.redhat.com/show_bug.cgi?id=1348160
Germano Massullo changed:
What|Removed |Added
Status|NEW |CLOSED
Resolution|---
https://bugzilla.redhat.com/show_bug.cgi?id=1348160
Fabio Alessandro Locati changed:
What|Removed |Added
Assignee|f...@redhat.com |nob...@fedoraproject.org
--
https://bugzilla.redhat.com/show_bug.cgi?id=1348160
Fabio Alessandro Locati changed:
What|Removed |Added
Flags||needinfo?(germano.massullo@
https://bugzilla.redhat.com/show_bug.cgi?id=1348160
Germano Massullo changed:
What|Removed |Added
Flags||fedora-review?
--
You are receivi
https://bugzilla.redhat.com/show_bug.cgi?id=1348160
--- Comment #14 from Fabio Alessandro Locati ---
ok, then use it, because ATM you are adding it to the package but you are not
using it
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about
https://bugzilla.redhat.com/show_bug.cgi?id=1348160
--- Comment #13 from Germano Massullo ---
(In reply to Fabio Alessandro Locati from comment #11)
> SOURCE1 is not used anywhere. It should be used somewhere or removed
it is still required in lastest em-proxy release
--
You are receiving th
https://bugzilla.redhat.com/show_bug.cgi?id=1348160
--- Comment #12 from Vít Ondruch ---
BTW it would be also good idea to execute some test suite, if that is available
...
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to thi
https://bugzilla.redhat.com/show_bug.cgi?id=1348160
--- Comment #11 from Fabio Alessandro Locati ---
SOURCE1 is not used anywhere. It should be used somewhere or removed
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this p
https://bugzilla.redhat.com/show_bug.cgi?id=1348160
Germano Massullo changed:
What|Removed |Added
Flags|needinfo?(germano.massullo@ |
|gmail.com)
https://bugzilla.redhat.com/show_bug.cgi?id=1348160
Vít Ondruch changed:
What|Removed |Added
CC||vondr...@redhat.com
--- Comment #9 from
https://bugzilla.redhat.com/show_bug.cgi?id=1348160
Fabio Alessandro Locati changed:
What|Removed |Added
CC||germano.massu...@gmail.com
https://bugzilla.redhat.com/show_bug.cgi?id=1348160
--- Comment #7 from Francesco Frassinelli (frafra) ---
I would suggest to avoid env for Fedora packages for two reasons:
1. Fedora wiki suggests to use #!/usr/bin/ruby and #!/usr/bin/python
2. You don't want to get different behaviour/errors
https://bugzilla.redhat.com/show_bug.cgi?id=1348160
--- Comment #6 from Germano Massullo ---
(In reply to Fabio Alessandro Locati from comment #5)
> @Germano: You _should_ probably rebase this package on 0.1.9
I know but I am waiting for Francesco Frassinelli comment about Ruby shebangs.
He cl
https://bugzilla.redhat.com/show_bug.cgi?id=1348160
--- Comment #5 from Fabio Alessandro Locati ---
@Germano: You _should_ probably rebase this package on 0.1.9
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product an
https://bugzilla.redhat.com/show_bug.cgi?id=1348160
--- Comment #4 from Germano Massullo ---
Personal reminder: LICENSE file included in rubygem version 0.1.9
https://github.com/igrigorik/em-proxy/issues/56
--
You are receiving this mail because:
You are on the CC list for the bug.
You are al
https://bugzilla.redhat.com/show_bug.cgi?id=1348160
--- Comment #3 from Fabio Alessandro Locati ---
@Germano: The result for both is the same, so I would say that it does not
really matter. If upstream uses `env ruby` is ok. That phrase compares using
ruby vs ruby-mri/jruby
--
You are receivi
https://bugzilla.redhat.com/show_bug.cgi?id=1348160
Igor Gnatenko changed:
What|Removed |Added
Summary|Review Request: |Review Request:
|ruby
https://bugzilla.redhat.com/show_bug.cgi?id=1348160
--- Comment #2 from Germano Massullo ---
Should I patch also other Shebangs in source code, as written in Ruby
guidelines?
https://fedoraproject.org/wiki/Packaging:Ruby#Shebang_lines
Example:
#!/usr/bin/ruby
instead of
#!/usr/bin/env ruby
--
https://bugzilla.redhat.com/show_bug.cgi?id=1348160
--- Comment #1 from Fabio Alessandro Locati ---
Things to fix/improve:
- Include the LICENSE file in the package
- Exclude the .rspec folder from the rpm package
- Fix script-without-shebang
/usr/share/gems/gems/em-proxy-0.1.8/examples/balanci
https://bugzilla.redhat.com/show_bug.cgi?id=1348160
Fabio Alessandro Locati changed:
What|Removed |Added
CC||f...@redhat.com
Ass
https://bugzilla.redhat.com/show_bug.cgi?id=1348160
Germano Massullo changed:
What|Removed |Added
Blocks||1348005
Referenced Bugs:
https:/
21 matches
Mail list logo