[Bug 815018] Review Request: nodejs - javascript fast build framework

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=815018

--- Comment #41 from Matthias Runge mru...@redhat.com ---
(In reply to comment #40)
 Stephen, Am sorry but I dont like the idea that you take over my package. We
 can work together on this am more than glad to add any suggestion but not to
 take it away from me. All my effort gonna be lost and I gonna continue
 working on it. Send me all the ideas and mods that u want to apply and I
 will be glad to do it

With my fedora packager hat on:

Adrian,

with all given respect, I think this package is a number too large for a
beginning packager.


Adrian, you tried to package this in a proper way for a half year now; the
progress I can see is, that your three proposals provided are basically the
same, for three different upstream versions. Apparently you didn't try to solve
any of the existing (listed) problems. During this half year, we didn't see
progress driven by you. Just hacking together a spec file is just a very small
part of the whole packaging thing.


I really urge you to hand over this package review to Stephen. 
He had shown the right way, communicated with upstream and his efforts already
showed progress there. 
Adrian, speaking about your lost work. It isn't your work and it's not lost;
since this is free software, everybody should use and improve it. For my
contributions, I'd like to see improvements, nothing is perfect.


With my Red Hat on:
We need this package *sooner* or later packaged in a proper and secure way, and
also could use some upstream involvement. Adrian, I currently can't see, how
you handle that. 


I'm sorry for these open and harsh words, but I didn't see insight in
thoughtful conversation, held in private and also public.

-- 
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 815018] Review Request: nodejs - javascript fast build framework

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=815018

--- Comment #42 from Adrian Alves aal...@gmail.com ---
Matthias am not a newbie I built packages for companys in USA since 2006 u can
check it on my fedora wiki. Am glad to apply or add any contribution that u
have we can co maintain the package please stop pushing me and send me all the
contributions that u want to add to the package this didnt help in anything.
Ill be waiting for any ur your patches but not more comments like this, If you
have doubts about my skill talk with Toshio my mentor!

-- 
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 760177] Review Request: knot - Authoritative DNS server

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=760177

--- Comment #33 from Stanislav Petr g...@glux.org ---
Fixed packages:

http://my.glux.org/fedora/packages/knot/knot.spec
http://my.glux.org/fedora/packages/knot/knot-1.1.1-1.fc16.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 873256] Review Request: python-cssselect - cssselect parses CSS3 Selectors and translates them to XPath 1.0

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=873256

Haïkel Guémar karlthe...@gmail.com changed:

   What|Removed |Added

 CC||karlthe...@gmail.com
   Assignee|nob...@fedoraproject.org|karlthe...@gmail.com
  Flags||fedora-review?

--- Comment #1 from Haïkel Guémar karlthe...@gmail.com ---
I'm reviewing this package since it is required for newer versions of Calibre
to work properly.

-- 
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 873738] Review Request: sequence-library - Textual diff and merge library

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=873738

Mikolaj Izdebski mizde...@redhat.com changed:

   What|Removed |Added

  Flags||needinfo?(ism...@olea.org)

-- 
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 760177] Review Request: knot - Authoritative DNS server

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=760177

--- Comment #34 from Stanislav Petr g...@glux.org ---
Created attachment 640075
  -- https://bugzilla.redhat.com/attachment.cgi?id=640075action=edit
spec for knot 1.1.1-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 874105] Review Request: qpid-proton - Proton is a high performance, lightweight messaging library

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=874105

Darryl L. Pierce dpie...@redhat.com changed:

   What|Removed |Added

  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 874105] New: Review Request: qpid-proton - Proton is a high performance, lightweight messaging library

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=874105

Bug ID: 874105
QA Contact: extras...@fedoraproject.org
  Severity: unspecified
   Version: rawhide
  Priority: unspecified
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
  Assignee: nob...@fedoraproject.org
   Summary: Review Request: qpid-proton - Proton is a high
performance, lightweight messaging library
Regression: ---
  Story Points: ---
Classification: Fedora
OS: Unspecified
  Reporter: dpie...@redhat.com
  Type: Bug
 Documentation: ---
  Hardware: Unspecified
Mount Type: ---
Status: NEW
 Component: Package Review
   Product: Fedora

Spec URL: http://mcpierce.fedorapeople.org/rpms/qpid-proton.spec
SRPM URL: http://mcpierce.fedorapeople.org/rpms/qpid-proton-0.2-1.fc17.src.rpm
Description: Proton is a high performance, lightweight messaging library. It
can be used in the widest range of messaging applications including brokers,
client libraries, routers, bridges, proxies, and more. Proton is based on the
AMQP 1.0 messaging standard. Using Proton it is trivial to integrate with the
AMQP 1.0 ecosystem from any platform, environment, or language.
Scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=4662498
Fedora Account System Username: mcpierce

-- 
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 873813] Review Request: gnome-shell-search-github-repositories - Search your Github Repos from the gnome-shell

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=873813

--- Comment #2 from Ralph Bean rb...@redhat.com ---
New version out with support for searching github Organizations.

Spec URL: http://threebean.org/rpm/gnome-shell-search-github-repositories.spec
SRPM URL:
http://threebean.org/rpm/gnome-shell-search-github-repositories-1.0.0-1.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 864535] Review Request: libvirt-designer - Libvirt configuration designer

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=864535

Michal Privoznik mpriv...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |NEXTRELEASE
Last Closed||2012-11-07 09:32:50

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

[Bug 836368] Review Request: rubygem-bicho - Library to access Bugzilla

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=836368

--- Comment #3 from Darryl L. Pierce dpie...@redhat.com ---
(In reply to comment #2)
 Also:
 
 * The spec file doesn't run tests. You should always run the test suite,
 especially if it's provided by upstream. Include the tests that come with
 the gem and run them in %check section. If they are based on Test::Unit you
 can use minitest to run them (which is a recommended way).
 
 In your spec file:
 
 - require minitest
 BuildRequires:  rubygem(minitest)
 
 - add check section that goes after %build
 %check
 pushd .%{gem_instdir}
 testrb -Ilib test

The test that fails has a dependency on Novell. So, for now, I'm going to skip
adding any test checks to the specfile.

Scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=4662553

-- 
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 874128] New: Review Request: php-horde-Horde-Stream - Horde stream handler

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=874128

Bug ID: 874128
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: php-horde-Horde-Stream - Horde stream
handler
Regression: ---
  Story Points: ---
Classification: Fedora
OS: Linux
  Reporter: fed...@famillecollet.com
  Type: ---
 Documentation: ---
  Hardware: All
Mount Type: ---
Status: NEW
 Component: Package Review
   Product: Fedora

Spec URL:
https://raw.github.com/remicollet/remirepo/master/php/horde/php-horde-Horde-Stream/php-horde-Horde-Stream.spec
SRPM URL:
http://rpms.famillecollet.com/SRPMS/php-horde-Horde-Stream-1.1.1-1.remi.src.rpm
Description: 
An object-oriented interface to assist in creating and storing PHP stream
resources, and to provide utility methods to access and manipulate the
stream contents.

Fedora Account System Username: remi

-- 
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 874128] Review Request: php-horde-Horde-Stream - Horde stream handler

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=874128

Remi Collet fed...@famillecollet.com changed:

   What|Removed |Added

 Depends On||785606 (horde-test)

-- 
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 785606] Review Request: php-horde-Horde-Test - Horde testing base classes

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=785606

Remi Collet fed...@famillecollet.com changed:

   What|Removed |Added

 Blocks||874128

-- 
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 836368] Review Request: rubygem-bicho - Library to access Bugzilla

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=836368

Bohuslav Slavek Kabrda bkab...@redhat.com changed:

   What|Removed |Added

 CC||bkab...@redhat.com

--- Comment #4 from Bohuslav Slavek Kabrda bkab...@redhat.com ---
Hi Darryl,
rubygem-bicho depends on rubygem-inifile in runtime. I don't see inifile in
Fedora, nor it has an opened review. I think you should first package all the
dependencies before actually finishing this review. Inifile seems to be the
only thing you will need.

-- 
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 785606] Review Request: php-horde-Horde-Test - Horde testing base classes

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=785606

Remi Collet fed...@famillecollet.com changed:

   What|Removed |Added

 Blocks||874172

-- 
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 874172] New: Review Request: php-horde-Horde-Secret - Secret Encryption API

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=874172

Bug ID: 874172
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: php-horde-Horde-Secret - Secret
Encryption API
Regression: ---
  Story Points: ---
Classification: Fedora
OS: Linux
  Reporter: fed...@famillecollet.com
  Type: ---
 Documentation: ---
  Hardware: All
Mount Type: ---
Status: NEW
 Component: Package Review
   Product: Fedora

Spec URL:
https://raw.github.com/remicollet/remirepo/master/php/horde/php-horde-Horde-Secret/php-horde-Horde-Secret.spec
SRPM URL:
http://rpms.famillecollet.com/SRPMS/php-horde-Horde-Secret-2.0.0-2.remi.src.rpm
Description: 
An API for encrypting and decrypting small pieces of data with the use of a
shared key.

Fedora Account System Username: remi

-- 
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 874172] Review Request: php-horde-Horde-Secret - Secret Encryption API

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=874172

Remi Collet fed...@famillecollet.com changed:

   What|Removed |Added

 CC||n...@fedoraproject.org
 Depends On||785606 (horde-test)

-- 
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 874184] Review Request: peervpn - A VPN software using full mesh network topology

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=874184

Jan Cholasta jchol...@redhat.com changed:

   What|Removed |Added

 Blocks||177841 (FE-NEEDSPONSOR)

-- 
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 874184] New: Review Request: peervpn - A VPN software using full mesh network topology

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=874184

Bug ID: 874184
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: peervpn - A VPN software using full
mesh network topology
Regression: ---
  Story Points: ---
Classification: Fedora
OS: Linux
  Reporter: jchol...@redhat.com
  Type: ---
 Documentation: ---
  Hardware: All
Mount Type: ---
Status: NEW
 Component: Package Review
   Product: Fedora

Spec URL: http://dl.dropbox.com/u/464637/peervpn.spec
SRPM URL: http://dl.dropbox.com/u/464637/peervpn-0.028-1.fc18.src.rpm
Description: PeerVPN is software that builds virtual Ethernet networks between
multiple computers. It uses full mesh network topology and can automatically
build tunnels through firewalls and NATs. It supports shared key encryption and
authentication.
Fedora Account System Username: jcholast

Rpmlint output:
$ rpmlint SPECS/peervpn.spec SRPMS/peervpn-0.028-1.fc18.src.rpm
RPMS/x86_64/peervpn-0.028-1.fc18.x86_64.rpm
peervpn.x86_64: W: only-non-binary-in-usr-lib
- This is caused by the included systemd unit file.
peervpn.x86_64: E: non-standard-dir-perm /etc/peervpn 0700L
- This is done on purpose, the directory contains configuration files which
contain sensitive information, such as passwords, in plain text.
peervpn.x86_64: W: no-manual-page-for-binary peervpn
- Upstream does not provide the manual page ATM.
2 packages and 1 specfiles checked; 1 errors, 2 warnings.

Koji build URL: http://koji.fedoraproject.org/koji/taskinfo?taskID=4662803

This is my first package, I'm looking for sponsorship.

-- 
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 869121] Review Request: php-sabredav-Sabre_CalDAV - CalDAV plugin for Sabre_DAV

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=869121

Matthew Miller mat...@redhat.com changed:

   What|Removed |Added

 CC||mat...@redhat.com
 Blocks||858841 (owncloud)

-- 
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 858841] Review Request: owncloud - Private file sync and share server

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

Matthew Miller mat...@redhat.com changed:

   What|Removed |Added

 Depends On||869123

-- 
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 869123] Review Request: php-sabredav-Sabre_CardDAV - CardDAV plugin for Sabre_DAV

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=869123

Matthew Miller mat...@redhat.com changed:

   What|Removed |Added

 CC||mat...@redhat.com
 Blocks||858841 (owncloud)

-- 
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 858841] Review Request: owncloud - Private file sync and share server

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

Matthew Miller mat...@redhat.com changed:

   What|Removed |Added

 Depends On||869121

-- 
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 858841] Review Request: owncloud - Private file sync and share server

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

--- Comment #25 from Matthew Miller mat...@redhat.com ---
Where is php-pear-MDB2-Driver-sqlite coming from? I don't see a review
request

-- 
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 865630] Review Request: python-pyvfs - simple python vfs module

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=865630

Saveliev Peter p...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |NEXTRELEASE
Last Closed||2012-11-07 12:12:40

-- 
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 865915] Review Request: python-py9p - Pure Python implementation of 9p protocol

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=865915

Saveliev Peter p...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |NEXTRELEASE
Last Closed||2012-11-07 12:12:49

-- 
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 867159] Review Request: zabbix20 - Open-source monitoring solution for your IT infrastructure

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=867159

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

   What|Removed |Added

 Status|MODIFIED|ON_QA

-- 
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 834481] Review Request: lttng-tools - LTTng control and utility programs

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=834481

--- Comment #11 from Fedora Update System upda...@fedoraproject.org ---
lttng-tools-2.0.4-2.el6 has been pushed to the Fedora EPEL 6 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 867159] Review Request: zabbix20 - Open-source monitoring solution for your IT infrastructure

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=867159

--- Comment #8 from Fedora Update System upda...@fedoraproject.org ---
zabbix20-2.0.3-3.el6 has been pushed to the Fedora EPEL 6 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 827810] Review Request: obnam - An easy, secure backup program

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=827810

Anton Shevtsov shevtsov.an...@gmail.com changed:

   What|Removed |Added

 CC||shevtsov.an...@gmail.com

--- Comment #24 from Anton Shevtsov shevtsov.an...@gmail.com ---
obnam does not work in CentOS6.
try simple command:

obnam backup --verbose --repository /tmp/OBNAM/ sftp://u...@domain.tld/tmp/

you get is 'timeout lock' always

-- 
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 873408] Review Request: php-horde-Horde-Role - PEAR installer role used to install Horde components

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=873408

--- Comment #1 from Remi Collet fed...@famillecollet.com ---
Minor fix to have scriplet succeed.
https://github.com/remicollet/remirepo/commit/5124d3a3b566a0d31c8e93f5335159e3ec5b7993

Spec URL:
https://raw.github.com/remicollet/remirepo/master/php/horde/php-horde-Horde-Role/php-horde-Horde-Role.spec
SRPM URL:
http://rpms.famillecollet.com/SRPMS/php-horde-Horde-Role-1.0.0-3.remi.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 858841] Review Request: owncloud - Private file sync and share server

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

--- Comment #26 from Remi Collet fed...@famillecollet.com ---
php-pear-MDB2-Driver-sqlite requires php sqlite extension (v2) which is
deprecated for years (pdo_sqlite and sqlite3 exists), moved from php 5.4
sources to pecl (but unmaintained), and removed from fedora (for years).

So the sqlite sub package could probably be dropped.

-- 
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 815018] Review Request: nodejs - javascript fast build framework

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=815018

Toshio Ernie Kuratomi a.bad...@gmail.com changed:

   What|Removed |Added

 CC||a.bad...@gmail.com

--- Comment #43 from Toshio Ernie Kuratomi a.bad...@gmail.com ---
Could we all take a moment to think of other ways to resolve our problems?  I
think that language differences are playing a role in inflating the differences
here but that there's also a basic conflict that we should try to address
calmly.

Fedora is a Project as well as a Product.  The project aims not just to produce
the Fedora distribution but also to teach people how to be better packagers and
to foster a collaborative environment for people to make that eventual Fedora
distribution the best that it can be.

@adrian - you do have experience with rpm packaging.  But there are many things
about packaging for Fedora and working on upstream projects that you may not
have experience with.  The free software ecosystem is very large and diverse so
this is always going to be the case.  I have been working on the Packaging
Guidelines since the beginning of Fedora and there are definitely types of
packages that I *would* be a newbie at packaging.  There's no shame in this.

With this package, we're running across some things that you might not have
dealt with before. What is a bundled library?  How do I deal with them?  What
changes to the upstream code do I have to make to fix those?  Some of those
answers are on the
http://fedoraproject.org/wiki/Packaging:Treatment_Of_Bundled_Libraries page
that has already been linked.  Others are expected to be general knowledge by
people who are doing packaging.  Still others require someone to learn the
programming language the upstream is written in, become involved with the
upstream communities, and submit patches to them to fix issues.

These are big commitments of time and effort.  sgallagh has started investing
that time.  Unless you are willing to put in that time and start examining the
source code, figuring out what changes have been made, looking for ways to
merge those patches upstream, and in general, becoming just as much a member of
the upstream community as you are a member of the Fedora community, it would be
extremely beneficial for you to do everything in your power to accommodate him. 

@mrunge - In Fedora, we give people the opportunity to succeed or fail and more
importantly, the opportunity to learn while they are failing and then correct
their mistakes.  We cannot pay people with money for the work that they are
willing to do.  Instead we pay them by teaching them to be better packagers
and helping them to achieve things that they could not do on their own.  In
some cases, like this, it may well be faster and more efficient to do the work
yourself than to have to teach another person as you're doing that work.  But
if we don't take the time to teach we'll gradually marginalize people outside
of our inner circle and then Fedora will shrink and die.

-- 
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 874238] New: Review Request: DOM Tooltip (aka domTT) is a Javascript widget

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=874238

Bug ID: 874238
QA Contact: extras...@fedoraproject.org
  Severity: unspecified
   Version: rawhide
  Priority: unspecified
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
  Assignee: nob...@fedoraproject.org
   Summary: Review Request: DOM Tooltip (aka domTT) is a
Javascript widget
Regression: ---
  Story Points: ---
Classification: Fedora
OS: Unspecified
  Reporter: mgans...@alice.de
  Type: Bug
 Documentation: ---
  Hardware: Unspecified
Mount Type: ---
Status: NEW
 Component: Package Review
   Product: Fedora

DOM Tooltip (aka domTT) is a Javascript widget, released under the Apache 2.0
license, which allows developers to add customized tool-tips to their web
sites.

Spec URL:
https://www.disk.dsl.o2online.de/FclyPlh/RPMS/VDR/vdr-live/domtt-0.7.3-1/domtt.spec?a=GlnyuPEdRDc

SRPM URL:
https://www.disk.dsl.o2online.de/FclyPlh/RPMS/VDR/vdr-live/domtt-0.7.3-1/domtt-0.7.3-1.fc17.src.rpm?a=1e5n1nyvDqk

rpmlint domtt-0.7.3-1.fc17.src.rpm
domtt.src:24: W: macro-in-comment %build
domtt.src: W: no-%build-section
1 packages and 0 specfiles checked; 0 errors, 2 warnings.

rpmlint domtt-0.7.3-1.fc17.x86_64.rpm 
domtt.x86_64: E: no-binary
1 packages and 0 specfiles checked; 1 errors, 0 warnings.

-- 
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 827810] Review Request: obnam - An easy, secure backup program

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=827810

--- Comment #25 from Ben Boeckel maths...@gmail.com ---
This is probably a bug for upstream. Depends on what --verbose output I
suppose.

-- 
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 874238] Review Request: DOM Tooltip (aka domTT) is a Javascript widget

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=874238

MartinKG mgans...@alice.de changed:

   What|Removed |Added

 CC||mru...@redhat.com
 Blocks||821233

-- 
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 859675] Review Request: wcd - chdir for DOS and Unix

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=859675

Erwin Waterlander water...@xs4all.nl changed:

   What|Removed |Added

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

--- Comment #23 from Erwin Waterlander water...@xs4all.nl ---
Package Change Request
==
Package Name: wcd
New Branches: f17
Owners: waterlan
InitialCC: mrunge twaugh

Make wcd available in the current f17 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 853514] Review Request: rubygem-clouddb - Ruby interface into the Rackspace Cloud DB service

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=853514

--- Comment #19 from Fedora Update System upda...@fedoraproject.org ---
Package rubygem-clouddb-0.0.1-3.fc18:
* should fix your issue,
* was pushed to the Fedora 18 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing rubygem-clouddb-0.0.1-3.fc18'
as soon as you are able to.
Please go to the following url:
https://admin.fedoraproject.org/updates/FEDORA-2012-17719/rubygem-clouddb-0.0.1-3.fc18
then log in and leave karma (feedback).

-- 
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 853514] Review Request: rubygem-clouddb - Ruby interface into the Rackspace Cloud DB service

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=853514

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

   What|Removed |Added

 Status|MODIFIED|ON_QA

-- 
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 821233] Review Request: vdr-live - An interactive web interface for VDR

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=821233

MartinKG mgans...@alice.de changed:

   What|Removed |Added

 Depends On||874238

-- 
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 859675] Review Request: wcd - chdir for DOS and Unix

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=859675

--- Comment #24 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 874249] New: Review Request: rubygem-inifile - INI file reader and writer

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=874249

Bug ID: 874249
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: rubygem-inifile - INI file reader and
writer
Regression: ---
  Story Points: ---
Classification: Fedora
OS: Linux
  Reporter: dpie...@redhat.com
  Type: ---
 Documentation: ---
  Hardware: All
Mount Type: ---
Status: NEW
 Component: Package Review
   Product: Fedora

Spec URL: http://mcpierce.fedorapeople.org/rpms/rubygem-inifile.spec
SRPM URL:
http://mcpierce.fedorapeople.org/rpms/rubygem-inifile-2.0.2-1.fc17.src.rpm
Scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=4663474
Description: Although made popular by Windows, INI files can be used on any
system thanks to their flexibility. They allow a program to store configuration
data, which can then be easily parsed and changed. Two notable systems that use
the INI format are Samba and Trac.
Fedora Account System Username: mcpierce

-- 
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 874105] Review Request: qpid-proton - Proton is a high performance, lightweight messaging library

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=874105

Darryl L. Pierce dpie...@redhat.com changed:

   What|Removed |Added

  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 858841] Review Request: owncloud - Private file sync and share server

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

--- Comment #27 from Gregor Tätzner gre...@freenet.de ---
Guys, did you spotted /owncloud/lib/MDB2/Driver/... ?
They have their own sqlite driver included. I'm not sure where did that came
from. Is it some sort of fork? If yes, I could move it to the owncloud-sqlite
subpackage.

So for the moment, I can just drop the req to php-pear-MDB2-Driver-sqlite
without disabling sqlite

(In reply to comment #26)
 So the sqlite sub package could probably be dropped.

We still want a sqlite subpackage in order to not pull in mysql or postgresql

-- 
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 756321] Review Request: csmith - Tool to generate random C programs for compiler testing

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=756321

--- Comment #20 from Jerry James loganje...@gmail.com ---
Hmmm, that results in the main csmith package creating both
/usr/share/doc/csmith and /usr/share/doc/csmith-2.1.0.  Can you get all of the
documentation files into one or the other?

Also, the BuildRoot tag in the spec file is unnecessary.  I recommend deleting
it.

This MUST item has not been addressedd:
[!]: If the package is under multiple licenses, the licensing breakdown must
 be documented in the spec.

Everything else looks good.

-- 
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 865535] Review Request: python-datanommer-models - SQLAlchemy models for datanommer

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=865535

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

   What|Removed |Added

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

--- Comment #8 from Mario Blättermann mario.blaetterm...@gmail.com ---
(In reply to comment #7)
 Mario, I thought I wrote a note to the list but it must have gotten lost in
 the tubes.. perhaps queued for moderation?
 
 Either way, do you think my removal of the explicit Conflicts tag is
 acceptable?

I think it's OK. Well, not really canonical, but I hope you will deprecate
datanommer in the foreseeable future ;)

Scratch build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=4663561


$ rpmlint -i -v *
python-datanommer-models.src: I: checking
python-datanommer-models.src: I: checking-url
http://pypi.python.org/pypi/datanommer.models (timeout 10 seconds)
python-datanommer-models.src: I: checking-url
http://pypi.python.org/packages/source/d/datanommer.models/datanommer.models-0.2.0.tar.gz
(timeout 10 seconds)
python-datanommer-models.noarch: I: checking
python-datanommer-models.noarch: I: checking-url
http://pypi.python.org/pypi/datanommer.models (timeout 10 seconds)
python-datanommer-models.spec: I: checking-url
http://pypi.python.org/packages/source/d/datanommer.models/datanommer.models-0.2.0.tar.gz
(timeout 10 seconds)
2 packages and 1 specfiles checked; 0 errors, 0 warnings.

No issues.


-
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.
GPLv3+
[+] 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 *
57f9486bea0c3854e6ca8dbdb454737bff6ab54083f1264795eacab6bd527d33 
datanommer.models-0.2.0.tar.gz
57f9486bea0c3854e6ca8dbdb454737bff6ab54083f1264795eacab6bd527d33 
datanommer.models-0.2.0.tar.gz.orig

[+] MUST: The package MUST successfully compile and build into binary rpms on
at least one primary architecture.
[.] MUST: If the package does not successfully compile, build or work on an
architecture, then those architectures should be listed in the spec in
ExcludeArch. Each architecture listed in ExcludeArch MUST have a bug filed in
bugzilla, describing the reason that the package does not compile/build/work on
that architecture. The bug number MUST be placed in a comment, next to the
corresponding ExcludeArch line.
[+] MUST: All build dependencies must be listed in BuildRequires, except for
any that are listed in the exceptions section of the Packaging Guidelines ;
inclusion of those as BuildRequires is optional. Apply common sense.
[.] MUST: The spec file MUST handle locales properly. This is done by using the
%find_lang macro. Using %{_datadir}/locale/* is strictly forbidden.
[.] MUST: Every binary RPM package (or subpackage) which stores shared library
files (not just symlinks) in any of the dynamic linker's default paths, must
call ldconfig in %post and %postun.
[.] MUST: Packages must NOT bundle copies of system libraries.
[.] MUST: If the package is designed to be relocatable, the packager must state
this fact in the request for review, along with the rationalization for
relocation of that specific package. Without this, use of Prefix: /usr is
considered a blocker.
[+] MUST: A package must own all directories that it creates. If it does not
create a directory that it uses, then it should require a package which does
create that directory.
[+] MUST: A Fedora package must not list a file more than once in the spec
file's %files listings. (Notable exception: license texts in specific
situations)
[+] MUST: Permissions on files must be set properly. Executables should be set
with executable 

[Bug 869088] Review Request: latte-integrale - Lattice point enumeration

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=869088

Jerry James loganje...@gmail.com changed:

   What|Removed |Added

  Flags|needinfo?   |

--- Comment #3 from Jerry James loganje...@gmail.com ---
Yes, this package is intended only for Fedora 18 and beyond.  I wasn't aware of
the cddlib situation; thanks for pointing that out.

-- 
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 873793] Review Request: skb - Simple keyboard layout indicator

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=873793

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

   What|Removed |Added

 CC||mario.blaetterm...@gmail.co
   ||m

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

$ rpmlint -i -v *
skb.src: I: checking
skb.src: I: checking-url http://plhk.ru/ (timeout 10 seconds)
skb.src: I: checking-url http://plhk.ru/static/skb/skb-0.4.tar.gz (timeout 10
seconds)
skb.i686: I: checking
skb.i686: I: checking-url http://plhk.ru/ (timeout 10 seconds)
skb.i686: E: incorrect-fsf-address /usr/share/doc/skb-0.4/LICENSE
The Free Software Foundation address in this file seems to be outdated or
misspelled.  Ask upstream to update the address, or if this is a license file,
possibly the entire file with a new copy available from the FSF.

skb.i686: W: no-manual-page-for-binary skb
Each executable in standard binary directories should have a man page.

skb.x86_64: I: checking
skb.x86_64: I: checking-url http://plhk.ru/ (timeout 10 seconds)
skb.x86_64: E: incorrect-fsf-address /usr/share/doc/skb-0.4/LICENSE
The Free Software Foundation address in this file seems to be outdated or
misspelled.  Ask upstream to update the address, or if this is a license file,
possibly the entire file with a new copy available from the FSF.

skb.x86_64: W: no-manual-page-for-binary skb
Each executable in standard binary directories should have a man page.

skb-debuginfo.i686: I: checking
skb-debuginfo.i686: I: checking-url http://plhk.ru/ (timeout 10 seconds)
skb-debuginfo.i686: E: empty-debuginfo-package
This debuginfo package contains no files.  This is often a sign of binaries
being unexpectedly stripped too early during the build, rpmbuild not being
able to strip the binaries, the package actually being a noarch one but
erratically packaged as arch dependent, or something else.  Verify what the
case is, and if there's no way to produce useful debuginfo out of it, disable
creation of the debuginfo package.

skb-debuginfo.x86_64: I: checking
skb-debuginfo.x86_64: I: checking-url http://plhk.ru/ (timeout 10 seconds)
skb-debuginfo.x86_64: E: empty-debuginfo-package
This debuginfo package contains no files.  This is often a sign of binaries
being unexpectedly stripped too early during the build, rpmbuild not being
able to strip the binaries, the package actually being a noarch one but
erratically packaged as arch dependent, or something else.  Verify what the
case is, and if there's no way to produce useful debuginfo out of it, disable
creation of the debuginfo package.

skb.spec: I: checking-url http://plhk.ru/static/skb/skb-0.4.tar.gz (timeout 10
seconds)
5 packages and 1 specfiles checked; 4 errors, 2 warnings.



Due to that CFLAGS is hardcoded in config.mk, the debuginfo packages are empty.
Call make as follows:

make CFLAGS=%{optflags} %{?_smp_mflags}

This should help to fix the issue. Moreover, the %{_prefix} macro is senseless,
because it is the default and the Makefile points to /usr anyway.

Regarding the incorrect FSF address: Don't touch it. Just inform the upstream
developer about the outdated address so that he can fix it in future releases.

(In reply to comment #1)
 - libX11-devel can be omitted
 https://fedoraproject.org/wiki/Packaging/Guidelines#BuildRequires

No it can't be omitted. It is needed for building the package and is not part
of the basic build environment:
http://fedoraproject.org/wiki/Packaging:Guidelines#Exceptions_2

Indeed, the description could be more informative. But the upstream developer
is very thriftily with additional info, and there seems to be no package in
other distributions where we could borrow the description from. In this case,
we can leave it as is. Anyway, if you have some idea to tweak it, do 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 874238] Review Request: DOM Tooltip (aka domTT) is a Javascript widget

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=874238

--- Comment #1 from MartinKG mgans...@alice.de ---
Spec URL:
https://www.disk.dsl.o2online.de/FclyPlh/RPMS/VDR/vdr-live/domtt-0.7.3-2/domtt.spec?a=NRE0_kqpfSs

SRPM URL:
https://www.disk.dsl.o2online.de/FclyPlh/RPMS/VDR/vdr-live/domtt-0.7.3-2/domtt-0.7.3-2.fc17.src.rpm?a=8NoeyePoKEs

%changelog
* Wed Nov 7 2012 Martin Gansser linux4martin[at]gmx.de - 0.7.3-2
- added js 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 821917] Review Request: mu - maildir utility with Emacs support

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=821917

--- Comment #10 from Jos de Kloe josdek...@gmail.com ---
Thanks for updating.

rpmbuild and mock results look fine again, and rpmlint outputs are identical to
your 0.9.902 version.

Looking at https://fedoraproject.org/wiki/Packaging:Emacs (section 1.4 package
contents), the Why package the source elisp? note says that the main purpose
of the elisp package is to get help on variables and provide debugging
information on the *.elc files.
Therefore the emacs-mu4e-el is only useful for this purpose when emacs-mu4e is
installed as well, so adding the directory ownership to emacs-mu4e is fine I
think.

Thinking about this, I think emacs-mu4e-el should explicitely require
emacs-mu4e.
I inspected randomly two other packages that package emacs extensions and they
do this as well (looked at mercurial and gnuplot).

-- 
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 865371] Review Request : SciD is a collection of numerical routines using Blas/Lapack

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=865371

Jos de Kloe josdek...@gmail.com changed:

   What|Removed |Added

 CC||josdek...@gmail.com

--- Comment #1 from Jos de Kloe josdek...@gmail.com ---
Your srpm and spec file seem no longer available.
Could you make them available again 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 858841] Review Request: owncloud - Private file sync and share server

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

--- Comment #28 from Matthew Miller mat...@redhat.com ---
(In reply to comment #27)
 Guys, did you spotted /owncloud/lib/MDB2/Driver/... ?
 They have their own sqlite driver included. I'm not sure where did that came
 from. Is it some sort of fork? If yes, I could move it to the
 owncloud-sqlite subpackage.


I dunno. I'm just looking at the dependencies in the spec file.

But if it's some sort of fork, we should look at whether it should be
unbundled.


 (In reply to comment #26)
  So the sqlite sub package could probably be dropped.
 We still want a sqlite subpackage in order to not pull in mysql or postgresql

+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 865371] Review Request : SciD is a collection of numerical routines using Blas/Lapack

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=865371

--- Comment #2 from MERCIER Jonathan bioinfornat...@gmail.com ---
yes sorry they just move to a subdirectory

http://bioinfornatics.fedorapeople.org/packages/scid.spec
http://bioinfornatics.fedorapeople.org/packages/scid-1-0.20120429git51236f3.fc17.1.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 846008] Review Request: dsqlite - DSQLite is a Hight level library for SQLite in D programming

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=846008

--- Comment #15 from MERCIER Jonathan bioinfornat...@gmail.com ---
little typo into french description 
bibliotheque - bibliothèque
language - langage

-- 
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 856238] Review Request: scratch - Programming language learning environment for stories, games, music and art

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=856238

Matthew Miller mat...@redhat.com changed:

   What|Removed |Added

 Depends On||861633

--- Comment #18 from Matthew Miller mat...@redhat.com ---
Fedora Legal has cleared the license as is. I've made an updated package which
assumes the squeak-vm in progress. This works with the latest from bug #861633,
with the caveat that you might need to edit /usr/bin/squeak to say lib64. 

Audio works and everything. Hopefully that package will land soon. In the
meantime:

Spec URL: http://mattdm.org/misc/fedora/scratch.spec
SRPM URL: http://mattdm.org/misc/fedora/scratch-1.4.0.7-4.fc18.mattdm.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 846008] Review Request: dsqlite - DSQLite is a Hight level library for SQLite in D programming

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=846008

--- Comment #16 from MERCIER Jonathan bioinfornat...@gmail.com ---
about unused-direct-shlib-dependency is maybe a ldc bug i wait upstream
https://github.com/ldc-developers/ldc/issues/228

so you can skip this as the bug is part of ldc an will be fixed

-- 
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 818457] Review Request: rubygem-ruby-debug-base19 - Fast Ruby debugger - core component

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=818457

--- Comment #10 from Fedora Update System upda...@fedoraproject.org ---
rubygem-ruby-debug19-0.11.6-3.fc17, rubygem-ruby-debug-base19-0.11.26-3.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 818458] Review Request: rubygem-ruby-debug19 - Command line interface (CLI) for ruby-debug-base

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=818458

--- Comment #15 from Fedora Update System upda...@fedoraproject.org ---
rubygem-ruby-debug19-0.11.6-3.fc17, rubygem-ruby-debug-base19-0.11.26-3.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 848388] Review Request: liblognorm - Tool to normalize log data

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

--- Comment #17 from Fedora Update System upda...@fedoraproject.org ---
liblognorm-0.3.4-4.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 848388] Review Request: liblognorm - Tool to normalize log data

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

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

   What|Removed |Added

 Status|MODIFIED|ON_QA

-- 
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 874105] Review Request: qpid-proton - Proton is a high performance, lightweight messaging library

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=874105

Sébastien Boisvert s...@boisvert.info changed:

   What|Removed |Added

 CC||s...@boisvert.info

--- Comment #1 from Sébastien Boisvert s...@boisvert.info ---
This is not a formal review as I am not sponsored yet.

$ rpmlint  qpid-proton-*rpm
qpid-proton.x86_64: W: shared-lib-calls-exit /usr/lib64/libqpid-proton.so.1.0.0
exit@GLIBC_2.2.5
qpid-proton.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/libqpid-proton.so
3 packages and 0 specfiles checked; 0 errors, 2 warnings.

Not sure why rpmlint says that %{_libdir}/libqpid-proton.so* should to in
devel.

Builds fine with mock.

mock -r  fedora-17-x86_64 rebuild qpid-proton-0.2-1.fc17.src.rpm  mock.qp

When I install qpid-proton and do man proton, I get this in DESCRIPTION:

  
/home/seb/rpmbuild/BUILDROOT/qpid-proton-0.2-1.fc17.x86_64/usr/bin/proton: 
error while loading shared libraries: libqpid-proton.so.1: can‐
   not open shared object file: No such file or directory

Same with man proton-dump.

-- 
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 865371] Review Request : SciD is a collection of numerical routines using Blas/Lapack

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=865371

Sébastien Boisvert s...@boisvert.info changed:

   What|Removed |Added

 CC||s...@boisvert.info

--- Comment #3 from Sébastien Boisvert s...@boisvert.info ---
*This is not a formal review because I am not sponsored yet.

Small mispelling / erreur d'ortographe:

 SciD est une collection de fonction numériques et de lien écrit enet pour le
de lien écrit en et pour le

 Summary:It is a hcollection of numerical routines for D language
collection

. /build lib %{_d_optflags} 
 pid_list=( ${pid_list[@]} ${!})

Is it useful to start process in background and then wait for them to finish ?


Builds fine with mock.

rpmlint throws a few warnings:

$ rpmlint scid.spec 
scid.spec: W: invalid-url Source0: scid-20120429git51236f3.tar.xz
0 packages and 1 specfiles checked; 0 errors, 1 warnings.

$ rpmlint  scid-*.rpm
scid.x86_64: I: enchant-dictionary-not-found fr_FR
scid.x86_64: W: unstripped-binary-or-object /usr/lib64/libscid-ldc.so.1.0.0
scid.x86_64: W: shared-lib-calls-exit /usr/lib64/libscid-ldc.so.1.0.0
exit@GLIBC_2.2.5
scid.x86_64: W: shared-lib-calls-exit /usr/lib64/libscid-ldc.so.1.0.0
exit@@GLIBC_2.2.5
scid-devel.x86_64: W: spelling-error Summary(en_US) hcollection - collection,
h collection, collectible
scid-devel.x86_64: I: enchant-dictionary-not-found fr
scid-devel.x86_64: W: no-documentation
2 packages and 0 specfiles checked; 0 errors, 5 warnings.

$ rpmlint scid scid-devel
scid.x86_64: I: enchant-dictionary-not-found fr_FR
scid.x86_64: W: unstripped-binary-or-object /usr/lib64/libscid-ldc.so.1.0.0
scid.x86_64: W: undefined-non-weak-symbol /usr/lib64/libscid-ldc.so.1.0.0
__data_start
scid.x86_64: W: undefined-non-weak-symbol /usr/lib64/libscid-ldc.so.1.0.0
_Dmain
scid.x86_64: W: unused-direct-shlib-dependency /usr/lib64/libscid-ldc.so.1.0.0
/lib64/librt.so.1
scid.x86_64: W: unused-direct-shlib-dependency /usr/lib64/libscid-ldc.so.1.0.0
/lib64/libdl.so.2
scid.x86_64: W: unused-direct-shlib-dependency /usr/lib64/libscid-ldc.so.1.0.0
/lib64/libpthread.so.0
scid.x86_64: W: unused-direct-shlib-dependency /usr/lib64/libscid-ldc.so.1.0.0
/lib64/libm.so.6
scid.x86_64: W: shared-lib-calls-exit /usr/lib64/libscid-ldc.so.1.0.0
exit@GLIBC_2.2.5
scid.x86_64: W: shared-lib-calls-exit /usr/lib64/libscid-ldc.so.1.0.0
exit@@GLIBC_2.2.5
scid-devel.x86_64: W: spelling-error Summary(en_US) hcollection - collection,
h collection, collectible
scid-devel.x86_64: I: enchant-dictionary-not-found fr
scid-devel.x86_64: W: no-documentation
2 packages and 0 specfiles checked; 0 errors, 11 warnings.

-- 
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 838775] Review Request: ghc-css-text - CSS parser and renderer

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=838775

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

   What|Removed |Added

 Status|ON_DEV  |MODIFIED

-- 
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 838775] Review Request: ghc-css-text - CSS parser and renderer

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=838775

--- Comment #5 from Fedora Update System upda...@fedoraproject.org ---
ghc-css-text-0.1.1-1.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/ghc-css-text-0.1.1-1.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 838775] Review Request: ghc-css-text - CSS parser and renderer

2012-11-07 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=838775

--- Comment #6 from Fedora Update System upda...@fedoraproject.org ---
ghc-css-text-0.1.1-1.fc18 has been submitted as an update for Fedora 18.
https://admin.fedoraproject.org/updates/ghc-css-text-0.1.1-1.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