[Bug 1501007] New: Review Request: golang-github-hashicorp-go-plugin - Golang plugin system over RPC
https://bugzilla.redhat.com/show_bug.cgi?id=1501007 Bug ID: 1501007 Summary: Review Request: golang-github-hashicorp-go-plugin - Golang plugin system over RPC Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: e...@logic.net QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://fedorapeople.org/~logic/golang-github-hashicorp-go-plugin/golang-github-hashicorp-go-plugin.spec SRPM URL: https://fedorapeople.org/~logic/golang-github-hashicorp-go-plugin/golang-github-hashicorp-go-plugin-0-0.1.20171028git3e6d191.fc28.src.rpm Description: Golang plugin system over RPC Fedora Account System Username: logic Koji scratch build: https://koji.fedoraproject.org/koji/taskinfo?taskID=22394070 The localhost tests are incredibly flaky on our builders (works ~75% of the time, usually failing on the arm or aarch builders, and local mock builds have been fine; I'm guessing it's load-related?), so I ended up having to disable them. :( -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1501002] New: Review Request: golang-github-pquerna-otp - TOTP Library for Go (Golang)
https://bugzilla.redhat.com/show_bug.cgi?id=1501002 Bug ID: 1501002 Summary: Review Request: golang-github-pquerna-otp - TOTP Library for Go (Golang) Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: e...@logic.net QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://fedorapeople.org/~logic/golang-github-pquerna-otp/golang-github-pquerna-otp-1.0.0-1.fc28.src.rpm SRPM URL: https://fedorapeople.org/~logic/golang-github-pquerna-otp/golang-github-pquerna-otp.spec Description: TOTP Library for Go (Golang) Fedora Account System Username: logic Koji scratch build: https://koji.fedoraproject.org/koji/taskinfo?taskID=22393770 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1500691] Review Request: perl-lib-relative - Add paths relative to the current file to @INC
https://bugzilla.redhat.com/show_bug.cgi?id=1500691 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #5 from Fedora Update System --- perl-lib-relative-0.002-1.fc27 has been pushed to the Fedora 27 testing repository. If problems still persist, please make note of it in this bug report. See https://fedoraproject.org/wiki/QA:Updates_Testing for instructions on how to install test updates. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2017-7f132d40d7 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1480966] Review Request: golang-github-sergi-go-diff - Diff, match and patch text in Go
https://bugzilla.redhat.com/show_bug.cgi?id=1480966 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #10 from Fedora Update System --- golang-github-sergi-go-diff-0-0.2.gitfeef008.fc27 has been pushed to the Fedora 27 testing repository. If problems still persist, please make note of it in this bug report. See https://fedoraproject.org/wiki/QA:Updates_Testing for instructions on how to install test updates. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2017-27e921ccc5 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1426972] Review Request: hugo - A Fast and Flexible Static Site Generator built with love in GoLang
https://bugzilla.redhat.com/show_bug.cgi?id=1426972 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #46 from Fedora Update System --- hugo-0.27.1-2.fc27 has been pushed to the Fedora 27 testing repository. If problems still persist, please make note of it in this bug report. See https://fedoraproject.org/wiki/QA:Updates_Testing for instructions on how to install test updates. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2017-7d38868ee1 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1485458] Review Request: orangefs - parallel network file system ( formerly PVFS2)
https://bugzilla.redhat.com/show_bug.cgi?id=1485458
--- Comment #21 from Martin Brandenburg ---
(In reply to Jonathan Dieter from comment #20)
> Thanks so much, this is looking better. I'm enclosing the rpmlint output
> because there are a number of problems that need to be investigated.
>
> You can ignore the shared-lib-calls-exit warnings, the
> script-without-shebang warning, the no-manual-page-for-binary warning
> (assuming you're not planning to write some new man pages), the
> useless-provides for the debuginfo subpackage, and the invalid-url warning.
>
> Please check the rest of them and either fix them or explain why they're
> there.
>
> Rpmlint
> ---
> Checking: orangefs-2.9.6-0.4.20171004svn.fc28.x86_64.rpm
> orangefs-debuginfo-2.9.6-0.4.20171004svn.fc28.x86_64.rpm
> orangefs-devel-2.9.6-0.4.20171004svn.fc28.x86_64.rpm
> orangefs-server-2.9.6-0.4.20171004svn.fc28.x86_64.rpm
> orangefs-fuse-2.9.6-0.4.20171004svn.fc28.x86_64.rpm
> orangefs-2.9.6-0.4.20171004svn.fc28.src.rpm
> orangefs.x86_64: W: shared-lib-calls-exit /usr/lib64/libofs.so.2.9.6
> exit@GLIBC_2.2.5
> orangefs.x86_64: W: shared-lib-calls-exit /usr/lib64/liborangefs.so.2.9.6
> exit@GLIBC_2.2.5
> orangefs.x86_64: W: shared-lib-calls-exit /usr/lib64/libpvfs2.so.2.9.6
> exit@GLIBC_2.2.5
> orangefs.x86_64: E: script-without-shebang
> /usr/lib64/liborangefsposix.so.2.9.6
> orangefs.x86_64: W: manual-page-warning
> /usr/share/man/man1/pvfs2-drop-caches.1.gz 13: warning: numeric expression
> expected (got `f')
> orangefs.x86_64: W: manual-page-warning
> /usr/share/man/man1/pvfs2-fs-dump.1.gz 15: warning: numeric expression
> expected (got `m')
These are fixed.
> orangefs.x86_64: W: no-manual-page-for-binary ofs_cp
> orangefs.x86_64: W: no-manual-page-for-binary ofs_graphite_driver
> orangefs.x86_64: W: no-manual-page-for-binary ofs_rm
> orangefs.x86_64: W: no-manual-page-for-binary ofs_setdirhint
I wrote a whole bunch of manpages before submitting for the first time,
so I will write more.
> orangefs-debuginfo.x86_64: E: useless-provides debuginfo(build-id)
> orangefs-devel.x86_64: W: no-dependency-on orangefs/orangefs-libs/liborangefs
Fixed.
> orangefs-devel.x86_64: W: only-non-binary-in-usr-lib
I'm guessing this is the linker script...
> orangefs-devel.x86_64: W: no-documentation
> orangefs-devel.x86_64: W: no-manual-page-for-binary pvfs2-config
> orangefs-server.x86_64: W: conffile-without-noreplace-flag
> /etc/orangefs/orangefs.conf
> orangefs-server.x86_64: W: conffile-without-noreplace-flag /etc/pvfs2tab
Fixed.
> orangefs-fuse.x86_64: W: only-non-binary-in-usr-lib
> orangefs-fuse.x86_64: W: no-documentation
> orangefs-fuse.x86_64: W: no-manual-page-for-binary pvfs2fuse
> orangefs.src:26: W: unversioned-explicit-provides libpvfs2.so()(64bit)
> orangefs.src:28: W: unversioned-explicit-provides libofs.so()(64bit)
> orangefs.src:28: W: unversioned-explicit-provides liborangefs.so()(64bit)
> orangefs.src:28: W: unversioned-explicit-provides libpvfs2.so()(64bit)
> orangefs.src:39: W: macro-in-comment %{version}
This will go away when the real release is made, but until then I don't
want to lose the real URL.
> orangefs.src:289: W: macro-in-%changelog %doc
I should write 'percent-doc' I guess?
> orangefs.src: E: specfile-error warning: bogus date in %changelog: Mon Oct
> 11 2017 Martin Brandenburg -
> 2.9.6-0.4.20171004svn
I guess it is Wednesday.
> 6 packages and 0 specfiles checked; 3 errors, 24 warnings.
>
>
>
>
> Rpmlint (debuginfo)
> ---
> Checking: orangefs-debuginfo-2.9.6-0.4.20171004svn.fc28.x86_64.rpm
> orangefs-server-debuginfo-2.9.6-0.4.20171004svn.fc28.x86_64.rpm
> orangefs-fuse-debuginfo-2.9.6-0.4.20171004svn.fc28.x86_64.rpm
> orangefs-debuginfo.x86_64: E: useless-provides debuginfo(build-id)
> orangefs-server-debuginfo.x86_64: E: useless-provides debuginfo(build-id)
> 3 packages and 0 specfiles checked; 2 errors, 0 warnings.
>
>
>
>
>
> Rpmlint (installed packages)
>
> sh: /usr/bin/python: No such file or directory
I don't know where this comes from?
> orangefs-fuse.x86_64: W: invalid-url URL: http://www.orangefs.org/ error [Errno -2] Name or service not known>
I assume the network is blocked wherever this is run?
> orangefs-fuse.x86_64: W: only-non-binary-in-usr-lib
> orangefs-fuse.x86_64: W: no-documentation
> orangefs-fuse.x86_64: W: no-manual-page-for-binary pvfs2fuse
> orangefs-fuse-debuginfo.x86_64: W: invalid-url URL: http://www.orangefs.org/
>
> orangefs-debuginfo.x86_64: W: invalid-url URL: http://www.orangefs.org/
>
> orangefs-debuginfo.x86_64: E: useless-provides debuginfo(build-id)
> orangefs-devel.x86_64: W: no-dependency-on orangefs/orangefs-libs/liborangefs
> orangefs-devel.x86_64: W: invalid-url URL: http://www.orangefs.org/ error [Errno -2] Name or service not known>
> orangefs-devel.x86_64: W: only-non-binary-in-usr-lib
> orangefs-devel.x86_64: W: no-doc
[Bug 1389311] Review Request: python-events - Bringing the elegance of C# EventHanlder to Python
https://bugzilla.redhat.com/show_bug.cgi?id=1389311 --- Comment #5 from Gwyn Ciesla --- (fedrepo-req-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/python-events -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1497349] Review Request: alignak - Alignak python Monitoring tool
https://bugzilla.redhat.com/show_bug.cgi?id=1497349 --- Comment #10 from Gwyn Ciesla --- (fedrepo-req-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/alignak -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1389311] Review Request: python-events - Bringing the elegance of C# EventHanlder to Python
https://bugzilla.redhat.com/show_bug.cgi?id=1389311 David Hannequin changed: What|Removed |Added Summary|Review Request: |Review Request: |python-Events - Bringing|python-events - Bringing |the elegance of C# |the elegance of C# |EventHanlder to Python |EventHanlder to Python -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1500958] Review Request: python-kiwi-gtk - Framework for Python GUI applications
https://bugzilla.redhat.com/show_bug.cgi?id=1500958 Haïkel Guémar changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|nob...@fedoraproject.org|ngomp...@gmail.com -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1500958] New: Review Request: python-kiwi-gtk - Framework for Python GUI applications
https://bugzilla.redhat.com/show_bug.cgi?id=1500958 Bug ID: 1500958 Summary: Review Request: python-kiwi-gtk - Framework for Python GUI applications Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: karlthe...@gmail.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://hguemar.fedorapeople.org/reviews/python-kiwi-gtk.spec SRPM URL: https://hguemar.fedorapeople.org/reviews/python-kiwi-gtk-1.11.1-1.fc28.src.rpm Description: Framework for Python GUI applications Fedora Account System Username: hguemar Rename due to name conflict, as kiwi has been renamed to kiwi-gtk upstream hence that review. Here is discussion on devel list https://lists.fedoraproject.org/archives/list/de...@lists.fedoraproject.org/thread/OSJHZJCEB4MJR4SQI7QCYBJJCXFH4WVX/?sort=date No Obsoletes as a different python-kiwi package will be introduced, any suggestion to a clean upgrade path is welcome -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1494960] Review Request: nodejs-csv - CSV parser with simple api, full of options and tested against large datasets
https://bugzilla.redhat.com/show_bug.cgi?id=1494960 Jared Smith changed: What|Removed |Added Status|MODIFIED|CLOSED Resolution|--- |RAWHIDE Last Closed||2017-10-11 13:37:47 --- Comment #4 from Jared Smith --- In rawhide, closing bug -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1442567] Review Request: nodejs-base-option - Adds a few options methods to base, like option, enable and disable
https://bugzilla.redhat.com/show_bug.cgi?id=1442567 Bug 1442567 depends on bug 1442531, which changed state. Bug 1442531 Summary: Review Request: nodejs-base-plugins - Adds smart plugin support to your base application https://bugzilla.redhat.com/show_bug.cgi?id=1442531 What|Removed |Added Status|MODIFIED|CLOSED Resolution|--- |RAWHIDE -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1442531] Review Request: nodejs-base-plugins - Adds smart plugin support to your base application
https://bugzilla.redhat.com/show_bug.cgi?id=1442531 Jared Smith changed: What|Removed |Added Status|MODIFIED|CLOSED Resolution|--- |RAWHIDE Last Closed||2017-10-11 13:30:02 --- Comment #4 from Jared Smith --- In rawhide, closing bug -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1494960] Review Request: nodejs-csv - CSV parser with simple api, full of options and tested against large datasets
https://bugzilla.redhat.com/show_bug.cgi?id=1494960 Bug 1494960 depends on bug 1494957, which changed state. Bug 1494957 Summary: Review Request: nodejs-stream-transform - Object transformations implementing the Node.js 'stream.Transform' API https://bugzilla.redhat.com/show_bug.cgi?id=1494957 What|Removed |Added Status|MODIFIED|CLOSED Resolution|--- |RAWHIDE -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1494957] Review Request: nodejs-stream-transform - Object transformations implementing the Node.js 'stream.Transform' API
https://bugzilla.redhat.com/show_bug.cgi?id=1494957 Jared Smith changed: What|Removed |Added Status|MODIFIED|CLOSED Resolution|--- |RAWHIDE Last Closed||2017-10-11 13:29:28 --- Comment #3 from Jared Smith --- In rawhide, closing bug -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1364172] Review Request: nodejs-braces - Fastest brace expansion for node.js
https://bugzilla.redhat.com/show_bug.cgi?id=1364172 Bug 1364172 depends on bug 1442424, which changed state. Bug 1442424 Summary: Review Request: nodejs-snapdragon - Easy-to-use plugin system for parsers and compilers https://bugzilla.redhat.com/show_bug.cgi?id=1442424 What|Removed |Added Status|MODIFIED|CLOSED Resolution|--- |RAWHIDE -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1442424] Review Request: nodejs-snapdragon - Easy-to-use plugin system for parsers and compilers
https://bugzilla.redhat.com/show_bug.cgi?id=1442424 Jared Smith changed: What|Removed |Added Status|MODIFIED|CLOSED Resolution|--- |RAWHIDE Last Closed|2017-09-26 13:43:50 |2017-10-11 13:29:14 --- Comment #7 from Jared Smith --- In rawhide, closing bug -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1495380] Review Request: nodejs-vlq - Generate, and decode, base64 VLQ mappings for source maps and other uses
https://bugzilla.redhat.com/show_bug.cgi?id=1495380 Bug 1495380 depends on bug 1442207, which changed state. Bug 1442207 Summary: Review Request: nodejs-rollup - Next-generation ES6 module bundler https://bugzilla.redhat.com/show_bug.cgi?id=1442207 What|Removed |Added Status|MODIFIED|CLOSED Resolution|--- |RAWHIDE -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1442207] Review Request: nodejs-rollup - Next-generation ES6 module bundler
https://bugzilla.redhat.com/show_bug.cgi?id=1442207 Jared Smith changed: What|Removed |Added Status|MODIFIED|CLOSED Resolution|--- |RAWHIDE Last Closed||2017-10-11 13:26:35 --- Comment #4 from Jared Smith --- In rawhide, closing bug -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1442528] Review Request: nodejs-is-registered - Optionally prevents a plugin from being registered more than once
https://bugzilla.redhat.com/show_bug.cgi?id=1442528 Jared Smith changed: What|Removed |Added Status|MODIFIED|CLOSED Resolution|--- |RAWHIDE Last Closed||2017-10-11 13:21:15 --- Comment #5 from Jared Smith --- In rawhide, closing bug -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1442531] Review Request: nodejs-base-plugins - Adds smart plugin support to your base application
https://bugzilla.redhat.com/show_bug.cgi?id=1442531 Bug 1442531 depends on bug 1442528, which changed state. Bug 1442528 Summary: Review Request: nodejs-is-registered - Optionally prevents a plugin from being registered more than once https://bugzilla.redhat.com/show_bug.cgi?id=1442528 What|Removed |Added Status|MODIFIED|CLOSED Resolution|--- |RAWHIDE -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1497349] Review Request: alignak - Alignak python Monitoring tool
https://bugzilla.redhat.com/show_bug.cgi?id=1497349 Robert-André Mauchin changed: What|Removed |Added Status|NEW |POST Assignee|nob...@fedoraproject.org|zebo...@gmail.com Flags||fedora-review+ --- Comment #9 from Robert-André Mauchin --- Ok, package accepted. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1485458] Review Request: orangefs - parallel network file system ( formerly PVFS2)
https://bugzilla.redhat.com/show_bug.cgi?id=1485458
--- Comment #20 from Jonathan Dieter ---
Thanks so much, this is looking better. I'm enclosing the rpmlint output
because there are a number of problems that need to be investigated.
You can ignore the shared-lib-calls-exit warnings, the script-without-shebang
warning, the no-manual-page-for-binary warning (assuming you're not planning to
write some new man pages), the useless-provides for the debuginfo subpackage,
and the invalid-url warning.
Please check the rest of them and either fix them or explain why they're there.
Rpmlint
---
Checking: orangefs-2.9.6-0.4.20171004svn.fc28.x86_64.rpm
orangefs-debuginfo-2.9.6-0.4.20171004svn.fc28.x86_64.rpm
orangefs-devel-2.9.6-0.4.20171004svn.fc28.x86_64.rpm
orangefs-server-2.9.6-0.4.20171004svn.fc28.x86_64.rpm
orangefs-fuse-2.9.6-0.4.20171004svn.fc28.x86_64.rpm
orangefs-2.9.6-0.4.20171004svn.fc28.src.rpm
orangefs.x86_64: W: shared-lib-calls-exit /usr/lib64/libofs.so.2.9.6
exit@GLIBC_2.2.5
orangefs.x86_64: W: shared-lib-calls-exit /usr/lib64/liborangefs.so.2.9.6
exit@GLIBC_2.2.5
orangefs.x86_64: W: shared-lib-calls-exit /usr/lib64/libpvfs2.so.2.9.6
exit@GLIBC_2.2.5
orangefs.x86_64: E: script-without-shebang /usr/lib64/liborangefsposix.so.2.9.6
orangefs.x86_64: W: manual-page-warning
/usr/share/man/man1/pvfs2-drop-caches.1.gz 13: warning: numeric expression
expected (got `f')
orangefs.x86_64: W: manual-page-warning /usr/share/man/man1/pvfs2-fs-dump.1.gz
15: warning: numeric expression expected (got `m')
orangefs.x86_64: W: no-manual-page-for-binary ofs_cp
orangefs.x86_64: W: no-manual-page-for-binary ofs_graphite_driver
orangefs.x86_64: W: no-manual-page-for-binary ofs_rm
orangefs.x86_64: W: no-manual-page-for-binary ofs_setdirhint
orangefs-debuginfo.x86_64: E: useless-provides debuginfo(build-id)
orangefs-devel.x86_64: W: no-dependency-on orangefs/orangefs-libs/liborangefs
orangefs-devel.x86_64: W: only-non-binary-in-usr-lib
orangefs-devel.x86_64: W: no-documentation
orangefs-devel.x86_64: W: no-manual-page-for-binary pvfs2-config
orangefs-server.x86_64: W: conffile-without-noreplace-flag
/etc/orangefs/orangefs.conf
orangefs-server.x86_64: W: conffile-without-noreplace-flag /etc/pvfs2tab
orangefs-fuse.x86_64: W: only-non-binary-in-usr-lib
orangefs-fuse.x86_64: W: no-documentation
orangefs-fuse.x86_64: W: no-manual-page-for-binary pvfs2fuse
orangefs.src:26: W: unversioned-explicit-provides libpvfs2.so()(64bit)
orangefs.src:28: W: unversioned-explicit-provides libofs.so()(64bit)
orangefs.src:28: W: unversioned-explicit-provides liborangefs.so()(64bit)
orangefs.src:28: W: unversioned-explicit-provides libpvfs2.so()(64bit)
orangefs.src:39: W: macro-in-comment %{version}
orangefs.src:289: W: macro-in-%changelog %doc
orangefs.src: E: specfile-error warning: bogus date in %changelog: Mon Oct 11
2017 Martin Brandenburg - 2.9.6-0.4.20171004svn
6 packages and 0 specfiles checked; 3 errors, 24 warnings.
Rpmlint (debuginfo)
---
Checking: orangefs-debuginfo-2.9.6-0.4.20171004svn.fc28.x86_64.rpm
orangefs-server-debuginfo-2.9.6-0.4.20171004svn.fc28.x86_64.rpm
orangefs-fuse-debuginfo-2.9.6-0.4.20171004svn.fc28.x86_64.rpm
orangefs-debuginfo.x86_64: E: useless-provides debuginfo(build-id)
orangefs-server-debuginfo.x86_64: E: useless-provides debuginfo(build-id)
3 packages and 0 specfiles checked; 2 errors, 0 warnings.
Rpmlint (installed packages)
sh: /usr/bin/python: No such file or directory
orangefs-fuse.x86_64: W: invalid-url URL: http://www.orangefs.org/
orangefs-fuse.x86_64: W: only-non-binary-in-usr-lib
orangefs-fuse.x86_64: W: no-documentation
orangefs-fuse.x86_64: W: no-manual-page-for-binary pvfs2fuse
orangefs-fuse-debuginfo.x86_64: W: invalid-url URL: http://www.orangefs.org/
orangefs-debuginfo.x86_64: W: invalid-url URL: http://www.orangefs.org/
orangefs-debuginfo.x86_64: E: useless-provides debuginfo(build-id)
orangefs-devel.x86_64: W: no-dependency-on orangefs/orangefs-libs/liborangefs
orangefs-devel.x86_64: W: invalid-url URL: http://www.orangefs.org/
orangefs-devel.x86_64: W: only-non-binary-in-usr-lib
orangefs-devel.x86_64: W: no-documentation
orangefs-devel.x86_64: W: no-manual-page-for-binary pvfs2-config
orangefs-server.x86_64: W: invalid-url URL: http://www.orangefs.org/
orangefs-server.x86_64: W: conffile-without-noreplace-flag
/etc/orangefs/orangefs.conf
orangefs-server.x86_64: W: conffile-without-noreplace-flag /etc/pvfs2tab
orangefs.x86_64: W: invalid-url URL: http://www.orangefs.org/
orangefs.x86_64: W: unused-direct-shlib-dependency /usr/lib64/libofs.so.2.9.6
/lib64/librt.so.1
orangefs.x86_64: W: unused-direct-shlib-dependency /usr/lib64/libofs.so.2.9.6
/lib64/libm.so.6
orangefs.x86_64: W: unused-direct-shlib-dependency /usr/lib64/libofs.so.2.9.6
/lib64/liblmdb.so.0.0.0
orangefs.x86_64: W: unused-direct-shlib-dependency /usr/lib64/libofs.so.2.9.6
/lib64/libssl.so.1.
[Bug 1500524] Review Request: gpxsee - GPS log file viewer and analyzer
https://bugzilla.redhat.com/show_bug.cgi?id=1500524 --- Comment #9 from Fedora Update System --- gpxsee-4.15-2.fc26 has been submitted as an update to Fedora 26. https://bodhi.fedoraproject.org/updates/FEDORA-2017-da608139c9 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1500524] Review Request: gpxsee - GPS log file viewer and analyzer
https://bugzilla.redhat.com/show_bug.cgi?id=1500524 --- Comment #10 from Fedora Update System --- gpxsee-4.15-2.fc25 has been submitted as an update to Fedora 25. https://bodhi.fedoraproject.org/updates/FEDORA-2017-3e4bc1aa15 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1500524] Review Request: gpxsee - GPS log file viewer and analyzer
https://bugzilla.redhat.com/show_bug.cgi?id=1500524 Fedora Update System changed: What|Removed |Added Status|POST|MODIFIED -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1500524] Review Request: gpxsee - GPS log file viewer and analyzer
https://bugzilla.redhat.com/show_bug.cgi?id=1500524 --- Comment #8 from Fedora Update System --- gpxsee-4.15-2.fc27 has been submitted as an update to Fedora 27. https://bodhi.fedoraproject.org/updates/FEDORA-2017-c330d8e5ef -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1497349] Review Request: alignak - Alignak python Monitoring tool
https://bugzilla.redhat.com/show_bug.cgi?id=1497349 --- Comment #8 from David Hannequin --- Hi, New version with last fix : Spec URL: https://hvad.fedorapeople.org/fedora/alignak/alignak.spec SRPM URL: https://hvad.fedorapeople.org/fedora/alignak/alignak-1.0.0-1.fc26.src.rpm I didn't know "spectool -g". Best regard -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1485458] Review Request: orangefs - parallel network file system ( formerly PVFS2)
https://bugzilla.redhat.com/show_bug.cgi?id=1485458 --- Comment #19 from Martin Brandenburg --- I have removed Karma, re-enabled aarch64 without the usrint, and dropped the start/stop scripts. I have also tested an EPEL build, which worked with no changes. Git: https://github.com/omnibond/orangefs-fedora Koji: https://koji.fedoraproject.org/koji/taskinfo?taskID=22385794 Spec: http://dev.orangefs.org/2017/marbran/1011/orangefs.spec SRPM: http://dev.orangefs.org/2017/marbran/1011/orangefs-2.9.6-0.4.20171004svn.fc26.src.rpm -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1474033] Review Request: ucx - Communication library implementing high-performance messaging
https://bugzilla.redhat.com/show_bug.cgi?id=1474033 --- Comment #20 from Michal Schmidt --- (In reply to Andrey Maslennikov from comment #18) > Supporting several specs is more complicated especially taking into account > that the diff will be in 1-2 lines only. > So I'd like to keep it this way, the line is required. It was proved by some > fails in our build env :) Of course if the build test was performed on a system with RPM < 4.6, then without explicit BuildRoot and Group tags it would fail. It's just that usually in Fedora spec files we do not care about such systems. I'm not going to block the review for this, so keep the tags if you want them. > Will add a comment. In fact, static libs are simply required to develop with > UCX. Do you mean it is impossible to develop with dynamic libraries? Where is the problem? I am looking at the contents of the ucx and ucx-devel binary packages. The contents of the /usr/share/ucx directory look like documentation to me. Would you consider moving those files under /usr/share/doc/ucx? (That would be an upstream change, not just a packaging change.) -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1500305] Review Request: python-os-service-types - Python library for consuming OpenStack sevice-types-authority data
https://bugzilla.redhat.com/show_bug.cgi?id=1500305 --- Comment #11 from Fedora Update System --- python-os-service-types-1.1.0-2.fc27 has been submitted as an update to Fedora 27. https://bodhi.fedoraproject.org/updates/FEDORA-2017-a7563de607 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1500305] Review Request: python-os-service-types - Python library for consuming OpenStack sevice-types-authority data
https://bugzilla.redhat.com/show_bug.cgi?id=1500305 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1500574] Review Request: golang-github-alecthomas-chroma - A general purpose syntax highlighter in pure Go
https://bugzilla.redhat.com/show_bug.cgi?id=1500574 Athos Ribeiro changed: What|Removed |Added Depends On||1500831 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1500831 [Bug 1500831] Review Request: douceur - A simple CSS parser and inliner in Go -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1500831] Review Request: douceur - A simple CSS parser and inliner in Go
https://bugzilla.redhat.com/show_bug.cgi?id=1500831 Athos Ribeiro changed: What|Removed |Added Blocks||1500574 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1500574 [Bug 1500574] Review Request: golang-github-alecthomas-chroma - A general purpose syntax highlighter in pure Go -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1500574] Review Request: golang-github-alecthomas-chroma - A general purpose syntax highlighter in pure Go
https://bugzilla.redhat.com/show_bug.cgi?id=1500574 --- Comment #2 from Athos Ribeiro --- Thanks for the input, Robert! This added a few new dependencies in the dependency chain -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1500821] Review Request: golang-github-gorilla-css - A CSS3 tokenizer
https://bugzilla.redhat.com/show_bug.cgi?id=1500821 Athos Ribeiro changed: What|Removed |Added Blocks||1500831 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1500831 [Bug 1500831] Review Request: douceur - A simple CSS parser and inliner in Go -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1500828] Review Request: golang-github-PuerkitoBio-goquery - A little like that j-thing, only in Go
https://bugzilla.redhat.com/show_bug.cgi?id=1500828 Athos Ribeiro changed: What|Removed |Added Blocks||1500831 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1500831 [Bug 1500831] Review Request: douceur - A simple CSS parser and inliner in Go -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1500831] New: Review Request: douceur - A simple CSS parser and inliner in Go
https://bugzilla.redhat.com/show_bug.cgi?id=1500831 Bug ID: 1500831 Summary: Review Request: douceur - A simple CSS parser and inliner in Go Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: athoscribe...@gmail.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://athoscr.fedorapeople.org/packaging/douceur.spec SRPM URL: https://athoscr.fedorapeople.org/packaging/douceur-0.2.0-1.fc26.src.rpm Description: A simple CSS parser and inliner in Go Fedora Account System Username: athoscr -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1500831] Review Request: douceur - A simple CSS parser and inliner in Go
https://bugzilla.redhat.com/show_bug.cgi?id=1500831 Athos Ribeiro changed: What|Removed |Added Depends On||1500821, 1500828 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1500821 [Bug 1500821] Review Request: golang-github-gorilla-css - A CSS3 tokenizer https://bugzilla.redhat.com/show_bug.cgi?id=1500828 [Bug 1500828] Review Request: golang-github-PuerkitoBio-goquery - A little like that j-thing, only in Go -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1500826] Review Request: golang-github-andybalholm-cascadia - CSS selector library in Go
https://bugzilla.redhat.com/show_bug.cgi?id=1500826 Athos Ribeiro changed: What|Removed |Added Blocks||1500828 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1500828 [Bug 1500828] Review Request: golang-github-PuerkitoBio-goquery - A little like that j-thing, only in Go -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1500828] Review Request: golang-github-PuerkitoBio-goquery - A little like that j-thing, only in Go
https://bugzilla.redhat.com/show_bug.cgi?id=1500828 Athos Ribeiro changed: What|Removed |Added Depends On||1500826 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1500826 [Bug 1500826] Review Request: golang-github-andybalholm-cascadia - CSS selector library in Go -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1500828] New: Review Request: golang-github-PuerkitoBio-goquery - A little like that j-thing, only in Go
https://bugzilla.redhat.com/show_bug.cgi?id=1500828 Bug ID: 1500828 Summary: Review Request: golang-github-PuerkitoBio-goquery - A little like that j-thing, only in Go Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: athoscribe...@gmail.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://athoscr.fedorapeople.org/packaging/golang-github-PuerkitoBio-goquery.spec SRPM URL: https://athoscr.fedorapeople.org/packaging/golang-github-PuerkitoBio-goquery-1.1.0-1.fc26.src.rpm Description: A little like that j-thing, only in Go Fedora Account System Username: athoscr -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1473314] Review Request: rclone - rsync for cloud storage
https://bugzilla.redhat.com/show_bug.cgi?id=1473314 Bug 1473314 depends on bug 1475832, which changed state. Bug 1475832 Summary: Review Request: golang-github-ncw-dropbox-sdk-go-unofficial - An unofficial Go SDK for integrating with the Dropbox API v2 https://bugzilla.redhat.com/show_bug.cgi?id=1475832 What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1473314] Review Request: rclone - rsync for cloud storage
https://bugzilla.redhat.com/show_bug.cgi?id=1473314 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA Last Closed||2017-10-11 10:47:23 --- Comment #17 from Fedora Update System --- rclone-1.38-1.fc27 has been pushed to the Fedora 27 stable repository. If problems still persist, please make note of it in this bug report. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1475832] Review Request: golang-github-ncw-dropbox-sdk-go-unofficial - An unofficial Go SDK for integrating with the Dropbox API v2
https://bugzilla.redhat.com/show_bug.cgi?id=1475832 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA Last Closed||2017-10-11 10:47:17 --- Comment #10 from Fedora Update System --- golang-github-ncw-dropbox-sdk-go-unofficial-0-0.1.20170530git5d9f46f.fc27 has been pushed to the Fedora 27 stable repository. If problems still persist, please make note of it in this bug report. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1417631] Review Request: mingw-libftdi - MinGW library to program and control the FTDI USB controller
https://bugzilla.redhat.com/show_bug.cgi?id=1417631 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA Last Closed||2017-10-11 10:47:11 --- Comment #12 from Fedora Update System --- mingw-libftdi-1.3-3.fc27 has been pushed to the Fedora 27 stable repository. If problems still persist, please make note of it in this bug report. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1417631] Review Request: mingw-libftdi - MinGW library to program and control the FTDI USB controller
https://bugzilla.redhat.com/show_bug.cgi?id=1417631 Bug 1417631 depends on bug 1417628, which changed state. Bug 1417628 Summary: Review Request: mingw-libconfuse - MinGW configuration file parser library https://bugzilla.redhat.com/show_bug.cgi?id=1417628 What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1417628] Review Request: mingw-libconfuse - MinGW configuration file parser library
https://bugzilla.redhat.com/show_bug.cgi?id=1417628 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA Last Closed||2017-10-11 10:47:05 --- Comment #14 from Fedora Update System --- mingw-libconfuse-3.2.1-3.fc27 has been pushed to the Fedora 27 stable repository. If problems still persist, please make note of it in this bug report. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1431423] Review Request: python-agate-excel - Adds read support for Excel files to agate
https://bugzilla.redhat.com/show_bug.cgi?id=1431423 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA Last Closed||2017-10-11 10:47:02 --- Comment #13 from Fedora Update System --- python-agate-excel-0.2.1-2.fc27 has been pushed to the Fedora 27 stable repository. If problems still persist, please make note of it in this bug report. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1431422] Review Request: python-agate-sql - Adds SQL read/ write support to agate
https://bugzilla.redhat.com/show_bug.cgi?id=1431422 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA Last Closed||2017-10-11 10:46:58 --- Comment #16 from Fedora Update System --- python-agate-sql-0.5.2-2.fc27 has been pushed to the Fedora 27 stable repository. If problems still persist, please make note of it in this bug report. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1431427] Review Request: python-agate-dbf - Adds read support for DBF files to agate
https://bugzilla.redhat.com/show_bug.cgi?id=1431427 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA Last Closed||2017-10-11 10:46:53 --- Comment #14 from Fedora Update System --- python-agate-dbf-0.2.0-2.fc27 has been pushed to the Fedora 27 stable repository. If problems still persist, please make note of it in this bug report. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1443523] Review Request: nodejs-progress - Flexible ascii progress bar
https://bugzilla.redhat.com/show_bug.cgi?id=1443523 Jared Smith changed: What|Removed |Added Status|MODIFIED|CLOSED Resolution|--- |RAWHIDE Last Closed||2017-10-11 10:44:46 --- Comment #5 from Jared Smith --- In rawhide, closing bug -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1500826] New: Review Request: golang-github-andybalholm-cascadia - CSS selector library in Go
https://bugzilla.redhat.com/show_bug.cgi?id=1500826 Bug ID: 1500826 Summary: Review Request: golang-github-andybalholm-cascadia - CSS selector library in Go Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: athoscribe...@gmail.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://athoscr.fedorapeople.org/packaging/golang-github-andybalholm-cascadia.spec SRPM URL: https://athoscr.fedorapeople.org/packaging/golang-github-andybalholm-cascadia-0-0.1.20161224git349dd02.fc26.src.rpm Koji build: https://koji.fedoraproject.org/koji/taskinfo?taskID=22384034 Description: CSS selector library in Go Fedora Account System Username: athoscr -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1500821] New: Review Request: golang-github-gorilla-css - A CSS3 tokenizer
https://bugzilla.redhat.com/show_bug.cgi?id=1500821 Bug ID: 1500821 Summary: Review Request: golang-github-gorilla-css - A CSS3 tokenizer Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: athoscribe...@gmail.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://athoscr.fedorapeople.org/packaging/golang-github-gorilla-css.spec SRPM URL: https://athoscr.fedorapeople.org/packaging/golang-github-gorilla-css-0-0.1.20170217git398b0b0.fc26.src.rpm Koji build: https://koji.fedoraproject.org/koji/taskinfo?taskID=22383825 Description: A CSS3 tokenizer Fedora Account System Username: athoscr -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1443523] Review Request: nodejs-progress - Flexible ascii progress bar
https://bugzilla.redhat.com/show_bug.cgi?id=1443523 --- Comment #4 from Gwyn Ciesla --- (fedrepo-req-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/nodejs-progress -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1495180] [PATCH] upgrade prometheus to 1.7.1
https://bugzilla.redhat.com/show_bug.cgi?id=1495180 Jan Chaloupka changed: What|Removed |Added Status|NEW |CLOSED Fixed In Version||golang-github-prometheus-pr ||ometheus-1.8.0-1.fc28 Resolution|--- |RAWHIDE Last Closed||2017-10-11 10:33:50 -- You are receiving this mail because: You are on the CC list for the bug. ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1443523] Review Request: nodejs-progress - Flexible ascii progress bar
https://bugzilla.redhat.com/show_bug.cgi?id=1443523 --- Comment #3 from Jared Smith --- Requested package in ticket https://pagure.io/releng/fedora-scm-requests/issue/2223 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1443523] Review Request: nodejs-progress - Flexible ascii progress bar
https://bugzilla.redhat.com/show_bug.cgi?id=1443523 Jared Smith changed: What|Removed |Added Assignee|nob...@fedoraproject.org|zebo...@gmail.com -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1444268] Review Request: nodejs-kuler2gpl - Converts Kuler ASE files to GIMP / Inkscape GPL color palette files
https://bugzilla.redhat.com/show_bug.cgi?id=1444268 Bug 1444268 depends on bug 1444256, which changed state. Bug 1444256 Summary: Review Request: nodejs-walker - A simple directory tree walker https://bugzilla.redhat.com/show_bug.cgi?id=1444256 What|Removed |Added Status|MODIFIED|CLOSED Resolution|--- |RAWHIDE -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1444256] Review Request: nodejs-walker - A simple directory tree walker
https://bugzilla.redhat.com/show_bug.cgi?id=1444256 Jared Smith changed: What|Removed |Added Status|MODIFIED|CLOSED Resolution|--- |RAWHIDE Last Closed||2017-10-11 10:26:24 --- Comment #4 from Jared Smith --- In rawhide, closing bug -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1495382] Review Request: nodejs-magic-string - Modify strings, generate sourcemaps
https://bugzilla.redhat.com/show_bug.cgi?id=1495382 Jared Smith changed: What|Removed |Added Status|MODIFIED|CLOSED Resolution|--- |RAWHIDE Last Closed||2017-10-11 10:23:21 --- Comment #4 from Jared Smith --- In rawhide, closing bug -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1495390] Review Request: nodejs-buble - The blazing fast, batteries-included ES2015 compiler
https://bugzilla.redhat.com/show_bug.cgi?id=1495390 Bug 1495390 depends on bug 1495382, which changed state. Bug 1495382 Summary: Review Request: nodejs-magic-string - Modify strings, generate sourcemaps https://bugzilla.redhat.com/show_bug.cgi?id=1495382 What|Removed |Added Status|MODIFIED|CLOSED Resolution|--- |RAWHIDE -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1437289] Review Request: nodejs-yeast - Tiny but linear growing unique id generator
https://bugzilla.redhat.com/show_bug.cgi?id=1437289 Jared Smith changed: What|Removed |Added Status|MODIFIED|CLOSED Resolution|--- |RAWHIDE Last Closed||2017-10-11 10:18:50 --- Comment #4 from Jared Smith --- In rawhide, closing bug -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1500305] Review Request: python-os-service-types - Python library for consuming OpenStack sevice-types-authority data
https://bugzilla.redhat.com/show_bug.cgi?id=1500305 --- Comment #10 from Gwyn Ciesla --- (fedrepo-req-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/python-os-service-types -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1500305] Review Request: python-os-service-types - Python library for consuming OpenStack sevice-types-authority data
https://bugzilla.redhat.com/show_bug.cgi?id=1500305 Alfredo Moralejo changed: What|Removed |Added Assignee|karlthe...@gmail.com|amora...@redhat.com -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1500305] Review Request: python-os-service-types - Python library for consuming OpenStack sevice-types-authority data
https://bugzilla.redhat.com/show_bug.cgi?id=1500305 --- Comment #9 from Haïkel Guémar --- I did agree with Alfredo to let him do the review, but we forgot the reassignement. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1474033] Review Request: ucx - Communication library implementing high-performance messaging
https://bugzilla.redhat.com/show_bug.cgi?id=1474033
--- Comment #19 from Andrey Maslennikov ---
Spec URL:
https://gist.github.com/amaslenn/3c847e0bdc063bcbb4b6507b5efbf6b9/raw/6acd2f4a4a5a34f6915e225b80c34d5ec7daf915/ucx.spec
SRPM URL:
https://gist.github.com/amaslenn/3c847e0bdc063bcbb4b6507b5efbf6b9/raw/6acd2f4a4a5a34f6915e225b80c34d5ec7daf915/ucx-1.2.2-1.fc25.src.rpm
Koji build: https://koji.fedoraproject.org/koji/taskinfo?taskID=22382583
Changes:
1. automake/autoconf/libtool --> gcc
-BuildRequires: automake autoconf libtool
+BuildRequires: gcc
2. fix . --> ,
-the following shared memory mechanisms: posix. sysv, cma, knem, xpmem.
+the following shared memory mechanisms: posix, sysv, cma, knem, xpmem.
3. Add a comment on static
+# Static libs are required for user apps to use UCX
4. make --> %make_build
-make %{?_smp_mflags} V=1
+%make_build V=1
5. make install --> %make_install
%install
-make DESTDIR=%{buildroot} install
+%make_install
6. Add empty lines, update date (release is the same)
-* Tue Sep 19 2017 Andrey Maslennikov 1.2.2-1
+* Wed Oct 11 2017 Andrey Maslennikov 1.2.2-1
- Spec file: new Source link, set default BuildRoot
+
* Mon Aug 21 2017 Andrey Maslennikov 1.2.1-1
- Spec file now complies with Fedora guidelines
+
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1500571] Review Request: golang-github-danwakefield-fnmatch - fnmatch implementation for Go
https://bugzilla.redhat.com/show_bug.cgi?id=1500571 --- Comment #2 from Gwyn Ciesla --- (fedrepo-req-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/golang-github-danwakefield-fnmatch -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1500573] Review Request: golang-github-dlclark-regexp2 - A regex engine for Go based on the .NET engine
https://bugzilla.redhat.com/show_bug.cgi?id=1500573 --- Comment #3 from Gwyn Ciesla --- (fedrepo-req-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/golang-github-dlclark-regexp2 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1500691] Review Request: perl-lib-relative - Add paths relative to the current file to @INC
https://bugzilla.redhat.com/show_bug.cgi?id=1500691 --- Comment #4 from Fedora Update System --- perl-lib-relative-0.002-1.fc27 has been submitted as an update to Fedora 27. https://bodhi.fedoraproject.org/updates/FEDORA-2017-7f132d40d7 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1500573] Review Request: golang-github-dlclark-regexp2 - A regex engine for Go based on the .NET engine
https://bugzilla.redhat.com/show_bug.cgi?id=1500573 --- Comment #2 from Athos Ribeiro --- Thanks for the review!!! -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1500691] Review Request: perl-lib-relative - Add paths relative to the current file to @INC
https://bugzilla.redhat.com/show_bug.cgi?id=1500691 Jitka Plesnikova changed: What|Removed |Added Status|POST|MODIFIED Fixed In Version||perl-lib-relative-0.002-1.f ||c28 --- Comment #3 from Jitka Plesnikova --- Thank you for the review and the repository. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1442241] Review Request: nodejs-cron - Cron jobs for your node
https://bugzilla.redhat.com/show_bug.cgi?id=1442241 Bug 1442241 depends on bug 1442234, which changed state. Bug 1442234 Summary: Review Request: nodejs-moment-timezone - Parse and display moments in any timezone https://bugzilla.redhat.com/show_bug.cgi?id=1442234 What|Removed |Added Status|MODIFIED|CLOSED Resolution|--- |RAWHIDE -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1442234] Review Request: nodejs-moment-timezone - Parse and display moments in any timezone
https://bugzilla.redhat.com/show_bug.cgi?id=1442234 Jared Smith changed: What|Removed |Added Status|MODIFIED|CLOSED Resolution|--- |RAWHIDE Last Closed||2017-10-11 09:42:36 --- Comment #5 from Jared Smith --- In rawhide, closing bug -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1500691] Review Request: perl-lib-relative - Add paths relative to the current file to @INC
https://bugzilla.redhat.com/show_bug.cgi?id=1500691 --- Comment #2 from Gwyn Ciesla --- (fedrepo-req-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/perl-lib-relative -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1500524] Review Request: gpxsee - GPS log file viewer and analyzer
https://bugzilla.redhat.com/show_bug.cgi?id=1500524 --- Comment #7 from Gwyn Ciesla --- (fedrepo-req-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/gpxsee. You may commit to the branch "f27" in about 10 minutes. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1421066] Review Request: deepin-wm-switcher - Window manager switcher for Deepin
https://bugzilla.redhat.com/show_bug.cgi?id=1421066 --- Comment #6 from Gwyn Ciesla --- (fedrepo-req-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/deepin-wm-switcher -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1468939] Review Request: ocaml-gen - Simple, efficient iterators for OCaml
https://bugzilla.redhat.com/show_bug.cgi?id=1468939 --- Comment #7 from Gwyn Ciesla --- (fedrepo-req-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/ocaml-gen -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1474033] Review Request: ucx - Communication library implementing high-performance messaging
https://bugzilla.redhat.com/show_bug.cgi?id=1474033
--- Comment #18 from Andrey Maslennikov ---
>> %{!?configure_options: %global configure_options %{nil}}
> How do you intend to use this? It will always be nil when the package is
> built in the Fedora build system.
>> Group: System Environment/Libraries
>> [...]
>> BuildRoot: %(mktemp -ud %{_tmppath}/%{name}-%{version}-%{release}-XX)
>
> "Group:" and "BuildRoot:" SHOULD NOT be used according to
> http://fedoraproject.org/wiki/Packaging:Guidelines#Tags_and_Sections
>
> "SHOULD NOT" is not as strong as "MUST NOT", but sharing the same spec file
> between Fedora and SLES does >not seem like a worthwhile goal in the long
> run. IMO it's not a valid reason for having these tags.
Supporting several specs is more complicated especially taking into account
that the diff will be in 1-2 lines only.
So I'd like to keep it this way, the line is required. It was proved by some
fails in our build env :)
>> BuildRequires: automake autoconf libtool
> It seems you're only running ./configure in the %build step, not regenerating
> it. Does the build really require autotools?
Will check.
>> the following shared memory mechanisms: posix. sysv, cma, knem, xpmem.
> Typo. This period should be a comma ---^
Will fix, thanks.
>> %package static
> The packaging guidelines say: "In general, packagers are strongly encouraged
> not to ship static libs unless a compelling reason exists."
> So a comment stating a reason for including the static libs would be nice.
Will add a comment. In fact, static libs are simply required to develop with
UCX.
>> make %{?_smp_mflags} V=1
> Just a minor suggestion: We have a macro called "make_build", defined as:
> %{__make} -O %{?_smp_mflags}
> It would be nice to use that macro:
> %make_build V=1
Will check if it works for as and switch if yes.
>> make DESTDIR=%{buildroot} install
> There is a macro for this as well: %make_install
Will update, thank you.
>> %{!?_licensedir:%global license %%doc}
> Is this to make the spec file work in older distros?
It is. Need a comment?
> Other:
> "[...] you must list a BuildRequires against gcc, gcc-c++ or clang"
> http://fedoraproject.org/wiki/Packaging:C_and_C%2B%2B#BuildRequires_and_Requires
Will add.
> Please insert empty lines between %changelog entries.
Will add.
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1499617] Review Request: adwaita-blue-gtk-theme - Adwaita Blue GTK theme variant
https://bugzilla.redhat.com/show_bug.cgi?id=1499617 --- Comment #4 from Gwyn Ciesla --- (fedrepo-req-admin): The Pagure repository was created at https://src.fedoraproject.org/rpms/adwaita-blue-gtk-theme -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1500691] Review Request: perl-lib-relative - Add paths relative to the current file to @INC
https://bugzilla.redhat.com/show_bug.cgi?id=1500691
Robert-André Mauchin changed:
What|Removed |Added
Status|NEW |POST
CC||zebo...@gmail.com
Assignee|nob...@fedoraproject.org|zebo...@gmail.com
Flags||fedora-review+
--- Comment #1 from Robert-André Mauchin ---
Package accepted.
Package Review
==
Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed
= MUST items =
Generic:
[x]: Package is licensed with an open-source compatible license and meets
other legal requirements as defined in the legal section of Packaging
Guidelines.
[x]: License field in the package spec file matches the actual license.
Note: Checking patched sources after %prep for licenses. Licenses
found: "Artistic (v2.0)", "Unknown or generated". 17 files have
unknown license. Detailed output of licensecheck in
/home/bob/packaging/review/perl-lib-relative/review-perl-lib-
relative/licensecheck.txt
[x]: Package does not own files or directories owned by other packages.
Note: Dirs in package are owned also by:
/usr/share/perl5/vendor_perl/lib(perl-local-lib, perl-lib-abs)
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[-]: Package contains desktop file if it is a GUI application.
[-]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[x]: Package consistently uses macros (instead of hard-coded directory
names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: Package is not known to require an ExcludeArch tag.
[-]: Large documentation must go in a -doc subpackage. Large could be size
(~1MB) or number of files.
Note: Documentation size is 10240 bytes in 2 files.
[x]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least
one supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
Note: There are rpmlint messages (see attachment).
[x]: If (and only if) the source package includes the text of the
license(s) in its own file, then that file, containing the text of the
license(s) for the package is included in %license.
[x]: Package requires other packages for directories it uses.
[x]: Package must own all directories that it creates.
[x]: Package uses either %{buildroot} or $RPM_BUILD_ROOT
[x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
beginning of %install.
[x]: Macros in Summary, %description expandable at SRPM build time.
[x]: Dist tag is present.
[x]: Package does not contain duplicates in %files.
[x]: Permissions on files are set properly.
[x]: Package use %makeinstall only when make install DESTDIR=... doesn't
work.
[x]: Package is named using only allowed ASCII characters.
[x]: Package does not use a name that already exists.
[x]: Package is not relocatable.
[x]: Sources used to build the package match the upstream source, as
provided in the spec URL.
[x]: Spec file name must match the spec package %{name}, in the format
%{name}.spec.
[x]: File names are valid UTF-8.
[x]: Packages must not store files under /srv, /opt or /usr/local
Perl:
[ ]: Package contains the mandatory BuildRequires and Requires:.
[ ]: CPAN urls should be non-versioned.
= SHOULD items =
Generic:
[-]: If the source package does not include license text(s) as a separate
file from upstream, the packager SHOULD query upstream to include it.
[x]: Final provides and requires are sane (see attachments).
[?]: Package functions as described.
[x]: Latest version is packaged.
[x]: Package does not include license text files separate from upstream.
[-]: Description and summary sections in the package spec file contains
translations for supported Non-English languages, if available.
[x]: Package should compile and build into binary rpms on all supported
architectures.
[x]: %check is present and all tests pass.
[x]: Packages should try to preserve timestamps of original installed
files.
[x]: Reviewer should test that the package builds in mock.
[x]: Buildroot is not present
[x]: Package has no %clean section with rm -rf %{buildroot} (or
$RPM_BUILD_ROOT
[Bug 1500574] Review Request: golang-github-alecthomas-chroma - A general purpose syntax highlighter in pure Go
https://bugzilla.redhat.com/show_bug.cgi?id=1500574 Robert-André Mauchin changed: What|Removed |Added CC||zebo...@gmail.com --- Comment #1 from Robert-André Mauchin --- This package is missing: DEBUG util.py:458: No matching package to install: 'golang(github.com/aymerick/douceur/css)' DEBUG util.py:458: No matching package to install: 'golang(github.com/aymerick/douceur/parser)' I guess you need to package golang-github-aymerick-douceur too. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1500573] Review Request: golang-github-dlclark-regexp2 - A regex engine for Go based on the .NET engine
https://bugzilla.redhat.com/show_bug.cgi?id=1500573 Robert-André Mauchin changed: What|Removed |Added Status|NEW |POST CC||zebo...@gmail.com Assignee|nob...@fedoraproject.org|zebo...@gmail.com Flags||fedora-review+ --- Comment #1 from Robert-André Mauchin --- Package accepted. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1500571] Review Request: golang-github-danwakefield-fnmatch - fnmatch implementation for Go
https://bugzilla.redhat.com/show_bug.cgi?id=1500571 Robert-André Mauchin changed: What|Removed |Added Status|NEW |POST CC||zebo...@gmail.com Assignee|nob...@fedoraproject.org|zebo...@gmail.com Flags||fedora-review+ --- Comment #1 from Robert-André Mauchin --- Package accepted. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1500524] Review Request: gpxsee - GPS log file viewer and analyzer
https://bugzilla.redhat.com/show_bug.cgi?id=1500524 --- Comment #6 from Nikola Forró --- Thanks for prompt review! -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1500524] Review Request: gpxsee - GPS log file viewer and analyzer
https://bugzilla.redhat.com/show_bug.cgi?id=1500524 Robert-André Mauchin changed: What|Removed |Added Status|NEW |POST Assignee|nob...@fedoraproject.org|zebo...@gmail.com Flags||fedora-review+ --- Comment #5 from Robert-André Mauchin --- All good then, package accepted. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1500524] Review Request: gpxsee - GPS log file viewer and analyzer
https://bugzilla.redhat.com/show_bug.cgi?id=1500524
--- Comment #4 from Nikola Forró ---
> There's another issue, you must own %{_datadir}/%{name}
Fixed.
Updated links:
Spec URL:
http://copr-dist-git.fedorainfracloud.org/cgit/nforro/GPXSee/gpxsee.git/plain/gpxsee.spec
SRPM URL:
https://copr-be.cloud.fedoraproject.org/results/nforro/GPXSee/fedora-rawhide-x86_64/00624102-gpxsee/gpxsee-4.15-1.fc28.src.rpm
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1500691] Review Request: perl-lib-relative - Add paths relative to the current file to @INC
https://bugzilla.redhat.com/show_bug.cgi?id=1500691 Jitka Plesnikova changed: What|Removed |Added Blocks||1500438 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1500438 [Bug 1500438] perl-Code-TidyAll-0.69 is available -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1500691] New: Review Request: perl-lib-relative - Add paths relative to the current file to @INC
https://bugzilla.redhat.com/show_bug.cgi?id=1500691 Bug ID: 1500691 Summary: Review Request: perl-lib-relative - Add paths relative to the current file to @INC Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: jples...@redhat.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://jplesnik.fedorapeople.org/perl-lib-relative/perl-lib-relative.spec SRPM URL: https://jplesnik.fedorapeople.org/perl-lib-relative/perl-lib-relative-0.002-1.fc28.src.rpm Description: This module proposes a straightforward method for adding local directory to @INC. It takes a path relative to the current file, absolutize it, and add it to @INC. Fedora Account System Username: jples...@redhat.com -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1421066] Review Request: deepin-wm-switcher - Window manager switcher for Deepin
https://bugzilla.redhat.com/show_bug.cgi?id=1421066 Zbigniew Jędrzejewski-Szmek changed: What|Removed |Added Status|NEW |POST Assignee|nob...@fedoraproject.org|zbys...@in.waw.pl Flags||fedora-review+ --- Comment #5 from Zbigniew Jędrzejewski-Szmek --- + package name is OK + license is acceptable for Fedora (GPLv3) + license is specified correctly + package builds in mock + BuildRequires / Requires / Provides look OK + latest version + fedora-review finds no issues + rpmlint output is OK (only some grumbling about missing man pages and documentation) + no scriptlets are necessary Package is APPROVED. I didn't test it, but the packaging is OK. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1500524] Review Request: gpxsee - GPS log file viewer and analyzer
https://bugzilla.redhat.com/show_bug.cgi?id=1500524
--- Comment #3 from Robert-André Mauchin ---
>The lang files are embedded into the executable as resources. Is that not
>allowed in Fedora? I could try to convince upstream to change that.
I didn't know that, sorry.
There's another issue, you must own %{_datadir}/%{name}: remove the asterisk *
here:
%{_datadir}/%{name}/*
Package Review
==
Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed
= MUST items =
C/C++:
[x]: Package does not contain kernel modules.
[x]: Package contains no static executables.
[x]: Package does not contain any libtool archives (.la)
[x]: Rpath absent or only used for internal libs.
Generic:
[x]: Package is licensed with an open-source compatible license and meets
other legal requirements as defined in the legal section of Packaging
Guidelines.
[x]: License field in the package spec file matches the actual license.
Note: Checking patched sources after %prep for licenses. Licenses
found: "Unknown or generated". 231 files have unknown license.
Detailed output of licensecheck in /home/bob/packaging/review/gpxsee
/review-gpxsee/licensecheck.txt
[x]: License file installed when any subpackage combination is installed.
[x]: Package requires other packages for directories it uses.
Note: No known owner of /usr/share/gpxsee
[!]: Package must own all directories that it creates.
Note: Directories without known owners: /usr/share/mime,
/usr/share/mime/packages, /usr/share/gpxsee
[x]: Package does not own files or directories owned by other packages.
[x]: %build honors applicable compiler flags or justifies otherwise.
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[x]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[x]: Package consistently uses macros (instead of hard-coded directory
names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: Useful -debuginfo package or justification otherwise.
[x]: Package is not known to require an ExcludeArch tag.
[-]: Large documentation must go in a -doc subpackage. Large could be size
(~1MB) or number of files.
Note: Documentation size is 10240 bytes in 1 files.
[x]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least
one supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
Note: There are rpmlint messages (see attachment).
[x]: If (and only if) the source package includes the text of the
license(s) in its own file, then that file, containing the text of the
license(s) for the package is included in %license.
[x]: All build dependencies are listed in BuildRequires, except for any
that are listed in the exceptions section of Packaging Guidelines.
[x]: Package uses either %{buildroot} or $RPM_BUILD_ROOT
[x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
beginning of %install.
[x]: Macros in Summary, %description expandable at SRPM build time.
[x]: Package contains desktop file if it is a GUI application.
[x]: Package installs a %{name}.desktop using desktop-file-install or
desktop-file-validate if there is such a file.
[x]: Dist tag is present.
[x]: Package does not contain duplicates in %files.
[x]: Permissions on files are set properly.
[x]: Package use %makeinstall only when make install DESTDIR=... doesn't
work.
[x]: Package is named using only allowed ASCII characters.
[x]: Package does not use a name that already exists.
[x]: Package is not relocatable.
[x]: Sources used to build the package match the upstream source, as
provided in the spec URL.
[x]: Spec file name must match the spec package %{name}, in the format
%{name}.spec.
[x]: File names are valid UTF-8.
[x]: Packages must not store files under /srv, /opt or /usr/local
= SHOULD items =
Generic:
[-]: If the source package does not include license text(s) as a separate
file from upstream, the packager SHOULD query upstream to include it.
[x]: Final provides and requires are sane (see attachments).
[x]: Fully versioned dependency in subpackages if applicable.
Note: No Requires: %{name}%{?_isa} = %{version}-%{release} in gpxsee-
debuginfo , gpxsee-debugsource
[?]: Package functions as described.
[x]: Latest version is packaged.
[x]: Package does not
[Bug 1376885] Dummy package review ticket for multiply-versioned packages
https://bugzilla.redhat.com/show_bug.cgi?id=1376885 Kalev Lember changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution|--- |NOTABUG Assignee|klem...@redhat.com |nob...@fedoraproject.org Summary|Review Request: libxml++30 |Dummy package review ticket |- C++ wrapper for the |for multiply-versioned |libxml2 XML parser library |packages Flags|fedora-review+ | Last Closed|2017-09-14 09:59:04 |2017-10-11 06:04:17 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1500524] Review Request: gpxsee - GPS log file viewer and analyzer
https://bugzilla.redhat.com/show_bug.cgi?id=1500524
--- Comment #2 from Nikola Forró ---
> Please use a more meaningful name for your archive
Will do. Thanks, I wasn't aware of this form of URL.
> You need to install the lang files in %{_datadir}/%{name}/locale/
The lang files are embedded into the executable as resources. Is that not
allowed in Fedora? I could try to convince upstream to change that.
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1493937] Review Request: plasma-drkonqi - crash handler for KF5/ Plasma5
https://bugzilla.redhat.com/show_bug.cgi?id=1493937 Jan Grulich changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #5 from Jan Grulich --- Looks good now. All issues have been fixed. Approved. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1500524] Review Request: gpxsee - GPS log file viewer and analyzer
https://bugzilla.redhat.com/show_bug.cgi?id=1500524
Robert-André Mauchin changed:
What|Removed |Added
CC||zebo...@gmail.com
--- Comment #1 from Robert-André Mauchin ---
Hello,
- Please use a more meaningful name for your archive, with:
Source0:
https://github.com/tumic0/%{appname}/archive/%{version}/%{name}-%{version}.tar.gz
- You need to install the lang files in %{_datadir}/%{name}/locale/ in
%install then run:
%find_lang %{name} --with-qt
And then add the lang file to %files
%files -f %{name}.lang
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1497349] Review Request: alignak - Alignak python Monitoring tool
https://bugzilla.redhat.com/show_bug.cgi?id=1497349
--- Comment #7 from Robert-André Mauchin ---
> Sorry but the first remark is not of my own. Archive name is create by
> upstream.
Github *does* support naming the archive like this:
Source0:
https://github.com/Alignak-monitoring/%{name}/archive/%{version}/%{name}-%{version}.tar.gz
Make the change in your SPEC, then run spectool -g on it: you will get the
archive with a proper name.
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1499676] Review Request: woff2 - Web Open Font Format 2.0 library
https://bugzilla.redhat.com/show_bug.cgi?id=1499676 Tomas Popela changed: What|Removed |Added Status|POST|CLOSED Resolution|--- |RAWHIDE Last Closed||2017-10-11 04:32:09 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1485721] Review Request: deepin-wm - Deepin Window Manager
https://bugzilla.redhat.com/show_bug.cgi?id=1485721 --- Comment #6 from sensor@gmail.com --- Thank you for reminding me. I fix golang-github-sergi-go-diff package provide for Rawhide and F27. Bodhi: https://bodhi.fedoraproject.org/updates/FEDORA-2017-27e921ccc5 Please add a ticket (karma) to make it faster. The directory owners of deepin-wm have also been fixed. https://github.com/FZUG/repo/commit/33f7f69909af0c7086218e850601e1e7be225254 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
[Bug 1480966] Review Request: golang-github-sergi-go-diff - Diff, match and patch text in Go
https://bugzilla.redhat.com/show_bug.cgi?id=1480966 --- Comment #9 from Fedora Update System --- golang-github-sergi-go-diff-0-0.2.gitfeef008.fc27 has been submitted as an update to Fedora 27. https://bodhi.fedoraproject.org/updates/FEDORA-2017-27e921ccc5 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list -- package-review@lists.fedoraproject.org To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
