Bug 389151
Summary: | Review Request: xchat-ruby - X-Chat plugin adding Ruby scripting functionality | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Conrad Meyer <cse.cem+redhatbugz> |
Component: | Package Review | Assignee: | Mamoru TASAKA <mtasaka> |
Status: | CLOSED ERRATA | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | low | ||
Version: | rawhide | CC: | fedora-package-review, kevin, lutter, mtasaka, notting |
Target Milestone: | --- | Flags: | mtasaka:
fedora-review+
kevin: fedora-cvs+ |
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | 1.2-5.fc8 | Doc Type: | Bug Fix |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2007-12-27 01:01:57 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
Conrad Meyer
2007-11-18 06:23:39 UTC
This is my first package and I need a sponsor. Thanks! CCing David Lutterkort (sponsor from Ruby SIG). David, are you willing to take this review/sponsorship request? License should be GPLv2+, not GPLv2, the sources explicitly allow any later version. (In reply to comment #3) > License should be GPLv2+, not GPLv2, the sources explicitly allow any later > version. Fixed in specfile; rebuilt and uploaded this modification to the same URLs. Thanks! OK, but next time please increase the Release number each time you make a change to the packaging, that's what it is for. :-) Spec URL: http://konrad.sobertillnoon.com/fedora/xchat-ruby.spec SRPM URL: http://konrad.sobertillnoon.com/fedora/xchat-ruby-1.2-2.fc8.src.rpm Changed some usages of /etc to %{_sysconfdir}. Well, I have not used xchat and so currently I don't know how to use this, however: - On x86_64 and ppc64, while arch dependent files (usually .so files) are installed under /usr/lib64 (i.e. %_libdir, for example ruby_sitearch), arch-independent files (.rb files, for example) are installed under /usr/lib (for example, ruby_sitelib) c.f. http://fedoraproject.org/wiki/Packaging/Ruby Does your xchat-rubyenv also work for 64 bits archs? - And currently the contents of xchat-rubyenv differs between on 32 bits archs and on 64 bits archs. In this case, it may be better that xchat-rubyenv is installed under %_libdir, not under %_sysconfdir. - build log shows that fedora specific compilation flags are not honored correctly. http://koji.fedoraproject.org/koji/taskinfo?taskID=255633 See build.log . 1) It didn't, but the next revision will. 2) This is also fixed. 3) This is also fixed :). SPEC file: http://konrad.sobertillnoon.com/fedora/xchat-ruby.spec SRPM: http://konrad.sobertillnoon.com/fedora/xchat-ruby-1.2-3.fc8.src.rpm Thanks for your feedback. Oops, xchat-rubyenv ought to include arch-specific load paths (for ruby libraries written in C). This is now fixed in 1.2-4. http://konrad.sobertillnoon.com/fedora/xchat-ruby.spec http://konrad.sobertillnoon.com/fedora/xchat-ruby-1.2-4.fc8.src.rpm For 1.2-4: * SourceURL - I suggest to use http://downloads.sourceforge.net/xchat-ruby/%{name}-%{version}-src.tar.gz With this, you don't have to rewrite sourceURL when new version is released. * Ruby API dependency for BuildRequires - While this is not written in ruby guidelines, I usually recommend to add also ------------------------------------------------------ BuildRequires: ruby(abi) = 1.8 ------------------------------------------------------ for the consistency with Requires. * Macros in %changelog ------------------------------------------------------ * Fri Nov 23 2007 Conrad Meyer <konrad> - 1.2-2 - Replaced /etc with %{_sysconfdir} ------------------------------------------------------ - Perhaps $ rpm -q --changelog xchat-ruby will show ------------------------------------------------------ * Fri Nov 23 2007 Conrad Meyer <konrad> - 1.2-2 - Replaced /etc with /etc ------------------------------------------------------ This is because macros are expanded even in %changelog. To avoild this, please use %% in changelog. i.e. ------------------------------------------------------ * Fri Nov 23 2007 Conrad Meyer <konrad> - 1.2-2 - Replaced /etc with %%{_sysconfdir} ------------------------------------------------------ Well, as this is SPONSORNEEDED ticket, ------------------------------------------------------------- NOTE: Before being sponsored: This package will be accepted with another few work. But before I accept this package, someone (I am a candidate) must sponsor you. Once you are sponsored, you have the right to review other submitters' review requests and approve the packages formally. For this reason, the person who want to be sponsored (like you) are required to "show that you have an understanding of the process and of the packaging guidelines" as is described on : http://fedoraproject.org/wiki/PackageMaintainers/HowToGetSponsored Usually there are two ways to show this. A. submit other review requests with enough quality. B. Do a "pre-review" of other person's review request (at the time you are not sponsored, you cannot do a formal review) When you have submitted a new review request or have pre-reviewed other person's review request, please write the bug number on this bug report so that I can check your comments or review request. Fedora package collection review requests which are waiting for someone to review can be checked on: http://fedoraproject.org/PackageReviewStatus/NEW.html (NOTE: please don't choose "Merge Review") Review guidelines are described mainly on: http://fedoraproject.org/wiki/Packaging/ReviewGuidelines http://fedoraproject.org/wiki/Packaging/Guidelines http://fedoraproject.org/wiki/Packaging/ScriptletSnippets ------------------------------------------------------------ Thanks you for the quick responses. Spec: http://konrad.sobertillnoon.com/fedora/xchat-ruby.spec SRPM: http://konrad.sobertillnoon.com/fedora/xchat-ruby-1.2-5.fc8.src.rpm I will work on "showing that I have an understanding of the process and of the packaging guidelines" now. :) Well, it may be better that you add ChangeLog to %doc (not a blocker) - This package itself is good - Your pre-review seems good for initial comments. -------------------------------------------------------- This package (xchat-ruby) is APPROVED by me -------------------------------------------------------- Please follow the procedure according to: http://fedoraproject.org/wiki/PackageMaintainers/Join from "Get a Fedora Account". At a point a mail should be sent to sponsor members which notifies that you need a sponsor (at the stage, please also write on this bug for confirmation that you requested for sponsorship) Then I will sponsor you. If you want to import this package into Fedora 7/8, you also have to look at http://fedoraproject.org/wiki/Infrastructure/UpdatesSystem/Bodhi-info-DRAFT (after once you rebuilt this package on Fedora rebuilding system). If you have questions, please ask me. I (konradm on the Fedora accounts system) confirm that I requested sponsorship :). Now I should be sponsoring you. Please continue "Join" process. New Package CVS Request ======================= Package Name: xchat-ruby Short Description: An X-Chat plugin providing scripting functionality with Ruby Owners: konradm Branches: F-7 F-8 InitialCC: Cvsextras Commits: yes cvs done. xchat-ruby-1.2-5.fc7 has been pushed to the Fedora 7 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 xchat-ruby' xchat-ruby-1.2-5.fc8 has been pushed to the Fedora 8 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 xchat-ruby' Closing. xchat-ruby-1.2-5.fc8 has been pushed to the Fedora 8 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 xchat-ruby' xchat-ruby-1.2-5.fc7 has been pushed to the Fedora 7 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 xchat-ruby' xchat-ruby-1.2-5.fc8 has been pushed to the Fedora 8 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 xchat-ruby' xchat-ruby-1.2-5.fc7 has been pushed to the Fedora 7 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 xchat-ruby' xchat-ruby-1.2-5.fc8 has been pushed to the Fedora 8 stable repository. If problems still persist, please make note of it in this bug report. xchat-ruby-1.2-5.fc7 has been pushed to the Fedora 7 stable repository. If problems still persist, please make note of it in this bug report. |