[Bug 860542] New: Review Request: kchildlock - KDE Parental Control Application

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=860542

Bug ID: 860542
QA Contact: extras...@fedoraproject.org
  Severity: medium
   Version: rawhide
  Priority: medium
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
  Assignee: nob...@fedoraproject.org
   Summary: Review Request: kchildlock - KDE Parental Control
Application
Regression: ---
  Story Points: ---
Classification: Fedora
OS: Linux
  Reporter: nlmin...@gmail.com
  Type: ---
 Documentation: ---
  Hardware: All
Mount Type: ---
Status: NEW
 Component: Package Review
   Product: Fedora

Spec URL:
https://raw.github.com/Ignotus/fedora-packages/be100d9cb3308f89ac5d7e85c3b2daba23279314/kchildlock/kchildlock.spec
SRPM URL:
https://dl.dropbox.com/s/p81m5zidqm4i4hd/kchildlock-0.90.4.2-1.fc17.src.rpm?dl=1
Description: KDE Parental Control Application
Fedora Account System Username: ignotusp

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 860542] Review Request: kchildlock - KDE Parental Control Application

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=860542

Minh Ngo nlmin...@gmail.com changed:

   What|Removed |Added

 Blocks||656997 (kde-reviews)
  Alias||kchildlock

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 857180] Review Request: gnome-clocks - Clock application designed for GNOME 3

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857180

--- Comment #6 from Yanko Kaneti yan...@declera.com ---
- Latest from upstream - 0.1.2

Spec URL: http://declera.com/~yaneti/gnome-clocks/gnome-clocks.spec
SRPM URL:
http://declera.com/~yaneti/gnome-clocks/gnome-clocks-0.1.2-1.fc19.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 860146] Review Request: vsqlite++ - A C++ Wrapper for SQLite

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=860146

--- Comment #8 from Vinzenz Feenstra eviliss...@gmail.com ---
SPEC:
https://raw.github.com/vinzenz/fedora-packages/vsqlite++/0.3.9-2/vsqlite++/vsqlite++.spec
SRPM:
https://github.com/downloads/vinzenz/fedora-packages/vsqlite++-0.3.9-2.fc17.src.rpm

ChangeLog:
* Wed Sep 26 2012 Vinzenz Feenstra eviliss...@gmail.com - 0.3.9-2
- Removed comment with macro - Not needed anymore (Fixed rpmlint warning)

Koji build results for vsqlite++-0.3.9-2.fc17.src.rpm:

F17: http://koji.fedoraproject.org/koji/taskinfo?taskID=4527693

F18: http://koji.fedoraproject.org/koji/taskinfo?taskID=4527692

Rawhide: http://koji.fedoraproject.org/koji/taskinfo?taskID=4527694

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 857487] Review Request: ht-alegreya-fonts - A Serif typeface originally intended for literature

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857487

Daiki Ueno du...@redhat.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||du...@redhat.com
   Assignee|nob...@fedoraproject.org|du...@redhat.com
  Flags||fedora-review+

--- Comment #1 from Daiki Ueno du...@redhat.com ---
Here is the review.

Package Review
==

Key:
[x] = Pass
[!] = Fail
[-] = Not applicable
[?] = Not evaluated
[ ] = Manual review needed


Issues:
===
[x]: License field in the package spec file matches the actual license.

= 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]: Package successfully compiles and builds into binary rpms on at least one
 supported primary architecture.
[-]: %build honors applicable compiler flags or justifies otherwise.
[x]: All build dependencies are listed in BuildRequires, except for any that
 are listed in the exceptions section of Packaging Guidelines.
[x]: Package contains no bundled libraries.
[x]: Changelog in prescribed format.
[x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
[x]: Sources contain only permissible code or content.
[x]: Each %files section contains %defattr if rpm  4.4
[x]: Macros in Summary, %description expandable at SRPM build time.
[-]: Package contains desktop file if it is a GUI application.
[-]: Development files must be in a -devel package
[x]: Package requires other packages for directories it uses.
[x]: Package uses nothing in %doc for runtime.
[x]: Package is not known to require ExcludeArch.
[x]: Package does not contain duplicates in %files.
[x]: Permissions on files are set properly.
[-]: Fully versioned dependency in subpackages, if present.
[x]: Package complies to the Packaging Guidelines
[x]: Spec file lacks Packager, Vendor, PreReq tags.
[-]: Large documentation files are in a -doc subpackage, if required.
[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]: License field in the package spec file matches the actual license.
[x]: License file installed when any subpackage combination is installed.
[x]: Package consistently uses macro is (instead of hard-coded directory
 names).
[x]: Package is named using only allowed ASCII characters.
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package do not use a name that already exist
[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]: Package must own all directories that it creates.
[x]: Package does not own files or directories owned by other packages.
[x]: Package installs properly.
[-]: Package is not relocatable.
[x]: Requires correct, justified where necessary.
[x]: CheckResultdir
[x]: Rpmlint is run on all rpms the build produces.
 Note: There are rpmlint messages (see attachment).
[x]: Sources used to build the package match the upstream source, as provided
 in the spec URL.
[x]: Spec file is legible and written in American English.
[x]: Spec file name must match the spec package %{name}, in the format
 %{name}.spec.
[-]: Package contains systemd file(s) if in need.
[x]: File names are valid UTF-8.

= SHOULD items =

Generic:
[x]: Reviewer should test that the package builds in mock.
[x]: Buildroot is not present
[x]: Package has no %clean section with rm -rf %{buildroot} (or
 $RPM_BUILD_ROOT)
[-]: 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]: Dist tag is present.
[x]: No file requires outside of /etc, /bin, /sbin, /usr/bin, /usr/sbin.
[x]: Final provides and requires are sane (rpm -q --provides and rpm -q
 --requires).
[x]: Package functions as described.
[x]: Latest version is packaged.
[x]: Package does not include license text files separate from upstream.
[x]: The placement of pkgconfig(.pc) files are correct.
[x]: SourceX tarball generation or download is documented.
[-]: SourceX / PatchY prefixed with %{name}.
 Note: Source2 (ht-alegreyaSC-fonts-fontconfig.conf) Source0
 (Alegreya.zip)
[x]: SourceX is a working URL.
[-]: Description and summary sections in the package spec file contains
 translations for supported Non-English languages, if available.
[x]: Package should compile and build into binary rpms on all supported
 architectures.
[-]: %check is present and all 

[Bug 858841] Review Request: owncloud - Private file sync and share server

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=858841

Matthias Runge mru...@redhat.com changed:

   What|Removed |Added

 CC||mru...@redhat.com

--- Comment #3 from Matthias Runge mru...@redhat.com ---
just a few drive-by comments:
- is that main package functional without any of those database sub-packages? 
- you also need to require rsyslog (for dir /etc/rsyslog.d), 
  note: there's also syslog-ng, you're fine to uninstall rsyslog from your
system
- same for that logrotate-snippet
- the svg-editor is also LGPLv3 and MIT
https://github.com/owncloud/apps/blob/master/files_svgedit/README

http://fedoraproject.org/wiki/Licensing#Good_Licenses lists, LGPLv3 is only
sort of compatible with AGPL.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 857992] Review Request: JQuery - Fast, concise library that simplifies how you use JavaScript

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857992

--- Comment #5 from Matthias Runge mru...@redhat.com ---
jquery: latest version is (currently) 1.8.2. , 1.8.1 was released on Aug 30, so
you'l need to update this package.

In my experience, esp. web developers are pretty ignorant in bundling/copying
external code. And also, most of them don't care for backwards compatibility.

For example: Django developers: they said about the request to unbundle: Not
having external dependencies is a design choice of Django.
https://code.djangoproject.com/ticket/17982
Django still uses jquery 1.4.2, and I don't see, this will change in the near
future.

jquery's release notes list some changes affecting backwards compatibility:
http://blog.jquery.com/2012/08/09/jquery-1-8-released/

When breaking other's packages by upgrading to newer versions, you're also
requested to fix those issues. I'm sure, I don't want to do that.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 856217] Review Request: quearcode - A tool for creating QR Codes

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856217

--- Comment #18 from Fedora Update System upda...@fedoraproject.org ---
quearcode-0.2-2.fc17 has been pushed to the Fedora 17 stable repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 860581] New: Review Request: dmlite-plugins-memcache - Memcached plug-in for dmlite

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=860581

Bug ID: 860581
QA Contact: extras...@fedoraproject.org
  Severity: medium
   Version: rawhide
  Priority: medium
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
  Assignee: nob...@fedoraproject.org
   Summary: Review Request: dmlite-plugins-memcache - Memcached
plug-in for dmlite
Regression: ---
  Story Points: ---
Classification: Fedora
OS: Linux
  Reporter: rocha.po...@gmail.com
  Type: ---
 Documentation: ---
  Hardware: All
Mount Type: ---
Status: NEW
 Component: Package Review
   Product: Fedora

Spec URL: http://rocha.web.cern.ch/rocha/fedora/dmlite-plugins-memcache.spec
SRPM URL:
http://rocha.web.cern.ch/rocha/fedora/dmlite-plugins-memcache-0.4.0-1.src.rpm
Description:
This package provides the memcached plug-in for dmlite. It provides a
memcached based implementation of the NS interface.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 855656] Review Request: perl-Safe-Isa - Call isa, can, does and DOES safely on things that may not be objects

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855656

--- Comment #9 from Fedora Update System upda...@fedoraproject.org ---
perl-Safe-Isa-1.02-1.fc16 has been pushed to the Fedora 16 stable
repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 854837] inkscape-sozi - Inkscape extension for creating animated presentations

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=854837

--- Comment #24 from Fedora Update System upda...@fedoraproject.org ---
inkscape-sozi-12.05-7.fc17 has been pushed to the Fedora 17 stable repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 853463] Review Request: php-redis - Extension for communicating with the Redis key-value store

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=853463

--- Comment #18 from Fedora Update System upda...@fedoraproject.org ---
php-redis-2.2.2-5.git6f7087f.fc17 has been pushed to the Fedora 17 stable
repository.  If problems still persist, please make note of it in this bug
report.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 851702] Review Request: python-acoustid - Python bindings for Chromaprint acoustic fingerprinting and the Acoustid API

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=851702

--- Comment #11 from Fedora Update System upda...@fedoraproject.org ---
python-acoustid-0.7-3.fc16 has been pushed to the Fedora 16 stable repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 855656] Review Request: perl-Safe-Isa - Call isa, can, does and DOES safely on things that may not be objects

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855656

--- Comment #10 from Fedora Update System upda...@fedoraproject.org ---
perl-Safe-Isa-1.02-1.fc17 has been pushed to the Fedora 17 stable
repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 845934] Review Request: wt - C++ library for developing web applications

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=845934

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Resolution|NEXTRELEASE |ERRATA

--- Comment #42 from Fedora Update System upda...@fedoraproject.org ---
wt-3.2.2-6.p1.fc16 has been pushed to the Fedora 16 stable repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 856217] Review Request: quearcode - A tool for creating QR Codes

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856217

--- Comment #19 from Fedora Update System upda...@fedoraproject.org ---
quearcode-0.2-2.fc16 has been pushed to the Fedora 16 stable repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 854691] Review Request: datovka - A free graphical interface for Czech Databox (Datové schránky)

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=854691

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #9 from Fedora Update System upda...@fedoraproject.org ---
python-sudsds-1.0-2.fc17, python-dslib-2.1.2-2.fc17, datovka-2.1.2-2.fc17 has
been pushed to the Fedora 17 testing repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 854690] Review Request: python-dslib - A Python library for accessing Czech Databox (Datové schránky)

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=854690

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #8 from Fedora Update System upda...@fedoraproject.org ---
python-sudsds-1.0-2.fc17, python-dslib-2.1.2-2.fc17, datovka-2.1.2-2.fc17 has
been pushed to the Fedora 17 testing repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 854679] Review Request: python-sudsds - A python SOAP client modified for usage with Czech Databox

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=854679

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #10 from Fedora Update System upda...@fedoraproject.org ---
python-sudsds-1.0-2.fc17, python-dslib-2.1.2-2.fc17, datovka-2.1.2-2.fc17 has
been pushed to the Fedora 17 testing repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 851734] Review Request: perl-Regexp-Grammars - Add grammatical parsing features to perl regular expressions

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=851734

--- Comment #13 from Fedora Update System upda...@fedoraproject.org ---
perl-Regexp-Grammars-1.021-4.fc17 has been pushed to the Fedora 17 stable
repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 847182] Review Request: nhn-nanum-gothic-light-fonts - Nanum Gothic Light family of Korean TrueType fonts

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=847182

--- Comment #9 from Fedora Update System upda...@fedoraproject.org ---
nhn-nanum-gothic-light-fonts-1.000-2.fc17 has been pushed to the Fedora 17
stable repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 845221] Review Request: ilbc - Internet Low Bitrate Codec

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=845221

--- Comment #22 from Fedora Update System upda...@fedoraproject.org ---
ilbc-1.1.1-3.fc17 has been pushed to the Fedora 17 stable repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 855665] Review Request: perl-Data-Validate-Type - Public interface to Params::Util offering data type validation functions

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=855665

--- Comment #8 from Fedora Update System upda...@fedoraproject.org ---
perl-Data-Validate-Type-1.3.0-2.fc17 has been pushed to the Fedora 17 stable
repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 854333] python-transifex

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=854333

--- Comment #25 from Fedora Update System upda...@fedoraproject.org ---
python-transifex-0.1.6-2.fc17 has been pushed to the Fedora 17 stable
repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 851702] Review Request: python-acoustid - Python bindings for Chromaprint acoustic fingerprinting and the Acoustid API

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=851702

--- Comment #12 from Fedora Update System upda...@fedoraproject.org ---
python-acoustid-0.7-3.fc17 has been pushed to the Fedora 17 stable repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 845934] Review Request: wt - C++ library for developing web applications

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=845934

--- Comment #43 from Fedora Update System upda...@fedoraproject.org ---
wt-3.2.2-6.p1.fc17 has been pushed to the Fedora 17 stable repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 847182] Review Request: nhn-nanum-gothic-light-fonts - Nanum Gothic Light family of Korean TrueType fonts

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=847182

--- Comment #10 from Fedora Update System upda...@fedoraproject.org ---
nhn-nanum-gothic-light-fonts-1.000-2.fc16 has been pushed to the Fedora 16
stable repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 860581] Review Request: dmlite-plugins-memcache - Memcached plug-in for dmlite

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=860581

--- Comment #1 from Ricardo Rocha rocha.po...@gmail.com ---
Koji builds (successful):

http://koji.fedoraproject.org/koji/taskinfo?taskID=4527957 (dist-5E-epel)
http://koji.fedoraproject.org/koji/taskinfo?taskID=4527929 (dist-6E-epel)
http://koji.fedoraproject.org/koji/taskinfo?taskID=4527934 (f17)
http://koji.fedoraproject.org/koji/taskinfo?taskID=4527937 (f18)
http://koji.fedoraproject.org/koji/taskinfo?taskID=4527940 (rawhide)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 860600] New: Review Request: dmlite-plugins-mysql - MySQL plug-in for dmlite

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=860600

Bug ID: 860600
QA Contact: extras...@fedoraproject.org
  Severity: medium
   Version: rawhide
  Priority: medium
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
  Assignee: nob...@fedoraproject.org
   Summary: Review Request: dmlite-plugins-mysql - MySQL plug-in
for dmlite
Regression: ---
  Story Points: ---
Classification: Fedora
OS: Linux
  Reporter: rocha.po...@gmail.com
  Type: ---
 Documentation: ---
  Hardware: All
Mount Type: ---
Status: NEW
 Component: Package Review
   Product: Fedora

Spec URL: http://rocha.web.cern.ch/rocha/fedora/dmlite-plugins-mysql.spec
SRPM URL:
http://rocha.web.cern.ch/rocha/fedora/dmlite-plugins-mysql-0.4.1-1.src.rpm
Description: 
This package provides the MySQL plug-in for dmlite.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 740660] Review Request: python-d2to1 - Allows using distutils2-like setup.cfg files with setup.py

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=740660

--- Comment #3 from Sergio Pascual sergio.pa...@gmail.com ---
Ok, thanks for the review. I have removed the egg and the defattr. I have also
updated to new upstream 0.2.7

Spec URL: http://guaix.fis.ucm.es/~spr/python-d2to1.spec
SRPM URL: http://guaix.fis.ucm.es/~spr/python-d2to1-0.2.7-1.fc17.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 860600] Review Request: dmlite-plugins-mysql - MySQL plug-in for dmlite

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=860600

--- Comment #1 from Ricardo Rocha rocha.po...@gmail.com ---
Koji builds (successful):

http://koji.fedoraproject.org/koji/taskinfo?taskID=4527964 (dist-5E-epel)
http://koji.fedoraproject.org/koji/taskinfo?taskID=4527974 (dist-6E-epel)
http://koji.fedoraproject.org/koji/taskinfo?taskID=4527996 (f17)
http://koji.fedoraproject.org/koji/taskinfo?taskID=4527999 (f18)
http://koji.fedoraproject.org/koji/taskinfo?taskID=4528002 (rawhide)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 860607] New: Review Request: dmlite-plugins-adapter - Adapter plug-in for dmlite

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=860607

Bug ID: 860607
QA Contact: extras...@fedoraproject.org
  Severity: medium
   Version: rawhide
  Priority: medium
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
  Assignee: nob...@fedoraproject.org
   Summary: Review Request: dmlite-plugins-adapter - Adapter
plug-in for dmlite
Regression: ---
  Story Points: ---
Classification: Fedora
OS: Linux
  Reporter: rocha.po...@gmail.com
  Type: ---
 Documentation: ---
  Hardware: All
Mount Type: ---
Status: NEW
 Component: Package Review
   Product: Fedora

Spec URL: http://rocha.web.cern.ch/rocha/fedora/dmlite-plugins-adapter.spec
SRPM URL:
http://rocha.web.cern.ch/rocha/fedora/dmlite-plugins-adapter-0.4.0-1.src.rpm
Description:
This package provides the adapter plug-in for dmlite. This plug-in provides
both
a name-space and pool management implementation which fallback to forwarding
calls to the old DPNS and DPM daemons.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 853692] Review Request: mate-settings-daemon - MATE Desktop settings daemon

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=853692

--- Comment #19 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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 856114] Review Request: qxmpp - Qt XMPP Libraries

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856114

--- Comment #7 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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 842379] Review Request: mcollective-qpid-plugin - Plugin to enable m-collective communication over amqp 1.0 enabled broker

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=842379

Steve Traylen steve.tray...@cern.ch changed:

   What|Removed |Added

 CC||steve.tray...@cern.ch

--- Comment #15 from Steve Traylen steve.tray...@cern.ch ---

Re RHEL5 then reguardless of the OS of message broker a RHEL5 package
would be needed if there is a single RHEL5 mcollective agent on your network.

Steve.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 813842] Review Request: glfw , A cross-platform multimedia library

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=813842

MERCIER Jonathan bioinfornat...@gmail.com changed:

   What|Removed |Added

  Flags||fedora-cvs?

--- Comment #24 from MERCIER Jonathan bioinfornat...@gmail.com ---


Spec: http://bioinfornatics.fedorapeople.org/glfw.spec
SRPMS :
http://bioinfornatics.fedorapeople.org/glfw-3.0-0.7.20120812gita9ed5b1.fc17.src.rpm
 

New Package SCM Request
===
Package Name: glfw
Short Description: A cross-platform multimedia library
Owners: bioinfornatics
Branches: f17 f18
InitialCC: bioinfornatics

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 847794] Review Request: gl3n An OpenGL Mathematics library for D

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=847794

--- Comment #7 from MERCIER Jonathan bioinfornat...@gmail.com ---
Done ldc own /usr/include/d 
I will see if gdc come into fedora

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 854553] Review Request: gr-air-modes - SDR receiver for Mode S transponder signals (ADS-B)

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=854553

--- Comment #12 from Fedora Update System upda...@fedoraproject.org ---
gr-air-modes-0-0.4.20120905git6c7a7370.fc18 has been submitted as an update for
Fedora 18.
https://admin.fedoraproject.org/updates/gr-air-modes-0-0.4.20120905git6c7a7370.fc18

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 854553] Review Request: gr-air-modes - SDR receiver for Mode S transponder signals (ADS-B)

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=854553

--- Comment #13 from Fedora Update System upda...@fedoraproject.org ---
gr-air-modes-0-0.4.20120905git6c7a7370.fc17 has been submitted as an update for
Fedora 17.
https://admin.fedoraproject.org/updates/gr-air-modes-0-0.4.20120905git6c7a7370.fc17

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 851800] Review Request: compiz - OpenGL window and compositing manager

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=851800

--- Comment #7 from Wolfgang Ulbrich chat-to...@raveit.de ---
(In reply to comment #5)
 This package was removed due to incompatibilities with the latest version of
 gnome, but Compiz works perfectly fine with other window managers (eg.
 XFCE). Don't retire a package just because it doesn't work for gnome users.

Sorry, this compiz version is designed for the Mate-Desktop as you could see in
the spec file.
Hopefuly it works also for xfce and lxde.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 846346] Review Request: halibut - TeX-like software manual tool

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=846346

Jaroslav Škarvada jskar...@redhat.com changed:

   What|Removed |Added

  Flags|fedora-cvs+ |
  Flags||fedora-cvs?

--- Comment #11 from Jaroslav Škarvada jskar...@redhat.com ---
Package Change Request
==
Package Name: halibut
New Branches: f16
Owners: jskarvad

I will also take f16 branch.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 813842] Review Request: glfw , A cross-platform multimedia library

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=813842

--- Comment #25 from Jon Ciesla limburg...@gmail.com ---
Git done (by process-git-requests).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 846346] Review Request: halibut - TeX-like software manual tool

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=846346

--- Comment #12 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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 851800] Review Request: compiz - OpenGL window and compositing manager

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=851800

--- Comment #8 from Wolfgang Ulbrich chat-to...@raveit.de ---
The package is ready for review now.

SPEC: http://raveit65.fedorapeople.org/compiz/SPEC/compiz.spec

SRPM: http://raveit65.fedorapeople.org/compiz/SRPM/compiz-0.8.8-4.fc18.src.rpm


%changelog
* Wed Sep 26 2012 Wolfgang Ulbrich chat-to...@raveit.de - 0.8.8-4
- remove upstreamed patches
- own include dir
- add compiz-mate-gtk source and compiz-mate-gtk.desktop file
- add keybinding sources
- change %%define to %%global entries
- rename no-more-gnome-wm-settings.patch to no-more-mate-wm-settings.patch
- add compiz-0.88_incorrect-fsf-address.patch
- clean up build section

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 832953] Review Request: Syntastic - A syntax checker for programming language in vim

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=832953

--- Comment #8 from MERCIER Jonathan bioinfornat...@gmail.com ---
SPEC: http://bioinfornatics.fedorapeople.org/syntastic.spec 
SRPM:
http://bioinfornatics.fedorapeople.org/syntastic-2.3.0-5.20120917git72856e6.fc17.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 851800] Review Request: compiz - OpenGL window and compositing manager

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=851800

--- Comment #9 from leigh scott leigh123li...@googlemail.com ---
(In reply to comment #8)
 The package is ready for review now.

There is still a block in place due to mate-control-center.

 
 SPEC: http://raveit65.fedorapeople.org/compiz/SPEC/compiz.spec
 
 SRPM:
 http://raveit65.fedorapeople.org/compiz/SRPM/compiz-0.8.8-4.fc18.src.rpm
 
 
 %changelog
 * Wed Sep 26 2012 Wolfgang Ulbrich chat-to...@raveit.de - 0.8.8-4
 - remove upstreamed patches
 - own include dir
 - add compiz-mate-gtk source and compiz-mate-gtk.desktop file
 - add keybinding sources
 - change %%define to %%global entries
 - rename no-more-gnome-wm-settings.patch to no-more-mate-wm-settings.patch
 - add compiz-0.88_incorrect-fsf-address.patch
 - clean up build section

Please remove compiz-0.88_incorrect-fsf-address.patch as it is against
guidelines to patch COPYING

https://fedoraproject.org/wiki/Common_Rpmlint_issues#incorrect-fsf-address

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 838901] Review Request: autotest-framework - Framework for fully automated testing

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=838901

Martin Krizek mkri...@redhat.com changed:

   What|Removed |Added

   Assignee|jpope...@redhat.com |nob...@fedoraproject.org
  Flags|fedora-review+  |
  Flags||fedora-review?

--- Comment #24 from Martin Krizek mkri...@redhat.com ---
Thanks everyone who participated in the review! (I am requesting branches only
now as I was waiting for a new Autotest release.)

New Package SCM Request
===
Package Name: autotest-framework
Short Description: Autotest is a framework for fully automated testing.
Owners: mkrizek
Branches: f16 f17 f18 el5 el6
InitialCC:

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 838901] Review Request: autotest-framework - Framework for fully automated testing

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=838901

Jiri Popelka jpope...@redhat.com changed:

   What|Removed |Added

   Assignee|nob...@fedoraproject.org|jpope...@redhat.com
  Flags|fedora-review?  |
  Flags||fedora-review+

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 860625] New: Review Request: pycanberra - A very basic (and incomplete) wrapper for libcanberra

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=860625

Bug ID: 860625
QA Contact: extras...@fedoraproject.org
  Severity: medium
   Version: rawhide
  Priority: medium
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
  Assignee: nob...@fedoraproject.org
   Summary: Review Request: pycanberra - A very basic (and
incomplete) wrapper for libcanberra
Regression: ---
  Story Points: ---
Classification: Fedora
OS: Linux
  Reporter: boche...@fedoraproject.org
  Type: ---
 Documentation: ---
  Hardware: All
Mount Type: ---
Status: NEW
 Component: Package Review
   Product: Fedora

Spec URL: http://bochecha.fedorapeople.org/packages/pycanberra.spec
SRPM URL:
http://bochecha.fedorapeople.org/packages/pycanberra-0-0.1.git65c3b3f.fc18.src.rpm

Description:
A very basic (and incomplete) wrapper for libcanberra.

This is necessary until libcanberra gets proper GObject Introspection support.

Fedora Account System Username: bochecha

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 857180] Review Request: gnome-clocks - Clock application designed for GNOME 3

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857180

Mathieu Bridon boche...@fedoraproject.org changed:

   What|Removed |Added

 CC||boche...@fedoraproject.org

--- Comment #7 from Mathieu Bridon boche...@fedoraproject.org ---
(In reply to comment #2)
 If anyone wishes to package pycanberra in some form this could be revisited.

I just submitted a review request for pycanberra:
https://bugzilla.redhat.com/show_bug.cgi?id=860625

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 860625] Review Request: pycanberra - A very basic (and incomplete) wrapper for libcanberra

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=860625

Yanko Kaneti yan...@declera.com changed:

   What|Removed |Added

 Blocks||857180

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 857180] Review Request: gnome-clocks - Clock application designed for GNOME 3

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857180

Yanko Kaneti yan...@declera.com changed:

   What|Removed |Added

 Depends On||860625

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 860625] Review Request: pycanberra - A very basic (and incomplete) wrapper for libcanberra

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=860625

Adel Gadllah adel.gadl...@gmail.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||adel.gadl...@gmail.com
  Flags||fedora-review+

--- Comment #1 from Adel Gadllah adel.gadl...@gmail.com ---
Looks good, rpmlint output is just noise:

---
pycanberra.src: W: spelling-error Summary(en_US) libcanberra - Canberra
pycanberra.src: W: spelling-error %description -l en_US libcanberra - Canberra
pycanberra.src: W: invalid-url Source0: pycanberra-git.65c3b3f.tar.xz
pycanberra.spec: W: invalid-url Source0: pycanberra-git.65c3b3f.tar.xz
pycanberra.noarch: W: spelling-error Summary(en_US) libcanberra - Canberra
pycanberra.noarch: W: spelling-error %description -l en_US libcanberra -
Canberra
2 packages and 1 specfiles checked; 0 errors, 6 warnings.


(You could just remove the empty %build but that would trigger another rpmlint
warning so just leave it as is).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 739347] Review Request: haveged - A Linux entropy source using the HAVEGE algorithm. Feed entropy into random pool

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=739347

Jussi Lehtola jussi.leht...@iki.fi changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution|--- |CURRENTRELEASE
Last Closed||2012-09-26 06:56:39

--- Comment #25 from Jussi Lehtola jussi.leht...@iki.fi ---
Odd that the update system hasn't closed this, even though the updates have hit
stable. Closing.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 860625] Review Request: pycanberra - A very basic (and incomplete) wrapper for libcanberra

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=860625

Mathieu Bridon boche...@fedoraproject.org changed:

   What|Removed |Added

  Flags||fedora-cvs?

--- Comment #2 from Mathieu Bridon boche...@fedoraproject.org ---
Thanks Adel!

New Package SCM Request
===
Package Name: pycanberra
Short Description: A very basic (and incomplete) wrapper for libcanberra
Owners: bochecha
Branches: devel f18
InitialCC:

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 857180] Review Request: gnome-clocks - Clock application designed for GNOME 3

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857180

--- Comment #8 from Yanko Kaneti yan...@declera.com ---
- Use packaged pycanberra

Spec URL: http://declera.com/~yaneti/gnome-clocks/gnome-clocks.spec
SRPM URL:
http://declera.com/~yaneti/gnome-clocks/gnome-clocks-0.1.2-2.fc19.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 860625] Review Request: pycanberra - A very basic (and incomplete) wrapper for libcanberra

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=860625

--- Comment #3 from Jon Ciesla limburg...@gmail.com ---
Git done (by process-git-requests).

Adel, please take ownership of review BZs, thanks!

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 832953] Review Request: Syntastic - A syntax checker for programming language in vim

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=832953

Pavel Raiskup prais...@redhat.com changed:

   What|Removed |Added

  Flags||needinfo?(bioinfornatics@gm
   ||ail.com)

--- Comment #9 from Pavel Raiskup prais...@redhat.com ---
The very last think is that the autoload directory should be owned by
vim-filesystem only (and with slash):

  rpm -qf /usr/share/vim/vimfiles/autoload/

  diff --git a/srpm/syntastic.spec b/srpm/syntastic.spec
  index 54ffc2d..9207e52 100644
  --- a/srpm/syntastic.spec
  +++ b/srpm/syntastic.spec
  @@ -190,7 +190,7 @@ exit 0

   %files
   %doc _assets/screenshot_1.png README.markdown LICENCE
  -%{vimfiles}/autoload
  +%dir %{vimfiles}/autoload/syntastic/
   %{vimfiles}/plugin/syntastic.vim
   %{vimfiles}/doc/syntastic.txt
   %dir %{vimfiles}/syntax_checkers/

I tried to find the reason why:

   $ rpm -qf rpm -qf /usr/share/vim/vimfiles/autoload/syntastic/c.vim
   syntastic-2.3.0-5.20120917git72856e6.fc17.noarch
   syntastic-c-2.3.0-5.20120917git72856e6.fc17.noarch

Could you repair it before approval or should I give you fedora-review+ flag
now and you will repair it later on?

Pavel

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 860625] Review Request: pycanberra - A very basic (and incomplete) wrapper for libcanberra

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=860625

Jon Ciesla limburg...@gmail.com changed:

   What|Removed |Added

 CC||limburg...@gmail.com
  Flags|fedora-cvs+ |
  Flags||fedora-cvs?

--- Comment #4 from Jon Ciesla limburg...@gmail.com ---
Also, please post more detail about exactly what you reviewed, etc.  There are
several templates available on the wiki as a starting point.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 857180] Review Request: gnome-clocks - Clock application designed for GNOME 3

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=857180

--- Comment #9 from Yanko Kaneti yan...@declera.com ---
0.1.2-3
- Forgot about the License tag 

Spec URL: http://declera.com/~yaneti/gnome-clocks/gnome-clocks.spec
SRPM URL:
http://declera.com/~yaneti/gnome-clocks/gnome-clocks-0.1.2-3.fc19.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 851800] Review Request: compiz - OpenGL window and compositing manager

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=851800

--- Comment #10 from Wolfgang Ulbrich chat-to...@raveit.de ---
(In reply to comment #9)
 (In reply to comment #8)
 
 Please remove compiz-0.88_incorrect-fsf-address.patch as it is against
 guidelines to patch COPYING
 
 https://fedoraproject.org/wiki/Common_Rpmlint_issues#incorrect-fsf-address

Thanks Leigh,
i remove the COPYING parts from the patch and notice upstream about this issue.

http://bugs.compiz.org/show_bug.cgi?id=92

New sources:
SPEC: http://raveit65.fedorapeople.org/compiz/SPEC/compiz.spec

SRPM: http://raveit65.fedorapeople.org/compiz/SRPM/compiz-0.8.8-5.fc18.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 785462] Review Request: sinfo - Show an overview of the available computers an their current load

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=785462

--- Comment #8 from Volker Fröhlich volke...@gmx.at ---
Guillermo Gómez, please explain why you think they are necessary.

[makerpm@desktop SPECS]$ rpm -qp --requires
/home/makerpm/rpmbuild/RPMS/x86_64/sinfo-0.0.47-1.fc16.x86_64.rpm
/bin/sh  
boost  
libboost_signals-mt.so.1.47.0()(64bit)  
libboost_system-mt.so.1.47.0()(64bit)  
libc.so.6()(64bit)  
libc.so.6(GLIBC_2.14)(64bit)  
libc.so.6(GLIBC_2.2.5)(64bit)  
libc.so.6(GLIBC_2.3.2)(64bit)  
libc.so.6(GLIBC_2.3.4)(64bit)  
libc.so.6(GLIBC_2.4)(64bit)  
libc.so.6(GLIBC_2.7)(64bit)  
libc.so.6(GLIBC_2.8)(64bit)  
libc.so.6(GLIBC_2.9)(64bit)  
libgcc_s.so.1()(64bit)  
libgcc_s.so.1(GCC_3.0)(64bit)  
libm.so.6()(64bit)  
libm.so.6(GLIBC_2.2.5)(64bit)  
libmessage.so.0()(64bit)  
libmessageio.so.0()(64bit)  
libmessageiobase.so.0()(64bit)  
libmessageparser.so.0()(64bit)  
libncurses.so.5()(64bit)  
libnsl.so.1()(64bit)  
libprotocol.so.0()(64bit)  
libprotocolasio.so.0()(64bit)  
libpthread.so.0()(64bit)  
libpthread.so.0(GLIBC_2.2.5)(64bit)  
libpthread.so.0(GLIBC_2.3.2)(64bit)  
libresolv.so.2()(64bit)  
librpc.so.0()(64bit)  
libsinfometer-0.0.47.so()(64bit)  
libsinfotypes-0.0.47.so()(64bit)  
libstdc++.so.6()(64bit)  
libstdc++.so.6(CXXABI_1.3)(64bit)  
libstdc++.so.6(CXXABI_1.3.1)(64bit)  
libstdc++.so.6(GLIBCXX_3.4)(64bit)  
libstdc++.so.6(GLIBCXX_3.4.11)(64bit)  
libstdc++.so.6(GLIBCXX_3.4.15)(64bit)  
libstdc++.so.6(GLIBCXX_3.4.9)(64bit)  
libtinfo.so.5()(64bit)  
ncurses  
rpmlib(CompressedFileNames) = 3.0.4-1
rpmlib(FileDigests) = 4.6.0-1
rpmlib(PayloadFilesHavePrefix) = 4.0-1
rtld(GNU_HASH)  
rpmlib(PayloadIsXz) = 5.2-1

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 860625] Review Request: pycanberra - A very basic (and incomplete) wrapper for libcanberra

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=860625

--- Comment #5 from Jon Ciesla limburg...@gmail.com ---
Fixing flag.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 818401] Review Request: 0ad - Cross-Platform RTS Game of Ancient Warfare

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=818401

--- Comment #54 from Bruno Wolff III br...@wolff.to ---
Don't forget to add it as a member to the games group in comps. You can already
do it for rawhide. It should be also done for other releases as it makes it to
stable / updates.
Thanks for packaging it.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 852174] Review Request: snapper - Tool for filesystem snapshot management

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=852174

--- Comment #6 from Ondrej Kozina okoz...@redhat.com ---
Update to latest devel version with dbus interface. I've done some basic code
review but it would be nice if someone would look at it as well.

Moreover I think selinux policy will have to be updated to enable dbus
communcation for the snapper package.

 I'm not sure if that file should be where it is:
 %{_libdir}/%{name}/bin/compare-dirs

The binary is considered obsolete and removed completely from dbus version.
Background tasks are handled by snapperd daemon.

Spec URL: http://okozina.fedorapeople.org/snapper/0.0.14/3/snapper.spec
SRPM URL:
http://okozina.fedorapeople.org/snapper/0.0.14/3/snapper-0.0.14-3.20120926git7918e5c.fc19.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 852174] Review Request: snapper - Tool for filesystem snapshot management

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=852174

--- Comment #7 from Ondrej Kozina okoz...@redhat.com ---
Hi David,

just wanted to let you know I request to add new dbus enabled service

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 785462] Review Request: sinfo - Show an overview of the available computers an their current load

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=785462

--- Comment #9 from Guillermo Gómez guillermo.go...@gmail.com ---
There are cases where the build process does not pickup the dependencies and
for new packagers that could confuse them, add or not the Requires.

In this particular case, there's no problem, both Requires can go away and in
general we should not include explicit Requires for cases like this (my only
motivation was educational).

Reference: http://fedoraproject.org/wiki/Packaging:Guidelines#Explicit_Requires

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 819951] Review Request: ostree - Linux-based operating system develop/build/deploy tool

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=819951

--- Comment #16 from Colin Walters walt...@redhat.com ---
New release:

SRPM: http://fedorapeople.org/~walters/ostree/ostree-2012.11-1.fc18.src.rpm
SPEC: http://fedorapeople.org/~walters/ostree/ostree.spec

Not much changed from the above other than a new upstream release.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 853252] Review Request: datanommer - A storage consumer for the Fedora Message Bus (fedmsg)

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=853252

Ralph Bean rb...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |ERRATA
Last Closed||2012-09-26 09:49:28

--- Comment #12 from Ralph Bean rb...@redhat.com ---
Updates - https://admin.fedoraproject.org/updates/datanommer

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 858060] qpid-snmpd - SNMP agent for qpid broker

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=858060

--- Comment #8 from Ernie eal...@redhat.com ---
Changed from an init.d script to a unit file.
Removed the step to zip the man page.

SPEC URL: http://eallen.fedorapeople.org/qpid-snmpd.spec
SRPM URL: http://eallen.fedorapeople.org/qpid-snmpd-1.0.0-4.fc17.src.rpm
TAR  URL: http://eallen.fedorapeople.org/qpid-snmpd-1.0.0.tar.bz2

One thing of note:
Installing the qpid-snmpd.service file in %{_unitdir} does generate an rpmlint
warning:

 [eallen@redhat x86_64]$ rpmlint *rpm ../../SPECS/qpid-snmpd.spec
 qpid-snmpd.x86_64: W: only-non-binary-in-usr-lib
 2 packages and 1 specfiles checked; 0 errors, 1 warnings.

But it appears that this warning is bogus:
https://bugzilla.redhat.com/show_bug.cgi?id=794777

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 860697] New: Review Request: tmw - The Mana World is a 2D MMORPG

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=860697

Bug ID: 860697
QA Contact: extras...@fedoraproject.org
  Severity: medium
   Version: rawhide
  Priority: medium
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
  Assignee: nob...@fedoraproject.org
   Summary: Review Request: tmw - The Mana World is a 2D MMORPG
Regression: ---
  Story Points: ---
Classification: Fedora
OS: Linux
  Reporter: martin.giesek...@uos.de
  Type: ---
 Documentation: ---
  Hardware: All
Mount Type: ---
Status: NEW
 Component: Package Review
   Product: Fedora

Spec URL: http://mgieseki.fedorapeople.org/review/tmw.spec
SRPM URL: http://mgieseki.fedorapeople.org/review/tmw-20110911-1.fc17.src.rpm

Description: 
The Mana World (TMW for short) is an innovative, free and open source MMORPG.
Besides the official game server, this client can connect to multiple
community-grown servers, which provide varied environments and further
challenge. In TMW, the players solve quests, fight monsters, practice skills
and study magic. Social activities include parties, trading and limited PvP in
designated areas. While there are no limits to solo play, collaborative
behavior such as healing others, fighting together and banding up against
tougher monsters are rewarded in the game. The Mana World graphics have been
inspired by 2D pixel art at its prime of the late 1990s, when many RPG
classics, such as Secret of Mana for the Super Nintendo Entertainment System,
were released.

Additional information:
tmw is supposed to replace the manaworld package which conflicts with mana. tmw
is based on mana and provides the manaworld branding/theme only.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 860703] New: Review Request: tmw-music - Music files for The Mana World

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=860703

Bug ID: 860703
QA Contact: extras...@fedoraproject.org
  Severity: medium
   Version: rawhide
  Priority: medium
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
  Assignee: nob...@fedoraproject.org
   Summary: Review Request: tmw-music - Music files for The Mana
World
Regression: ---
  Story Points: ---
Classification: Fedora
OS: Linux
  Reporter: martin.giesek...@uos.de
  Type: ---
 Documentation: ---
  Hardware: All
Mount Type: ---
Status: NEW
 Component: Package Review
   Product: Fedora

Spec URL: http://mgieseki.fedorapeople.org/review/tmw-music.spec
SRPM URL: http://mgieseki.fedorapeople.org/review/tmw-music-0.3-5.fc17.src.rpm

Description: 
This package contains the optional music files for The Mana World (TMW).

Additional information:
tmw-music is supposed to replace manaworld-music, an optional music package for
manaworld. manaworld is going to be replaced with tmw because of file
conflicts.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 860703] Review Request: tmw-music - Music files for The Mana World

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=860703

Martin Gieseking martin.giesek...@uos.de changed:

   What|Removed |Added

 Depends On||860697

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 860697] Review Request: tmw - The Mana World is a 2D MMORPG

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=860697

Martin Gieseking martin.giesek...@uos.de changed:

   What|Removed |Added

 Blocks||860703

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 860697] Review Request: tmw - The Mana World is a 2D MMORPG

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=860697

Martin Gieseking martin.giesek...@uos.de changed:

   What|Removed |Added

  Alias||tmw

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 860703] Review Request: tmw-music - Music files for The Mana World

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=860703

Martin Gieseking martin.giesek...@uos.de changed:

   What|Removed |Added

  Alias||tmw-music

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 853694] Review Request: mate-control-center - MATE Desktop control center

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=853694

--- Comment #5 from Rex Dieter rdie...@math.unl.edu ---
In order of preference, you'll need to:
* find a way for the buildsys not to create those files (patching maybe)
* manually delete them
or 
* use %exclude on those files.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 848388] Review Request: liblognorm - Tool to normalize log data

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=848388

Miloslav Trmač m...@redhat.com changed:

   What|Removed |Added

 CC||m...@redhat.com
  Component|pacemaker   |Package Review

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 847811] Review Request: libee - An event expression library

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=847811

Miloslav Trmač m...@redhat.com changed:

   What|Removed |Added

 CC||m...@redhat.com
  Component|pacemaker   |Package Review

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 847817] Review Request: libestr - A library to handle strings

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=847817

Miloslav Trmač m...@redhat.com changed:

   What|Removed |Added

 CC||m...@redhat.com
  Component|pacemaker   |Package Review

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 832635] Review Request: pencil - A sketching and GUI prototyping tool

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=832635

--- Comment #6 from Truong Anh Tuan tua...@iwayvietnam.com ---
All necessary changes have been made.

Updated .spec file: http://tuanta.fedorapeople.org/pencil/pencil.spec
And new .src file:
http://tuanta.fedorapeople.org/pencil/pencil-2.0-2.fc17.src.rpm

Please help me to review the package.

Rgds,
Tuan

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 832635] Review Request: pencil - A sketching and GUI prototyping tool

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=832635

--- Comment #7 from Volker Fröhlich volke...@gmx.at ---
No prebuilt binaries!

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 832635] Review Request: pencil - A sketching and GUI prototyping tool

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=832635

--- Comment #8 from Truong Anh Tuan tua...@iwayvietnam.com ---
You meant the file /usr/bin/pencil, right?
It's just a shell script file, not a binary.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 832635] Review Request: pencil - A sketching and GUI prototyping tool

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=832635

--- Comment #9 from Volker Fröhlich volke...@gmx.at ---
I'm sorry!

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 836821] Review Request: libcec - Library for HDMI-CEC device control

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=836821

Peter Robinson pbrobin...@gmail.com changed:

   What|Removed |Added

  Flags|needinfo?(pbrobinson@gmail. |
   |com)|

--- Comment #2 from Peter Robinson pbrobin...@gmail.com ---
Sorry about delayed response. 1.8.x had build issues, and I've been away. 1.9.0
now builds so updated.

(In reply to comment #1)
 The license is GPLv2+, but it is actually dual-licensed:

Fedora doesn't use the commercial license option so it's the only relevant
license for our use case

 The FSF address is outdated and should be replaced. Also let upstream know.

I'll fix it when it's fixed upstream (it might be already in 1.9.0) but I'll
let them know

 If this is a snapshot, the release should be different:
 http://fedoraproject.org/wiki/Packaging:NamingGuidelines#NonNumericRelease

No it's not a snapshot. It's the release using their git tag. Theit github
generated archives have issues so we generate them from the git source but it's
still a tagged release so we can still use the standard NVRs

 The version number in the changelog also doesn't match.

Fixed.

 Please add ?_isa, see
 http://fedoraproject.org/wiki/Packaging:Guidelines#Requiring_Base_Package

Added

 Why do you require systemd-devel?

Because that's where the udev library headers are now and it uses udev.

 rpmlint complains about mixed-use-of-spaces-and-tabs (spaces: line 1, tab:
 line 1) and the non-american spelling of utilizing. cec-config should be
 in the main package.

Line 1 is a comment so it's completely irrelevant. I've updated the comment
further.

cec-config IS in the main package:
$ rpm -qlp libcec-1.9.0-1.fc17.x86_64.rpm 
/usr/bin/cec-client
/usr/bin/cec-config
/usr/lib64/libcec.so.1
/usr/lib64/libcec.so.1.0.9

 defattr is no longer necessary. Consider to add ChangeLog as documentation.

I thought I had done both of those. Fixed.

SPEC: as above
SRPM: http://pbrobinson.fedorapeople.org/libcec-1.9.0-1.fc17.src.rpm
koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=4529045

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 836821] Review Request: libcec - Library for HDMI-CEC device control

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=836821

--- Comment #3 from Peter Robinson pbrobin...@gmail.com ---
Volker: are you doing a proper review on this? if so please assign as
appropriate

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 843997] Review Request: mlpack - scalable C++ machine learning library

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=843997

--- Comment #11 from José Matos jama...@fc.up.pt ---
There is no need to add the files as attachments. :-)

Some more low hanging fruit.

1) There is no need to put the license file in place in the %install section.

It is enough to do this in the %files section:

%files
...
%doc LICENSE.txt

and voila... this replaces all the logic you have now and it does precisely the
same. :-)

2) In reference to point 2 from #6 above:
 The documentation is Doxygen-generated HTML which a user could more easily 
 access at mlpack.org (it is prettier there, too).  I don't think it's 
 necessary to include that amount of stuff with the distribution, especially 
 when a user could generate it themselves using Doxygen painlessly.

I have a different opinion regarding this. If the documentation is provided by
upstream it is my opinion that we should provide it, even if we place it on a
-doc sub-package that it is not required by the main package.
This subject is discussed here:
https://fedoraproject.org/wiki/Packaging:Guidelines#Documentation

As a rationale for the above sometimes I am in places with small or no
connectivity at all and having an update documentation is a boon. I am
perfectly able to run doxygen but if this is a new package that I am exploring
pre-available off-line documentation is a bonus in my book. :-)

3) Running rpmlint on the generated rpms shows that libxml is a superfluous
requirement and it should be dropped since that dependency is automatically
taken from BuildRequires.
The same point applies to the boost* libraries, to armadillo and to lapack.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 836821] Review Request: libcec - Library for HDMI-CEC device control

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=836821

--- Comment #4 from Volker Fröhlich volke...@gmx.at ---
GPLv2+ -- not GPLv2

If I'll take the review, I'll assign myself. For now I'm commenting.

It's easier for the reviewer if you bump the release on changes and leave
changelog entries.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 848142] Review Request: maliit-plugins - Input method plugins

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=848142

Peter Robinson pbrobin...@gmail.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
  Flags||fedora-cvs-

--- Comment #3 from Peter Robinson pbrobin...@gmail.com ---
Will fix the comments above on commit. Thanks for the review

New Package GIT Request
===
Package Name: maliit-plugins
Short Description: Input method plugins
Owners: pbrobinson
Branches: F-18 F-17
InitialCC:

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 848139] Review Request: maliit-framework - Input method framework

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=848139

Peter Robinson pbrobin...@gmail.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
  Flags||fedora-cvs?

--- Comment #2 from Peter Robinson pbrobin...@gmail.com ---
Will fix the comments above on commit. Thanks for the review

New Package GIT Request
===
Package Name: maliit-framework
Short Description: Input method framework
Owners: pbrobinson
Branches: F-18 F-17
InitialCC:

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 848139] Review Request: maliit-framework - Input method framework

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=848139

--- 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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 852174] Review Request: snapper - Tool for filesystem snapshot management

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=852174

--- Comment #8 from David Zeuthen dav...@redhat.com ---
(In reply to comment #7)
 just wanted to let you know I request to add new dbus enabled service

Any package can include D-Bus services, there is no central registry or
anything like that. So it doesn't make sense to request one...

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 848142] Review Request: maliit-plugins - Input method plugins

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=848142

Peter Robinson pbrobin...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-cvs- |
  Flags||fedora-cvs?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 848142] Review Request: maliit-plugins - Input method plugins

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=848142

--- Comment #4 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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 860146] Review Request: vsqlite++ - A C++ Wrapper for SQLite

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=860146

Stanislav Ochotnicky socho...@redhat.com changed:

   What|Removed |Added

  Flags|fedora-review?  |
  Flags||fedora-review+

--- Comment #9 from Stanislav Ochotnicky socho...@redhat.com ---
Package looks good now, except one tiny thing. doc subpackage should be noarch.
But that's indeed small issue which I believe you'll fix without problems.

I have approved the packages so now you can follow:
https://fedoraproject.org/wiki/Package_SCM_admin_requests

Please add me to the initial cc for the package so I can tiptoe around you for
a while :-)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 823458] Review Request: erlang-erlsha2 - SHA-224, SHA-256, SHA-384, SHA-512 implemented in Erlang NIFs

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=823458

Peter Lemenkov lemen...@gmail.com changed:

   What|Removed |Added

 Whiteboard||NotReady

--- Comment #1 from Peter Lemenkov lemen...@gmail.com ---
This package is slowly merging into base Erlang distribution (since R15B02) so
perhaps it's better to withdraw it. I will reconsider it soon. Do NOT start
review, please.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 860521] Review Request: mate-session-manager - MATE Desktop session manager

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=860521

Rex Dieter rdie...@math.unl.edu changed:

   What|Removed |Added

Version|17  |rawhide

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 860515] Review Request: mate-themes - MATE Desktop themes

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=860515

Rex Dieter rdie...@math.unl.edu changed:

   What|Removed |Added

Version|17  |rawhide
   Assignee|nob...@fedoraproject.org|rdie...@math.unl.edu
  Flags||fedora-review?

--- Comment #1 from Rex Dieter rdie...@math.unl.edu ---
moar fun to do!

-- 
You are receiving this mail because:
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 834681] Review Request: python-sieve - XML Comparison Utils

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=834681

Mario Blättermann mario.blaetterm...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-review?  |
  Flags||fedora-review+

--- Comment #6 from Mario Blättermann mario.blaetterm...@gmail.com ---
$ rpmlint -i -v *
python3-sieve.noarch: I: checking
python3-sieve.noarch: W: spelling-error %description -l en_US webapp - web
app, web-app, weapon
The value of this tag appears to be misspelled. Please double-check.

python3-sieve.noarch: W: spelling-error %description -l en_US eq - e, q, seq
The value of this tag appears to be misspelled. Please double-check.

python3-sieve.noarch: W: spelling-error %description -l en_US xml - XML, ml, x
ml
The value of this tag appears to be misspelled. Please double-check.

python3-sieve.noarch: W: spelling-error %description -l en_US html - HTML, ht
ml, ht-ml
The value of this tag appears to be misspelled. Please double-check.

python3-sieve.noarch: I: checking-url http://pypi.python.org/pypi/sieve
(timeout 10 seconds)
python-sieve.src: I: checking
python-sieve.src: W: spelling-error %description -l en_US webapp - web app,
web-app, weapon
The value of this tag appears to be misspelled. Please double-check.

python-sieve.src: W: spelling-error %description -l en_US eq - e, q, seq
The value of this tag appears to be misspelled. Please double-check.

python-sieve.src: W: spelling-error %description -l en_US xml - XML, ml, x ml
The value of this tag appears to be misspelled. Please double-check.

python-sieve.src: W: spelling-error %description -l en_US html - HTML, ht ml,
ht-ml
The value of this tag appears to be misspelled. Please double-check.

python-sieve.src: I: checking-url http://pypi.python.org/pypi/sieve (timeout 10
seconds)
python-sieve.src: I: checking-url
http://pypi.python.org/packages/source/s/sieve/sieve-0.1.6.tar.gz (timeout 10
seconds)
python-sieve.noarch: I: checking
python-sieve.noarch: W: spelling-error %description -l en_US webapp - web app,
web-app, weapon
The value of this tag appears to be misspelled. Please double-check.

python-sieve.noarch: W: spelling-error %description -l en_US eq - e, q, seq
The value of this tag appears to be misspelled. Please double-check.

python-sieve.noarch: W: spelling-error %description -l en_US xml - XML, ml, x
ml
The value of this tag appears to be misspelled. Please double-check.

python-sieve.noarch: W: spelling-error %description -l en_US html - HTML, ht
ml, ht-ml
The value of this tag appears to be misspelled. Please double-check.

python-sieve.noarch: I: checking-url http://pypi.python.org/pypi/sieve (timeout
10 seconds)
python-sieve.spec: I: checking-url
http://pypi.python.org/packages/source/s/sieve/sieve-0.1.6.tar.gz (timeout 10
seconds)
(none): E: no installed packages by name sieve-0.1.6.tar.gz.packaged
3 packages and 1 specfiles checked; 0 errors, 12 warnings.

No real issues anymore.


-
key:

[+] OK
[.] OK, not applicable
[X] needs work
-

[+] MUST: rpmlint must be run on the source rpm and all binary rpms the build
produces. The output should be posted in the review.
[+] MUST: The package must be named according to the Package Naming Guidelines.
[+] MUST: The spec file name must match the base package %{name}, in the format
%{name}.spec unless your package has an exemption.
[+] MUST: The package must meet the Packaging Guidelines.
[+] MUST: The package must be licensed with a Fedora approved license and meet
the Licensing Guidelines.
[+] MUST: The License field in the package spec file must match the actual
license.
MIT
[+] MUST: If (and only if) the source package includes the text of the
license(s) in its own file, then that file, containing the text of the
license(s) for the package must be included in %doc.
[+] MUST: The spec file must be written in American English.
[+] MUST: The spec file for the package MUST be legible.
[+] MUST: The sources used to build the package must match the upstream source,
as provided in the spec URL. Reviewers should use sha256sum for this task as it
is used by the sources file once imported into git. If no upstream URL can be
specified for this package, please see the Source URL Guidelines for how to
deal with this.
$ sha256sum *
bb840f52804617f683ff84ab02e71f4f25ef5f568d658d7637846bb5d238c068 
sieve-0.1.6.tar.gz
bb840f52804617f683ff84ab02e71f4f25ef5f568d658d7637846bb5d238c068 
sieve-0.1.6.tar.gz.packaged

[+] MUST: The package MUST successfully compile and build into binary rpms on
at least one primary architecture.
[.] MUST: If the package does not successfully compile, build or work on an
architecture, then those architectures should be listed in the spec in
ExcludeArch. Each architecture listed in ExcludeArch MUST have a bug filed in
bugzilla, describing the reason 

[Bug 740660] Review Request: python-d2to1 - Allows using distutils2-like setup.cfg files with setup.py

2012-09-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=740660

Mario Blättermann mario.blaetterm...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-review?  |
  Flags||fedora-review+

--- Comment #4 from Mario Blättermann mario.blaetterm...@gmail.com ---
Scratch build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=4529544

$ rpmlint -i -v *
python3-d2to1.noarch: I: checking
python3-d2to1.noarch: W: spelling-error Summary(en_US) cfg - cf, cg, cog
The value of this tag appears to be misspelled. Please double-check.

python3-d2to1.noarch: W: spelling-error Summary(en_US) py - pt, p, y
The value of this tag appears to be misspelled. Please double-check.

python3-d2to1.noarch: W: spelling-error %description -l en_US cfg - cf, cg,
cog
The value of this tag appears to be misspelled. Please double-check.

python3-d2to1.noarch: W: spelling-error %description -l en_US metadata - meta
data, meta-data, metatarsal
The value of this tag appears to be misspelled. Please double-check.

python3-d2to1.noarch: W: spelling-error %description -l en_US setuptools -
setup tools, setup-tools, toadstools
The value of this tag appears to be misspelled. Please double-check.

python3-d2to1.noarch: W: spelling-error %description -l en_US py - pt, p, y
The value of this tag appears to be misspelled. Please double-check.

python3-d2to1.noarch: I: checking-url http://pypi.python.org/pypi/d2to1
(timeout 10 seconds)
python-d2to1.src: I: checking
python-d2to1.src: W: spelling-error Summary(en_US) cfg - cf, cg, cog
The value of this tag appears to be misspelled. Please double-check.

python-d2to1.src: W: spelling-error Summary(en_US) py - pt, p, y
The value of this tag appears to be misspelled. Please double-check.

python-d2to1.src: W: spelling-error %description -l en_US cfg - cf, cg, cog
The value of this tag appears to be misspelled. Please double-check.

python-d2to1.src: W: spelling-error %description -l en_US metadata - meta
data, meta-data, metatarsal
The value of this tag appears to be misspelled. Please double-check.

python-d2to1.src: W: spelling-error %description -l en_US setuptools - setup
tools, setup-tools, toadstools
The value of this tag appears to be misspelled. Please double-check.

python-d2to1.src: W: spelling-error %description -l en_US py - pt, p, y
The value of this tag appears to be misspelled. Please double-check.

python-d2to1.src: I: checking-url http://pypi.python.org/pypi/d2to1 (timeout 10
seconds)
python-d2to1.src: I: checking-url
http://pypi.python.org/packages/source/d/d2to1/d2to1-0.2.7.tar.gz (timeout 10
seconds)
python-d2to1.noarch: I: checking
python-d2to1.noarch: W: spelling-error Summary(en_US) cfg - cf, cg, cog
The value of this tag appears to be misspelled. Please double-check.

python-d2to1.noarch: W: spelling-error Summary(en_US) py - pt, p, y
The value of this tag appears to be misspelled. Please double-check.

python-d2to1.noarch: W: spelling-error %description -l en_US cfg - cf, cg, cog
The value of this tag appears to be misspelled. Please double-check.

python-d2to1.noarch: W: spelling-error %description -l en_US metadata - meta
data, meta-data, metatarsal
The value of this tag appears to be misspelled. Please double-check.

python-d2to1.noarch: W: spelling-error %description -l en_US setuptools -
setup tools, setup-tools, toadstools
The value of this tag appears to be misspelled. Please double-check.

python-d2to1.noarch: W: spelling-error %description -l en_US py - pt, p, y
The value of this tag appears to be misspelled. Please double-check.

python-d2to1.noarch: I: checking-url http://pypi.python.org/pypi/d2to1 (timeout
10 seconds)
python-d2to1.spec: I: checking-url
http://pypi.python.org/packages/source/d/d2to1/d2to1-0.2.7.tar.gz (timeout 10
seconds)
3 packages and 1 specfiles checked; 0 errors, 18 warnings.

Some ignorable spelling errors, nothing of interest so far.


-
key:

[+] OK
[.] OK, not applicable
[X] needs work
-

[+] MUST: rpmlint must be run on the source rpm and all binary rpms the build
produces. The output should be posted in the review.
[+] MUST: The package must be named according to the Package Naming Guidelines.
[+] MUST: The spec file name must match the base package %{name}, in the format
%{name}.spec unless your package has an exemption.
[+] MUST: The package must meet the Packaging Guidelines.
[+] MUST: The package must be licensed with a Fedora approved license and meet
the Licensing Guidelines.
[+] MUST: The License field in the package spec file must match the actual
license.
BSD
[+] MUST: If (and only if) the source package includes the text of the
license(s) in its own file, then that file, containing the text of the
license(s) for the package must be included in %doc.
[+] MUST: The spec file must be written in American 

  1   2   >