[Bug 1119003] Review Request: minutor - Qt5 Minecraft Mapping tool
https://bugzilla.redhat.com/show_bug.cgi?id=1119003 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|minutor-2.0.2-0.1.20141209g |minutor-2.0.2-0.1.20141209g |itce24a81.fc21 |itce24a81.fc20 --- Comment #11 from Fedora Update System upda...@fedoraproject.org --- minutor-2.0.2-0.1.20141209gitce24a81.fc20 has been pushed to the Fedora 20 stable 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1190431] Review Request: golang-github-bmizerany-perks - Effective Computation of Things
https://bugzilla.redhat.com/show_bug.cgi?id=1190431 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||golang-github-bmizerany-per ||ks-0-0.1.gitd9a9656.fc21 Resolution|--- |ERRATA Last Closed||2015-02-25 08:30:16 --- Comment #8 from Fedora Update System upda...@fedoraproject.org --- golang-github-bmizerany-perks-0-0.1.gitd9a9656.fc21 has been pushed to the Fedora 21 stable 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1190442] Review Request: golang-github-prometheus-client_golang - Prometheus instrumentation library for Go applications
https://bugzilla.redhat.com/show_bug.cgi?id=1190442 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||golang-github-prometheus-cl ||ient_golang-0-0.1.git52186f ||c.fc20 Resolution|--- |ERRATA Last Closed||2015-02-25 08:30:12 --- Comment #8 from Fedora Update System upda...@fedoraproject.org --- golang-github-prometheus-client_golang-0-0.1.git52186fc.fc20 has been pushed to the Fedora 20 stable 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1196186] Review Request: perl-DBIx-Class-Schema-Config - Credential Management for DBIx::Class
https://bugzilla.redhat.com/show_bug.cgi?id=1196186 --- Comment #1 from Petr Šabata psab...@redhat.com --- Missing BRs: - Module::Install::Metadata, via various calls in Makefile.PL - Module::Install::WriteAll, via a call in Makefile.PL You don't have to filter out ^perl(DBIx::Class)$; it's not used anywhere in the lib code (directly). -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1190426] Review Request: golang-github-prometheus-prometheus - The Prometheus monitoring system and time series database
https://bugzilla.redhat.com/show_bug.cgi?id=1190426 Bug 1190426 depends on bug 1190418, which changed state. Bug 1190418 Summary: Review Request: golang-github-syndtr-goleveldb - LevelDB key/value database in Go https://bugzilla.redhat.com/show_bug.cgi?id=1190418 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1120882] Review Request: golang-googlecode-gogoprotobuf - A fork of goprotobuf with several extra features
https://bugzilla.redhat.com/show_bug.cgi?id=1120882 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version||golang-googlecode-gogoproto ||buf-0-0.10.gitbc946d0.fc20 Resolution|NEXTRELEASE |ERRATA --- Comment #22 from Fedora Update System upda...@fedoraproject.org --- golang-googlecode-gogoprotobuf-0-0.10.gitbc946d0.fc20 has been pushed to the Fedora 20 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1190442] Review Request: golang-github-prometheus-client_golang - Prometheus instrumentation library for Go applications
https://bugzilla.redhat.com/show_bug.cgi?id=1190442 Bug 1190442 depends on bug 1190413, which changed state. Bug 1190413 Summary: Review Request: golang-github-prometheus-procfs - Functions to retrieve system, kernel and process metrics from the /proc fs https://bugzilla.redhat.com/show_bug.cgi?id=1190413 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1190413] Review Request: golang-github-prometheus-procfs - Functions to retrieve system, kernel and process metrics from the /proc fs
https://bugzilla.redhat.com/show_bug.cgi?id=1190413 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||golang-github-prometheus-pr ||ocfs-0-0.1.git92faa30.fc20 Resolution|--- |ERRATA Last Closed||2015-02-25 08:25:00 --- Comment #8 from Fedora Update System upda...@fedoraproject.org --- golang-github-prometheus-procfs-0-0.1.git92faa30.fc20 has been pushed to the Fedora 20 stable 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1151818] Review Request: nodejs-typeahead.js - Autocomplete library for JavaScript
https://bugzilla.redhat.com/show_bug.cgi?id=1151818 Bug 1151818 depends on bug 1151817, which changed state. Bug 1151817 Summary: Review Request: nodejs-grunt-sed - Grunt task for search and replace https://bugzilla.redhat.com/show_bug.cgi?id=1151817 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1190437] Review Request: golang-github-prometheus-client_model - Data model artifacts for Prometheus
https://bugzilla.redhat.com/show_bug.cgi?id=1190437 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||golang-github-prometheus-cl ||ient_model-0-0.1.gitbc9454c ||.fc20 Resolution|--- |ERRATA Last Closed||2015-02-25 08:25:25 --- Comment #8 from Fedora Update System upda...@fedoraproject.org --- golang-github-prometheus-client_model-0-0.1.gitbc9454c.fc20 has been pushed to the Fedora 20 stable 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1190426] Review Request: golang-github-prometheus-prometheus - The Prometheus monitoring system and time series database
https://bugzilla.redhat.com/show_bug.cgi?id=1190426 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||golang-github-prometheus-pr ||ometheus-0-0.1.git4e6a807.f ||c20 Resolution|--- |ERRATA Last Closed||2015-02-25 08:24:28 --- Comment #9 from Fedora Update System upda...@fedoraproject.org --- golang-github-prometheus-prometheus-0-0.1.git4e6a807.fc20 has been pushed to the Fedora 20 stable 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1190418] Review Request: golang-github-syndtr-goleveldb - LevelDB key/value database in Go
https://bugzilla.redhat.com/show_bug.cgi?id=1190418 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||golang-github-syndtr-goleve ||ldb-0-0.1.gite9e2c8f.fc20 Resolution|--- |ERRATA Last Closed||2015-02-25 08:24:24 --- Comment #8 from Fedora Update System upda...@fedoraproject.org --- golang-github-syndtr-goleveldb-0-0.1.gite9e2c8f.fc20 has been pushed to the Fedora 20 stable 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1195835] Review Request: python-qrencode - A simple wrapper for the C qrencode
https://bugzilla.redhat.com/show_bug.cgi?id=1195835
Victoria Martinez de la Cruz vimar...@redhat.com changed:
What|Removed |Added
CC||vimar...@redhat.com
--- Comment #2 from Victoria Martinez de la Cruz vimar...@redhat.com ---
This is an unofficial review only.
Package Review
==
Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
Issues:
===
- Permissions on files are set properly.
Note: See rpmlint output
See: http://fedoraproject.org/wiki/Packaging/Guidelines#FilePermissions
- 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 %doc.
Note: Cannot find LICENSE-2.0 in rpm(s)
See:
http://fedoraproject.org/wiki/Packaging/LicensingGuidelines#License_Text
- %{__python} deprecated. The unversioned macros, %{__python},
%{python_sitelib}, and %{python_sitearch} are deprecated. You should use
%{__python2}, %{python2_sitelib}, and %{python2_sitearch} to explicitly
reference the python2 interpreter instead. This is future proofing for the
time when things will be switched over to python3 by default instead of
python2.
- Packages should try to preserve timestamps of original installed files
http://fedoraproject.org/wiki/Packaging:Guidelines#Timestamps.
Add -p flag to cp.
- %check is present and all tests pass. There are no tests for the submitted
package.
= MUST items =
C/C++:
[x]: Package does not contain kernel modules.
[x]: Package contains no static executables.
[!]: Development (unversioned) .so files in -devel subpackage, if present.
Note: Unversioned so-files in private %_libdir subdirectory (see
attachment). Verify they are not in ld path.
[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.
[!]: License field in the package spec file matches the actual license.
Note: Checking patched sources after %prep for licenses. Licenses found:
Unknown or generated. 7 files have unknown license.
[x]: License file installed when any subpackage combination is installed.
[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.
[-]: 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.
[-]: Useful -debuginfo package or justification otherwise.
[x]: Package is not known to require an ExcludeArch tag.
[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.
[!]: Rpmlint is run on all rpms the build produces.
Note: There are rpmlint messages (see attachment).
[x]: Package requires other packages for directories it uses.
[x]: Package must own all directories that it creates.
[x]: Package does not own files or directories owned by other packages.
[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 does not contain duplicates in %files.
[x]: Package use %makeinstall only when make install' ' DESTDIR=... doesn't
work.
[x]: Package is named using only allowed ASCII characters.
[x]: Package do not use a name that already exist
[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]: Large documentation must go in a -doc subpackage. Large could be size
(~1MB) or number of files.
[Bug 1141494] Review Request: python-flask-whooshalchemy - Whoosh extension to Flask/SQLAlchemy
https://bugzilla.redhat.com/show_bug.cgi?id=1141494 Jon Ciesla limburg...@gmail.com changed: What|Removed |Added Flags|fedora-cvs? |fedora-cvs+ -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1141494] Review Request: python-flask-whooshalchemy - Whoosh extension to Flask/SQLAlchemy
https://bugzilla.redhat.com/show_bug.cgi?id=1141494 --- Comment #25 from Jon Ciesla limburg...@gmail.com --- Git done (by process-git-requests). -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1171205] Review Request: perl-Monitoring-Plugin - Family of modules to streamline writing plugins for various monitoring systems
https://bugzilla.redhat.com/show_bug.cgi?id=1171205 Jon Ciesla limburg...@gmail.com changed: What|Removed |Added Flags|fedora-cvs? |fedora-cvs+ -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1190361] Review Request: clutter-gst3 - GStreamer integration library for Clutter
https://bugzilla.redhat.com/show_bug.cgi?id=1190361 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #10 from Fedora Update System upda...@fedoraproject.org --- clutter-gst3-3.0.4-1.fc21 has been pushed to the Fedora 21 testing 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1183286] Review Request: perl-Crypt-PWSafe3 - Read and write Passwordsafe v3 files
https://bugzilla.redhat.com/show_bug.cgi?id=1183286 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #8 from Fedora Update System upda...@fedoraproject.org --- perl-Crypt-PWSafe3-1.16-1.fc20 has been pushed to the Fedora 20 testing 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1196186] Review Request: perl-DBIx-Class-Schema-Config - Credential Management for DBIx::Class
https://bugzilla.redhat.com/show_bug.cgi?id=1196186 --- Comment #2 from Jitka Plesnikova jples...@redhat.com --- Updated. Links are same. -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1183422] Review Request: python-ghp-pages - Copies docs directly to the gh-pages branch of a GitHub repo
https://bugzilla.redhat.com/show_bug.cgi?id=1183422 José Matos jama...@fc.up.pt changed: What|Removed |Added Status|NEW |ASSIGNED CC||jama...@fc.up.pt --- Comment #3 from José Matos jama...@fc.up.pt --- Hi Pete. Have you looked further to comment#2? This package is simple and I am willing to review it. I saw it from nikola copr. -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1196186] New: Review Request: perl-DBIx-Class-Schema-Config - Credential Management for DBIx::Class
https://bugzilla.redhat.com/show_bug.cgi?id=1196186 Bug ID: 1196186 Summary: Review Request: perl-DBIx-Class-Schema-Config - Credential Management for DBIx::Class 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-DBIx-Class-Schema-Config/perl-DBIx-Class-Schema-Config.spec SRPM URL: https://jplesnik.fedorapeople.org/perl-DBIx-Class-Schema-Config/perl-DBIx-Class-Schema-Config-0.001011-1.fc23.src.rpm Description: DBIx::Class::Schema::Config is a subclass of DBIx::Class::Schema that allows the loading of credentials configuration from a file. The actual code itself would only need to know about the name used in the configuration file. This aims to make it simpler for operations teams to manage database credentials. Fedora Account System Username: jplesnik -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1190437] Review Request: golang-github-prometheus-client_model - Data model artifacts for Prometheus
https://bugzilla.redhat.com/show_bug.cgi?id=1190437 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|golang-github-prometheus-cl |golang-github-prometheus-cl |ient_model-0-0.1.gitbc9454c |ient_model-0-0.1.gitbc9454c |.fc20 |.fc21 --- Comment #9 from Fedora Update System upda...@fedoraproject.org --- golang-github-prometheus-client_model-0-0.1.gitbc9454c.fc21 has been pushed to the Fedora 21 stable 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1190413] Review Request: golang-github-prometheus-procfs - Functions to retrieve system, kernel and process metrics from the /proc fs
https://bugzilla.redhat.com/show_bug.cgi?id=1190413 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|golang-github-prometheus-pr |golang-github-prometheus-pr |ocfs-0-0.1.git92faa30.fc20 |ocfs-0-0.1.git92faa30.fc21 --- Comment #9 from Fedora Update System upda...@fedoraproject.org --- golang-github-prometheus-procfs-0-0.1.git92faa30.fc21 has been pushed to the Fedora 21 stable 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1190438] Review Request: golang-github-matttproud-golang_protobuf_extensions - Support for streaming Protocol Buffer messages for the Go language (golang)
https://bugzilla.redhat.com/show_bug.cgi?id=1190438 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|golang-github-matttproud-go |golang-github-matttproud-go |lang_protobuf_extensions-0- |lang_protobuf_extensions-0- |0.1.git7a864a0.fc21 |0.1.git7a864a0.fc20 --- Comment #9 from Fedora Update System upda...@fedoraproject.org --- golang-github-matttproud-golang_protobuf_extensions-0-0.1.git7a864a0.fc20 has been pushed to the Fedora 20 stable 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1186499] Review Request: tfdocgen - TiLP framework documentation generator
https://bugzilla.redhat.com/show_bug.cgi?id=1186499 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|tfdocgen-20150202git-2.fc20 |tfdocgen-20150202git-2.fc21 --- Comment #14 from Fedora Update System upda...@fedoraproject.org --- tfdocgen-20150202git-2.fc21 has been pushed to the Fedora 21 stable 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1190426] Review Request: golang-github-prometheus-prometheus - The Prometheus monitoring system and time series database
https://bugzilla.redhat.com/show_bug.cgi?id=1190426 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|golang-github-prometheus-pr |golang-github-prometheus-pr |ometheus-0-0.1.git4e6a807.f |ometheus-0-0.1.git4e6a807.f |c20 |c21 --- Comment #10 from Fedora Update System upda...@fedoraproject.org --- golang-github-prometheus-prometheus-0-0.1.git4e6a807.fc21 has been pushed to the Fedora 21 stable 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1156176] Review Request: golang-github-russross-blackfriday - Markdown processor implemented in Go
https://bugzilla.redhat.com/show_bug.cgi?id=1156176 --- Comment #11 from Fedora Update System upda...@fedoraproject.org --- golang-github-russross-blackfriday-1.2-4.el6 has been submitted as an update for Fedora EPEL 6. https://admin.fedoraproject.org/updates/golang-github-russross-blackfriday-1.2-4.el6 -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 665733] Review Request: Coin3 - High-level 3D visualization library
https://bugzilla.redhat.com/show_bug.cgi?id=665733 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #38 from Fedora Update System upda...@fedoraproject.org --- Coin3-3.1.3-6.fc20 has been pushed to the Fedora 20 testing repository. -- You are receiving this mail because: You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1119003] Review Request: minutor - Qt5 Minecraft Mapping tool
https://bugzilla.redhat.com/show_bug.cgi?id=1119003 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||minutor-2.0.2-0.1.20141209g ||itce24a81.fc21 Resolution|--- |ERRATA Last Closed||2015-02-25 08:26:48 --- Comment #10 from Fedora Update System upda...@fedoraproject.org --- minutor-2.0.2-0.1.20141209gitce24a81.fc21 has been pushed to the Fedora 21 stable 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1195835] Review Request: python-qrencode - A simple wrapper for the C qrencode
https://bugzilla.redhat.com/show_bug.cgi?id=1195835
--- Comment #3 from Victoria Martinez de la Cruz vimar...@redhat.com ---
Update:
[!]: Package complies to the Packaging Guidelines
The packaging guidelines require using commit-based tarballs when using Github
generated sources, not versions:
https://fedoraproject.org/wiki/Packaging:SourceURL#Github
You should use a tarball derived from a commit hash even for stable releases,
unless upstream provides tarballs in some way other than Github's generation
system. You can version the package as 1.0-1 , but the Source0 URL should use a
commit hash.
So, SourceO should look like: %{url}/archive/%{commit}/%{name}-%{commit}.tar.gz
--
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
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1171205] Review Request: perl-Monitoring-Plugin - Family of modules to streamline writing plugins for various monitoring systems
https://bugzilla.redhat.com/show_bug.cgi?id=1171205 --- Comment #9 from Jon Ciesla limburg...@gmail.com --- Git done (by process-git-requests). -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1190431] Review Request: golang-github-bmizerany-perks - Effective Computation of Things
https://bugzilla.redhat.com/show_bug.cgi?id=1190431 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|golang-github-bmizerany-per |golang-github-bmizerany-per |ks-0-0.1.gitd9a9656.fc21|ks-0-0.1.gitd9a9656.fc20 --- Comment #9 from Fedora Update System upda...@fedoraproject.org --- golang-github-bmizerany-perks-0-0.1.gitd9a9656.fc20 has been pushed to the Fedora 20 stable 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1188763] Review Request: nodejs-ultron - Ultron is a high-intelligence robot
https://bugzilla.redhat.com/show_bug.cgi?id=1188763 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|nodejs-ultron-1.0.1-3.fc20 |nodejs-ultron-1.0.1-3.fc21 --- Comment #12 from Fedora Update System upda...@fedoraproject.org --- nodejs-ultron-1.0.1-3.fc21 has been pushed to the Fedora 21 stable 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1156492] Review Request: golang-github-cpuguy83-go-md2man - Process markdown into manpages
https://bugzilla.redhat.com/show_bug.cgi?id=1156492 --- Comment #15 from Fedora Update System upda...@fedoraproject.org --- golang-github-cpuguy83-go-md2man-1-3.fc21 has been submitted as an update for Fedora 21. https://admin.fedoraproject.org/updates/golang-github-cpuguy83-go-md2man-1-3.fc21 -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1156492] Review Request: golang-github-cpuguy83-go-md2man - Process markdown into manpages
https://bugzilla.redhat.com/show_bug.cgi?id=1156492 --- Comment #14 from Fedora Update System upda...@fedoraproject.org --- golang-github-cpuguy83-go-md2man-1-3.fc20 has been submitted as an update for Fedora 20. https://admin.fedoraproject.org/updates/golang-github-cpuguy83-go-md2man-1-3.fc20 -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1190442] Review Request: golang-github-prometheus-client_golang - Prometheus instrumentation library for Go applications
https://bugzilla.redhat.com/show_bug.cgi?id=1190442 Bug 1190442 depends on bug 1190431, which changed state. Bug 1190431 Summary: Review Request: golang-github-bmizerany-perks - Effective Computation of Things https://bugzilla.redhat.com/show_bug.cgi?id=1190431 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1190442] Review Request: golang-github-prometheus-client_golang - Prometheus instrumentation library for Go applications
https://bugzilla.redhat.com/show_bug.cgi?id=1190442 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|golang-github-prometheus-cl |golang-github-prometheus-cl |ient_golang-0-0.1.git52186f |ient_golang-0-0.1.git52186f |c.fc20 |c.fc21 --- Comment #9 from Fedora Update System upda...@fedoraproject.org --- golang-github-prometheus-client_golang-0-0.1.git52186fc.fc21 has been pushed to the Fedora 21 stable 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1188763] Review Request: nodejs-ultron - Ultron is a high-intelligence robot
https://bugzilla.redhat.com/show_bug.cgi?id=1188763 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||nodejs-ultron-1.0.1-3.fc20 Resolution|--- |ERRATA Last Closed||2015-02-25 08:31:06 --- Comment #11 from Fedora Update System upda...@fedoraproject.org --- nodejs-ultron-1.0.1-3.fc20 has been pushed to the Fedora 20 stable 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1120882] Review Request: golang-googlecode-gogoprotobuf - A fork of goprotobuf with several extra features
https://bugzilla.redhat.com/show_bug.cgi?id=1120882 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|golang-googlecode-gogoproto |golang-googlecode-gogoproto |buf-0-0.10.gitbc946d0.fc20 |buf-0-0.10.gitbc946d0.fc21 --- Comment #23 from Fedora Update System upda...@fedoraproject.org --- golang-googlecode-gogoprotobuf-0-0.10.gitbc946d0.fc21 has been pushed to the Fedora 21 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1196186] Review Request: perl-DBIx-Class-Schema-Config - Credential Management for DBIx::Class
https://bugzilla.redhat.com/show_bug.cgi?id=1196186 Petr Šabata psab...@redhat.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||psab...@redhat.com Assignee|nob...@fedoraproject.org|psab...@redhat.com Flags||fedora-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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1190438] Review Request: golang-github-matttproud-golang_protobuf_extensions - Support for streaming Protocol Buffer messages for the Go language (golang)
https://bugzilla.redhat.com/show_bug.cgi?id=1190438 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||golang-github-matttproud-go ||lang_protobuf_extensions-0- ||0.1.git7a864a0.fc21 Resolution|--- |ERRATA Last Closed||2015-02-25 08:26:24 --- Comment #8 from Fedora Update System upda...@fedoraproject.org --- golang-github-matttproud-golang_protobuf_extensions-0-0.1.git7a864a0.fc21 has been pushed to the Fedora 21 stable 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1190442] Review Request: golang-github-prometheus-client_golang - Prometheus instrumentation library for Go applications
https://bugzilla.redhat.com/show_bug.cgi?id=1190442 Bug 1190442 depends on bug 1190437, which changed state. Bug 1190437 Summary: Review Request: golang-github-prometheus-client_model - Data model artifacts for Prometheus https://bugzilla.redhat.com/show_bug.cgi?id=1190437 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1156176] Review Request: golang-github-russross-blackfriday - Markdown processor implemented in Go
https://bugzilla.redhat.com/show_bug.cgi?id=1156176 --- Comment #9 from Fedora Update System upda...@fedoraproject.org --- golang-github-russross-blackfriday-1.2-4.fc20 has been submitted as an update for Fedora 20. https://admin.fedoraproject.org/updates/golang-github-russross-blackfriday-1.2-4.fc20 -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1151817] Review Request: nodejs-grunt-sed - Grunt task for search and replace
https://bugzilla.redhat.com/show_bug.cgi?id=1151817 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||nodejs-grunt-sed-0.1.1-1.fc ||21 Resolution|--- |ERRATA Last Closed||2015-02-25 08:24:56 --- Comment #8 from Fedora Update System upda...@fedoraproject.org --- nodejs-grunt-sed-0.1.1-1.fc21 has been pushed to the Fedora 21 stable 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1156176] Review Request: golang-github-russross-blackfriday - Markdown processor implemented in Go
https://bugzilla.redhat.com/show_bug.cgi?id=1156176 --- Comment #10 from Fedora Update System upda...@fedoraproject.org --- golang-github-russross-blackfriday-1.2-4.fc21 has been submitted as an update for Fedora 21. https://admin.fedoraproject.org/updates/golang-github-russross-blackfriday-1.2-4.fc21 -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1190442] Review Request: golang-github-prometheus-client_golang - Prometheus instrumentation library for Go applications
https://bugzilla.redhat.com/show_bug.cgi?id=1190442 Bug 1190442 depends on bug 1190438, which changed state. Bug 1190438 Summary: Review Request: golang-github-matttproud-golang_protobuf_extensions - Support for streaming Protocol Buffer messages for the Go language (golang) https://bugzilla.redhat.com/show_bug.cgi?id=1190438 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1190418] Review Request: golang-github-syndtr-goleveldb - LevelDB key/value database in Go
https://bugzilla.redhat.com/show_bug.cgi?id=1190418 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|golang-github-syndtr-goleve |golang-github-syndtr-goleve |ldb-0-0.1.gite9e2c8f.fc20 |ldb-0-0.1.gite9e2c8f.fc21 --- Comment #9 from Fedora Update System upda...@fedoraproject.org --- golang-github-syndtr-goleveldb-0-0.1.gite9e2c8f.fc21 has been pushed to the Fedora 21 stable 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1186494] Review Request: libticonv - Texas Instruments calculators charsets library
https://bugzilla.redhat.com/show_bug.cgi?id=1186494 Bug 1186494 depends on bug 1186499, which changed state. Bug 1186499 Summary: Review Request: tfdocgen - TiLP framework documentation generator https://bugzilla.redhat.com/show_bug.cgi?id=1186499 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1186499] Review Request: tfdocgen - TiLP framework documentation generator
https://bugzilla.redhat.com/show_bug.cgi?id=1186499 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||tfdocgen-20150202git-2.fc20 Resolution|--- |ERRATA Last Closed||2015-02-25 08:26:29 --- Comment #13 from Fedora Update System upda...@fedoraproject.org --- tfdocgen-20150202git-2.fc20 has been pushed to the Fedora 20 stable 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1156492] Review Request: golang-github-cpuguy83-go-md2man - Process markdown into manpages
https://bugzilla.redhat.com/show_bug.cgi?id=1156492 --- Comment #16 from Fedora Update System upda...@fedoraproject.org --- golang-github-cpuguy83-go-md2man-1-3.el6 has been submitted as an update for Fedora EPEL 6. https://admin.fedoraproject.org/updates/golang-github-cpuguy83-go-md2man-1-3.el6 -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1196186] Review Request: perl-DBIx-Class-Schema-Config - Credential Management for DBIx::Class
https://bugzilla.redhat.com/show_bug.cgi?id=1196186 Petr Šabata psab...@redhat.com changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #3 from Petr Šabata psab...@redhat.com --- Ack, approving. -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1196289] New: Review Request: nodejs-defaults - Merge single level defaults over a config object
https://bugzilla.redhat.com/show_bug.cgi?id=1196289 Bug ID: 1196289 Summary: Review Request: nodejs-defaults - Merge single level defaults over a config object Product: Fedora Version: rawhide Component: Package Review Severity: medium Assignee: nob...@fedoraproject.org Reporter: zsvet...@redhat.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://fedorapeople.org/~humaton/rpms/nodejs-defaults.spec SRPM URL: https://fedorapeople.org/~humaton/rpms/nodejs-defaults-1.0.0-3.fc23.src.rpm Description: Merge single level defaults over a config object. Fedora Account System Username: zvetlik -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1196186] Review Request: perl-DBIx-Class-Schema-Config - Credential Management for DBIx::Class
https://bugzilla.redhat.com/show_bug.cgi?id=1196186 Jitka Plesnikova jples...@redhat.com changed: What|Removed |Added Flags||fedora-cvs? --- Comment #4 from Jitka Plesnikova jples...@redhat.com --- New Package SCM Request === Package Name: perl-DBIx-Class-Schema-Config Short Description: Credential Management for DBIx::Class Upstream URL: http://search.cpan.org/dist/DBIx-Class-Schema-Config/ Owners: jplesnik ppisar psabata Branches: f22 InitialCC: perl-sig -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 665733] Review Request: Coin3 - High-level 3D visualization library
https://bugzilla.redhat.com/show_bug.cgi?id=665733 --- Comment #40 from Fedora Update System upda...@fedoraproject.org --- Coin3-3.1.3-7.fc21 has been submitted as an update for Fedora 21. https://admin.fedoraproject.org/updates/Coin3-3.1.3-7.fc21 -- You are receiving this mail because: You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 665733] Review Request: Coin3 - High-level 3D visualization library
https://bugzilla.redhat.com/show_bug.cgi?id=665733 --- Comment #39 from Fedora Update System upda...@fedoraproject.org --- Coin3-3.1.3-7.fc20 has been submitted as an update for Fedora 20. https://admin.fedoraproject.org/updates/Coin3-3.1.3-7.fc20 -- You are receiving this mail because: You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 989791] Review Request: doublecmd-qt4 - Twin-panel (commander-style) file manager(Qt4)
https://bugzilla.redhat.com/show_bug.cgi?id=989791 Raphael Groner projects...@smart.ms changed: What|Removed |Added Flags||needinfo?(i...@cicku.me) --- Comment #12 from Raphael Groner projects...@smart.ms --- Ping? Again? -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1196290] New: Review Request: nodejs-wcwidth - Determine columns needed for a fixed-size wide-character string
https://bugzilla.redhat.com/show_bug.cgi?id=1196290 Bug ID: 1196290 Summary: Review Request: nodejs-wcwidth - Determine columns needed for a fixed-size wide-character string Product: Fedora Version: rawhide Component: Package Review Severity: medium Assignee: nob...@fedoraproject.org Reporter: zsvet...@redhat.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://fedorapeople.org/~humaton/rpms/nodejs-wcwidth.spec SRPM URL: https://fedorapeople.org/~humaton/rpms/nodejs-wcwidth-1.0.0-3.fc21.src.rpm Description: Determine columns needed for a fixed-size wide-character string. Fedora Account System Username: zvetlik -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 665733] Review Request: Coin3 - High-level 3D visualization library
https://bugzilla.redhat.com/show_bug.cgi?id=665733 --- Comment #41 from Fedora Update System upda...@fedoraproject.org --- Coin3-3.1.3-7.fc22 has been submitted as an update for Fedora 22. https://admin.fedoraproject.org/updates/Coin3-3.1.3-7.fc22 -- You are receiving this mail because: You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1189269] Review Request: qparted - Visual partition editor based on Qt framework [a fork of qtparted]
https://bugzilla.redhat.com/show_bug.cgi?id=1189269 Raphael Groner projects...@smart.ms changed: What|Removed |Added Depends On||885365, 1188780 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=885365 [Bug 885365] qtparted does not work on USB flash because no partition table, just FS https://bugzilla.redhat.com/show_bug.cgi?id=1188780 [Bug 1188780] [qtparted] No menu icon -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1155376] Review Request: rnetclient - Program to submit the Brazilian Tax Report
https://bugzilla.redhat.com/show_bug.cgi?id=1155376 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #12 from Fedora Update System upda...@fedoraproject.org --- rnetclient-2014.2-1.fc22 has been pushed to the Fedora 22 testing 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 709328] Review Request: psi-plus - Jabber client based on Qt
https://bugzilla.redhat.com/show_bug.cgi?id=709328 Raphael Groner projects...@smart.ms changed: What|Removed |Added Flags||needinfo?(dr...@land.ru) --- Comment #107 from Raphael Groner projects...@smart.ms --- Ping? -- You are receiving this mail because: You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1150637] Review Request: paintown - action beatem-up game
https://bugzilla.redhat.com/show_bug.cgi?id=1150637 Bug 1150637 depends on bug 1150241, which changed state. Bug 1150241 Summary: [abrt] scons: Executor.py:396:prepare:StopError: Source `/usr/lib64/libpng.a' not found, needed by target `misc/libpng.a'. https://bugzilla.redhat.com/show_bug.cgi?id=1150241 What|Removed |Added Status|NEW |CLOSED Resolution|--- |WONTFIX -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1195853] Review Request: OpenGPG Key Exchange for Humans - OpenGPG Key Exchange for Humans
https://bugzilla.redhat.com/show_bug.cgi?id=1195853
Florian der-flo Lehner d...@der-flo.net changed:
What|Removed |Added
Status|NEW |ASSIGNED
CC||d...@der-flo.net
Assignee|nob...@fedoraproject.org|d...@der-flo.net
Flags||fedora-review?
--- Comment #1 from Florian der-flo Lehner d...@der-flo.net ---
Hi Igor!
Is this for python2 or python3? because there are different macros[1]?
The unversioned macros, %{__python}, %{python_sitelib}, and %{python_sitearch}
are deprecated.
And %license is missing in the files-section.
Cheers,
Florian
[1] https://fedoraproject.org/wiki/Packaging:Python#Macros
--
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
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1193986] Review Request: openstack-rally - Benchmark as a service for OpenStack
https://bugzilla.redhat.com/show_bug.cgi?id=1193986 --- Comment #4 from Victoria Martinez de la Cruz vimar...@redhat.com --- (In reply to Haïkel Guémar from comment #1) I'm taking care of the sponsorship process for Victoria. @Victoria: when you'll have finished with your informal reviews (at least 2), please link them in this ticket. http://fedoraproject.org/PackageReviewStatus/NEW.html Thanks Haïkel! https://bugzilla.redhat.com/show_bug.cgi?id=1195835 https://bugzilla.redhat.com/show_bug.cgi?id=1195573 --- Comment #5 from Pranav Kant pranav...@gmail.com --- I didn't run mock separately. It was done under fedora-review tool -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1080411] Review Request: trojita - Qt IMAP e-mail client
https://bugzilla.redhat.com/show_bug.cgi?id=1080411 --- Comment #28 from Kevin Kofler ke...@tigcc.ticalc.org --- As I already said in comment #15 and explained in comment #17, I also agree that you MUST require hicolor-icon-theme instead of owning the same directories. -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1193986] Review Request: openstack-rally - Benchmark as a service for OpenStack
https://bugzilla.redhat.com/show_bug.cgi?id=1193986
--- Comment #3 from Victoria Martinez de la Cruz vimar...@redhat.com ---
(In reply to Pranav Kant from comment #2)
This is an unofficial review only.
Package Review
==
Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
Issues:
===
- Package doesn't installs properly in my case.
- Package uses either %{buildroot} or $RPM_BUILD_ROOT
Please be consistent with only one of them.
See: http://fedoraproject.org/wiki/Packaging/Guidelines#macros
- Please use %license for your LICENSE file. This has changed rececntly.
See:
http://fedoraproject.org/wiki/Changes/
Use_license_macro_in_RPMs_for_packages_in_Cloud_Image
- You are using too much asterisks in your description of the spec file.
The description in your summary makes extensive use of asterisks. Please
note that these are not translated to anything. So, IMHO, their use should
be minimal in spec file.
= 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: Licenses found:
Apache (v2.0), Unknown or generated, *No copyright* Apache (v2.0).
21 files have unknown license.
[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.
[!]: Package consistently uses macros (instead of hard-coded directory
names).
I am not sure about this. But your spec file makes considerable use of
hard-coded directory names. Though, I see no advantage of using macros
in
this case, so this might be ok.
[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.
[x]: If the package is a rename of another package, proper Obsoletes and
Provides are present.
[x]: Requires correct, justified where necessary.
[!]: Spec file is legible and written in American English.
See Issues above.
[-]: 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.
[x]: Packages must not store files under /srv, /opt or /usr/local
[x]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least
one
supported primary architecture.
[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.
See Issues above.
[x]: Package requires other packages for directories it uses.
[x]: Package must own all directories that it creates.
[x]: Package does not own files or directories owned by other packages.
[x]: All build dependencies are listed in BuildRequires, except for any that
are listed in the exceptions section of Packaging Guidelines.
[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 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 do not use a name that already exist
[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.
Python:
[?]: Binary eggs must be removed in %prep
[x]: Python eggs must not download any dependencies during the build process.
[x]: A package which is used by another package via an egg interface should
provide egg info.
[x]: Package meets the Packaging Guidelines::Python
[x]: Package contains BR: python2-devel or python3-devel
= 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]: Package functions as described.
[x]: Latest version is packaged.
[x]: Package does not include license
[Bug 1170875] Review Request: budgie-desktop - An elegant desktop with GNOME integration
https://bugzilla.redhat.com/show_bug.cgi?id=1170875 --- Comment #20 from Michel Alexandre Salim mic...@michel-slm.name --- Will try and find the time to do this in the next couple of days - I don't have access to Fedora at work. -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1193986] Review Request: openstack-rally - Benchmark as a service for OpenStack
https://bugzilla.redhat.com/show_bug.cgi?id=1193986 --- Comment #4 from Victoria Martinez de la Cruz vimar...@redhat.com --- (In reply to Haïkel Guémar from comment #1) I'm taking care of the sponsorship process for Victoria. @Victoria: when you'll have finished with your informal reviews (at least 2), please link them in this ticket. http://fedoraproject.org/PackageReviewStatus/NEW.html Thanks Haïkel! https://bugzilla.redhat.com/show_bug.cgi?id=1195835 https://bugzilla.redhat.com/show_bug.cgi?id=1195573 -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1196353] Review Request: main package name here - short summary here
https://bugzilla.redhat.com/show_bug.cgi?id=1196353
Christian Dersch lupi...@mailbox.org changed:
What|Removed |Added
Flags|fedora-review? |fedora-review+
--- Comment #2 from Christian Dersch lupi...@mailbox.org ---
Package looks fine to me, there are two details to be fixed before SCM import:
* The package should be noarch
* Please fix the mixed-use-of-spaces-and-tabs warning rpmlint found
Solution: Approved!
Package Review
==
Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed
Issues:
===
- 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 %doc.
Note: Cannot find LICENSE in rpm(s)
See:
http://fedoraproject.org/wiki/Packaging/LicensingGuidelines#License_Text
** Not an issue, package uses %license macro as recommended now
= 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. No licenses
found. Please check the source files for licenses manually.
[-]: %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.
[-]: 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.
[-]: 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.
[x]: Packages must not store files under /srv, /opt or /usr/local
[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]: Package requires other packages for directories it uses.
[x]: Package must own all directories that it creates.
[x]: Package does not own files or directories owned by other packages.
[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 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 do not use a name that already exist
[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.
= 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]: 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.
* Note: IMHO the package should be noarch
[-]: %check is present and all tests pass.
[x]: Packages should try to preserve timestamps of original installed files.
[x]: Packager, Vendor, PreReq, Copyright tags should not be in spec file
[x]: Sources can be downloaded from URI
[Bug 1196361] New: Review Request: golang-github-gorilla-websocket - A WebSocket implementation for Go
https://bugzilla.redhat.com/show_bug.cgi?id=1196361 Bug ID: 1196361 Summary: Review Request: golang-github-gorilla-websocket - A WebSocket implementation for Go Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: jchal...@redhat.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://jchaloup.fedorapeople.org/reviews/golang-github-gorilla-websocket/golang-github-gorilla-websocket.spec SRPM URL: https://jchaloup.fedorapeople.org/reviews/golang-github-gorilla-websocket/golang-github-gorilla-websocket-0-0.1.gitab5b3a6.fc20.src.rpm Description: A WebSocket implementation for Go Fedora Account System Username: jchaloup Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=9068575 $ rpmlint golang-github-gorilla-websocket-0-0.1.gitab5b3a6.fc20.src.rpm golang-github-gorilla-websocket-devel-0-0.1.gitab5b3a6.fc20.noarch.rpm golang-github-gorilla-websocket.spec 2 packages and 1 specfiles checked; 0 errors, 0 warnings. Upstream: https://github.com/gorilla/websocket -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1196363] New: Review Request: golang-github-docker-spdystream - A multiplexed stream library using spdy
https://bugzilla.redhat.com/show_bug.cgi?id=1196363
Bug ID: 1196363
Summary: Review Request: golang-github-docker-spdystream - A
multiplexed stream library using spdy
Product: Fedora
Version: rawhide
Component: Package Review
Severity: medium
Priority: medium
Assignee: nob...@fedoraproject.org
Reporter: jchal...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org
Spec URL:
https://jchaloup.fedorapeople.org/reviews/golang-github-docker-spdystream/golang-github-docker-spdystream.spec
SRPM URL:
https://jchaloup.fedorapeople.org/reviews/golang-github-docker-spdystream/golang-github-docker-spdystream-0-0.1.git29e1da2.fc20.src.rpm
Description: A multiplexed stream library using spdy
Fedora Account System Username: jchaloup
$ rpmlint golang-github-docker-spdystream-0-0.1.git29e1da2.fc20.src.rpm
golang-github-docker-spdystream-devel-0-0.1.git29e1da2.fc20.noarch.rpm
golang-github-docker-spdystream.spec
golang-github-docker-spdystream.src: W: spelling-error Summary(en_US) spdy -
spy, spay, spry
golang-github-docker-spdystream.src: W: spelling-error %description -l en_US
spdy - spy, spay, spry
golang-github-docker-spdystream.src:55: W: macro-in-comment %{buildroot}
golang-github-docker-spdystream.src:55: W: macro-in-comment %{gopath}
golang-github-docker-spdystream.src:55: W: macro-in-comment %{gopath}
golang-github-docker-spdystream.src:55: W: macro-in-comment %{import_path}
golang-github-docker-spdystream-devel.noarch: W: spelling-error Summary(en_US)
spdy - spy, spay, spry
golang-github-docker-spdystream-devel.noarch: W: spelling-error %description -l
en_US spdy - spy, spay, spry
golang-github-docker-spdystream.spec:55: W: macro-in-comment %{buildroot}
golang-github-docker-spdystream.spec:55: W: macro-in-comment %{gopath}
golang-github-docker-spdystream.spec:55: W: macro-in-comment %{gopath}
golang-github-docker-spdystream.spec:55: W: macro-in-comment %{import_path}
2 packages and 1 specfiles checked; 0 errors, 12 warnings.
Upstream: https://github.com/docker/spdystream
Notes: Depends on github.com/gorilla/websocket, which is on review (#1196361).
Locally it builds fine.
--
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
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1187869] Review Request: kdocker - Dock any application in the system tray
https://bugzilla.redhat.com/show_bug.cgi?id=1187869 Raphael Groner projects...@smart.ms changed: What|Removed |Added Alias||kdocker -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1196366] New: Review Request: openstack-designateclient - Client library for OpenStack Designate DNS API
https://bugzilla.redhat.com/show_bug.cgi?id=1196366 Bug ID: 1196366 Summary: Review Request: openstack-designateclient - Client library for OpenStack Designate DNS API Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: vimar...@redhat.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: http://vmartinezdelacruz.com/rpms/python-designateclient/python-designateclient.spec SRPM URL: http://vmartinezdelacruz.com/rpms/python-designateclient/python-designateclient-1.1.1-1.fc21.src.rpm Description: Client library for OpenStack Designate DNS API Fedora Account System Username: vkmc -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1196367] New: Review Request: qtkeychain - QtKeychain is a Qt API to store passwords and other secret data securely
https://bugzilla.redhat.com/show_bug.cgi?id=1196367 Bug ID: 1196367 Summary: Review Request: qtkeychain - QtKeychain is a Qt API to store passwords and other secret data securely Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: he...@kde.org QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://fedorapeople.org/~heliocastro/pkgreview/qtkeychain.spec SRPM URL: https://fedorapeople.org/~heliocastro/pkgreview/qtkeychain-0.4.0-1.fc21.src.rpm Description: QtKeychain is a Qt API to store passwords and other secret data securely Fedora Account System Username: heliocastro -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1196368] New: Review Request: golang-github-golang-groupcache - Caching and cache-filling library as a replacement for memcached in many cases
https://bugzilla.redhat.com/show_bug.cgi?id=1196368 Bug ID: 1196368 Summary: Review Request: golang-github-golang-groupcache - Caching and cache-filling library as a replacement for memcached in many cases Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: jchal...@redhat.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://jchaloup.fedorapeople.org/reviews/golang-github-golang-groupcache/golang-github-golang-groupcache.spec SRPM URL: https://jchaloup.fedorapeople.org/reviews/golang-github-golang-groupcache/golang-github-golang-groupcache-0-0.1.git604ed57.fc20.src.rpm Description: Caching and cache-filling library as a replacement for memcached in many cases Fedora Account System Username: jchaloup Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=9074078 $ rpmlint golang-github-golang-groupcache-0-0.1.git604ed57.fc20.src.rpm golang-github-golang-groupcache-devel-0-0.1.git604ed57.fc20.noarch.rpm golang-github-golang-groupcache.spec golang-github-golang-groupcache.src: W: spelling-error Summary(en_US) memcached - schemed golang-github-golang-groupcache.src: W: spelling-error %description -l en_US memcached - schemed golang-github-golang-groupcache-devel.noarch: W: spelling-error Summary(en_US) memcached - schemed golang-github-golang-groupcache-devel.noarch: W: spelling-error %description -l en_US memcached - schemed 2 packages and 1 specfiles checked; 0 errors, 4 warnings. Upstream: https://github.com/golang/groupcache -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1196373] Review Request: stockfish - Powerful open source chess engine
https://bugzilla.redhat.com/show_bug.cgi?id=1196373 Raphael Groner projects...@smart.ms changed: What|Removed |Added See Also||https://bugzilla.redhat.com ||/show_bug.cgi?id=820796 -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1196366] Review Request: openstack-designateclient - Client library for OpenStack Designate DNS API
https://bugzilla.redhat.com/show_bug.cgi?id=1196366 Victoria Martinez de la Cruz vimar...@redhat.com changed: What|Removed |Added Blocks||1193986 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1193986 [Bug 1193986] Review Request: openstack-rally - Benchmark as a service for OpenStack -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1196353] Review Request: itop - Interrupts 'top-like' utility for Linux
https://bugzilla.redhat.com/show_bug.cgi?id=1196353 Christian Dersch lupi...@mailbox.org changed: What|Removed |Added Summary|Review Request: main |Review Request: itop - |package name here - short |Interrupts 'top-like' |summary here |utility for Linux -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1196353] Review Request: itop - Interrupts 'top-like' utility for Linux
https://bugzilla.redhat.com/show_bug.cgi?id=1196353 Florian der-flo Lehner d...@der-flo.net changed: What|Removed |Added Flags||fedora-cvs? --- Comment #3 from Florian der-flo Lehner d...@der-flo.net --- Thanks Christian for the review! New Package SCM Request === Package Name: itop Short Description: Interactive interrupt viewer Upstream URL: https://github.com/kargig/itop Owners: flo Branches: f20 f21 f22 el6 epel7 InitialCC: -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1006197] Review Request: stockfish - Powerful open source chess engine
https://bugzilla.redhat.com/show_bug.cgi?id=1006197 Raphael Groner projects...@smart.ms changed: What|Removed |Added Blocks||1196373 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1196373 [Bug 1196373] Review Request: stockfish - Powerful open source chess engine -- You are receiving this mail because: You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1196373] New: Review Request: stockfish - Powerful open source chess engine
https://bugzilla.redhat.com/show_bug.cgi?id=1196373 Bug ID: 1196373 Summary: Review Request: stockfish - Powerful open source chess engine Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: projects...@smart.ms QA Contact: extras...@fedoraproject.org CC: bress...@redhat.com, dh...@openwall.com, dkho...@redhat.com, extras...@fedoraproject.org, i...@cicku.me, mattga...@gmail.com, package-review@lists.fedoraproject.org, projects...@smart.ms, rc040...@freenet.de Depends On: 1006197 Blocks: 201449 (FE-DEADREVIEW) +++ This bug was initially created as a clone of Bug #1006197 +++ Stockfish is a free UCI chess engine derived from Glaurung 2.1. It is not a complete chess program, but requires some UCI compatible GUI (like XBoard with PolyGlot, eboard, Arena, Sigma Chess, Shredder, Chess Partner or Fritz) in order to be used comfortably. Read the documentation for your GUI of choice for information about how to use Stockfish with your GUI. Original RFE: https://bugzilla.redhat.com/show_bug.cgi?id=820796 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=201449 [Bug 201449] FE-DEADREVIEW -- Reviews stalled due to lack of submitter response should be blocking this bug. https://bugzilla.redhat.com/show_bug.cgi?id=1006197 [Bug 1006197] Review Request: stockfish - Powerful open source chess engine -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1179982] Review Request: docsis-config-encoder - Encode a DOCSIS binary configuration file
https://bugzilla.redhat.com/show_bug.cgi?id=1179982 Raphael Groner projects...@smart.ms changed: What|Removed |Added Summary|Review Request: docsis -|Review Request: |Encode a DOCSIS binary |docsis-config-encoder - |configuration file |Encode a DOCSIS binary ||configuration file --- Comment #5 from Raphael Groner projects...@smart.ms --- (In reply to Simon Farnsworth from comment #3) Three things to fix up: 1. Source10 (the master branch version of COPYING) isn't good; it's a github page, not the raw text, and it changes every time it's downloaded. Could you get the raw text instead? Fixed. There's no sense in linking any COPYING separately. Dunno why I did that. 2. You have a bundled md5 implementation, but don't have the appropriate provides: https://fedoraproject.org/wiki/Packaging: No_Bundled_Libraries#Packages_granted_exceptions - I think it's bundled(md5-plumb) in this case. Fixed. 3. I don't understand why you've put some of the documentation into /usr/share/docsis (the examples and the config format documentation), rather than into /usr/share/doc/docsis or into a separate docsis-doc subpackage. Fixed. And some personal notes: * I don't like naming the package docsis - it's too general for my liking, given how big the DOCSIS spec is. Perhaps docsis-config-encoder? Hmm, you should better discuss that with upstream. But okay, fixed. * 1.4M of the resulting installed package is noarch MIBs, to 500k of arch-dependent binaries. Is there any use for these MIBs without the config encoder? If so, it might be nice to split them out into their own noarch subpackage. Fixed. -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1179982] Review Request: docsis-config-encoder - Encode a DOCSIS binary configuration file
https://bugzilla.redhat.com/show_bug.cgi?id=1179982 --- Comment #7 from Raphael Groner projects...@smart.ms --- rawhide scratch: http://koji.fedoraproject.org/koji/taskinfo?taskID=9072993 -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1194330] Review Request: kate4 - Advanced Text Editor
https://bugzilla.redhat.com/show_bug.cgi?id=1194330 --- Comment #4 from Rex Dieter rdie...@math.unl.edu --- As far as license files, we have one already, did you miss? %files part %doc COPYING.LIB ? Spec URL: https://rdieter.fedorapeople.org/rpms/plasma5/kate4.spec SRPM URL: https://rdieter.fedorapeople.org/rpms/plasma5/kate4-4.14.3-11.fc20.src.rpm %changelog * Wed Feb 25 2015 Rex Dieter rdie...@fedoraproject.org 4.14.3-11 - improve pkg %%summary, %%description -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1187869] Review Request: kdocker - Dock any application in the system tray
https://bugzilla.redhat.com/show_bug.cgi?id=1187869 Rex Dieter rdie...@math.unl.edu changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #9 from Rex Dieter rdie...@math.unl.edu --- Thanks, looks good, APPROVED. Please add more detailed changelog next time, instead of fix review issues, list the specific items changed. -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1156492] Review Request: golang-github-cpuguy83-go-md2man - Process markdown into manpages
https://bugzilla.redhat.com/show_bug.cgi?id=1156492 --- Comment #19 from Fedora Update System upda...@fedoraproject.org --- golang-github-cpuguy83-go-md2man-1-4.el6 has been submitted as an update for Fedora EPEL 6. https://admin.fedoraproject.org/updates/golang-github-cpuguy83-go-md2man-1-4.el6 -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1178912] Review Request: cairo-dock-plug-ins - Plug-ins files for Cairo-Dock
https://bugzilla.redhat.com/show_bug.cgi?id=1178912 Jan Pokorný jpoko...@redhat.com changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #38 from Jan Pokorný jpoko...@redhat.com --- ACK, it's in a good shape now. Also checked it (c-d + c-d-p-i together) works well on F21/LXDE. re CDBashApplet.sh: Ok, there is apparently more going on, also when a comment for INSTALL_PREFIX = os.path.abspath(..) in CDApplet.py taken into account. Note on the demo_* applets usage: Performed $ cp -r /usr/share/doc/cairo-dock-plug-ins-dbus/Dbus/demos/* \ ~/.config/cairo-dock/third-party and restarted cairo-dock. I am able to add these applets to the main dock but they seem to be only half-working for me. Per, e.g., demo_python description: This is a distant applet It simulates a counter: Scroll up/down to increase/decrease the counter, Click/middle-click to increase/decrease the counter by 10 Drop some text to set it as the label. I am observing no such behaviour, just a plain icon of two gears. Is it expected? -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1193986] Review Request: openstack-rally - Benchmark as a service for OpenStack
https://bugzilla.redhat.com/show_bug.cgi?id=1193986 --- Comment #6 from Victoria Martinez de la Cruz vimar...@redhat.com --- (In reply to Pranav Kant from comment #5) I didn't run mock separately. It was done under fedora-review tool You are right... apparently there are some missing requirements. Precisely, python-designateclient. I'll take a look, thanks! -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1186497] Review Request: libtifiles2 - Texas Instruments calculator files library
https://bugzilla.redhat.com/show_bug.cgi?id=1186497
--- Comment #7 from Mario Blättermann mario.blaetterm...@gmail.com ---
Scratch build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=9072904
$ rpmlint -i -v *
libtifiles2.src: I: checking
libtifiles2.src: W: spelling-error %description -l en_US tifiles - ti files,
ti-files, stifles
The value of this tag appears to be misspelled. Please double-check.
libtifiles2.src: W: spelling-error %description -l en_US ungroup - grouping,
group, roundup
The value of this tag appears to be misspelled. Please double-check.
libtifiles2.src: I: checking-url https://sourceforge.net/projects/tilp/
(timeout 10 seconds)
libtifiles2.src: I: checking-url
http://sourceforge.net/projects/tilp/files/tilp2-linux/tilp2-1.17/libtifiles2-1.1.6.tar.bz2
(timeout 10 seconds)
libtifiles2.x86_64: I: checking
libtifiles2.x86_64: W: spelling-error %description -l en_US tifiles - ti
files, ti-files, stifles
The value of this tag appears to be misspelled. Please double-check.
libtifiles2.x86_64: W: spelling-error %description -l en_US ungroup -
grouping, group, roundup
The value of this tag appears to be misspelled. Please double-check.
libtifiles2.x86_64: I: checking-url https://sourceforge.net/projects/tilp/
(timeout 10 seconds)
libtifiles2-debuginfo.x86_64: I: checking
libtifiles2-debuginfo.x86_64: I: checking-url
https://sourceforge.net/projects/tilp/ (timeout 10 seconds)
libtifiles2-devel.x86_64: I: checking
libtifiles2-devel.x86_64: W: spelling-error %description -l en_US libtifiles -
lifestyles
The value of this tag appears to be misspelled. Please double-check.
libtifiles2-devel.x86_64: I: checking-url
https://sourceforge.net/projects/tilp/ (timeout 10 seconds)
libtifiles2-devel.x86_64: W: only-non-binary-in-usr-lib
There are only non binary files in /usr/lib so they should be in /usr/share.
libtifiles2-devel.x86_64: W: no-documentation
The package contains no documentation (README, doc, etc). You have to include
documentation files.
libtifiles2-doc.x86_64: I: checking
libtifiles2-doc.x86_64: I: checking-url https://sourceforge.net/projects/tilp/
(timeout 10 seconds)
libtifiles2.spec: I: checking
libtifiles2.spec: I: checking-url
http://sourceforge.net/projects/tilp/files/tilp2-linux/tilp2-1.17/libtifiles2-1.1.6.tar.bz2
(timeout 10 seconds)
5 packages and 1 specfiles checked; 0 errors, 7 warnings.
Rpmlint is quite silent.
-
key:
[+] OK
[.] OK, not applicable
[X] needs work
-
[+] MUST: rpmlint must be run on the source rpm and all binary rpms the build
produces. The output should be posted in the review.
[+] MUST: The package must be named according to the Package Naming Guidelines.
[+] MUST: The spec file name must match the base package %{name}, in the format
%{name}.spec unless your package has an exemption.
[+] MUST: The package must meet the Packaging Guidelines.
[+] MUST: The package must be licensed with a Fedora approved license and meet
the Licensing Guidelines.
[+] MUST: The License field in the package spec file must match the actual
license.
GPLv2+
[+] MUST: 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 must be included in %license.
[+] MUST: The spec file must be written in American English.
[+] MUST: The spec file for the package MUST be legible.
[+] MUST: The sources used to build the package must match the upstream source,
as provided in the spec URL. Reviewers should use sha256sum for this task as it
is used by the sources file once imported into git. If no upstream URL can be
specified for this package, please see the Source URL Guidelines for how to
deal with this.
$ sha256sum *
ec4d6acd1d135ee04256d04a5e1512c1ccaf82ad01e48f5e24d5c4da31ce898f
libtifiles2-1.1.6.tar.bz2
ec4d6acd1d135ee04256d04a5e1512c1ccaf82ad01e48f5e24d5c4da31ce898f
libtifiles2-1.1.6.tar.bz2.orig
[+] MUST: The package MUST successfully compile and build into binary rpms on
at least one primary architecture.
[.] MUST: If the package does not successfully compile, build or work on an
architecture, then those architectures should be listed in the spec in
ExcludeArch. Each architecture listed in ExcludeArch MUST have a bug filed in
bugzilla, describing the reason that the package does not compile/build/work on
that architecture. The bug number MUST be placed in a comment, next to the
corresponding ExcludeArch line.
[+] MUST: All build dependencies must be listed in BuildRequires, except for
any that are listed in the exceptions section of the Packaging Guidelines ;
inclusion of those as BuildRequires is optional. Apply common sense.
[X] MUST: The spec file MUST handle locales properly. This is done by using the
%find_lang macro. Using %{_datadir}/locale/* is strictly forbidden.
[.] MUST: Every binary RPM package (or subpackage) which stores shared library
files (not just symlinks) in any of
[Bug 1193986] Review Request: openstack-rally - Benchmark as a service for OpenStack
https://bugzilla.redhat.com/show_bug.cgi?id=1193986 Victoria Martinez de la Cruz vimar...@redhat.com changed: What|Removed |Added Depends On||1196366 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1196366 [Bug 1196366] Review Request: openstack-designateclient - Client library for OpenStack Designate DNS API -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1187869] Review Request: kdocker - Dock any application in the system tray
https://bugzilla.redhat.com/show_bug.cgi?id=1187869 --- Comment #8 from Raphael Groner projects...@smart.ms --- Spec URL: https://raphgro.fedorapeople.org/review/qt/kdocker/kdocker.spec SRPM URL: https://raphgro.fedorapeople.org/review/qt/kdocker/kdocker-4.8-2.fc21.src.rpm rawhide scratch: http://koji.fedoraproject.org/koji/taskinfo?taskID=9073658 Mario Rex, thank you both 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1196353] Review Request: main package name here - short summary here
https://bugzilla.redhat.com/show_bug.cgi?id=1196353 Christian Dersch lupi...@mailbox.org changed: What|Removed |Added Flags||fedora-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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1194330] Review Request: kate4 - Advanced Text Editor
https://bugzilla.redhat.com/show_bug.cgi?id=1194330 --- Comment #5 from Rex Dieter rdie...@math.unl.edu --- (and if you want to help finish the review, set the fedora-review flag to ?) -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1187869] Review Request: kdocker - Dock any application in the system tray
https://bugzilla.redhat.com/show_bug.cgi?id=1187869 Raphael Groner projects...@smart.ms changed: What|Removed |Added Status|ASSIGNED|POST Flags||fedora-cvs? --- Comment #10 from Raphael Groner projects...@smart.ms --- New Package SCM Request === Package Name: kdocker Short Description: Dock any application in the system tray Upstream URL: https://launchpad.net/kdocker Owners: raphgro Branches: f21 f22 InitialCC: rdieter -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1179982] Review Request: docsis-config-encoder - Encode a DOCSIS binary configuration file
https://bugzilla.redhat.com/show_bug.cgi?id=1179982 --- Comment #6 from Raphael Groner projects...@smart.ms --- Spec URL: https://raphgro.fedorapeople.org/review/net/docsis/docsis-config-encoder.spec SRPM URL: https://raphgro.fedorapeople.org/review/net/docsis/docsis-config-encoder-0.9.6-3.20150218git.fc21.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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1179982] Review Request: docsis-config-encoder - Encode a DOCSIS binary configuration file
https://bugzilla.redhat.com/show_bug.cgi?id=1179982 --- Comment #8 from Raphael Groner projects...@smart.ms --- Uh… for armv7 we get: RunTests.sh: line 34: 21971 Segmentation fault (core dumped) $DOCSIS -e $TEST.txt $KEYFILE $TEST.cm.new Interesting. -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1196353] New: Review Request: main package name here - short summary here
https://bugzilla.redhat.com/show_bug.cgi?id=1196353 Bug ID: 1196353 Summary: Review Request: main package name here - short summary here Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: d...@der-flo.net QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://flo.fedorapeople.org/itop.spec SRPM URL: https://flo.fedorapeople.org/itop-20150225git6dbb3c42-1.fc21.src.rpm Description: Interrupts 'top-like' utility for Linux Fedora Account System Username: flo Koji-Build: http://koji.fedoraproject.org/koji/taskinfo?taskID=9073430 -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1196353] Review Request: main package name here - short summary here
https://bugzilla.redhat.com/show_bug.cgi?id=1196353 Christian Dersch lupi...@mailbox.org changed: What|Removed |Added Status|NEW |ASSIGNED CC||lupi...@mailbox.org Assignee|nob...@fedoraproject.org|lupi...@mailbox.org --- Comment #1 from Christian Dersch lupi...@mailbox.org --- Taken! Review will follow soon -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1156492] Review Request: golang-github-cpuguy83-go-md2man - Process markdown into manpages
https://bugzilla.redhat.com/show_bug.cgi?id=1156492 --- Comment #17 from Fedora Update System upda...@fedoraproject.org --- golang-github-cpuguy83-go-md2man-1-4.fc20 has been submitted as an update for Fedora 20. https://admin.fedoraproject.org/updates/golang-github-cpuguy83-go-md2man-1-4.fc20 -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1196347] New: Review Request: f22-backgrounds – Fedora 22 default desktop background
https://bugzilla.redhat.com/show_bug.cgi?id=1196347 Bug ID: 1196347 Summary: Review Request: f22-backgrounds – Fedora 22 default desktop background Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: martin.sour...@gmail.com QA Contact: extras...@fedoraproject.org CC: extras...@fedoraproject.org, package-review@lists.fedoraproject.org, pin...@pingoured.fr Depends On: 1133217 +++ This bug was initially created as a clone of Bug #1133217 +++ Spec URL: https://mso.fedorapeople.org/packages/SPECS/f22-backgrounds.spec SRPM URL: https://mso.fedorapeople.org/packages/SRPMS/f22-backgrounds-21.91.0-1.el7.centos.src.rpm Description: This package contains desktop backgrounds for the Fedora 22 default theme. Fedora Account System Username: mso Additional info: Package structurally same as previous default themes, supplemental wallpapers will come later (I already have the images, but they don't need to be in alpha). Tested in XFCE (on CentOS 7, but should work on Fedora as well). Rpmlint output comment: missing Docs OK, as the affected packages are dependant on the f22-backgrounds-base (sub)package that has the Docs, same with dangling symlinks. Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1133217 [Bug 1133217] Review Request: f21-backgrounds - Fedora 21 default desktop background -- 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 https://admin.fedoraproject.org/mailman/listinfo/package-review
