Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
Summary: Review Request: rubygem-open4 - Manage child processes and their IO handles easily
https://bugzilla.redhat.com/show_bug.cgi?id=642577
Summary: Review Request: rubygem-open4 - Manage child processes and their IO handles easily Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: mfojtik@redhat.com QAContact: extras-qa@fedoraproject.org CC: notting@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Target Release: ---
Spec URL: http://mifo.sk/RPMS/rubygem-open4.spec SRPM URL: http://mifo.sk/RPMS/rubygem-open4-1.0.1-1.fc13.src.rpm Description:
Open child process with handles on pid, stdin, stdout, and stderr. Manage child processes and their IO handles easily.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=642577
Jozef Zigmund jzigmund@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |jzigmund@redhat.com Flag| |fedora-review?
--- Comment #1 from Jozef Zigmund jzigmund@redhat.com 2010-10-13 08:46:12 EDT --- i will take this review.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=642577
Mamoru Tasaka mtasaka@ioa.s.u-tokyo.ac.jp changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |mtasaka@ioa.s.u-tokyo.ac.jp Flag| |needinfo?(jzigmund@redhat.c | |om)
--- Comment #2 from Mamoru Tasaka mtasaka@ioa.s.u-tokyo.ac.jp 2010-10-13 14:18:51 EDT --- Hello, Jozef:
As far as I checked FAS (Fedora Account System), it seems that you are not sponsored and are not in packager group. The person who is going to formally review review requests must be packager group. Would you tell me if you are sponsored?
http://fedoraproject.org/wiki/PackageMaintainers/HowToGetSponsored
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=642577
Jozef Zigmund jzigmund@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|needinfo?(jzigmund@redhat.c | |om) |
--- Comment #3 from Jozef Zigmund jzigmund@redhat.com 2010-10-14 11:29:34 EDT --- MUST: rpmlint must be run on every package. The output should be posted in the review.
rpmlint rubygem-open4-1.0.1-1.fc13.src.rpm rubygem-open4.src: W: spelling-error %description -l en_US pid -> peed, dip, pud rubygem-open4.src: W: spelling-error %description -l en_US stdin -> stein, stain, stdio rubygem-open4.src: W: spelling-error %description -l en_US stdout -> stout, std out, std-out rubygem-open4.src: W: spelling-error %description -l en_US stderr -> std err, std-err, stander rubygem-open4.src: W: no-buildroot-tag 1 packages and 0 specfiles checked; 0 errors, 5 warnings. [OK]
Each Ruby package MUST indicate the Ruby ABI version it depends on with a line like: Requires: ruby(abi) = 1.8
[OK - this MUST condition is written via macro in this spec file]
Packages that contain Ruby Gems MUST be called rubygem-%{gemname} where gemname is the name from the Gem's specification.
%global gemname open4 Name: rubygem-%{gemname} [OK]
The Source of the package MUST be the full URL to the released Gem archive
URL: http://github.com/ahoward/open4/tree/master Source0: http://gems.rubyforge.org/gems/%%7Bgemname%7D-%%7Bversion%7D.gem [OK]
The version of the package MUST be the Gem's version
Version: 1.0.1 [OK]
The package MUST have a Requires and a BuildRequires on rubygems
%global rubyabi 1.8
Requires: ruby(abi) = %{rubyabi} Requires: rubygems BuildRequires: rubygems [OK]
The Gem MUST be installed into %{gemdir} defined as: %global gemdir %(ruby -rubygems -e 'puts Gem::dir' 2>/dev/null)
[OK]
If the Gem only contains pure Ruby code, it MUST be marked as BuildArch: noarch
BuildArch: noarch [OK]
The package MUST own the following files and directories:
%{gemdir}/gems/%{gemname}-%{version}/ %{gemdir}/cache/%{gemname}-%{version}.gem %{gemdir}/specifications/%{gemname}-%{version}.gemspec [OK]
The install SHOULD be performed with the command
gem install --local --install-dir %{buildroot}%{gemdir} --force %{SOURCE0} [OK]
Note: At the end of %description is '.' character and i'm not sure if it's OK.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=642577
--- Comment #4 from Jozef Zigmund jzigmund@redhat.com 2010-10-14 11:57:23 EDT --- Yes i checked this wiki page before, so i've written unformally review to this package, and now i hope that somebody will add me to Fedora Packager Group. And of course i can tell you if i will have sponsoring. (In reply to comment #2)
Hello, Jozef:
As far as I checked FAS (Fedora Account System), it seems that you are not sponsored and are not in packager group. The person who is going to formally review review requests must be packager group. Would you tell me if you are sponsored?
http://fedoraproject.org/wiki/PackageMaintainers/HowToGetSponsored
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=642577
Marek Mahut mmahut@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |mmahut@redhat.com AssignedTo|nobody@fedoraproject.org |mmahut@redhat.com
--- Comment #5 from Marek Mahut mmahut@redhat.com 2010-10-15 09:21:12 EDT --- I will be sponsoring Jozef, taking over the review.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=642577
Marek Mahut mmahut@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review? |fedora-review+
--- Comment #6 from Marek Mahut mmahut@redhat.com 2010-10-15 09:46:33 EDT --- * The dot at the end of the description is OK, we avoid it only in Summary * Use %doc under your %files to specify documentation files
The review from Jozef looks sane. Package approved.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=642577
--- Comment #7 from Michal Fojtik mfojtik@redhat.com 2010-10-15 11:19:44 EDT --- (In reply to comment #6)
- The dot at the end of the description is OK, we avoid it only in Summary
- Use %doc under your %files to specify documentation files
Could you please point me out, which files you mean ? I throught that there is no need to mark files under -doc sub package with %doc.
The review from Jozef looks sane. Package approved.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=642577
Michal Fojtik mfojtik@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs?
--- Comment #8 from Michal Fojtik mfojtik@redhat.com 2010-10-15 11:48:54 EDT --- Anyway thank you all for a review :-)
New Package SCM Request ======================= Package Name: rubygem-open4 Short Description: Manage child processes and their IO handles easily Owners: mfojtik Branches: f12 f13 f14
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=642577
--- Comment #9 from Kevin Fenzi kevin@tummy.com 2010-10-16 17:20:20 EDT --- Git done (by process-git-requests).
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=642577
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |MODIFIED
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=642577
--- Comment #10 from Fedora Update System updates@fedoraproject.org 2010-10-17 10:32:52 EDT --- rubygem-open4-1.0.1-1.fc13 has been submitted as an update for Fedora 13. https://admin.fedoraproject.org/updates/rubygem-open4-1.0.1-1.fc13
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=642577
--- Comment #11 from Fedora Update System updates@fedoraproject.org 2010-10-17 10:37:37 EDT --- rubygem-open4-1.0.1-1.fc14 has been submitted as an update for Fedora 14. https://admin.fedoraproject.org/updates/rubygem-open4-1.0.1-1.fc14
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=642577
--- Comment #12 from Fedora Update System updates@fedoraproject.org 2010-10-17 10:42:35 EDT --- rubygem-open4-1.0.1-1.fc12 has been submitted as an update for Fedora 12. https://admin.fedoraproject.org/updates/rubygem-open4-1.0.1-1.fc12
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=642577
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|MODIFIED |ON_QA
--- Comment #13 from Fedora Update System updates@fedoraproject.org 2010-10-17 11:42:04 EDT --- rubygem-open4-1.0.1-1.fc14 has been pushed to the Fedora 14 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update rubygem-open4'. You can provide feedback for this update here: https://admin.fedoraproject.org/updates/rubygem-open4-1.0.1-1.fc14
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=642577
--- Comment #14 from Fedora Update System updates@fedoraproject.org 2010-10-27 18:32:47 EDT --- rubygem-open4-1.0.1-1.fc12 has been pushed to the Fedora 12 stable repository. If problems still persist, please make note of it in this bug report.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=642577
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ON_QA |CLOSED Fixed In Version| |rubygem-open4-1.0.1-1.fc12 Resolution| |ERRATA Last Closed| |2010-10-27 18:32:52
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=642577
--- Comment #15 from Fedora Update System updates@fedoraproject.org 2010-10-27 18:39:47 EDT --- rubygem-open4-1.0.1-1.fc13 has been pushed to the Fedora 13 stable repository. If problems still persist, please make note of it in this bug report.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=642577
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Fixed In Version|rubygem-open4-1.0.1-1.fc12 |rubygem-open4-1.0.1-1.fc13
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=642577
--- Comment #16 from Fedora Update System updates@fedoraproject.org 2010-11-04 19:31:05 EDT --- rubygem-open4-1.0.1-1.fc14 has been pushed to the Fedora 14 stable repository. If problems still persist, please make note of it in this bug report.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=642577
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Fixed In Version|rubygem-open4-1.0.1-1.fc13 |rubygem-open4-1.0.1-1.fc14
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=642577
Vít Ondruch vondruch@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |vondruch@redhat.com
--- Comment #17 from Vít Ondruch vondruch@redhat.com 2011-06-02 07:06:49 EDT --- *** Bug 708990 has been marked as a duplicate of this bug. ***
package-review@lists.fedoraproject.org