[Bug 1178912] Review Request: cairo-dock-plug-ins - Plug-ins files for Cairo-Dock
https://bugzilla.redhat.com/show_bug.cgi?id=1178912
--- Comment #23 from Mamoru TASAKA mtas...@fedoraproject.org ---
See comment 16, one can install A.i686 B.noarch, C.x86_64 manually using rpm -i
(normal install using default yum option won't do that, dnf - I don't know
well) and this won't work because actually C.x86_64 needs A.i686.
One can write Requires: A%{?_isa} on C explicitly, however this is redundant.
If B is _VERY_ large (like game data), it is preferable to make it noarch,
however this case I see no gain.
--
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 1121601] Review Request: rt - request tracker
https://bugzilla.redhat.com/show_bug.cgi?id=1121601 --- Comment #36 from Alex Vandiver ale...@bestpractical.com --- (In reply to Ralf Corsepius from comment #34) Fedora's perl-HTML-FormatText-WithLinks-AndTable has 2 patches applied: c.f. http://pkgs.fedoraproject.org/cgit/perl-HTML-FormatText-WithLinks-AndTables. git/tree/ Reverting one of these (col_0_fix.patch), lets the t/mail/html-outgoing.t test succeed - Unfortunately, this patch isn't documented at all - No idea what it is trying to address nor about its origins :( It's from https://rt.cpan.org/Public/Bug/Display.html?id=55919#txn-753466 , which is a (IMHO) worse fix than we attempted to supply to the module author in https://rt.cpan.org/Public/Bug/Display.html?id=63555 Reverting col_0_fix.patch in Fedora is likely not the correct fix here -- the patch in perl-HTML-FormatText-WithLinks-AndTables is absolutely fixing a bug, and one that the original CPAN author has failed to address. The RT tests are merely being pessimistic and were expecting that the module would _always_ fail; the Fedora patched version doesn't so do, which isn't really a failure of the Fedora version of the module -- we should be fixing the test to mark those tests as TODO, or not test them at all. For the short term, you can remove lines 85-106 of t/mail/html-outgoing.t in Fedora. I'll ponder what the most right fix is for code. = Something for me to bugzilla. As noted above, I disagree that 1185427 is a bug that Fedora needs to address. A pretty complex html page with embedded js. I am going to attach it. Thanks. The core of it is Failed to connect to database: unable to open database file. Can you think of anything that might prevent the webserver from writing to $RT::VarPath (which I believe is /var/lib/rt under the Fedora layout)? Presumably selinux kicking in? - Alex -- 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 592500] Review Request: libeatmydata - Disable fsync() and friends
https://bugzilla.redhat.com/show_bug.cgi?id=592500 Dave Love d.l...@liverpool.ac.uk changed: What|Removed |Added CC||d.l...@liverpool.ac.uk --- Comment #5 from Dave Love d.l...@liverpool.ac.uk --- For what it's worth, there's a currently-current srpm with several updates at https://loveshack.fedorapeople.org/copr/libeatmydata-105-1.el6.src.rpm and a copr build under http://copr.fedoraproject.org/coprs/loveshack/livhpc -- 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 1185424] New: Review Request: mate-user-guide - User Guide for MATE desktop
https://bugzilla.redhat.com/show_bug.cgi?id=1185424 Bug ID: 1185424 Summary: Review Request: mate-user-guide - User Guide for MATE desktop Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: chat-to...@raveit.de QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://raveit65.fedorapeople.org/Mate/SPECS/mate-user-guide.spec SRPM URL: https://raveit65.fedorapeople.org/Mate/SRPM/mate-user-guide-1.9.0-1.fc22.src.rpm Description: Documentations for MATE desktop Fedora Account System Username: raveit65 -- 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 1183191] Review Request: ocaml-cudf - Common Upgradeability Description Format (CUDF) library
https://bugzilla.redhat.com/show_bug.cgi?id=1183191 Richard W.M. Jones rjo...@redhat.com changed: What|Removed |Added CC||rjo...@redhat.com --- Comment #4 from Richard W.M. Jones rjo...@redhat.com --- You can probably ignore the error ocaml-cudf.x86_64: E: no-binary rpmlint has some problems with OCaml packages, although it has improved over time. The spec file looks reasonable to me. -- 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 --- Comment #22 from Jan Pokorný jpoko...@redhat.com --- Not just %_isa but marking subpackages as noarch properly. IIUIC, it would save some space on the mirrors, etc. so in case there is no strong argument not to make it noarch (which may still be the case and cairo-dock-python2 may qualify) I don't like much. Not a blocker, but would like to know what issues you've hit that are being prevented by not using noarch. -- 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 1179336] Review Request: python-pytest-multihost - Utility for writing multi-host tests for pytest
https://bugzilla.redhat.com/show_bug.cgi?id=1179336 Stephen Gallagher sgall...@redhat.com changed: What|Removed |Added CC||sgall...@redhat.com Assignee|nob...@fedoraproject.org|sgall...@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 1179336] Review Request: python-pytest-multihost - Utility for writing multi-host tests for pytest
https://bugzilla.redhat.com/show_bug.cgi?id=1179336
Stephen Gallagher sgall...@redhat.com changed:
What|Removed |Added
CC||pvikt...@redhat.com
Flags||needinfo?(pviktori@redhat.c
||om)
--- Comment #1 from Stephen Gallagher sgall...@redhat.com ---
Package Review
==
Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed
= Issues
- Use %license instead of %doc for COPYING. This will ensure that it's still
installed, even if someone uses --nodocs with RPM (to maintain legal
requirements. Also, in the future, RPM will treat %license as a hardlink
opportunity to avoid duplicating licenses on the disk.
- Upstream tarball contains tests but they are not run in %check
If they cannot be run in %check, a comment to that effect should be
provided.
= MUST items =
Generic:
[X]: Package is licensed with an open-source compatible license and meets
other legal requirements as defined in the legal section of Packaging
Guidelines.
[X]: License field in the package spec file matches the actual license.
Note: Checking patched sources after %prep for licenses. Licenses found:
Unknown or generated. 20 files have unknown license. Detailed output of
licensecheck in /dev/shm/reviews/1179336-python-pytest-
multihost/licensecheck.txt
[!]: License file installed when any subpackage combination is installed.
[X]: Package contains no bundled libraries without FPC exception.
[X]: Changelog in prescribed format.
[X]: Sources contain only permissible code or content.
[-]: Package contains desktop file if it is a GUI application.
[-]: Development files must be in a -devel package
[X]: Package uses nothing in %doc for runtime.
[X]: Package consistently uses macros (instead of hard-coded directory names).
[X]: Package is named according to the Package Naming Guidelines.
[X]: Package does not generate any conflict.
[X]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
Provides are present.
[X]: Requires correct, justified where necessary.
[X]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[X]: Package is not known to require an ExcludeArch tag.
[-]: Large documentation must go in a -doc subpackage. Large could be size
(~1MB) or number of files.
Note: Documentation size is 102400 bytes in 4 files.
[X]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least one
supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
Note: There are rpmlint messages (see attachment).
[x]: If (and only if) the source package includes the text of the license(s)
in its own file, then that file, containing the text of the license(s)
for the package is included in %doc.
[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.
[x]: Packages must not store files under /srv, /opt or /usr/local
Python:
[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
[x]: Binary eggs must be removed in %prep
= 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).
[-]: Fully versioned
[Bug 1178940] Review Request: libqhyccd - Library to control QHY CCD devices
https://bugzilla.redhat.com/show_bug.cgi?id=1178940 Wolfgang Ulbrich chat-to...@raveit.de changed: What|Removed |Added Status|NEW |ASSIGNED CC||chat-to...@raveit.de Assignee|nob...@fedoraproject.org|chat-to...@raveit.de -- 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 1185424] Review Request: mate-user-guide - User Guide for MATE desktop
https://bugzilla.redhat.com/show_bug.cgi?id=1185424 Christian Dersch lupi...@mailbox.org changed: What|Removed |Added Status|NEW |ASSIGNED -- 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 1168333] Review Request: vagrant-libvirt - Vagrant provider for libvirt
https://bugzilla.redhat.com/show_bug.cgi?id=1168333 --- Comment #11 from Michael Adam ma...@redhat.com --- Change history for this version on top of Vít's is to be found here: https://fedorapeople.org/cgit/obnox/public_git/fedora-vagrant-libvirt.git/ Michael -- 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 1185424] Review Request: mate-user-guide - User Guide for MATE desktop
https://bugzilla.redhat.com/show_bug.cgi?id=1185424
Christian Dersch lupi...@mailbox.org changed:
What|Removed |Added
Flags|fedora-review? |fedora-review+
--- Comment #2 from Christian Dersch lupi...@mailbox.org ---
Package looks nice, I have one suggestion: Starting with Fedora 21 there is a
special %license macro for the license files, see
https://fedoraproject.org/wiki/Packaging:LicensingGuidelines#License_Text and
https://fedorahosted.org/fpc/ticket/411
Solution: APPROVED
Package Review
==
Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed
= MUST items =
Generic:
[x]: Package is licensed with an open-source compatible license and meets
other legal requirements as defined in the legal section of Packaging
Guidelines.
[x]: License field in the package spec file matches the actual license.
Note: Checking patched sources after %prep for licenses. No licenses
found. Please check the source files for licenses manually.
[x]: Package must own all directories that it creates.
Note: Directories without known owners: /usr/share/help,
/usr/share/help/C
Seems to be the same problem as here (false positive):
https://bugzilla.redhat.com/show_bug.cgi?format=multipleid=1043290
[x]: Package does not own files or directories owned by other packages.
Note: Dirs in package are owned also by: /usr/share/help/C/mate-user-
guide/figures(mate-desktop-libs, mate-desktop), /usr/share/help/C/mate-
user-guide(mate-desktop-libs, mate-desktop)
Seems to be the same problem as here (false positive):
https://bugzilla.redhat.com/show_bug.cgi?format=multipleid=1043290
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[-]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[x]: Package consistently uses macros (instead of hard-coded directory names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: Package is not known to require an ExcludeArch tag.
[-]: Large documentation must go in a -doc subpackage. Large could be size
(~1MB) or number of files.
[-]: 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]: 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.
[x]: Package requires other packages for directories it uses.
[x]: All build dependencies are listed in BuildRequires, except for any that
are listed in the exceptions section of Packaging Guidelines.
[x]: Package uses either %{buildroot} or $RPM_BUILD_ROOT
[x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
beginning of %install.
[x]: Macros in Summary, %description expandable at SRPM build time.
[x]: Package contains desktop file if it is a GUI application.
[x]: Package installs a %{name}.desktop using desktop-file-install or desktop-
file-validate if there is such a file.
[x]: 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
[Bug 1185301] Review Request: gnome-builder - IDE for writing GNOME-based software
https://bugzilla.redhat.com/show_bug.cgi?id=1185301 David King amigad...@amigadave.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Fixed In Version||gnome-builder-3.15.4.1-1.fc ||22 Resolution|--- |NEXTRELEASE Last Closed||2015-01-23 12:57:07 --- Comment #13 from David King amigad...@amigadave.com --- Thanks for the reviews! -- 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 1185099] Review Request: opam - A source-based package manager for OCaml
https://bugzilla.redhat.com/show_bug.cgi?id=1185099 --- Comment #3 from Richard W.M. Jones rjo...@redhat.com --- Patch file should be *.patch. The rest of the spec file looks reasonable. -- 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 1168333] Review Request: vagrant-libvirt - Vagrant provider for libvirt
https://bugzilla.redhat.com/show_bug.cgi?id=1168333 Michael Adam ma...@redhat.com changed: What|Removed |Added Assignee|ma...@redhat.com|vondr...@redhat.com Flags||fedora-review+ --- Comment #12 from Michael Adam ma...@redhat.com --- Vít, Setting the review flag and reassigning to you for final review of my changes and further processing. If you don't like something I changed, please bounce back to me. I'd be glad if you would incorporate the changes from my git in the scm if possible. (I was not aware how I can create a repo in the exact format for fedpkg.) Thanks - Michael -- 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 1181193] Review Request: cdsclient - Tools to query databases at CDS
https://bugzilla.redhat.com/show_bug.cgi?id=1181193 Wolfgang Ulbrich chat-to...@raveit.de changed: What|Removed |Added Status|NEW |ASSIGNED CC||chat-to...@raveit.de Assignee|nob...@fedoraproject.org|chat-to...@raveit.de -- 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 1183825] Review Request: ocaml-jsonm - Non-blocking streaming JSON codec for OCaml
https://bugzilla.redhat.com/show_bug.cgi?id=1183825 Richard W.M. Jones rjo...@redhat.com changed: What|Removed |Added Blocks||1185099 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1185099 [Bug 1185099] Review Request: opam - A source-based package manager for OCaml -- 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 1174036] Review Request: ocaml-re - OCaml regular expression library
https://bugzilla.redhat.com/show_bug.cgi?id=1174036 Richard W.M. Jones rjo...@redhat.com changed: What|Removed |Added Blocks||1185099 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1185099 [Bug 1185099] Review Request: opam - A source-based package manager for OCaml -- 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 1185099] Review Request: opam - A source-based package manager for OCaml
https://bugzilla.redhat.com/show_bug.cgi?id=1185099 Richard W.M. Jones rjo...@redhat.com changed: What|Removed |Added CC||rjo...@redhat.com Depends On||1172771, 1174036, 1183191, ||1183195, 1183825, 1183826 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1172771 [Bug 1172771] Review Request: ocaml-cmdliner - OCaml library for dealing with command line arguments https://bugzilla.redhat.com/show_bug.cgi?id=1174036 [Bug 1174036] Review Request: ocaml-re - OCaml regular expression library https://bugzilla.redhat.com/show_bug.cgi?id=1183191 [Bug 1183191] Review Request: ocaml-cudf - Common Upgradeability Description Format (CUDF) library https://bugzilla.redhat.com/show_bug.cgi?id=1183195 [Bug 1183195] Review Request: ocaml-uutf - Non-blocking streaming codec for UTF-8, UTF-16, UTF-16LE and UTF-16BE https://bugzilla.redhat.com/show_bug.cgi?id=1183825 [Bug 1183825] Review Request: ocaml-jsonm - Non-blocking streaming JSON codec for OCaml https://bugzilla.redhat.com/show_bug.cgi?id=1183826 [Bug 1183826] Review Request: ocaml-dose3 - a framework for managing distribution packages and their dependencies -- 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 1183826] Review Request: ocaml-dose3 - a framework for managing distribution packages and their dependencies
https://bugzilla.redhat.com/show_bug.cgi?id=1183826 Richard W.M. Jones rjo...@redhat.com changed: What|Removed |Added Blocks||1185099 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1185099 [Bug 1185099] Review Request: opam - A source-based package manager for OCaml -- 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 1183195] Review Request: ocaml-uutf - Non-blocking streaming codec for UTF-8, UTF-16, UTF-16LE and UTF-16BE
https://bugzilla.redhat.com/show_bug.cgi?id=1183195 Richard W.M. Jones rjo...@redhat.com changed: What|Removed |Added Blocks||1185099 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1185099 [Bug 1185099] Review Request: opam - A source-based package manager for OCaml -- 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 1183191] Review Request: ocaml-cudf - Common Upgradeability Description Format (CUDF) library
https://bugzilla.redhat.com/show_bug.cgi?id=1183191 Richard W.M. Jones rjo...@redhat.com changed: What|Removed |Added Blocks||1185099 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1185099 [Bug 1185099] Review Request: opam - A source-based package manager for OCaml -- 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 1172771] Review Request: ocaml-cmdliner - OCaml library for dealing with command line arguments
https://bugzilla.redhat.com/show_bug.cgi?id=1172771 Richard W.M. Jones rjo...@redhat.com changed: What|Removed |Added Blocks||1185099 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1185099 [Bug 1185099] Review Request: opam - A source-based package manager for OCaml -- 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 1185424] Review Request: mate-user-guide - User Guide for MATE desktop
https://bugzilla.redhat.com/show_bug.cgi?id=1185424 Christian Dersch lupi...@mailbox.org changed: What|Removed |Added CC||lupi...@mailbox.org Assignee|nob...@fedoraproject.org|lupi...@mailbox.org Flags||fedora-review? --- 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 1121601] Review Request: rt - request tracker
https://bugzilla.redhat.com/show_bug.cgi?id=1121601
--- Comment #35 from Ralf Corsepius rc040...@freenet.de ---
Created attachment 983480
-- https://bugzilla.redhat.com/attachment.cgi?id=983480action=edit
$m-content as requested in comment#31
Contents of t/tmp/web-install.t-*/rt.debug.log:
[3022] [Fri Jan 23 17:35:34 2015] [warning]: DBI
connect('dbname=rt4test;host=localhost','urt4test',...) failed: Unknown
database 'rt4test' at /usr/share/perl5/vendor_perl/DBIx/SearchBuilder/Handle.pm
line 105. (/usr/share/perl5/vendor_perl/Carp.pm:168)
[3022] [Fri Jan 23 17:35:35 2015] [warning]: DBI
connect('dbname=rt4test;host=localhost','urt4test',...) failed: Unknown
database 'rt4test' at /usr/share/perl5/vendor_perl/DBIx/SearchBuilder/Handle.pm
line 105. (/usr/share/perl5/vendor_perl/Carp.pm:168)
--
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 1183191] Review Request: ocaml-cudf - Common Upgradeability Description Format (CUDF) library
https://bugzilla.redhat.com/show_bug.cgi?id=1183191 Richard W.M. Jones rjo...@redhat.com changed: What|Removed |Added Blocks||1183826 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1183826 [Bug 1183826] Review Request: ocaml-dose3 - a framework for managing distribution packages and their dependencies -- 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 1183825] Review Request: ocaml-jsonm - Non-blocking streaming JSON codec for OCaml
https://bugzilla.redhat.com/show_bug.cgi?id=1183825 --- Comment #2 from Richard W.M. Jones rjo...@redhat.com --- Defining 'native_compiler' symbol but not using it. [Uutf] .. [] doesn't mean anything to rpm - this is not ocamldoc! It's probably better just to remove the square brackets. The rest looks good to me. -- 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 1183826] Review Request: ocaml-dose3 - a framework for managing distribution packages and their dependencies
https://bugzilla.redhat.com/show_bug.cgi?id=1183826 Richard W.M. Jones rjo...@redhat.com changed: What|Removed |Added CC||rjo...@redhat.com Depends On||1183191 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1183191 [Bug 1183191] Review Request: ocaml-cudf - Common Upgradeability Description Format (CUDF) library -- 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 1121601] Review Request: rt - request tracker
https://bugzilla.redhat.com/show_bug.cgi?id=1121601 --- Comment #34 from Ralf Corsepius rc040...@freenet.de --- (In reply to Alex Vandiver from comment #33) Thanks, for your hints, Alex! The first test failure on 4.2 is due to something _not_ failing as it does everywhere else. That is, RT uses the HTML::FormatText::WithLinks::AndTables module to render HTML to plain text -- which unfortunately fails spectacularly if there are tables involved. Does Fedora have local patches applied to the HTML::FormatText module, or HTML::FormatText::WithLinks::AndTables ? You are right on the spot! Fedora's perl-HTML-FormatText-WithLinks-AndTable has 2 patches applied: c.f. http://pkgs.fedoraproject.org/cgit/perl-HTML-FormatText-WithLinks-AndTables.git/tree/ Reverting one of these (col_0_fix.patch), lets the t/mail/html-outgoing.t test succeed - Unfortunately, this patch isn't documented at all - No idea what it is trying to address nor about its origins :( = Something for me to bugzilla. The second failure (with the web-based installer) is also odd -- it's a failure to create a test SQLite DB. What does adding a die $m-content; on line 81 yield? A pretty complex html page with embedded js. I am going to attach it. -- 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 1121601] Review Request: rt - request tracker
https://bugzilla.redhat.com/show_bug.cgi?id=1121601 Ralf Corsepius rc040...@freenet.de changed: What|Removed |Added Depends On||1185427 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1185427 [Bug 1185427] col_0_fix.patch breaks rt-4.2.x -- 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 1172771] Review Request: ocaml-cmdliner - OCaml library for dealing with command line arguments
https://bugzilla.redhat.com/show_bug.cgi?id=1172771 Richard W.M. Jones rjo...@redhat.com changed: What|Removed |Added CC||rjo...@redhat.com --- Comment #7 from Richard W.M. Jones rjo...@redhat.com --- In general, the spec file looks OK. I can't find any specific problem to point out. -- 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 1183825] Review Request: ocaml-jsonm - Non-blocking streaming JSON codec for OCaml
https://bugzilla.redhat.com/show_bug.cgi?id=1183825 Richard W.M. Jones rjo...@redhat.com changed: What|Removed |Added CC||rjo...@redhat.com Depends On||1183195 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1183195 [Bug 1183195] Review Request: ocaml-uutf - Non-blocking streaming codec for UTF-8, UTF-16, UTF-16LE and UTF-16BE -- 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 1183195] Review Request: ocaml-uutf - Non-blocking streaming codec for UTF-8, UTF-16, UTF-16LE and UTF-16BE
https://bugzilla.redhat.com/show_bug.cgi?id=1183195 Richard W.M. Jones rjo...@redhat.com changed: What|Removed |Added Blocks||1183825 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1183825 [Bug 1183825] Review Request: ocaml-jsonm - Non-blocking streaming JSON codec for OCaml -- 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 1183195] Review Request: ocaml-uutf - Non-blocking streaming codec for UTF-8, UTF-16, UTF-16LE and UTF-16BE
https://bugzilla.redhat.com/show_bug.cgi?id=1183195 Richard W.M. Jones rjo...@redhat.com changed: What|Removed |Added CC||rjo...@redhat.com --- Comment #2 from Richard W.M. Jones rjo...@redhat.com --- Patch file should probably be named 'something.patch'. You're defining 'native_compiler' but not actually using it. -- 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 1174036] Review Request: ocaml-re - OCaml regular expression library
https://bugzilla.redhat.com/show_bug.cgi?id=1174036 Richard W.M. Jones rjo...@redhat.com changed: What|Removed |Added CC||rjo...@redhat.com --- Comment #3 from Richard W.M. Jones rjo...@redhat.com --- Usually better for patches to be '*.patch' since it makes it clearer what the file is when looking at the git repo. You probably don't need a newline after %description. Not sure what rpm does - whether it ignores it or adds whitespace to the rpm header. Rest looks fine to me. -- 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 1183826] Review Request: ocaml-dose3 - a framework for managing distribution packages and their dependencies
https://bugzilla.redhat.com/show_bug.cgi?id=1183826 --- Comment #2 from Richard W.M. Jones rjo...@redhat.com --- Spec file looks OK to me. -- 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 1182261] Review Request: libabigail - Tool for constructing, manipulating, serializing and de-serializing ABI-relevant artifacts
https://bugzilla.redhat.com/show_bug.cgi?id=1182261
--- Comment #15 from Sinny Kumari ksi...@gmail.com ---
Updated spec file and also ran basic checks on generated packages. Updated spec
and rpms are:
SPEC url -
https://github.com/sinnykumari/libabigail-package/raw/master/spec/libabigail.spec
SRPM url -
https://github.com/sinnykumari/libabigail-package/raw/master/srpm/libabigail-1.0-0.2.20150114git63c81f0.fc21.src.rpm
Koji Build - http://koji.fedoraproject.org/koji/taskinfo?taskID=8708846
(In reply to Parag AN(पराग) from comment #9)
Few more fixes required for above updated package.
1) When you prepare updated package its a good practice to do koji scratch
build and if its a successful build then check rpmlint output for all
generated rpm files. Or you can use fedora-review tool on your own updated
package.
Yes, rpmlint ran on generated package
= You need to fix the changelog version-release to
1.0-0.1.git.20150114git63c81f0
Fixed
= Then the wrong-file-end-of-line-encoding warning can be fixed by adding
BuildRequires: dos2unix
and at the end of %install
dos2unix doc/manuals/html/_static/jquery.js
Added dos2unix
3) As per new guidelines
https://fedoraproject.org/wiki/Packaging:LicensingGuidelines#License_Text ,
you need to use %license macro for license file instead of %doc
%license COPYING-LGPLV3
Fixed
4) Follow the texinfo guidelines
http://fedoraproject.org/wiki/Packaging:ScriptletSnippets#Texinfo
You have missing Requires lines and need to use preun (not postun)
Using preun instead of postun.
Not adding
Requires(post): info
Requires(preun): info
because this dependency should be handled by yum.
5) I had a look at source archive and it contains approximately 2.9MB of
.git directory which we don't need. So change the source generation step like
tar -cvzf %%{name}-%%{version}.tar.gz %%{name}-%%{version} --exclude .git
and generate new tarball, use it and submit new package for further review.
Thank you for telling about it. I really didn't notice it. Fixed.
6) Everytime you update the spec file, increase the release number tag (so
next will be 0.2.%{checkout}%{?dist} and add corresponding changes
information in %changelog
Updated Release number tag and %changelog information
(In reply to Richard Shaw from comment #12)
(In reply to Parag AN(पराग) from comment #9)
= You need to fix the changelog version-release to
1.0-0.1.git.20150114git63c81f0
The first git is redundant, just use:
%global checkout %{date}git%{git_revision}
Fixed
Probably easier to use git archvie... at this point. Something like:
# This tarball was constructed from pulling the source code of
# libabigail from its Git repository by doing:
#git clone git://sourceware.org/git/libabigail.git
#pushd libabigail
#git archive --prefix %%{name}-%%{version}/ -o
%%{name}-%%{version}.tar.gz %%{git_revision}
Yes, using it to generate archive and it works absolutely fine :)
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 1181793] Review Request: mp - An open-source library for mathematical programming
https://bugzilla.redhat.com/show_bug.cgi?id=1181793
--- Comment #4 from Paulo Andrade paulo.cesar.pereira.de.andr...@gmail.com ---
Thanks for the comments!
(In reply to Antonio Trande from comment #3)
- There is code licensed with BSD and involved in the compilation.
Change the License to MIT and BSD. Otherwise, it is a free license
https://fedoraproject.org/wiki/Licensing
- It's not necessary to list documentation files in the -devel package
since it depends by the main one.
I prefer to keep it for files that are not really required in -devel, e.g.
one may want to create build chroots passing --excludedocs as an extra install
rpm option
- Please, use %license macro.
Done
- 'environment-modules' is a 'Requires' package.
'mp' installs files in the '/usr/share/Modules' directory.
Corrected. it was incorrectly set as BuildRequires, not Requires.
- There are some unused-direct-shlib-dependency warnings.
It comes from CMakeLists.tst
---8---
# Link with librt for clock_gettime (Linux on i386).
find_library(RT_LIBRARY rt)
if (RT_LIBRARY)
target_link_libraries(mp ${RT_LIBRARY})
endif ()
---8---
But it generates two shared libraries, while it is only required for one
of them:
---8---
$ objdump -d /lib64/libmp.so|grep clock_gettime
00016ab0 clock_gettime@plt:
16ebc: e8 ef fb ff ff callq 16ab0 clock_gettime@plt
---8---
I think this is not a big issue, otherwise, Fedora would have
--as-needed in __global_ldflags, but my experience is that one
just trade a kind of problems for others with that...
Anyway, I added
-DCMAKE_SHARED_LINKER_FLAGS=%{__global_ldflags} -Wl,--as-needed \
to mp.spec, so the rpmlint warning should no longer happen.
- Instructions to package %{_javadir}/jacop/jacop.jar file are missing.
Added information to the spec, I would appreciate a review, did
not ask earlier because it is optional :)
# Waiting for review https://bugzilla.redhat.com/show_bug.cgi?id=1177191
Spec URL: https://pcpa.fedorapeople.org/mp.spec
SRPM URL: https://pcpa.fedorapeople.org/mp-1.3.0-2.fc22.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 1183218] Review Request: perl-Crypt-Random-TESHA2 - Random numbers using timer/schedule entropy
https://bugzilla.redhat.com/show_bug.cgi?id=1183218 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 1121601] Review Request: rt - request tracker
https://bugzilla.redhat.com/show_bug.cgi?id=1121601 --- Comment #38 from Alex Vandiver ale...@bestpractical.com --- (In reply to Jason Tibbitts from comment #37) We could pretty easily mess with the packaging of that perl module if any of this makes a difference. It appears that it's used only by publican (our docbook publication system) so we'd have to talk to them. Maybe someone just needs to fork the module. Unfortunately, all of the perl-based HTML - text converters seem to be poorly-mainatined, and prone to crashing on fairly simple input. As a result, RT 4.2 is moving towards instead adding an optional dependency on HTML::FormatExternal, which shells out to w3m or elinks -- we're not interested in forking, developing, and supporting a text-based HTML rendering engine when there already exist several in the wild (aka browsers). I expect that 4.4 will drop HTML::FormatText::WithTables::AndLinks entirely. And selinux would definitely keep the webserver from writing to an unlabeled location under /var (or anywhere else; the web server is rather strictly confined). Now, when I look in the current F21 policy, I see the following rt-related labels: /var/cache/rt(3|4)(/.*)? all files system_u:object_r:httpd_cache_t:s0 /var/lib/rt(3|4)/data/RT-Shredder(/.*)?all files system_u:object_r:httpd_var_lib_t:s0 Which makes it pretty obvious where the problems lie. Since we're using rt and not rt4 for these directories, none of this matches, and even if it were fixed, the labeling for /var/lib/rt would be a bit too restrictive, I think. The selinux folks are very happy to tweak policy and they usually do it rather quickly. If we could just get a list of everywhere rt is expected to write, it would be pretty easy to get them to patch things up. Alex, would you happen to know that off the top of your head? /var/lib/rt needs to be writable for SQLite; the database is a file named /var/lib/rt/rt4 (assuming that $DatabaseName is set to rt4). Since SQLite is defined to be not for production there's some slack here in how much we care, though. If file-based logging is enabled, writing to /var/log/rt is also necessary. The above rules (fixed for rt not rt4) cover Mason's cache. The shredder directories also need to be writable. I _believe_ that to be sufficient -- in the past we've simply set httpd_sys_rw_content_t on all of /opt/rt4/var, which is a big-ish hammer. - alex -- 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 1183218] Review Request: perl-Crypt-Random-TESHA2 - Random numbers using timer/schedule entropy
https://bugzilla.redhat.com/show_bug.cgi?id=1183218 --- Comment #4 from Paul Howarth p...@city-fan.org --- Review == rpmlint output == perl-Crypt-Random-TESHA2.noarch: W: spelling-error %description -l en_US jitter - hitter, jotter, jitters perl-Crypt-Random-TESHA2.src: W: spelling-error %description -l en_US jitter - hitter, jotter, jitters checklist = - rpmlint OK - package and spec file naming OK - package meets guidelines - license is same as perl, OK for Fedora, spec matches - upstream LICENSE file packaged - spec file written in English and is legible - source matches upstream - package builds OK in mock for Rawhide x86_64 - buildreqs OK - no locales, libraries etc. to worry about - package is not intended to be relocatable - directory ownership and permissions OK - no duplicate files - macro usage is consistent - code, not content - no large docs, docs don't affect runtime - no sub-packages present or warranted - not a GUI app, no desktop file needed - filenames are all ASCII notes = BR: perl(Config) only seems to be used in examples/threads.pl so is probably not needed Consider using %license LICENSE for EPEL-7 and all Fedora releases Package APPROVED. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1183218] Review Request: perl-Crypt-Random-TESHA2 - Random numbers using timer/schedule entropy
https://bugzilla.redhat.com/show_bug.cgi?id=1183218 Paul Howarth p...@city-fan.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 1183218] Review Request: perl-Crypt-Random-TESHA2 - Random numbers using timer/schedule entropy
https://bugzilla.redhat.com/show_bug.cgi?id=1183218 David Dick dd...@cpan.org changed: What|Removed |Added Flags||fedora-cvs? --- Comment #5 from David Dick dd...@cpan.org --- Thanks Paul. I've removed perl(Config). New Package SCM Request === Package Name: perl-Crypt-Random-TESHA2 Short Description: Random numbers using timer/schedule entropy Upstream URL: http://search.cpan.org/dist/Crypt-Random-TESHA2/ Owners: ddick Branches: f20, f21, el6, epel7 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 1121601] Review Request: rt - request tracker
https://bugzilla.redhat.com/show_bug.cgi?id=1121601 --- Comment #37 from Jason Tibbitts ti...@math.uh.edu --- We could pretty easily mess with the packaging of that perl module if any of this makes a difference. It appears that it's used only by publican (our docbook publication system) so we'd have to talk to them. Maybe someone just needs to fork the module. And selinux would definitely keep the webserver from writing to an unlabeled location under /var (or anywhere else; the web server is rather strictly confined). Now, when I look in the current F21 policy, I see the following rt-related labels: /var/cache/rt(3|4)(/.*)? all files system_u:object_r:httpd_cache_t:s0 /var/lib/rt(3|4)/data/RT-Shredder(/.*)?all files system_u:object_r:httpd_var_lib_t:s0 Which makes it pretty obvious where the problems lie. Since we're using rt and not rt4 for these directories, none of this matches, and even if it were fixed, the labeling for /var/lib/rt would be a bit too restrictive, I think. The selinux folks are very happy to tweak policy and they usually do it rather quickly. If we could just get a list of everywhere rt is expected to write, it would be pretty easy to get them to patch things up. Alex, would you happen to know that off the top of your head? -- 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 1181793] Review Request: mp - An open-source library for mathematical programming
https://bugzilla.redhat.com/show_bug.cgi?id=1181793
Antonio Trande anto.tra...@gmail.com changed:
What|Removed |Added
Flags|fedora-review? |fedora-review+
--- Comment #5 from Antonio Trande anto.tra...@gmail.com ---
Package 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.rst in rpm(s)
See:
http://fedoraproject.org/wiki/Packaging/LicensingGuidelines#License_Text
= MUST items =
C/C++:
[x]: Package does not contain kernel modules.
[x]: Package contains no static executables.
[x]: Header files in -devel subpackage, if present.
[x]: ldconfig called in %post and %postun if required.
[x]: Package does not contain any libtool archives (.la)
[x]: Rpath absent or only used for internal libs.
[x]: Development (unversioned) .so files in -devel subpackage, if present.
Generic:
[x]: Package is licensed with an open-source compatible license and meets
other legal requirements as defined in the legal section of Packaging
Guidelines.
[x]: License field in the package spec file matches the actual license.
Note: Checking patched sources after %prep for licenses. Licenses found:
BSD (3 clause), BSD (2 clause), LGPL (v2 or later), GPL (v3 or
later), Unknown or generated. 275 files have unknown license. Detailed
output of licensecheck in /home/sagitter/1181793-mp/licensecheck.txt
[x]: License file installed when any subpackage combination is installed.
[-]: If the package is under multiple licenses, the licensing breakdown must
be documented in the spec.
[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.
[x]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[x]: Package consistently uses macros (instead of hard-coded directory names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: Useful -debuginfo package or justification otherwise.
[x]: Package is not known to require an ExcludeArch tag.
Note: Test run failed
[-]: Large documentation must go in a -doc subpackage. Large could be size
(~1MB) or number of files.
Note: Test run failed
[x]: Packages must not store files under /srv, /opt or /usr/local
Note: Test run failed
[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:
[x]: Uses parallel make %{?_smp_mflags} macro.
[-]: If the source package does not include license text(s) as a separate file
from upstream, the packager SHOULD query upstream to include it.
[x]: Final provides and requires are sane (see attachments).
[ ]: Package functions as described.
[x]:
[Bug 1180698] Review Request: sflphone - SIP/IAX2 compatible enterprise-class software phone
https://bugzilla.redhat.com/show_bug.cgi?id=1180698 Sandro Mani manisan...@gmail.com changed: What|Removed |Added Flags||fedora-cvs? --- Comment #7 from Sandro Mani manisan...@gmail.com --- Oh, never noticed that before, will need to fix it in a number of other packages as well! Thanks for the review! New Package SCM Request === Package Name: sflphone Short Description: SIP/IAX2 compatible enterprise-class software phone Owners: smani Branches: f20 f21 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 1183235] Review Request: kdeedu-data - Shared icons, artwork and data files for educational applications
https://bugzilla.redhat.com/show_bug.cgi?id=1183235
Mario Blättermann mario.blaetterm...@gmail.com changed:
What|Removed |Added
Flags|fedora-review? |fedora-review+
--- Comment #5 from Mario Blättermann mario.blaetterm...@gmail.com ---
Scratch build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=8710097
$ rpmlint -i -v *
kdeedu-data.src: I: checking
kdeedu-data.src: I: checking-url
https://projects.kde.org/projects/kde/kdeedu/kdeedu-data (timeout 10 seconds)
kdeedu-data.src: I: checking-url
http://download.kde.org/stable/applications/14.12.1/src/kdeedu-data-14.12.1.tar.xz
(timeout 10 seconds)
kdeedu-data.noarch: I: checking
kdeedu-data.noarch: I: checking-url
https://projects.kde.org/projects/kde/kdeedu/kdeedu-data (timeout 10 seconds)
kdeedu-data.noarch: W: no-documentation
The package contains no documentation (README, doc, etc). You have to include
documentation files.
kdeedu-data.noarch: W: gzipped-svg-icon
/usr/share/icons/hicolor/scalable/actions/integral_func.svgz
Not all desktop environments that support SVG icons support them gzipped
(.svgz). Install the icon as plain uncompressed SVG.
kdeedu-data.noarch: W: dangling-relative-symlink /usr/share/apps/kvtml
../../kde4/apps/kvtml
The target of the symbolic link does not exist within this package or its file
based dependencies. Verify spelling of the link target and that the target is
included in a package in this package's dependency chain.
kdeedu-data.noarch: W: gzipped-svg-icon
/usr/share/icons/hicolor/scalable/actions/editplots.svgz
Not all desktop environments that support SVG icons support them gzipped
(.svgz). Install the icon as plain uncompressed SVG.
kdeedu-data.noarch: W: gzipped-svg-icon
/usr/share/icons/hicolor/scalable/actions/maximum.svgz
Not all desktop environments that support SVG icons support them gzipped
(.svgz). Install the icon as plain uncompressed SVG.
kdeedu-data.noarch: W: gzipped-svg-icon
/usr/share/icons/hicolor/scalable/actions/minimum.svgz
Not all desktop environments that support SVG icons support them gzipped
(.svgz). Install the icon as plain uncompressed SVG.
kdeedu-data.noarch: W: gzipped-svg-icon
/usr/share/icons/hicolor/scalable/actions/newpolar.svgz
Not all desktop environments that support SVG icons support them gzipped
(.svgz). Install the icon as plain uncompressed SVG.
kdeedu-data.noarch: W: gzipped-svg-icon
/usr/share/icons/hicolor/scalable/actions/newfunction.svgz
Not all desktop environments that support SVG icons support them gzipped
(.svgz). Install the icon as plain uncompressed SVG.
kdeedu-data.noarch: W: gzipped-svg-icon
/usr/share/icons/hicolor/scalable/actions/deriv_func.svgz
Not all desktop environments that support SVG icons support them gzipped
(.svgz). Install the icon as plain uncompressed SVG.
kdeedu-data.noarch: W: gzipped-svg-icon
/usr/share/icons/hicolor/scalable/actions/newparametric.svgz
Not all desktop environments that support SVG icons support them gzipped
(.svgz). Install the icon as plain uncompressed SVG.
kdeedu-data.noarch: W: gzipped-svg-icon
/usr/share/icons/hicolor/scalable/actions/functionhelp.svgz
Not all desktop environments that support SVG icons support them gzipped
(.svgz). Install the icon as plain uncompressed SVG.
kdeedu-data.noarch: W: gzipped-svg-icon
/usr/share/icons/hicolor/scalable/actions/resetview.svgz
Not all desktop environments that support SVG icons support them gzipped
(.svgz). Install the icon as plain uncompressed SVG.
kdeedu-data.noarch: W: gzipped-svg-icon
/usr/share/icons/hicolor/scalable/actions/editconstants.svgz
Not all desktop environments that support SVG icons support them gzipped
(.svgz). Install the icon as plain uncompressed SVG.
kdeedu-data.spec: I: checking
kdeedu-data.spec: I: checking-url
http://download.kde.org/stable/applications/14.12.1/src/kdeedu-data-14.12.1.tar.xz
(timeout 10 seconds)
2 packages and 1 specfiles checked; 0 errors, 13 warnings.
Most warnings refer to the svgz icons. I've thought about again. The installed
icons are intended to be used in kdeedu apps. All of them are in the actions
folders, so they don't affect the display in simple window manager menus. This
would be the only usecase which I can imagine as a problem. Leave the
compressed icons untouched.
-
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
[Bug 1183235] Review Request: kdeedu-data - Shared icons, artwork and data files for educational applications
https://bugzilla.redhat.com/show_bug.cgi?id=1183235 Rex Dieter rdie...@math.unl.edu changed: What|Removed |Added Flags||fedora-cvs? --- Comment #6 from Rex Dieter rdie...@math.unl.edu --- New Package SCM Request === Package Name: kdeedu-data Short Description: Shared icons, artwork and data files for educational applications Upstream URL: https://projects.kde.org/projects/kde/kdeedu/kdeedu-data Owners: group::kde-sig Branches: f20 f21 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 1121601] Review Request: rt - request tracker
https://bugzilla.redhat.com/show_bug.cgi?id=1121601 --- Comment #39 from Jason Tibbitts ti...@math.uh.edu --- Shelling out will spell super fun for selinux, I'm sure. So, basically we need to fix the policy to allow writes to /var/lib/rt and /var/log/rt (which shouldn't be too difficult). I'll bug the selinux folks. -- 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 1177191] Review Request: jacop - Java Constraint Programming solver
https://bugzilla.redhat.com/show_bug.cgi?id=1177191 Antonio Trande anto.tra...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|nob...@fedoraproject.org|anto.tra...@gmail.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 1183218] Review Request: perl-Crypt-Random-TESHA2 - Random numbers using timer/schedule entropy
https://bugzilla.redhat.com/show_bug.cgi?id=1183218 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1183218] Review Request: perl-Crypt-Random-TESHA2 - Random numbers using timer/schedule entropy
https://bugzilla.redhat.com/show_bug.cgi?id=1183218 --- Comment #7 from Fedora Update System upda...@fedoraproject.org --- perl-Crypt-Random-TESHA2-0.01-1.fc21 has been submitted as an update for Fedora 21. https://admin.fedoraproject.org/updates/perl-Crypt-Random-TESHA2-0.01-1.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 1183218] Review Request: perl-Crypt-Random-TESHA2 - Random numbers using timer/schedule entropy
https://bugzilla.redhat.com/show_bug.cgi?id=1183218 --- Comment #8 from Fedora Update System upda...@fedoraproject.org --- perl-Crypt-Random-TESHA2-0.01-1.fc20 has been submitted as an update for Fedora 20. https://admin.fedoraproject.org/updates/perl-Crypt-Random-TESHA2-0.01-1.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 1180698] Review Request: sflphone - SIP/IAX2 compatible enterprise-class software phone
https://bugzilla.redhat.com/show_bug.cgi?id=1180698 Jerry James loganje...@gmail.com changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #6 from Jerry James loganje...@gmail.com --- Hmmm, the -Wl,--as-needed addition stomped on Fedora's relro flags. How about doing this instead? LDFLAGS=$RPM_LD_FLAGS -Wl,--as-needed %configure No need to delay the review for that change, though. Please do that when you import the package into git. This package is APPROVED. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1168333] Review Request: vagrant-libvirt - Vagrant provider for libvirt
https://bugzilla.redhat.com/show_bug.cgi?id=1168333 James (purpleidea) jshu...@redhat.com changed: What|Removed |Added CC||jshu...@redhat.com --- Comment #13 from James (purpleidea) jshu...@redhat.com --- FYI: this RPM seems to be missing the work maxamillion and I did ~7 months ago: https://github.com/maxamillion/vagrant-libvirt-rpm/ In particular it's missing: .bashrc_vagrant vagrant-libvirt.pkla Please include these two changes. They're quite vital for making vagrant on fedora suck less :) -- 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 1141841] Review Request: golang-googlecode-google-api-client - Go libraries for new style Google APIs
https://bugzilla.redhat.com/show_bug.cgi?id=1141841 --- Comment #11 from Fedora Update System upda...@fedoraproject.org --- golang-googlecode-google-api-client-0-0.4.gitfc402b0.fc20 has been submitted as an update for Fedora 20. https://admin.fedoraproject.org/updates/golang-googlecode-google-api-client-0-0.4.gitfc402b0.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 1141841] Review Request: golang-googlecode-google-api-client - Go libraries for new style Google APIs
https://bugzilla.redhat.com/show_bug.cgi?id=1141841 --- Comment #12 from Fedora Update System upda...@fedoraproject.org --- golang-googlecode-google-api-client-0-0.4.gitfc402b0.fc21 has been submitted as an update for Fedora 21. https://admin.fedoraproject.org/updates/golang-googlecode-google-api-client-0-0.4.gitfc402b0.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 1185301] Review Request: gnome-builder - IDE for writing GNOME-based software
https://bugzilla.redhat.com/show_bug.cgi?id=1185301 Yanko Kaneti yan...@declera.com changed: What|Removed |Added CC||yan...@declera.com --- Comment #1 from Yanko Kaneti yan...@declera.com --- gnome-builder.x86_64: E: invalid-appdata-file /usr/share/appdata/org.gnome.Builder.appdata.xml $ appstream-util validate /usr/share/appdata/org.gnome.Builder.appdata.xml /usr/share/appdata/org.gnome.Builder.appdata.xml: FAILED: • style-invalid : Not enough p content before ul minor: gnome-builder.spec:6: W: mixed-use-of-spaces-and-tabs (spaces: line 6, tab: line 1) -- 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 1185301] Review Request: gnome-builder - IDE for writing GNOME-based software
https://bugzilla.redhat.com/show_bug.cgi?id=1185301 --- Comment #2 from David King amigad...@amigadave.com --- Thanks for the review! The AppData guidelines suggest that validate-relax is sufficient (which is included in the spec, and passes locally), although it would of course be nice if there were a few more paragraphs. I fixed the tabs-and-spaces warning and reuploaded the spec and srpm in place. -- 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 1177926] Review Request: smuxi - Powerful, flexible, user-friendly chat client
https://bugzilla.redhat.com/show_bug.cgi?id=1177926 --- Comment #18 from Antonio Trande anto.tra...@gmail.com --- (In reply to Mirco Bauer from comment #17) I explained 2 times why my proposal is the optimal solution. I am not going to invest more time on this. Do whatever is right for Fedora. Disclaimer: I don't know the criteria Fedora uses to split packages, I only do know those practice/rules for Debian. Nonetheless, thank you a lot for caring for Smuxi and packaging it for Fedora! My intention was not underestimate what you say. I modified package according to your advices: http://pkgs.fedoraproject.org/cgit/smuxi.git/tree/smuxi.spec -- 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 1128754] Review Request: mozilla-requestpolicy - Firefox and Seamonkey extension that gives you control over cross-site requests
https://bugzilla.redhat.com/show_bug.cgi?id=1128754
--- Comment #4 from Paulo Andrade paulo.cesar.pereira.de.andr...@gmail.com ---
I would like some comments about these = MUST items =:
1. It says License GPLv3+ and LGPLv3+ MPLv2
I believe it is LGPLv2.1+ (the icons) and not LGPLv3+. Please verify.
I believe the MPLv2 are only the tests, so, MPLv2 should be only sources,
not installed files. Please verify.
[ ]: If the package is under multiple licenses, the licensing breakdown must
be documented in the spec.
The license tag should also have an extra and if keeping as is, that is:
-GPLv3+ and LGPLv3+ MPLv2
+GPLv3+ and LGPLv3+ and MPLv2
2. It is the owner of %{firefox_inst_dir} and %{seamonkey_inst_dir}
This looks wrong. Too bad firefox-filesystem is not the onwer of
%{firefox_inst_dir}, but there are other packages that think they are the
owner.
seamonkey is the owner of %{seamonkey_inst_dir}.
I think it should be worth a bug report for firefox-filesystem for it to become
owner of %{firefox_inst_dir}
[ ]: Package must own all directories that it creates.
Note: Directories without known owners:
/usr/share/mozilla/extensions/{ec8030f7-c20a-464f-9b0e-13a3a9e97384}
[ ]: Package does not own files or directories owned by other packages.
Note: Dirs in package are owned also by: /usr/share/mozilla/extensions
/{92650c4d-4b8e-4d2a-b7eb-24ecf4f6b63a}(mozilla-esteid, seamonkey,
mozilla-https-everywhere)
3. Please correct:
[ ]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
beginning of %install.
Note: rm -rf %{buildroot} present but not required
--
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 1184600] Review Request: python-sep -Astronomical source extraction and photometry in Python
https://bugzilla.redhat.com/show_bug.cgi?id=1184600 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1184600] Review Request: python-sep -Astronomical source extraction and photometry in Python
https://bugzilla.redhat.com/show_bug.cgi?id=1184600 --- Comment #5 from Fedora Update System upda...@fedoraproject.org --- python-sep-0.2.0-1.fc21 has been submitted as an update for Fedora 21. https://admin.fedoraproject.org/updates/python-sep-0.2.0-1.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 1185509] New: Review Request: python-photutils - Astropy affiliated package for image photometry tasks
https://bugzilla.redhat.com/show_bug.cgi?id=1185509 Bug ID: 1185509 Summary: Review Request: python-photutils - Astropy affiliated package for image photometry tasks Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: sergio.pa...@gmail.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: http://guaix.fis.ucm.es/~spr/fedora/python-photutils.spec SRPM URL: http://guaix.fis.ucm.es/~spr/fedora/python-photutils-0.1-1.fc21.src.rpm Description: Photutils contains functions for: * estimating the background and background rms in astronomical images * detecting sources in astronomical images * estimating morphological parameters of those sources (e.g., centroid and shape parameters) * performing aperture and PSF photometry Fedora Account System Username: sergiopr -- 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 1166851] Review Request: python-husl - A Python implementation of HUSL
https://bugzilla.redhat.com/show_bug.cgi?id=1166851 Sergio Pascual sergio.pa...@gmail.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution|--- |NEXTRELEASE Last Closed||2015-01-23 19:45:31 -- 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 1167136] Review Request: python-seaborn - Statistical data visualization in Python
https://bugzilla.redhat.com/show_bug.cgi?id=1167136 Bug 1167136 depends on bug 1166851, which changed state. Bug 1166851 Summary: Review Request: python-husl - A Python implementation of HUSL https://bugzilla.redhat.com/show_bug.cgi?id=1166851 What|Removed |Added Status|ASSIGNED|CLOSED Resolution|--- |NEXTRELEASE -- 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 1185015] Review Request: gap-pkg-browse - GAP browser for 2-dimensional arrays of data
https://bugzilla.redhat.com/show_bug.cgi?id=1185015
--- Comment #2 from Jerry James loganje...@gmail.com ---
(In reply to Sandro Mani from comment #1)
Same comments as in https://bugzilla.redhat.com/show_bug.cgi?id=1185014#c1
(except for license), but all in all ok.
I have made the same fixes to this spec file, namely marking the documentation
directory as %docdir and adding Requires(post) and Requires(postun).
Only additional thing:
[!]: %build honors applicable compiler flags or justifies otherwise.
Consider adding
LDFLAGS=%{__global_ldflags}
to make (despite these currently being the same as the default flags, but
i.e. %cmake etc. also explicitly set these).
Done (although I used $RPM_LD_FLAGS, but it has the same effect).
New URLs:
Spec URL: https://jjames.fedorapeople.org/gap-pkg-browse/gap-pkg-browse.spec
SRPM URL:
https://jjames.fedorapeople.org/gap-pkg-browse/gap-pkg-browse-1.8.6-2.fc22.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 1185509] Review Request: python-photutils - Astropy affiliated package for image photometry tasks
https://bugzilla.redhat.com/show_bug.cgi?id=1185509 Sergio Pascual sergio.pa...@gmail.com changed: What|Removed |Added Blocks||115 (Astronomy-SIG) Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=115 [Bug 115] Astronomy SIG - review tracker -- 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 1159829] Review Request: lxqt-openssh-askpass - Askpass openssh transition dialog for LXQt desktop suite
https://bugzilla.redhat.com/show_bug.cgi?id=1159829 Jonathan Underwood jonathan.underw...@gmail.com changed: What|Removed |Added CC||jonathan.underw...@gmail.co ||m --- Comment #10 from Jonathan Underwood jonathan.underw...@gmail.com --- Shouldn't this be closed? -- 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 1159044] Review Request: lxqt-notificationd - Notification daemon for LXQt desktop suite
https://bugzilla.redhat.com/show_bug.cgi?id=1159044 Jonathan Underwood jonathan.underw...@gmail.com changed: What|Removed |Added CC||jonathan.underw...@gmail.co ||m --- Comment #8 from Jonathan Underwood jonathan.underw...@gmail.com --- Shouldn't this be closed? -- 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 976714] Review Request: python-scikit-learn - Machine learning in Python
https://bugzilla.redhat.com/show_bug.cgi?id=976714 Sergio Pascual sergio.pa...@gmail.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution|--- |NEXTRELEASE Last Closed|2013-07-30 11:50:18 |2015-01-23 20:05:09 -- 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 1185014] Review Request: gap-pkg-atlasrep - GAP interface to the Atlas of Group Representations
https://bugzilla.redhat.com/show_bug.cgi?id=1185014 --- Comment #2 from Jerry James loganje...@gmail.com --- (In reply to Sandro Mani from comment #1) From the review (see below), two points: MUST [?]: Large documentation must go in a -doc subpackage. Large could be size (~1MB) or number of files. == I guess the location of /usr/lib/gap/pkg/atlasrep/doc/ is predictated by gap, so this is probably a non-issue. I have marked /usr/lib/gap/pkg/atlasrep/doc as %docdir to partially address this. SHOULD [!]: If the source package does not include license text(s) as a separate file from upstream, the packager SHOULD query upstream to include it. Okay, I have sent an email to upstream asking them to add the license text in the next release. Looking at the spec, unless I'm mistaken about the purpose of these, I think you need: Requires(pre): gap-core Requires(postun): gap-core You're right! I've added that in all of the gap-pkg-* spec files. Good catch. Not strictly related to this package, but I noticed this when uninstalling: - Running transaction (shutdown inhibited) Erasing: gap-pkg-atlasrep-1.5.0-1.fc22.noarch 1/4 Updating GAP workspace /var/lib/gap/workspace.gz: done. Erasing: GAPDoc-1.5.1-5.fc21.noarch 2/4 Updating GAP workspace /var/lib/gap/workspace.gz: Error, failed to load needed package `gapdoc' (version = 1.2) called from brk true brk done. Oops. That's a bug in the gap and GAPDoc packages. I will push a fix for this to Rawhide shortly. New URLs: Spec URL: https://jjames.fedorapeople.org/gap-pkg-atlasrep/gap-pkg-atlasrep.spec SRPM URL: https://jjames.fedorapeople.org/gap-pkg-atlasrep/gap-pkg-atlasrep-1.5.0-2.fc22.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 1184130] Review Request: rubygem-rails-deprecated_sanitizer - Deprecated sanitizer API extracted from Action View
https://bugzilla.redhat.com/show_bug.cgi?id=1184130 Vít Ondruch vondr...@redhat.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Fixed In Version||rubygem-rails-deprecated_sa ||nitizer-1.0.3-2.fc22 Resolution|--- |RAWHIDE Last Closed||2015-01-23 04:02:44 -- 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 1183235] Review Request: kdeedu-data - Shared icons, artwork and data files for educational applications
https://bugzilla.redhat.com/show_bug.cgi?id=1183235 Mario Blättermann mario.blaetterm...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||mario.blaetterm...@gmail.co ||m Assignee|nob...@fedoraproject.org|mario.blaetterm...@gmail.co ||m Flags||fedora-review? --- Comment #2 from Mario Blättermann mario.blaetterm...@gmail.com --- Rpmlint complains about some things: $ rpmlint -i -v * kdeedu-data.src: I: checking kdeedu-data.src: I: checking-url https://projects.kde.org/projects/kde/kdeedu/kdeedu-data (timeout 10 seconds) kdeedu-data.src: W: invalid-url URL: https://projects.kde.org/projects/kde/kdeedu/kdeedu-data The read operation timed out The value should be a valid, public HTTP, HTTPS, or FTP URL. kdeedu-data.src:29: W: setup-not-quiet Use the -q option to the %setup macro to avoid useless build output from unpacking the sources. kdeedu-data.src: I: checking-url http://download.kde.org/stable/applications/14.12.1/src/kdeedu-data-14.12.1.tar.xz (timeout 10 seconds) kdeedu-data.noarch: I: checking kdeedu-data.noarch: I: checking-url https://projects.kde.org/projects/kde/kdeedu/kdeedu-data (timeout 10 seconds) kdeedu-data.noarch: W: no-documentation The package contains no documentation (README, doc, etc). You have to include documentation files. kdeedu-data.noarch: W: gzipped-svg-icon /usr/share/icons/hicolor/scalable/actions/integral_func.svgz Not all desktop environments that support SVG icons support them gzipped (.svgz). Install the icon as plain uncompressed SVG. kdeedu-data.noarch: W: dangling-relative-symlink /usr/share/apps/kvtml ../../kde4/apps/kvtml The target of the symbolic link does not exist within this package or its file based dependencies. Verify spelling of the link target and that the target is included in a package in this package's dependency chain. kdeedu-data.noarch: W: gzipped-svg-icon /usr/share/icons/hicolor/scalable/actions/editplots.svgz Not all desktop environments that support SVG icons support them gzipped (.svgz). Install the icon as plain uncompressed SVG. kdeedu-data.noarch: W: gzipped-svg-icon /usr/share/icons/hicolor/scalable/actions/maximum.svgz Not all desktop environments that support SVG icons support them gzipped (.svgz). Install the icon as plain uncompressed SVG. kdeedu-data.noarch: W: gzipped-svg-icon /usr/share/icons/hicolor/scalable/actions/minimum.svgz Not all desktop environments that support SVG icons support them gzipped (.svgz). Install the icon as plain uncompressed SVG. kdeedu-data.noarch: W: gzipped-svg-icon /usr/share/icons/hicolor/scalable/actions/newpolar.svgz Not all desktop environments that support SVG icons support them gzipped (.svgz). Install the icon as plain uncompressed SVG. kdeedu-data.noarch: W: gzipped-svg-icon /usr/share/icons/hicolor/scalable/actions/newfunction.svgz Not all desktop environments that support SVG icons support them gzipped (.svgz). Install the icon as plain uncompressed SVG. kdeedu-data.noarch: W: gzipped-svg-icon /usr/share/icons/hicolor/scalable/actions/deriv_func.svgz Not all desktop environments that support SVG icons support them gzipped (.svgz). Install the icon as plain uncompressed SVG. kdeedu-data.noarch: W: gzipped-svg-icon /usr/share/icons/hicolor/scalable/actions/newparametric.svgz Not all desktop environments that support SVG icons support them gzipped (.svgz). Install the icon as plain uncompressed SVG. kdeedu-data.noarch: W: gzipped-svg-icon /usr/share/icons/hicolor/scalable/actions/functionhelp.svgz Not all desktop environments that support SVG icons support them gzipped (.svgz). Install the icon as plain uncompressed SVG. kdeedu-data.noarch: W: gzipped-svg-icon /usr/share/icons/hicolor/scalable/actions/resetview.svgz Not all desktop environments that support SVG icons support them gzipped (.svgz). Install the icon as plain uncompressed SVG. kdeedu-data.noarch: W: gzipped-svg-icon /usr/share/icons/hicolor/scalable/actions/editconstants.svgz Not all desktop environments that support SVG icons support them gzipped (.svgz). Install the icon as plain uncompressed SVG. kdeedu-data.spec: I: checking kdeedu-data.spec:29: W: setup-not-quiet Use the -q option to the %setup macro to avoid useless build output from unpacking the sources. kdeedu-data.spec: I: checking-url http://download.kde.org/stable/applications/14.12.1/src/kdeedu-data-14.12.1.tar.xz (timeout 10 seconds) 2 packages and 1 specfiles checked; 0 errors, 16 warnings. The warning about the not downloadable tarball is false positive. I can download it using wget. Regarding the svgz icons, you should install them uncompressed. I don't know about the problems
[Bug 1182261] Review Request: libabigail - Tool for constructing, manipulating, serializing and de-serializing ABI-relevant artifacts
https://bugzilla.redhat.com/show_bug.cgi?id=1182261 --- Comment #10 from Dodji Seketeli do...@seketeli.org --- (In reply to Parag AN(पराग) from comment #9) 2) I see you actually need libzip not gzip. You can add following BuildRequires: libzip to enable optional feature. Thank you for your attention to details, that is appreciated. However, the optional feature you are talking about is meant *not* to be activated in the current context. As the upstream maintainer of this package, I think it's a good thing to disable it. I even plan to disable the feature by default upstream (even when libzip is installed) in the future. So I think the BuildRequires: libzip should not be added. -- 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 1185281] Review Request: golang-google-golangorg-cloud - Go App Engine for Managed VMs
https://bugzilla.redhat.com/show_bug.cgi?id=1185281 Jan Chaloupka jchal...@redhat.com changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|nob...@fedoraproject.org|l...@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 1163816] Review Request: sonar-runner - Default launcher to analyze a project with SonarQube
https://bugzilla.redhat.com/show_bug.cgi?id=1163816 Michael Simacek msima...@redhat.com changed: What|Removed |Added Status|NEW |CLOSED Resolution|--- |RAWHIDE Last Closed||2015-01-23 07:19:11 -- 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 1177926] Review Request: smuxi - Powerful, flexible, user-friendly chat client
https://bugzilla.redhat.com/show_bug.cgi?id=1177926 --- Comment #17 from Mirco Bauer mmmba...@gnome.org --- I explained 2 times why my proposal is the optimal solution. I am not going to invest more time on this. Do whatever is right for Fedora. Disclaimer: I don't know the criteria Fedora uses to split packages, I only do know those practice/rules for Debian. Nonetheless, thank you a lot for caring for Smuxi and packaging it for Fedora! -- 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 1185281] Review Request: golang-google-golangorg-cloud - Go App Engine for Managed VMs
https://bugzilla.redhat.com/show_bug.cgi?id=1185281 Jan Chaloupka jchal...@redhat.com changed: What|Removed |Added Depends On||1185076, 1185082 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1185076 [Bug 1185076] Review Request: golang-github-golang-protobuf - Go support for Google's protocol buffers https://bugzilla.redhat.com/show_bug.cgi?id=1185082 [Bug 1185082] Review Request: golang-github-golang-appengine - Go App Engine for Managed VMs -- 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 1185082] Review Request: golang-github-golang-appengine - Go App Engine for Managed VMs
https://bugzilla.redhat.com/show_bug.cgi?id=1185082 Jan Chaloupka jchal...@redhat.com changed: What|Removed |Added Blocks||1185281 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1185281 [Bug 1185281] Review Request: golang-google-golangorg-cloud - Go App Engine for Managed VMs -- 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 1185281] New: Review Request: golang-google-golangorg-cloud - Go App Engine for Managed VMs
https://bugzilla.redhat.com/show_bug.cgi?id=1185281 Bug ID: 1185281 Summary: Review Request: golang-google-golangorg-cloud - Go App Engine for Managed VMs 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-google-golangorg-cloud/golang-google-golangorg-cloud.spec SRPM URL: https://jchaloup.fedorapeople.org/reviews/golang-google-golangorg-cloud/golang-google-golangorg-cloud-0-0.1.git2e43671.fc20.src.rpm Description: Go App Engine for Managed VMs Fedora Account System Username: jchaloup $ rpmlint golang-google-golangorg-cloud-0-0.1.git2e43671.fc20.src.rpm golang-google-golangorg-cloud-devel-0-0.1.git2e43671.fc20.noarch.rpm golang-google-golangorg-cloud.spec golang-google-golangorg-cloud.src: W: file-size-mismatch 2e43671e4ad874a7bca65746ff3edb38e6e93762.tar.gz = 85837, https://code.googlesource.com/gocloud/+archive/2e43671e4ad874a7bca65746ff3edb38e6e93762.tar.gz = 86094 2 packages and 1 specfiles checked; 0 errors, 1 warnings. Koji: depends on google.golang.org/appengine (bz#1185082), google.golang.org/api (update) and github.com/golang/protobuf (bz#1185076), which are on review and build update. Locally it builds fine. Upstream: https://code.googlesource.com/gocloud Notes: New schema of import path, the spec file's head is 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 1185076] Review Request: golang-github-golang-protobuf - Go support for Google's protocol buffers
https://bugzilla.redhat.com/show_bug.cgi?id=1185076 Jan Chaloupka jchal...@redhat.com changed: What|Removed |Added Blocks||1185281 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1185281 [Bug 1185281] Review Request: golang-google-golangorg-cloud - Go App Engine for Managed VMs -- 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 1185076] Review Request: golang-github-golang-protobuf - Go support for Google's protocol buffers
https://bugzilla.redhat.com/show_bug.cgi?id=1185076 Jan Chaloupka jchal...@redhat.com changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|nob...@fedoraproject.org|l...@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 1185082] Review Request: golang-github-golang-appengine - Go App Engine for Managed VMs
https://bugzilla.redhat.com/show_bug.cgi?id=1185082 Jan Chaloupka jchal...@redhat.com changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|nob...@fedoraproject.org|l...@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 1163739] Review Request: gradle - Build automation tool
https://bugzilla.redhat.com/show_bug.cgi?id=1163739 Bug 1163739 depends on bug 1163816, which changed state. Bug 1163816 Summary: Review Request: sonar-runner - Default launcher to analyze a project with SonarQube https://bugzilla.redhat.com/show_bug.cgi?id=1163816 What|Removed |Added Status|NEW |CLOSED Resolution|--- |RAWHIDE -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1158188] Review Request: sonar-plugins-parent - Sonar Plugins Parent POM
https://bugzilla.redhat.com/show_bug.cgi?id=1158188 Michael Simacek msima...@redhat.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution|--- |RAWHIDE Last Closed||2015-01-23 07:18:15 -- 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 1185301] New: Review Request: gnome-builder - IDE for writing GNOME-based software
https://bugzilla.redhat.com/show_bug.cgi?id=1185301 Bug ID: 1185301 Summary: Review Request: gnome-builder - IDE for writing GNOME-based software Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: amigad...@amigadave.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://amigadave.fedorapeople.org/gnome-builder.spec SRPM URL: https://amigadave.fedorapeople.org/gnome-builder-3.15.4.1-1.fc22.src.rpm Description: Builder attempts to be an IDE for writing software for GNOME. It does not try to be a generic IDE, but one specialized for writing GNOME software. Fedora Account System Username: amigadave -- 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 1181050] Review Request: python-XStatic-smart-table - smart-table (XStatic packaging standard)
https://bugzilla.redhat.com/show_bug.cgi?id=1181050 Matthias Runge mru...@redhat.com changed: What|Removed |Added Status|ASSIGNED|POST Flags||fedora-cvs? --- Comment #2 from Matthias Runge mru...@redhat.com --- Thanks again, Haikel and thank you for your kind offer. It's nothing urgent, and I'm not trying to put additional work on an already overloaded person! New Package SCM Request === Package Name: python-XStatic-smart-table Short Description: smart-table (XStatic packaging standard) Upstream URL: https://github.com/lorenzofox3/Smart-Table Owners: mrunge Branches: epel7 f21 -- 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 1185275] New: Review Request: rome-utils - Utility classes for ROME projects
https://bugzilla.redhat.com/show_bug.cgi?id=1185275 Bug ID: 1185275 Summary: Review Request: rome-utils - Utility classes for ROME projects Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: punto...@libero.it QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://gil.fedorapeople.org/rome-utils.spec SRPM URL: https://gil.fedorapeople.org/rome-utils-1.5.0-1.fc20.src.rpm Description: Utility classes for ROME projects Fedora Account System Username: gil -- 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 1060436] Review Request: opam - A Package Manger for Ocaml
https://bugzilla.redhat.com/show_bug.cgi?id=1060436 --- Comment #7 from Jon Ludlam j...@recoil.org --- I've made a review request for OPAM without the bundled dependencies here: https://bugzilla.redhat.com/show_bug.cgi?id=1185099 It depends upon a few other packages, linked to from that ticket. -- 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 1181050] Review Request: python-XStatic-smart-table - smart-table (XStatic packaging standard)
https://bugzilla.redhat.com/show_bug.cgi?id=1181050 --- Comment #3 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 1184600] Review Request: python-sep -Astronomical source extraction and photometry in Python
https://bugzilla.redhat.com/show_bug.cgi?id=1184600 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 1181050] Review Request: python-XStatic-smart-table - smart-table (XStatic packaging standard)
https://bugzilla.redhat.com/show_bug.cgi?id=1181050 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 1172414] Review Request: ltp - Linux Test Project
https://bugzilla.redhat.com/show_bug.cgi?id=1172414 Mamoru TASAKA mtas...@fedoraproject.org changed: What|Removed |Added CC||ktdre...@ktdreyer.com Flags||needinfo?(ktdreyer@ktdreyer ||.com) --- Comment #3 from Mamoru TASAKA mtas...@fedoraproject.org --- Would you update the status? -- 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 1184920] Review Request: rubygem-rails-dom-testing - Compares doms and assert certain elements exists in doms using Nokogiri
https://bugzilla.redhat.com/show_bug.cgi?id=1184920 Bug 1184920 depends on bug 1184130, which changed state. Bug 1184130 Summary: Review Request: rubygem-rails-deprecated_sanitizer - Deprecated sanitizer API extracted from Action View https://bugzilla.redhat.com/show_bug.cgi?id=1184130 What|Removed |Added Status|ASSIGNED|CLOSED Resolution|--- |RAWHIDE -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1182261] Review Request: libabigail - Tool for constructing, manipulating, serializing and de-serializing ABI-relevant artifacts
https://bugzilla.redhat.com/show_bug.cgi?id=1182261
--- Comment #12 from Richard Shaw hobbes1...@gmail.com ---
(In reply to Parag AN(पराग) from comment #9)
= You need to fix the changelog version-release to
1.0-0.1.git.20150114git63c81f0
The first git is redundant, just use:
%global checkout %{date}git%{git_revision}
5) I had a look at source archive and it contains approximately 2.9MB of
.git directory which we don't need. So change the source generation step like
tar -cvzf %%{name}-%%{version}.tar.gz %%{name}-%%{version} --exclude .git
and generate new tarball, use it and submit new package for further review.
Probably easier to use git archvie... at this point. Something like:
# This tarball was constructed from pulling the source code of
# libabigail from its Git repository by doing:
#git clone git://sourceware.org/git/libabigail.git
#pushd libabigail
#git archive --prefix %%{name}-%%{version}/ -o %%{name}-%%{version}.tar.gz
%%{git_revision}
--
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 1177926] Review Request: smuxi - Powerful, flexible, user-friendly chat client
https://bugzilla.redhat.com/show_bug.cgi?id=1177926 --- Comment #19 from Fedora Update System upda...@fedoraproject.org --- smuxi-0.11-3.fc20 has been submitted as an update for Fedora 20. https://admin.fedoraproject.org/updates/smuxi-0.11-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 1177926] Review Request: smuxi - Powerful, flexible, user-friendly chat client
https://bugzilla.redhat.com/show_bug.cgi?id=1177926 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org 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
--- Comment #13 from Jan Pokorný jpoko...@redhat.com ---
As for cairo-dock-plug-ins-common (only license files for now, which is
OK), I think it should rather be norach (and, consequently, internal
Requires without %{?_isa}).
--
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 1128754] Review Request: mozilla-requestpolicy - Firefox and Seamonkey extension that gives you control over cross-site requests
https://bugzilla.redhat.com/show_bug.cgi?id=1128754
--- Comment #5 from Antonio Trande anto.tra...@gmail.com ---
(In reply to Paulo Andrade from comment #4)
2. It is the owner of %{firefox_inst_dir} and %{seamonkey_inst_dir}
This looks wrong. Too bad firefox-filesystem is not the onwer of
%{firefox_inst_dir}, but there are other packages that think they are the
owner.
seamonkey is the owner of %{seamonkey_inst_dir}.
I think it should be worth a bug report for firefox-filesystem for it to
become owner of %{firefox_inst_dir}
[ ]: Package must own all directories that it creates.
Note: Directories without known owners:
/usr/share/mozilla/extensions/{ec8030f7-c20a-464f-9b0e-13a3a9e97384}
[ ]: Package does not own files or directories owned by other packages.
Note: Dirs in package are owned also by: /usr/share/mozilla/extensions
/{92650c4d-4b8e-4d2a-b7eb-24ecf4f6b63a}(mozilla-esteid, seamonkey,
mozilla-https-everywhere)
%{firefox_inst_dir}=%{moz_extensions}/%{firefox_app_id}/requestpol...@requestpolicy.com
must be owned by this package.
%{moz_extensions}/%{firefox_app_id} is co-owned with 'firefox'.
%{moz_extensions} is owned by 'mozilla-filesystem' that is required by
'firefox'.
Is it not right?
--
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
