[Bug 694479] Review Request: pcl - Library for point cloud processing
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=694479 --- Comment #16 from Fedora Update System upda...@fedoraproject.org 2012-01-05 15:57:00 EST --- pcl-1.3.1-2.fc15 has been pushed to the Fedora 15 stable repository. -- 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 694479] Review Request: pcl - Library for point cloud processing
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=694479 --- Comment #16 from Fedora Update System upda...@fedoraproject.org 2012-01-05 15:57:00 EST --- pcl-1.3.1-2.fc15 has been pushed to the Fedora 15 stable repository. -- 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 694479] Review Request: pcl - Library for point cloud processing
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=694479 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||pcl-1.3.1-2.fc16 Resolution||ERRATA Last Closed||2012-01-05 16:00:40 --- Comment #17 from Fedora Update System upda...@fedoraproject.org 2012-01-05 16:00:40 EST --- pcl-1.3.1-2.fc16 has been pushed to the Fedora 16 stable repository. -- 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 694479] Review Request: pcl - Library for point cloud processing
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=694479 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||pcl-1.3.1-2.fc16 Resolution||ERRATA Last Closed||2012-01-05 16:00:40 --- Comment #17 from Fedora Update System upda...@fedoraproject.org 2012-01-05 16:00:40 EST --- pcl-1.3.1-2.fc16 has been pushed to the Fedora 16 stable repository. -- 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 694479] Review Request: pcl - Library for point cloud processing
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=694479 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #15 from Fedora Update System upda...@fedoraproject.org 2011-12-27 17:56:55 EST --- pcl-1.3.1-2.fc16 has been pushed to the Fedora 16 testing repository. -- 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 694479] Review Request: pcl - Library for point cloud processing
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=694479 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #15 from Fedora Update System upda...@fedoraproject.org 2011-12-27 17:56:55 EST --- pcl-1.3.1-2.fc16 has been pushed to the Fedora 16 testing repository. -- 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 694479] Review Request: pcl - Library for point cloud processing
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=694479 --- Comment #13 from Fedora Update System upda...@fedoraproject.org 2011-12-27 07:58:28 EST --- pcl-1.3.1-2.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/pcl-1.3.1-2.fc15 -- 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 694479] Review Request: pcl - Library for point cloud processing
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=694479 --- Comment #14 from Fedora Update System upda...@fedoraproject.org 2011-12-27 07:59:14 EST --- pcl-1.3.1-2.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/pcl-1.3.1-2.fc16 -- 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 694479] Review Request: pcl - Library for point cloud processing
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=694479 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 694479] Review Request: pcl - Library for point cloud processing
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=694479 --- Comment #13 from Fedora Update System upda...@fedoraproject.org 2011-12-27 07:58:28 EST --- pcl-1.3.1-2.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/pcl-1.3.1-2.fc15 -- 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 694479] Review Request: pcl - Library for point cloud processing
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=694479 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 694479] Review Request: pcl - Library for point cloud processing
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=694479 --- Comment #14 from Fedora Update System upda...@fedoraproject.org 2011-12-27 07:59:14 EST --- pcl-1.3.1-2.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/pcl-1.3.1-2.fc16 -- 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 694479] Review Request: pcl - Library for point cloud processing
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=694479 --- Comment #12 from Jon Ciesla limburg...@gmail.com 2011-12-26 18:02:21 EST --- 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 694479] Review Request: pcl - Library for point cloud processing
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=694479 --- Comment #12 from Jon Ciesla limburg...@gmail.com 2011-12-26 18:02:21 EST --- 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 694479] Review Request: pcl - Library for point cloud processing
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=694479 Tim Niemueller t...@niemueller.de changed: What|Removed |Added Flag||fedora-cvs? --- Comment #11 from Tim Niemueller t...@niemueller.de 2011-12-25 05:56:56 EST --- New Package SCM Request === Package Name: pcl Short Description: Library for point cloud processing Owners: timn Branches: f15 f16 el6 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 694479] Review Request: pcl - Library for point cloud processing
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=694479 Tim Niemueller t...@niemueller.de changed: What|Removed |Added Flag||fedora-cvs? --- Comment #11 from Tim Niemueller t...@niemueller.de 2011-12-25 05:56:56 EST --- New Package SCM Request === Package Name: pcl Short Description: Library for point cloud processing Owners: timn Branches: f15 f16 el6 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 694479] Review Request: pcl - Library for point cloud processing
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=694479 --- Comment #9 from Tim Niemueller t...@niemueller.de 2011-12-24 04:52:56 EST --- Thanks for the review Rich. I have uploaded the new version 1.3.1-2 which fixes the issues, especially the docs issue. Thanks for pointing this out, I just missed it. Additionally I have sorted out the OpenNI problems and published a new OpenNI (and PrimeSense) update. It is tagged for the next four days into the buildroot of f15 and f16. I have done scratch builds on both, f15 and f16. The OpenNI updates will land in f15/f16 updates before pcl, so I think this is no longer a reason not to complete this review. I added a short comment why there are no .desktop files in the spec file (I did this after the scratch builds hence it is only in the spec, not in the srpms). New SRPM is at http://fedorapeople.org/~timn/robotics/pcl-1.3.1-2.fc15.src.rpm, spec file changed in place. Scratch builds: f15: https://koji.fedoraproject.org/koji/taskinfo?taskID=3604183 f16: https://koji.fedoraproject.org/koji/taskinfo?taskID=3604194 I hope these takes the pcl package to an acceptable state. Merry Christmas/happy holidays. -- 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 694479] Review Request: pcl - Library for point cloud processing
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=694479 --- Comment #9 from Tim Niemueller t...@niemueller.de 2011-12-24 04:52:56 EST --- Thanks for the review Rich. I have uploaded the new version 1.3.1-2 which fixes the issues, especially the docs issue. Thanks for pointing this out, I just missed it. Additionally I have sorted out the OpenNI problems and published a new OpenNI (and PrimeSense) update. It is tagged for the next four days into the buildroot of f15 and f16. I have done scratch builds on both, f15 and f16. The OpenNI updates will land in f15/f16 updates before pcl, so I think this is no longer a reason not to complete this review. I added a short comment why there are no .desktop files in the spec file (I did this after the scratch builds hence it is only in the spec, not in the srpms). New SRPM is at http://fedorapeople.org/~timn/robotics/pcl-1.3.1-2.fc15.src.rpm, spec file changed in place. Scratch builds: f15: https://koji.fedoraproject.org/koji/taskinfo?taskID=3604183 f16: https://koji.fedoraproject.org/koji/taskinfo?taskID=3604194 I hope these takes the pcl package to an acceptable state. Merry Christmas/happy holidays. -- 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 694479] Review Request: pcl - Library for point cloud processing
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=694479 Rich Mattes richmat...@gmail.com changed: What|Removed |Added AssignedTo|nob...@fedoraproject.org|richmat...@gmail.com -- 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 694479] Review Request: pcl - Library for point cloud processing
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=694479 Rich Mattes richmat...@gmail.com changed: What|Removed |Added AssignedTo|nob...@fedoraproject.org|richmat...@gmail.com -- 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 694479] Review Request: pcl - Library for point cloud processing
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=694479 Rich Mattes richmat...@gmail.com changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #10 from Rich Mattes richmat...@gmail.com 2011-12-24 17:17:27 EST --- I've reviewed and tested your changes. The documentation packaging issue and specfile comments are taken care of, and the pending openni update+buildroot override take care of the primary arch build requirement. With these changes, this package is APPROVED. Happy holidays to you too! -- 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 694479] Review Request: pcl - Library for point cloud processing
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=694479 Rich Mattes richmat...@gmail.com changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #10 from Rich Mattes richmat...@gmail.com 2011-12-24 17:17:27 EST --- I've reviewed and tested your changes. The documentation packaging issue and specfile comments are taken care of, and the pending openni update+buildroot override take care of the primary arch build requirement. With these changes, this package is APPROVED. Happy holidays to you too! -- 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 694479] Review Request: pcl - Library for point cloud processing
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=694479 Rich Mattes richmat...@gmail.com changed: What|Removed |Added 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 694479] Review Request: pcl - Library for point cloud processing
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=694479
Rich Mattes richmat...@gmail.com changed:
What|Removed |Added
CC||package-review@lists.fedora
||project.org
Component|0x |Package Review
AssignedTo|richmat...@gmail.com|nob...@fedoraproject.org
--- Comment #8 from Rich Mattes richmat...@gmail.com 2011-12-17 01:06:06 EST
---
I was able to build this package locally on f16 without any errors, so I can
(finally) proceed with a review!
[+]: rpmlint must be run on the source rpm and all binary rpms the build
produces. The output should be posted in the review.
$ rpmlint ../RPMS/x86_64/pcl-* ../SRPMS/pcl*
pcl.x86_64: E: zero-length
/usr/share/doc/pcl-1.3.1/tutorials/sources/pcl_visualizer/CMakeLists.txt.simple
pcl.x86_64: E: zero-length
/usr/share/doc/pcl-1.3.1/tutorials/sources/pcl_visualizer/pcl_visualizer_simple.cpp
pcl-devel.x86_64: W: no-documentation
pcl-tools.x86_64: W: no-documentation
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_passthrough_filter
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_openni_mls_smoothing
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_extract_feature
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_mesh2pcd
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_openni_change_viewer
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_surfel_smoothing_test
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_fpfh_estimation
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_openni_uniform_sampling
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_openni_grabber_example
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_pcd_grabber_viewer
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_icp2d
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_openni_planar_segmentation
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_gp3_surface
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_spin_estimation
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_openni_io
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_mls_smoothing
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_openni_passthrough
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_elch
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_registration_visualizer
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_icp
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_cluster_extraction
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_pcd_convert_NaN_nan
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_nn_classification_example
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_convert_pcd_ascii_binary
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_pcd_viewer
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_openni_ii_normal_estimation
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_octree_viewer
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_ppf_object_recognition
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_transform_point_cloud
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_openni_viewer
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_plane_projection
pcl-tools.x86_64: W: no-manual-page-for-binary
pcl_statistical_multiscale_interest_region_extraction_example
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_voxel_grid
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_openni_voxel_grid
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_openni_feature_persistence
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_mesh_sampling
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_openni_grab_frame
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_add_gaussian_noise
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_openni_3d_convex_hull
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_compute_cloud_error
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_boundary_estimation
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_openni_stream_compression
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_oni_viewer
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_vfh_estimation
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_openni_image
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_normal_estimation
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_pyramid_surface_matching
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_openni_boundary_estimation
pcl-tools.x86_64: W: no-manual-page-for-binary
pcl_multiscale_feature_persistence_example
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_openni_3d_concave_hull
pcl.src:126: W: macro-in-comment %{_libdir}
pcl.src:126: W: macro-in-comment %{_libdir}
pcl.src:127: W: macro-in-comment %{_libdir}
pcl.src:130: W: macro-in-comment %{_libdir}
pcl.src:134: W:
[Bug 694479] Review Request: pcl - Library for point cloud processing
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=694479
Rich Mattes richmat...@gmail.com changed:
What|Removed |Added
CC||package-review@lists.fedora
||project.org
Component|0x |Package Review
AssignedTo|richmat...@gmail.com|nob...@fedoraproject.org
--- Comment #8 from Rich Mattes richmat...@gmail.com 2011-12-17 01:06:06 EST
---
I was able to build this package locally on f16 without any errors, so I can
(finally) proceed with a review!
[+]: rpmlint must be run on the source rpm and all binary rpms the build
produces. The output should be posted in the review.
$ rpmlint ../RPMS/x86_64/pcl-* ../SRPMS/pcl*
pcl.x86_64: E: zero-length
/usr/share/doc/pcl-1.3.1/tutorials/sources/pcl_visualizer/CMakeLists.txt.simple
pcl.x86_64: E: zero-length
/usr/share/doc/pcl-1.3.1/tutorials/sources/pcl_visualizer/pcl_visualizer_simple.cpp
pcl-devel.x86_64: W: no-documentation
pcl-tools.x86_64: W: no-documentation
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_passthrough_filter
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_openni_mls_smoothing
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_extract_feature
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_mesh2pcd
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_openni_change_viewer
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_surfel_smoothing_test
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_fpfh_estimation
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_openni_uniform_sampling
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_openni_grabber_example
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_pcd_grabber_viewer
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_icp2d
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_openni_planar_segmentation
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_gp3_surface
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_spin_estimation
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_openni_io
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_mls_smoothing
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_openni_passthrough
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_elch
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_registration_visualizer
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_icp
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_cluster_extraction
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_pcd_convert_NaN_nan
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_nn_classification_example
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_convert_pcd_ascii_binary
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_pcd_viewer
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_openni_ii_normal_estimation
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_octree_viewer
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_ppf_object_recognition
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_transform_point_cloud
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_openni_viewer
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_plane_projection
pcl-tools.x86_64: W: no-manual-page-for-binary
pcl_statistical_multiscale_interest_region_extraction_example
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_voxel_grid
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_openni_voxel_grid
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_openni_feature_persistence
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_mesh_sampling
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_openni_grab_frame
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_add_gaussian_noise
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_openni_3d_convex_hull
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_compute_cloud_error
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_boundary_estimation
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_openni_stream_compression
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_oni_viewer
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_vfh_estimation
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_openni_image
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_normal_estimation
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_pyramid_surface_matching
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_openni_boundary_estimation
pcl-tools.x86_64: W: no-manual-page-for-binary
pcl_multiscale_feature_persistence_example
pcl-tools.x86_64: W: no-manual-page-for-binary pcl_openni_3d_concave_hull
pcl.src:126: W: macro-in-comment %{_libdir}
pcl.src:126: W: macro-in-comment %{_libdir}
pcl.src:127: W: macro-in-comment %{_libdir}
pcl.src:130: W: macro-in-comment %{_libdir}
pcl.src:134: W:
[Bug 694479] Review Request: pcl - Library for point cloud processing
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=694479 Rich Mattes richmat...@gmail.com changed: What|Removed |Added 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 694479] Review Request: pcl - Library for point cloud processing
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=694479 Tim Niemueller t...@niemueller.de changed: What|Removed |Added CC||dw...@infradead.org Component|Package Review |0x Flag|fedora-review? | --- Comment #7 from Tim Niemueller t...@niemueller.de 2011-12-04 20:00:12 EST --- I have upgraded the package to PCL 1.3.1. It also addresses the rpath problem. Besides that it removes the version suffix from pkg-config files and removes the openni-dev requirement from pcl_io pkg-config file (not available on Fedora). It also fixes the Requires of the devel sub-package. I have also prefixed all installed binaries with pcl_ (just like the libraries). There is an increasing number with ambiguous names like add_gaussian_noise and hence this is needed. I'll also propose this upstream. The new SRPM is at http://fedorapeople.org/~timn/robotics/pcl-1.3.1-1.fc15.src.rpm, the spec file changed in place. Scratch build does not work atm because it needs openni-devel 1.3.2.1 from updates-testing, but this has a problem on its own that I need to fix first. -- 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 694479] Review Request: pcl - Library for point cloud processing
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=694479 Gabor Papp gabor.p...@gmail.com changed: What|Removed |Added CC||gabor.p...@gmail.com --- Comment #6 from Gabor Papp gabor.p...@gmail.com 2011-11-13 02:19:26 EST --- Adding the -DCMAKE_SKIP_RPATH=ON switch to the %cmake line solved the rpath issue for me. -- 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 694479] Review Request: pcl - Library for point cloud processing
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=694479
Rich Mattes richmat...@gmail.com changed:
What|Removed |Added
Flag||fedora-review?
--- Comment #5 from Rich Mattes richmat...@gmail.com 2011-11-02 20:16:09 EDT
---
I tried building the package in mock (fedora-16-x86_64) and got some internal
compiler errors each time I ran the build. I removed the smp_mflags and tried
again, and managed to get the package to build.
After scaling that hurdle, I'm getting some rpath errors in f16 from rpmlint:
$ rpmlint *.rpm
pcl.src:118: W: macro-in-comment %{_libdir}
pcl.src:118: W: macro-in-comment %{_libdir}
pcl.src:119: W: macro-in-comment %{_libdir}
pcl.x86_64: E: binary-or-shlib-defines-rpath
/usr/lib64/libpcl_features.so.1.2.0 ['/usr/lib64', '/usr/lib64/vtk-5.6']
pcl.x86_64: E: binary-or-shlib-defines-rpath
/usr/lib64/libpcl_sample_consensus.so.1.2.0 ['/usr/lib64',
'/usr/lib64/vtk-5.6']
pcl.x86_64: E: binary-or-shlib-defines-rpath /usr/lib64/libpcl_common.so.1.2.0
['/usr/lib64', '/usr/lib64/vtk-5.6']
pcl.x86_64: E: binary-or-shlib-defines-rpath
/usr/lib64/libpcl_range_image.so.1.2.0 ['/usr/lib64', '/usr/lib64/vtk-5.6']
pcl.x86_64: E: binary-or-shlib-defines-rpath /usr/bin/nn_classification_example
['/usr/lib64', '/usr/lib64/vtk-5.6']
pcl.x86_64: E: binary-or-shlib-defines-rpath
/usr/lib64/libpcl_registration.so.1.2.0 ['/usr/lib64', '/usr/lib64/vtk-5.6']
pcl.x86_64: E: binary-or-shlib-defines-rpath
/usr/lib64/libpcl_visualization.so.1.2.0 ['/usr/lib64', '/usr/lib64/vtk-5.6',
'/usr/lib64/mysql']
pcl.x86_64: E: binary-or-shlib-defines-rpath /usr/lib64/libpcl_kdtree.so.1.2.0
['/usr/lib64', '/usr/lib64/vtk-5.6']
pcl.x86_64: E: binary-or-shlib-defines-rpath /usr/lib64/libpcl_io.so.1.2.0
['/usr/lib64', '/usr/lib64/vtk-5.6']
pcl.x86_64: E: binary-or-shlib-defines-rpath
/usr/lib64/libpcl_range_image_border_extractor.so.1.2.0 ['/usr/lib64',
'/usr/lib64/vtk-5.6']
pcl.x86_64: E: binary-or-shlib-defines-rpath /usr/bin/convert_pcd_ascii_binary
['/usr/lib64', '/usr/lib64/vtk-5.6']
pcl.x86_64: E: binary-or-shlib-defines-rpath
/usr/lib64/libpcl_segmentation.so.1.2.0 ['/usr/lib64', '/usr/lib64/vtk-5.6']
pcl.x86_64: E: binary-or-shlib-defines-rpath /usr/lib64/libpcl_surface.so.1.2.0
['/usr/lib64', '/usr/lib64/vtk-5.6']
pcl.x86_64: E: binary-or-shlib-defines-rpath /usr/lib64/libpcl_filters.so.1.2.0
['/usr/lib64', '/usr/lib64/vtk-5.6']
pcl.x86_64: E: binary-or-shlib-defines-rpath /usr/bin/pcd_convert_NaN_nan
['/usr/lib64', '/usr/lib64/vtk-5.6']
pcl.x86_64: E: binary-or-shlib-defines-rpath
/usr/lib64/libpcl_keypoints.so.1.2.0 ['/usr/lib64', '/usr/lib64/vtk-5.6']
pcl.x86_64: E: binary-or-shlib-defines-rpath /usr/lib64/libpcl_octree.so.1.2.0
['/usr/lib64', '/usr/lib64/vtk-5.6']
pcl.x86_64: E: zero-length
/usr/share/doc/pcl-1.2.0/tutorials/sources/pcl_visualizer/pcl_visualizer_simple.cpp
pcl.x86_64: E: zero-length
/usr/share/doc/pcl-1.2.0/tutorials/sources/pcl_visualizer/CMakeLists.txt.simple
pcl.x86_64: W: no-manual-page-for-binary nn_classification_example
pcl.x86_64: W: no-manual-page-for-binary convert_pcd_ascii_binary
pcl.x86_64: W: no-manual-page-for-binary pcd_convert_NaN_nan
pcl-devel.x86_64: W: no-documentation
pcl-doc.noarch: E: zero-length
/usr/share/doc/pcl-1.2.0/tutorials/sources/pcl_visualizer/pcl_visualizer_simple.cpp
pcl-doc.noarch: E: zero-length
/usr/share/doc/pcl-1.2.0/tutorials/sources/pcl_visualizer/CMakeLists.txt.simple
pcl-viewers.x86_64: E: binary-or-shlib-defines-rpath
/usr/bin/openni_viewer_simple ['/usr/lib64', '/usr/lib64/vtk-5.6',
'/usr/lib64/mysql']
pcl-viewers.x86_64: E: binary-or-shlib-defines-rpath /usr/bin/openni_image
['/usr/lib64', '/usr/lib64/vtk-5.6', '/usr/lib64/mysql']
pcl-viewers.x86_64: E: binary-or-shlib-defines-rpath
/usr/bin/openni_passthrough ['/usr/lib64', '/usr/lib64/vtk-5.6',
'/usr/lib64/mysql']
pcl-viewers.x86_64: E: binary-or-shlib-defines-rpath /usr/bin/pcd_viewer_simple
['/usr/lib64', '/usr/lib64/vtk-5.6', '/usr/lib64/mysql']
pcl-viewers.x86_64: E: binary-or-shlib-defines-rpath /usr/bin/openni_grab_frame
['/usr/lib64', '/usr/lib64/vtk-5.6']
pcl-viewers.x86_64: E: binary-or-shlib-defines-rpath
/usr/bin/openni_3d_convex_hull ['/usr/lib64', '/usr/lib64/vtk-5.6',
'/usr/lib64/mysql']
pcl-viewers.x86_64: E: binary-or-shlib-defines-rpath
/usr/bin/openni_stream_compression ['/usr/lib64', '/usr/lib64/vtk-5.6',
'/usr/lib64/mysql']
pcl-viewers.x86_64: E: binary-or-shlib-defines-rpath
/usr/bin/openni_ii_normal_estimation ['/usr/lib64', '/usr/lib64/vtk-5.6',
'/usr/lib64/mysql']
pcl-viewers.x86_64: E: binary-or-shlib-defines-rpath /usr/bin/openni_voxel_grid
['/usr/lib64', '/usr/lib64/vtk-5.6', '/usr/lib64/mysql']
pcl-viewers.x86_64: E: binary-or-shlib-defines-rpath
/usr/bin/pcd_grabber_viewer ['/usr/lib64',
[Bug 694479] Review Request: pcl - Library for point cloud processing
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=694479 --- Comment #4 from Tim Niemueller t...@niemueller.de 2011-10-27 14:59:05 EDT --- Hi Rich. Thanks for reviewing this. The OpenNI version currently in F-16 is not up2date. I thought I had pushed all updates, but obviously I only built them. Going to push them. You can get the updated packages from http://koji.fedoraproject.org/koji/buildinfo?buildID=269182. Please remember to set the review flag to ?. -- 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 694479] Review Request: pcl - Library for point cloud processing
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=694479 Rich Mattes richmat...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||richmat...@gmail.com AssignedTo|nob...@fedoraproject.org|richmat...@gmail.com Bug 694479 depends on bug 690025, which changed state. Bug 690025 Summary: Review Request: cminpack - Solver for nonlinear equations and nonlinear least squares problems https://bugzilla.redhat.com/show_bug.cgi?id=690025 What|Old Value |New Value Status|ASSIGNED|MODIFIED Status|MODIFIED|ON_QA Resolution||ERRATA Status|ON_QA |CLOSED --- Comment #3 from Rich Mattes richmat...@gmail.com 2011-10-26 22:38:18 EDT --- I'll review this for you. I'm having some issues building this package for f16. I did a scratch build for f16 on koji, you can take a look at the build logs at http://koji.fedoraproject.org/koji/taskinfo?taskID=3464287 -- 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 694479] Review Request: pcl - Library for point cloud processing
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=694479 --- Comment #2 from Tim Niemueller t...@niemueller.de 2011-10-13 11:49:22 EDT --- I have updated the package to the latest stable release 1.2.0. Spec changed in place, the new SRPM is at http://fedorapeople.org/~timn/robotics/pcl-1.2.0-1.fc15.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 694479] Review Request: pcl - Library for point cloud processing
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=694479 Bug 694479 depends on bug 672395, which changed state. Bug 672395 Summary: Review Request: eigen3 - A lightweight C++ template library for vector and matrix math https://bugzilla.redhat.com/show_bug.cgi?id=672395 What|Old Value |New Value Status|ASSIGNED|MODIFIED Status|MODIFIED|ON_QA Resolution||ERRATA Status|ON_QA |CLOSED --- Comment #1 from Tim Niemueller t...@niemueller.de 2011-09-22 14:02:06 EDT --- I have updated the package to the latest stable release 1.1.1. Spec changed in place, the new SRPM is at http://fedorapeople.org/~timn/robotics/pcl-1.1.1-1.fc15.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 694479] Review Request: pcl - Library for point cloud processing
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=694479 Tim Niemueller t...@niemueller.de changed: What|Removed |Added Depends on||690025, 672395 -- 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 694479] Review Request: pcl - Library for point cloud processing
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=694479 Tim Niemueller t...@niemueller.de changed: What|Removed |Added Depends on|690025 | -- 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 694479] Review Request: pcl - Library for point cloud processing
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=694479 Tim Niemueller t...@niemueller.de changed: What|Removed |Added Depends on||690025 -- 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
