[Bug 545046] Review Request: CVector - ANSI C API for Dynamic Arrays
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=545046 --- Comment #28 from Fedora Update System upda...@fedoraproject.org 2010-09-16 12:30:51 EDT --- CVector-1.0.3-1.5Aug09.el5 has been pushed to the Fedora EPEL 5 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 545046] Review Request: CVector - ANSI C API for Dynamic Arrays
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=545046 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|CVector-1.0.3-1.5Aug09.fc13 |CVector-1.0.3-1.5Aug09.el5 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 545046] Review Request: CVector - ANSI C API for Dynamic Arrays
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=545046 --- Comment #26 from Fedora Update System upda...@fedoraproject.org 2010-09-11 04:58:33 EDT --- CVector-1.0.3-1.5Aug09.fc12 has been pushed to the Fedora 12 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 545046] Review Request: CVector - ANSI C API for Dynamic Arrays
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=545046 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|CVector-1.0.3-1.5Aug09.fc14 |CVector-1.0.3-1.5Aug09.fc12 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 545046] Review Request: CVector - ANSI C API for Dynamic Arrays
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=545046 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|CVector-1.0.3-1.5Aug09.fc12 |CVector-1.0.3-1.5Aug09.fc13 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 545046] Review Request: CVector - ANSI C API for Dynamic Arrays
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=545046 --- Comment #27 from Fedora Update System upda...@fedoraproject.org 2010-09-11 05:03:08 EDT --- CVector-1.0.3-1.5Aug09.fc13 has been pushed to the Fedora 13 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 545046] Review Request: CVector - ANSI C API for Dynamic Arrays
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=545046 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||CVector-1.0.3-1.5Aug09.fc14 Resolution||ERRATA Last Closed||2010-09-10 23:32:56 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 545046] Review Request: CVector - ANSI C API for Dynamic Arrays
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=545046 --- Comment #20 from Fedora Update System upda...@fedoraproject.org 2010-09-01 06:05:15 EDT --- CVector-1.0.3-1.5Aug09.fc14 has been submitted as an update for Fedora 14. https://admin.fedoraproject.org/updates/CVector-1.0.3-1.5Aug09.fc14 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 545046] Review Request: CVector - ANSI C API for Dynamic Arrays
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=545046 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 545046] Review Request: CVector - ANSI C API for Dynamic Arrays
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=545046 --- Comment #21 from Fedora Update System upda...@fedoraproject.org 2010-09-01 06:08:49 EDT --- CVector-1.0.3-1.5Aug09.fc12 has been submitted as an update for Fedora 12. https://admin.fedoraproject.org/updates/CVector-1.0.3-1.5Aug09.fc12 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 545046] Review Request: CVector - ANSI C API for Dynamic Arrays
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=545046 --- Comment #22 from Fedora Update System upda...@fedoraproject.org 2010-09-01 06:10:10 EDT --- CVector-1.0.3-1.5Aug09.el5 has been submitted as an update for Fedora EPEL 5. https://admin.fedoraproject.org/updates/CVector-1.0.3-1.5Aug09.el5 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 545046] Review Request: CVector - ANSI C API for Dynamic Arrays
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=545046 --- Comment #23 from Fedora Update System upda...@fedoraproject.org 2010-09-01 06:11:40 EDT --- CVector-1.0.3-1.5Aug09.fc13 has been submitted as an update for Fedora 13. https://admin.fedoraproject.org/updates/CVector-1.0.3-1.5Aug09.fc13 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 545046] Review Request: CVector - ANSI C API for Dynamic Arrays
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=545046 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #24 from Fedora Update System upda...@fedoraproject.org 2010-09-01 22:28:18 EDT --- CVector-1.0.3-1.5Aug09.el5 has been pushed to the Fedora EPEL 5 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 CVector'. You can provide feedback for this update here: https://admin.fedoraproject.org/updates/CVector-1.0.3-1.5Aug09.el5 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 545046] Review Request: CVector - ANSI C API for Dynamic Arrays
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=545046 Takanori MATSUURA t.mat...@gmail.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #18 from Takanori MATSUURA t.mat...@gmail.com 2010-08-30 06:09:56 EDT --- New Package SCM Request === Package Name: CVector Short Description: ANSI C API for Dynamic Arrays Owners: tmatsuu Branches: f12 f13 f14 el5 el6 InitialCC: (In reply to comment #15) Versioning and patch rather than sed issues are fixed. (In reply to comment #17) Useless sed command has been removed. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 545046] Review Request: CVector - ANSI C API for Dynamic Arrays
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=545046 --- Comment #19 from Kevin Fenzi ke...@tummy.com 2010-08-30 13:29:45 EDT --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 545046] Review Request: CVector - ANSI C API for Dynamic Arrays
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=545046 Chen Lei supercyp...@gmail.com changed: What|Removed |Added CC||supercyp...@gmail.com --- Comment #17 from Chen Lei supercyp...@gmail.com 2010-08-25 02:21:35 EDT --- sed -e 's,-dynamic,-rdynamic,g' -i Makefile is useless at all, (grep dynamic Makefile) find nothing. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 545046] Review Request: CVector - ANSI C API for Dynamic Arrays
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=545046 Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed: What|Removed |Added Flag||needinfo?(jussi.leht...@iki ||.fi) --- Comment #13 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2010-08-24 11:48:14 EDT --- So Jussi, would you make it clear if you are going to sponsor Matsuura-san or if you are asking him to do some other things before sponsoring him? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 545046] Review Request: CVector - ANSI C API for Dynamic Arrays
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=545046 Jussi Lehtola jussi.leht...@iki.fi changed: What|Removed |Added Status|NEW |ASSIGNED Blocks|177841(FE-NEEDSPONSOR) | AssignedTo|nob...@fedoraproject.org|jussi.leht...@iki.fi Flag|needinfo?(jussi.leht...@iki |fedora-review? |.fi)| --- Comment #14 from Jussi Lehtola jussi.leht...@iki.fi 2010-08-24 11:55:30 EDT --- Oh, I'm terribly sorry - this one totally slipped under my radar. Review coming right up. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 545046] Review Request: CVector - ANSI C API for Dynamic Arrays
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=545046
--- Comment #15 from Jussi Lehtola jussi.leht...@iki.fi 2010-08-24 13:28:35
EDT ---
rpmlint output:
CVector-devel.x86_64: W: no-documentation
4 packages and 0 specfiles checked; 0 errors, 1 warnings.
This is OK.
- Normally it's better to use a patch than sed, since sed can fail silently.
YMMV.
- There are test binaries that you could run in the build.
%check
make tests
although make install already seems to run them.
MUST: The package does not yet exist in Fedora. The Review Request is not a
duplicate. OK
MUST: The spec file for the package is legible and macros are used
consistently. OK
MUST: The package must be named according to the Package Naming Guidelines. ~OK
- You should keep the original naming scheme. This means
Release:4.20090805%{?dist}
should be
Release:4.5Aug09%{?dist}
and so on. So the first number is the release number, the second is the date
from the tarball.
MUST: The spec file name must match the base package %{name}. OK
MUST: The package must be licensed with a Fedora approved license and meet the
Licensing Guidelines. OK
MUST: The License field in the package spec file must match the actual license.
OK
MUST: The sources used to build the package must match the upstream source, as
provided in the spec URL. OK
00d4399e74d9c7bc6e0bb72804990c3e CVector-1.0.3-5Aug09.tar.gz
00d4399e74d9c7bc6e0bb72804990c3e ../SOURCES/CVector-1.0.3-5Aug09.tar.gz
MUST: The package MUST successfully compile and build into binary rpms. OK
MUST: The spec file MUST handle locales properly. N/A
MUST: Optflags are used and time stamps preserved. OK
MUST: Packages containing shared library files must call ldconfig. OK
MUST: A package must own all directories that it creates or require the package
that owns the directory. OK
MUST: Files only listed once in %files listings. OK
MUST: Debuginfo package is complete. OK
MUST: Permissions on files must be set properly. OK
MUST: Large documentation files must go in a -doc subpackage. N/A
MUST: All relevant items are included in %doc. Items in %doc do not affect
runtime of application. OK
MUST: Header files must be in a -devel package. OK
MUST: Static libraries must be in a -static package. N/A (removed)
MUST: If a package contains library files with a suffix then library files
ending in .so must go in a -devel package. OK
MUST: In the vast majority of cases, devel packages must require the base
package using a fully versioned dependency. OK
MUST: Packages does not contain any .la libtool archives. OK
MUST: Desktop files are installed properly. N/A
MUST: No file conflicts with other packages and no general names. OK
SHOULD: %{?dist} tag is used in release. OK
SHOULD: If the package does not include license text(s) as separate files from
upstream, the packager should query upstream to include it. OK
SHOULD: The package builds in mock. OK
EPEL: Clean section exists. OK
EPEL: Buildroot cleaned before install. OK
EPEL: Packages containing pkgconfig(.pc) files must 'Requires: pkgconfig'. N/A
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 545046] Review Request: CVector - ANSI C API for Dynamic Arrays
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=545046 Jussi Lehtola jussi.leht...@iki.fi changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #16 from Jussi Lehtola jussi.leht...@iki.fi 2010-08-24 13:36:08 EDT --- Please change the versioning scheme before import to git. This package has been APPROVED. Also, I've just sponsored you in FAS. You seem to have a pretty good hang on packaging. We are short on reviewers, so you're most warmly invited to take on formal reviews of your own. Just pay a bit more attention to the exact words of the licenses, and the honoring of build flags. Reviewing is a one-time task, whereas maintaining a package is an ongoing process. Reviewing is the way of least resistance to improve Fedora to your liking! If you have any problems feel free to ask me for help. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 545046] Review Request: CVector - ANSI C API for Dynamic Arrays
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=545046 --- Comment #12 from Takanori MATSUURA t.mat...@gmail.com 2010-06-02 01:37:41 EDT --- (In reply to comment #11) I'll try bug 597307 for informal review. and bug 598511. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 545046] Review Request: CVector - ANSI C API for Dynamic Arrays
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=545046 --- Comment #9 from Takanori MATSUURA t.mat...@gmail.com 2010-05-06 00:14:07 EDT --- (In reply to comment #8) I've selected bug 587315 as informal review. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 545046] Review Request: CVector - ANSI C API for Dynamic Arrays
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=545046 --- Comment #10 from Jussi Lehtola jussi.leht...@iki.fi 2010-05-06 01:33:07 EDT --- Well, that took you a while.. Anyway, you will still need to do another submission and another informal review. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
