[Bug 577152] Review Request: apiextractor - Library headers parser to extract API information
https://bugzilla.redhat.com/show_bug.cgi?id=577152 Jiri Kastner changed: What|Removed |Added CC||jkast...@redhat.com --- Comment #23 from Jiri Kastner --- Package Change Request == Package Name: apiextractor New Branches: epel7 Owners: than jreznik ltinkl kkofler geertj rdieter -- You are receiving this mail because: You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 577152] Review Request: apiextractor - Library headers parser to extract API information
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=577152 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||apiextractor-0.6.0-4.fc13 Resolution||ERRATA -- 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 577152] Review Request: apiextractor - Library headers parser to extract API information
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=577152 --- Comment #20 from Fedora Update System 2010-06-29 11:38:11 EDT --- apiextractor-0.6.0-4.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 577152] Review Request: apiextractor - Library headers parser to extract API information
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=577152 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|ON_QA --- Comment #19 from Fedora Update System 2010-06-21 17:28:11 EDT --- apiextractor-0.6.0-4.fc13 has been pushed to the Fedora 13 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 apiextractor'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/apiextractor-0.6.0-4.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 577152] Review Request: apiextractor - Library headers parser to extract API information
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=577152 --- Comment #18 from Fedora Update System 2010-06-21 00:53:56 EDT --- apiextractor-0.6.0-4.fc13 has been submitted as an update for Fedora 13. http://admin.fedoraproject.org/updates/apiextractor-0.6.0-4.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 577152] Review Request: apiextractor - Library headers parser to extract API information
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=577152 --- Comment #17 from Kevin Fenzi 2010-06-20 21:55:14 EDT --- CVS done (by process-cvs-requests.py). -- 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 577152] Review Request: apiextractor - Library headers parser to extract API information
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=577152
Chen Lei changed:
What|Removed |Added
Status|NEW |ASSIGNED
--- Comment #16 from Chen Lei 2010-06-20 07:46:07 EDT
---
(In reply to comment #14)
> (In reply to comment #13)
> > Group for apoextractor should be Group: System
> > Environment/Libraries,
> > Development/Libraries is for -devel only.
> I'll change that, thanks.
> By the way, do you know of any Fedora guidelines for choosing RPM groups? I
> guess there are none, since RPM groups are pretty much obsoleted by comps.
NO guideline as I known, but comps can not replace rpm group because most
libraries are not registerd in comps.
> > I noticed apiextractor rpm for opensuse patched the place of cmake modules
> > from
> > %{_libdir}/cmake/ApiExtractor-%{version}/ to %{_libdir}/cmake/ApiExtractor,
> > I'
> > not sure if it's needed.
> No idea why they would want to do that. CMake searches both directories for
> module configuration files, so there is really no advantage using one over
> other.
> Do you have a link to that opensuse package?
No difference between the two places for cmake modules I think, but we still
need a convention to treat cmake modules.
See
http://www.pyside.org/downloads/
http://download.opensuse.org/repositories/home:/cgoncalves:/pyside:/
--
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 577152] Review Request: apiextractor - Library headers parser to extract API information
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=577152 --- Comment #15 from Kalev Lember 2010-06-20 07:01:29 EDT --- Updated owners list. New Package CVS Request === Package Name: apiextractor Short Description: Library headers parser to extract API information Owners: kalev rdieter kkofler than ltinkl Branches: F-13 InitialCC: -- 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 577152] Review Request: apiextractor - Library headers parser to extract API information
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=577152
--- Comment #14 from Kalev Lember 2010-06-20 06:26:38 EDT
---
(In reply to comment #13)
> Group for apoextractor should be Group: System Environment/Libraries,
> Development/Libraries is for -devel only.
I'll change that, thanks.
By the way, do you know of any Fedora guidelines for choosing RPM groups? I
guess there are none, since RPM groups are pretty much obsoleted by comps.
> I noticed apiextractor rpm for opensuse patched the place of cmake modules
> from
> %{_libdir}/cmake/ApiExtractor-%{version}/ to %{_libdir}/cmake/ApiExtractor, I'
> not sure if it's needed.
No idea why they would want to do that. CMake searches both directories for
module configuration files, so there is really no advantage using one over
other.
Do you have a link to that opensuse package?
--
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 577152] Review Request: apiextractor - Library headers parser to extract API information
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=577152
--- Comment #13 from Chen Lei 2010-06-19 22:15:39 EDT
---
Some suggestions:
Group for apoextractor should be Group: System Environment/Libraries,
Development/Libraries is for -devel only.
Also, some rpm warnings can be ignored, e.g. spelling-error, it's Okay.
I noticed apiextractor rpm for opensuse patched the place of cmake modules from
%{_libdir}/cmake/ApiExtractor-%{version}/ to %{_libdir}/cmake/ApiExtractor, I'
not sure if it's needed.
--
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 577152] Review Request: apiextractor - Library headers parser to extract API information
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=577152 Kalev Lember changed: What|Removed |Added Flag||fedora-cvs? --- Comment #12 from Kalev Lember 2010-06-19 20:12:13 EDT --- Thank you for the review, Jonathan! New Package CVS Request === Package Name: apiextractor Short Description: Library headers parser to extract API information Owners: kalev Branches: F-13 InitialCC: -- 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 577152] Review Request: apiextractor - Library headers parser to extract API information
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=577152 Jonathan Steffan changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #11 from Jonathan Steffan 2010-06-19 19:44:04 EDT --- 4 packages and 0 specfiles checked; 0 errors, 0 warnings. Looks good. Passes all MUSTS and all SHOULDS. -- 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 577152] Review Request: apiextractor - Library headers parser to extract API information
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=577152 Jonathan Steffan changed: What|Removed |Added AssignedTo|nob...@fedoraproject.org|jonathanstef...@gmail.com Flag||fedora-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 577152] Review Request: apiextractor - Library headers parser to extract API information
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=577152 --- Comment #10 from Kalev Lember 2010-06-19 19:33:54 EDT --- * Sun Jun 20 2010 Kalev Lember - 0.6.0-3 - Removed .buildinfo file from documentation directory Spec URL: http://kalev.fedorapeople.org/apiextractor.spec SRPM URL: http://kalev.fedorapeople.org/apiextractor-0.6.0-3.fc14.src.rpm -- 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 577152] Review Request: apiextractor - Library headers parser to extract API information
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=577152 --- Comment #9 from Jonathan Steffan 2010-06-19 19:25:39 EDT --- apiextractor-devel.x86_64: W: hidden-file-or-dir /usr/share/doc/apiextractor-devel-0.6.0/html/.buildinfo 4 packages and 0 specfiles checked; 0 errors, 1 warnings. Please remove this directory. -- 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 577152] Review Request: apiextractor - Library headers parser to extract API information
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=577152 Kalev Lember changed: What|Removed |Added Flag|needinfo?(ka...@smartlink.e | |e) | --- Comment #8 from Kalev Lember 2010-06-19 19:18:47 EDT --- * Sun Jun 20 2010 Kalev Lember - 0.6.0-2 - Build html documentation and ship it in -devel subpackage - typo fix in description Spec URL: http://kalev.fedorapeople.org/apiextractor.spec SRPM URL: http://kalev.fedorapeople.org/apiextractor-0.6.0-2.fc14.src.rpm Scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=2259495 -- 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 577152] Review Request: apiextractor - Library headers parser to extract API information
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=577152 Jonathan Steffan changed: What|Removed |Added CC||jonathanstef...@gmail.com Flag||needinfo?(ka...@smartlink.e ||e) --- Comment #7 from Jonathan Steffan 2010-06-19 18:54:49 EDT --- apiextractor.src: W: spelling-error %description -l en_US typesystem -> type system, type-system, typesetter apiextractor.x86_64: W: spelling-error %description -l en_US typesystem -> type system, type-system, typesetter apiextractor-devel.x86_64: W: no-documentation 4 packages and 0 specfiles checked; 0 errors, 3 warnings. MUST: rpmlint must be run on every package. The output should be posted in the review. Please update spec to include the spelling fixes. The no-documentation warning is something that should be addressed. Maybe ship the "doc" folder in the -devel subpackage as %doc ? -- 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 577152] Review Request: apiextractor - Library headers parser to extract API information
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=577152 Kalev Lember changed: What|Removed |Added Blocks||604695 -- 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 577152] Review Request: apiextractor - Library headers parser to extract API information
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=577152 --- Comment #6 from Kalev Lember 2010-06-15 22:45:01 EDT --- Updated to 0.6.0. Spec URL: http://kalev.fedorapeople.org/apiextractor.spec SRPM URL: http://kalev.fedorapeople.org/apiextractor-0.6.0-1.fc14.src.rpm Scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=2252774 -- 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 577152] Review Request: apiextractor - Library headers parser to extract API information
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=577152 --- Comment #5 from Chen Lei 2010-03-28 08:05:56 EDT --- (In reply to comment #2) > (In reply to comment #1) > > Would you be interested in packaging the whole pyside set? > Yes, that's my plan. Would you be interested in reviewing the whole set? I'd like to review pyside set several weeks later after I finish package reviews that were assigned to me already. -- 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 577152] Review Request: apiextractor - Library headers parser to extract API information
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=577152
--- Comment #4 from Kalev Lember 2010-03-27 08:03:08 EDT
---
(In reply to comment #3)
> Thanks for clarification. currently fedora doesn't have a packaging guideline
> according to cmake modules(even a draft), so I'm a little confusion about
> this,
> historically cmake package didn't own {_libdir}/cmake and some packages store
> cmake modules on %{_datadir}/cmake/%{name} (e.g. boost 1.41).
Right, cmake searches in both %{_datadir}/cmake/ and %{_libdir}/cmake/ (and
some more places), it's just that you can't put arch-specific configuration in
%{_datadir}.
Just for reference I looked up what packages install in these directories.
Packages which install cmake modules in %{_datadir}/cmake/:
shared-desktop-ontologies-devel
boost-devel
Packages which install cmake modules in %{_libdir}/cmake/:
kdepimlibs-devel
kdevplatform-devel
kdebase-workspace-devel
akonadi-devel
--
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 577152] Review Request: apiextractor - Library headers parser to extract API information
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=577152
--- Comment #3 from Chen Lei 2010-03-27 07:41:34 EDT ---
(In reply to comment #2)
> (In reply to comment #1)
> > Would you be interested in packaging the whole pyside set?
> Yes, that's my plan. Would you be interested in reviewing the whole set?
> > BTW, %{_libdir}/cmake/ApiExtractor-%{version}/ seems not the right place for
> > cmake modules.
> This is the place where cmake-aware projects put their arch-specific
> configuration [1] (the files may contain references to %{_libdir}, for
> example). The directory %{_libdir}/cmake/ is roughly similar to
> %{_libdir}/pkgconfig/ -- both are placed under %{_libdir} for the same reason.
> Several KDE packages install files in this directory and Fedora cmake package
> recently started owning the directory [2].
> [1] http://www.cmake.org/Wiki/CMake_2.6_Notes#Packages
> [2]
> http://cvs.fedoraproject.org/viewvc/rpms/cmake/devel/cmake.spec?r1=1.76&r2=1.77
>
Thanks for clarification. currently fedora doesn't have a packaging guideline
according to cmake modules(even a draft), so I'm a little confusion about this,
historically cmake package didn't own {_libdir}/cmake and some packages store
cmake modules on %{_datadir}/cmake/%{name} (e.g. boost 1.41).
--
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 577152] Review Request: apiextractor - Library headers parser to extract API information
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=577152
--- Comment #2 from Kalev Lember 2010-03-26 15:36:27 EDT
---
(In reply to comment #1)
> Would you be interested in packaging the whole pyside set?
Yes, that's my plan. Would you be interested in reviewing the whole set?
> BTW, %{_libdir}/cmake/ApiExtractor-%{version}/ seems not the right place for
> cmake modules.
This is the place where cmake-aware projects put their arch-specific
configuration [1] (the files may contain references to %{_libdir}, for
example). The directory %{_libdir}/cmake/ is roughly similar to
%{_libdir}/pkgconfig/ -- both are placed under %{_libdir} for the same reason.
Several KDE packages install files in this directory and Fedora cmake package
recently started owning the directory [2].
[1] http://www.cmake.org/Wiki/CMake_2.6_Notes#Packages
[2]
http://cvs.fedoraproject.org/viewvc/rpms/cmake/devel/cmake.spec?r1=1.76&r2=1.77
--
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 577152] Review Request: apiextractor - Library headers parser to extract API information
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=577152
Chen Lei changed:
What|Removed |Added
CC||supercy...@163.com
--- Comment #1 from Chen Lei 2010-03-26 13:29:55 EDT ---
Hi Kalev,
Would you be interested in packaging the whole pyside set?
BTW, %{_libdir}/cmake/ApiExtractor-%{version}/ seems not the right place for
cmake modules.
--
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
