[Bug 1196763] New: Review Request: perl-Tie-Cache - LRU Cache in Memory

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1196763

Bug ID: 1196763
   Summary: Review Request: perl-Tie-Cache - LRU Cache in Memory
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: de...@fateyev.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: http://www.fateyev.com/RPMS/Fedora20/testing/perl-Tie-Cache.spec
SRPM URL:
http://www.fateyev.com/RPMS/Fedora20/testing/SRPMS/perl-Tie-Cache-0.21-1.fc20.denf.src.rpm
Description: This module implements a least recently used (LRU) cache in memory
through a tie interface. Any time data is stored in the tied hash, that
key/value pair has an entry time associated with it, and as the cache fills up,
those members of the cache that are the oldest are removed to make room for new
entries.
Fedora Account System Username: dfateyev

Koji scratch build:
https://koji.fedoraproject.org/koji/taskinfo?taskID=9082740 (Rawhide)
https://koji.fedoraproject.org/koji/taskinfo?taskID=9082720 (Fedora 22)
https://koji.fedoraproject.org/koji/taskinfo?taskID=9082732 (EPEL 7)
https://koji.fedoraproject.org/koji/taskinfo?taskID=9082756 (EPEL 6)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1196551] Review Request: golang-github-shurcooL-sanitized_anchor_name - Package sanitized_anchor_name provides a func to create sanitized anchor names

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1196551

Jan Chaloupka  changed:

   What|Removed |Added

  Flags|fedora-cvs+ |fedora-cvs?



--- Comment #5 from Jan Chaloupka  ---
New Package SCM Request
===
Package Name: golang-github-shurcooL-sanitized_anchor_name
Short Description: Package sanitized_anchor_name provides a func to create
sanitized anchor names
Upstream URL: https://github.com/shurcooL/sanitized_anchor_name
Owners: jchaloup lsm5
Branches: f22 f21 f20 el6
InitialCC: golang-sig

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1196551] Review Request: golang-github-shurcooL-sanitized_anchor_name - Package sanitized_anchor_name provides a func to create sanitized anchor names

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1196551



--- Comment #4 from Jan Chaloupka  ---
$ fedprep.sh golang-github-shurcooL-sanitized_anchor_name
Cloning into 'golang-github-shurcooL-sanitized_anchor_name'...
warning: You appear to have cloned an empty repository.
Checking connectivity... done.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1196432] Review Request: golang-github-beorn7-perks - Effective Computation of Things

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1196432

Jan Chaloupka  changed:

   What|Removed |Added

  Flags|fedora-cvs+ |fedora-cvs?



--- Comment #4 from Jan Chaloupka  ---
Getting this error:

$ fedpkg clone golang-github-beorn7-perks
Cloning into 'golang-github-beorn7-perks'...
warning: You appear to have cloned an empty repository.
Checking connectivity... done.

$ gitk --all show no branches

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1196432] Review Request: golang-github-beorn7-perks - Effective Computation of Things

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1196432



--- Comment #5 from Jan Chaloupka  ---
New Package SCM Request
===
Package Name: golang-github-beorn7-perks
Short Description: Effective Computation of Things
Upstream URL: https://github.com/beorn7/perks
Owners: jchaloup lsm5
Branches: f22 f21 f20 el6
InitialCC: golang-sig

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1196363] Review Request: golang-github-docker-spdystream - A multiplexed stream library using spdy

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1196363

Jon Ciesla  changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1196551] Review Request: golang-github-shurcooL-sanitized_anchor_name - Package sanitized_anchor_name provides a func to create sanitized anchor names

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1196551

Jon Ciesla  changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1196551] Review Request: golang-github-shurcooL-sanitized_anchor_name - Package sanitized_anchor_name provides a func to create sanitized anchor names

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1196551



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

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1196432] Review Request: golang-github-beorn7-perks - Effective Computation of Things

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1196432

Jon Ciesla  changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1196432] Review Request: golang-github-beorn7-perks - Effective Computation of Things

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1196432



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

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1196368] Review Request: golang-github-golang-groupcache - Caching and cache-filling library as a replacement for memcached in many cases

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1196368



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

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1196368] Review Request: golang-github-golang-groupcache - Caching and cache-filling library as a replacement for memcached in many cases

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1196368

Jon Ciesla  changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1196361] Review Request: golang-github-gorilla-websocket - A WebSocket implementation for Go

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1196361

Jon Ciesla  changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1196361] Review Request: golang-github-gorilla-websocket - A WebSocket implementation for Go

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1196361



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

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 830869] Review Request: hpl - A Portable Implementation of the High-Performance Linpack Benchmark

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=830869



--- Comment #22 from Jaroslav Škarvada  ---
(In reply to Dave Love from comment #21)
> The current version doesn't build, with 
> 
> error: Directory not found:
> /builddir/build/BUILDROOT/hpl-2.1-4.fc23.x86_64/etc/mpich-x86_64/hpl
> 
> etc.

Probably bug in mpich, reported as bug 1196728.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1036754] Review Request: ttembed - Remove embedding limitations from TrueType fonts

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1036754



--- Comment #15 from Orion Poplawski  ---
Thank you!

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1181726] Merge Review Request: dejavu-fonts - DejaVu fonts

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1181726



--- Comment #16 from Fedora Update System  ---
dejavu-fonts-2.33-5.el5 has been submitted as an update for Fedora EPEL 5.
https://admin.fedoraproject.org/updates/dejavu-fonts-2.33-5.el5

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1196347] Review Request: f22-backgrounds – Fedora 22 default desktop background

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1196347

Jaroslav Reznik  changed:

   What|Removed |Added

 Blocks||1135100 (F22Blocker-kde)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1135100
[Bug 1135100] Fedora 22 Blocker KDE Tracker
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1181726] Merge Review Request: dejavu-fonts - DejaVu fonts

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1181726

Fedora Update System  changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1196361] Review Request: golang-github-gorilla-websocket - A WebSocket implementation for Go

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1196361

Jan Chaloupka  changed:

   What|Removed |Added

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



--- Comment #2 from Jan Chaloupka  ---
New Package SCM Request
===
Package Name: golang-github-gorilla-websocket
Short Description: A WebSocket implementation for Go
Upstream URL: https://github.com/gorilla/websocket
Owners: jchaloup lsm5
Branches: f22 f21 f20 el6
InitialCC: golang-sig

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1196368] Review Request: golang-github-golang-groupcache - Caching and cache-filling library as a replacement for memcached in many cases

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1196368

Jan Chaloupka  changed:

   What|Removed |Added

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



--- Comment #2 from Jan Chaloupka  ---
New Package SCM Request
===
Package Name: golang-github-golang-groupcache
Short Description: Caching and cache-filling library as a replacement for
memcached in many cases
Upstream URL: https://github.com/golang/groupcache
Owners: jchaloup lsm5
Branches: f22 f21 f20 el6
InitialCC: golang-sig

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1196432] Review Request: golang-github-beorn7-perks - Effective Computation of Things

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1196432

Jan Chaloupka  changed:

   What|Removed |Added

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



--- Comment #2 from Jan Chaloupka  ---
New Package SCM Request
===
Package Name: golang-github-beorn7-perks
Short Description: Effective Computation of Things
Upstream URL: https://github.com/beorn7/perks
Owners: jchaloup lsm5
Branches: f22 f21 f20 el6
InitialCC: golang-sig

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1196551] Review Request: golang-github-shurcooL-sanitized_anchor_name - Package sanitized_anchor_name provides a func to create sanitized anchor names

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1196551

Jan Chaloupka  changed:

   What|Removed |Added

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



--- Comment #2 from Jan Chaloupka  ---
New Package SCM Request
===
Package Name: golang-github-shurcooL-sanitized_anchor_name
Short Description: Package sanitized_anchor_name provides a func to create
sanitized anchor names
Upstream URL: https://github.com/shurcooL/sanitized_anchor_name
Owners: jchaloup lsm5
Branches: f22 f21 f20 el6
InitialCC: golang-sig

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1196347] Review Request: f22-backgrounds – Fedora 22 default desktop background

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1196347

Fedora Blocker Bugs Application  changed:

   What|Removed |Added

 Blocks||1043121
   ||(AlphaBlocker,F22AlphaBlock
   ||er)



--- Comment #4 from Fedora Blocker Bugs Application 
 ---
Proposed as a Blocker for 22-alpha by Fedora user jreznik using the blocker
tracking app because:

 The default desktop background must be different from that of the two previous
stable releases.

Default background has to be set up in release blocking desktops once the
package is built.


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1043121
[Bug 1043121] Fedora 22 Alpha blocker bug tracker
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1196363] Review Request: golang-github-docker-spdystream - A multiplexed stream library using spdy

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1196363

Jan Chaloupka  changed:

   What|Removed |Added

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



--- Comment #2 from Jan Chaloupka  ---
New Package SCM Request
===
Package Name: golang-github-docker-spdystream
Short Description: A multiplexed stream library using spdy
Upstream URL: https://github.com/docker/spdystream
Owners: jchaloup lsm5
Branches: f22 f21 f20 el6
InitialCC: golang-sig

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1195279] Review Request: preupgrade-assistant - Preupgrade assistant a tool for assess system before an upgrade

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1195279



--- Comment #7 from Zbigniew Jędrzejewski-Szmek  ---
(In reply to Petr Hracek from comment #6)
> As you can see in koji Python3 in rawhide hasn't built correctly and it
> still fails.
Oh, I see it now. But the previous build should still be available, so it
should not matter for you.

> Regarding configparser. All is working properly.
> configparser.py provides class ConfigParser in F22 and in F23 too.
> This was not changed.
The class name did not change. But the module name did.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1196363] Review Request: golang-github-docker-spdystream - A multiplexed stream library using spdy

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1196363

Lokesh Mandvekar  changed:

   What|Removed |Added

 CC||l...@redhat.com
   Assignee|nob...@fedoraproject.org|l...@redhat.com
  Flags||fedora-review+



--- Comment #1 from Lokesh Mandvekar  ---
- specfile conforms to golang packaging draft
- license correct and valid

package approved

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1196368] Review Request: golang-github-golang-groupcache - Caching and cache-filling library as a replacement for memcached in many cases

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1196368

Lokesh Mandvekar  changed:

   What|Removed |Added

 CC||l...@redhat.com
   Assignee|nob...@fedoraproject.org|l...@redhat.com
  Flags||fedora-review+



--- Comment #1 from Lokesh Mandvekar  ---
- specfile conforms to current golang packaging draft
- license correct and valid

package approved

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1196432] Review Request: golang-github-beorn7-perks - Effective Computation of Things

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1196432

Lokesh Mandvekar  changed:

   What|Removed |Added

 CC||l...@redhat.com
   Assignee|nob...@fedoraproject.org|l...@redhat.com
  Flags||fedora-review+



--- Comment #1 from Lokesh Mandvekar  ---
- specfile conforms to current golang packaging draft
- license correct and valid

package approved

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1196551] Review Request: golang-github-shurcooL-sanitized_anchor_name - Package sanitized_anchor_name provides a func to create sanitized anchor names

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1196551

Lokesh Mandvekar  changed:

   What|Removed |Added

 CC||l...@redhat.com
   Assignee|nob...@fedoraproject.org|l...@redhat.com
  Flags||fedora-review+



--- Comment #1 from Lokesh Mandvekar  ---
- specfile conforms to current golang packaging draft
- license correct and valid

package approved

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1196361] Review Request: golang-github-gorilla-websocket - A WebSocket implementation for Go

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1196361

Lokesh Mandvekar  changed:

   What|Removed |Added

 CC||l...@redhat.com
   Assignee|nob...@fedoraproject.org|l...@redhat.com
  Flags||fedora-review+



--- Comment #1 from Lokesh Mandvekar  ---
- specfile conforms to current golang packaging draft
- license correct and valid

package approved

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1195279] Review Request: preupgrade-assistant - Preupgrade assistant a tool for assess system before an upgrade

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1195279



--- Comment #6 from Petr Hracek  ---
Thanks for huge comments.

As you can see in koji Python3 in rawhide hasn't built correctly and it still
fails.

I am going to change LICENSE file as you recommended.

Regarding configparser. All is working properly.
configparser.py provides class ConfigParser in F22 and in F23 too.
This was not changed.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1196366] Review Request: openstack-designateclient - Client library for OpenStack Designate DNS API

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1196366



--- Comment #2 from Victoria Martinez de la Cruz  ---
(In reply to Haïkel Guémar from comment #1)
> One small typo: sed -i s/REDHATDESIGNATECLIENTVERSION/%{version}/
> designateclient/version.py (patch is correct)
> 
> Everything else looks good.

Oops, fixed, thanks Haïkel!

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1016677] Review Request: mathjax - JavaScript library to render math in the browser

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1016677



--- Comment #24 from Fedora Update System  ---
mathjax-2.4.0-1.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/mathjax-2.4.0-1.el6

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1189015] Review Request: python-MDAnalysis - Analyze and manipulate molecular dynamics trajectories

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1189015



--- Comment #1 from Antonio Trande  ---
Hi Dominik.

Building this package none test is performed:

...
Ran 0 tests in 0.757s

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1195279] Review Request: preupgrade-assistant - Preupgrade assistant a tool for assess system before an upgrade

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1195279



--- Comment #5 from Zbigniew Jędrzejewski-Szmek  ---
(In reply to Petr Hracek from comment #4)
> Today finally my mock is running.
> 
> But python3 does not exist rawhide yet.
> http://koji.fedoraproject.org/koji/packageinfo?packageID=9781
> And therefore ConfigParser is not available yet:(
When I look at the list, I see three F23 builds right at the top. I'm not sure
why do you think that there are no rawhide builds.

> ConfigParser class is defined in configparser.py file, though.
> ConfigParser (configparser.py) is part of subpackage python3-libs.
Yeah, it got renamed. You need to either rename it too, or if you're providing
Py2 compat, use something like:
  try:
import configparser
  except ImportError:
import ConfigParser as configparser

> Thanks for good idea with renaming binaries. It really makes sense.
> I am going to do that.
Cool, thanks.

> I tried to build package in Copr, but F22 branch does not exist yet.
You might need to update you mock package first. It carries the buildroot
definitions. F22 and F23 build roots have been working for the last few weeks
or so.

> I have updated SPEC file so that in F22 it uses python2. But Since F23 is
> uses python3.
This makes things much more complicated for no good reason. The spec file is
much more complicated, and it'll also be confusing for users. I'd strongly
advise against that.

Although not strictly required, the intent of %license is to be used with bare
filename, so that the file is installed in /usr/share/licenses. Please remove
LICENSE from %{_pkgdocdir}
and add in %files:
  %license LICENSE

fedora-review says:
- Sources used to build the package match the upstream source, as provided in
  the spec URL.
  Note: Upstream MD5sum check error, diff is in /var/tmp/1195279-preupgrade-
  assistant/diff.txt
  See: http://fedoraproject.org/wiki/Packaging/SourceURL

E: explicit-lib-dependency python3-libs
Yep, such dependencies are added automatically.

W: spurious-executable-perm /usr/share/man/man1/preupg.1.gz

E: non-executable-script
/usr/lib/python3.4/site-packages/preuputils/postupgrade.py 0644L
/usr/bin/python2

preupgrade-assistant.src:96: W: macro-in-comment %{__python2}
preupgrade-assistant.src:97: W: macro-in-comment %else
preupgrade-assistant.src:98: W: macro-in-comment %{__python3}
preupgrade-assistant.src:99: W: macro-in-comment %endif

I hope you can reverse the Python2 / 3 split. I'll hold on actually testing the
package and further review until you make the decision. Testing both versions
is twice as much work :)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1196347] Review Request: f22-backgrounds – Fedora 22 default desktop background

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1196347

Jon Ciesla  changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1187869] Review Request: kdocker - Dock any application in the system tray

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1187869



--- Comment #16 from Jon Ciesla  ---
Git done (by process-git-requests).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1196347] Review Request: f22-backgrounds – Fedora 22 default desktop background

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1196347



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

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1178912] Review Request: cairo-dock-plug-ins - Plug-ins files for Cairo-Dock

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1178912

Jon Ciesla  changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1178911] Review Request: cairo-dock - Light eye-candy fully themable animated dock

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1178911



--- Comment #20 from Jon Ciesla  ---
Git done (by process-git-requests).

Unretired.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1178912] Review Request: cairo-dock-plug-ins - Plug-ins files for Cairo-Dock

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1178912



--- Comment #42 from Jon Ciesla  ---
Git done (by process-git-requests).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1178911] Review Request: cairo-dock - Light eye-candy fully themable animated dock

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1178911

Jon Ciesla  changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1016677] Review Request: mathjax - JavaScript library to render math in the browser

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1016677



--- Comment #23 from Jon Ciesla  ---
Git done (by process-git-requests).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1016677] Review Request: mathjax - JavaScript library to render math in the browser

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1016677

Jon Ciesla  changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1187869] Review Request: kdocker - Dock any application in the system tray

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1187869

Raphael Groner  changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1187869] Review Request: kdocker - Dock any application in the system tray

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1187869



--- Comment #15 from Fedora Update System  ---
kdocker-4.8-2.fc21 has been submitted as an update for Fedora 21.
https://admin.fedoraproject.org/updates/kdocker-4.8-2.fc21

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1187869] Review Request: kdocker - Dock any application in the system tray

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1187869



--- Comment #14 from Fedora Update System  ---
kdocker-4.8-2.fc22 has been submitted as an update for Fedora 22.
https://admin.fedoraproject.org/updates/kdocker-4.8-2.fc22

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1187869] Review Request: kdocker - Dock any application in the system tray

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1187869

Fedora Update System  changed:

   What|Removed |Added

 Status|POST|MODIFIED



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1191498] Review Request: safelease - legacy locking mechanism for VDSM

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1191498

Douglas Schilling Landgraf  changed:

   What|Removed |Added

   Assignee|nob...@fedoraproject.org|dougsl...@redhat.com



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1188514] Review Request: nodejs-time -time.h bindings for Node.js

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1188514



--- Comment #2 from anish  ---
Hi Piotr,

Thank you for your comments, please find new SRPM and spec file on 


Spec URL: https://anishpatil.fedorapeople.org/nodejs-time.spec
SRPM URL: https://anishpatil.fedorapeople.org/nodejs-time-0.11.1-1.fc21.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1189015] Review Request: python-MDAnalysis - Analyze and manipulate molecular dynamics trajectories

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1189015

Antonio Trande  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Assignee|nob...@fedoraproject.org|anto.tra...@gmail.com
  Flags||fedora-review?



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1186501] Review Request: libticables2 - Texas Instruments link cables library

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1186501



--- Comment #3 from Antonio Trande  ---
1- %define tilp_version 1.17
   %define _udevdir /lib/udev/rules.d

http://fedoraproject.org/wiki/Packaging:Guidelines#.25global_preferred_over_.25define

Why /lib/udev/rules.d and not %{_prefix}/lib/udev/rules.d ?

2- Release: 0%{?dist}

http://fedoraproject.org/wiki/Packaging:NamingGuidelines#Release_Tag

3-  cd docs
make install DESTDIR=%{buildroot}

Compact version is

 make -C docs install DESTDIR=%{buildroot}

4-
   mkdir -p %{buildroot}/lib/udev/rules.d/
   cp %SOURCE1 %{buildroot}%{_udevdir}/69-libticables.rules

Compact (without 'mkdir' command) and correct version is 

   cp -a %SOURCE1 %{buildroot}%{_udevdir}/69-libticables.rules

5- %lang(fr) %{_datadir}/locale/fr/LC_MESSAGES/libticables2.mo

http://fedoraproject.org/wiki/Packaging:Guidelines#Handling_Locale_Files

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1196186] Review Request: perl-DBIx-Class-Schema-Config - Credential Management for DBIx::Class

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1196186



--- Comment #8 from Fedora Update System  ---
perl-DBIx-Class-Schema-Config-0.001011-1.fc21 has been submitted as an update
for Fedora 21.
https://admin.fedoraproject.org/updates/perl-DBIx-Class-Schema-Config-0.001011-1.fc21

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1196186] Review Request: perl-DBIx-Class-Schema-Config - Credential Management for DBIx::Class

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1196186



--- Comment #7 from Fedora Update System  ---
perl-DBIx-Class-Schema-Config-0.001011-1.fc20 has been submitted as an update
for Fedora 20.
https://admin.fedoraproject.org/updates/perl-DBIx-Class-Schema-Config-0.001011-1.fc20

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1186501] Review Request: libticables2 - Texas Instruments link cables library

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1186501

Antonio Trande  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1186501] Review Request: libticables2 - Texas Instruments link cables library

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1186501

Antonio Trande  changed:

   What|Removed |Added

   Assignee|nob...@fedoraproject.org|anto.tra...@gmail.com
  Flags||fedora-review?



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1196366] Review Request: openstack-designateclient - Client library for OpenStack Designate DNS API

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1196366



--- Comment #1 from Haïkel Guémar  ---
One small typo: sed -i s/REDHATDESIGNATECLIENTVERSION/%{version}/
designateclient/version.py (patch is correct)

Everything else looks good.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1183193] Review Request: ceres-solver - A non-linear least squares minimizer

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1183193



--- Comment #9 from Antonio Trande  ---
(In reply to Taylor Braun-Jones from comment #8)
> 
> Now to find someone to review this package request...

http://fedoraproject.org/wiki/How_to_get_sponsored_into_the_packager_group
;)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1196347] Review Request: f22-backgrounds – Fedora 22 default desktop background

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1196347

Martin Sourada  changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #2 from Martin Sourada  ---
New Package SCM Request
===
Package Name: f22-backgrounds
Short Description: Fedora 22 default desktop background
Upstream URL: https://fedoraproject.org/wiki/F22_Artwork
Owners: mso
Branches: f22 epel7
InitialCC: design-sw

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1185099] Review Request: opam - A source-based package manager for OCaml

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1185099



--- Comment #6 from Jon Ludlam  ---
New version posted. I've made it install the cmxs files as well, and also
changed the Requires line in ocaml-opam-lib-devel - previously it was requiring
opam, now it requires ocaml-opam-lib%{?_isa}.

Also, I tested the smp flags with make, but the parallel build fails. I think
this still fails on the master branch, so I'll double check then make an issue
on github.


Spec URL: http://www.recoil.org/~jon/opam.spec
SRPM URL: http://www.recoil.org/~jon/opam-1.2.0-4.fc21.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1195279] Review Request: preupgrade-assistant - Preupgrade assistant a tool for assess system before an upgrade

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1195279



--- Comment #4 from Petr Hracek  ---
Today finally my mock is running.

But python3 does not exist rawhide yet.
http://koji.fedoraproject.org/koji/packageinfo?packageID=9781
And therefore ConfigParser is not available yet:(

ConfigParser class is defined in configparser.py file, though.
ConfigParser (configparser.py) is part of subpackage python3-libs.

On Fedora 22 ConfigParser is here:
http://koji.fedoraproject.org/koji/rpminfo?fileStart=450&rpmID=5873335&fileOrder=name&buildrootOrder=-id&buildrootStart=50#filelist

Thanks for good idea with renaming binaries. It really makes sense.
I am going to do that.

I tried to build package in Copr, but F22 branch does not exist yet.

I have updated SPEC file so that in F22 it uses python2. But Since F23 is uses
python3.

Spec URL: https://phracek.fedorapeople.org/preupgrade-assistant.spec
SRPM URL:
https://phracek.fedorapeople.org/preupgrade-assistant-0.11.6-1.fc21.src.rpm

Hopefully the latest correction.
Both mock builds were successful (rawhide and F22).

Rawhide build in Copr is also done.
http://copr-fe.cloud.fedoraproject.org/coprs/phracek/preupgrade-assistant/builds/

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1036754] Review Request: ttembed - Remove embedding limitations from TrueType fonts

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1036754



--- Comment #14 from Petr Vobornik  ---
Package Change Request
==
Package Name: ttembed
New Branches: el6
Owners: pvoborni

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1196347] Review Request: f22-backgrounds – Fedora 22 default desktop background

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1196347

Jaroslav Reznik  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #1 from Jaroslav Reznik  ---
(In reply to Martin Sourada from comment #0)
> +++ This bug was initially created as a clone of Bug #1133217 +++
> Additional info:
> Package structurally same as previous default themes, supplemental
> wallpapers will come later (I already have the images, but they don't need
> to be in alpha).

Confirmed the package structure is as in the previous version (except
supplemental wallpapers that are still undergoing collection). All strings are
correctly updated to F22.

Package builds - http://koji.fedoraproject.org/koji/taskinfo?taskID=9080407 -
and installs correctly.

> Rpmlint output comment: 
> missing Docs OK, as the affected packages are dependant on the
> f22-backgrounds-base (sub)package that has the Docs, same with dangling
> symlinks.

No new rpmlint issues found except docs and symlinks mentioned above.

Just one thing I spotted for KDE sub-package. 

%files kde
%{_kde4_datadir}/wallpapers/%{Bg_Name}/

KDE 4 is going to be replaced with Plasma 5 in Fedora 22 but the wallpapers
location is still the same. I confirmed it with KDE SIG and tested - it works
as expected. So it's not a blocker for review but would be nice to sync with
KDE SIG (KDE 4 macros are not going away in the near future but...)

As there are minimal changes in SPEC file (just version update to match current
release, no extras yet) and all know rpm issues are known (and correct as
stated above), package is APPROVED.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1196186] Review Request: perl-DBIx-Class-Schema-Config - Credential Management for DBIx::Class

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1196186

Jitka Plesnikova  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
   Fixed In Version||perl-DBIx-Class-Schema-Conf
   ||ig-0.001011-1.fc23
 Resolution|--- |RAWHIDE
Last Closed||2015-02-26 05:00:41



--- Comment #6 from Jitka Plesnikova  ---
Thank you for the review and the repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1180723] Review Request: clufter - Tool for transforming/analyzing cluster configuration formats

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1180723



--- Comment #23 from Mamoru TASAKA  ---
Well, at lease please increase release number
even during review process (i.e. 0.3.XXX -> 0.4.XXX)

* Calling "rpm" inside rpmbuild is forbidden
  - While I don't remember where it is written as document
(currently with google I just found
   
http://www.redhat.com/promo/summit/2008/downloads/pdf/Wednesday_130pm_Tom_Callaway_OSS.pdf
page 25), actually I remember that calling "rpm" inside
rpmbuild process is forbidden.

One of the reason is that rpmdb created inside mock chroot is
created by rpm "outside" mock chroot - and calling rpm
inside mock chroot may just see rpmdb corruption.

* bash completion file treatment on scriptlets
  - Well, on second thought, I think that writing to modify
the "actual" file (i.e. %_sysconfdir/%name/bash-completion)
rather than ${bashcomp} (this is symlink) is preferable,
- as it clearly shows that scriptlet is modifying files
  under %_sysconfdir and actually not modifying the files
  under %_datadir.

I think only the above two issues are left.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1196347] Review Request: f22-backgrounds – Fedora 22 default desktop background

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1196347

Jaroslav Reznik  changed:

   What|Removed |Added

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



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1196551] New: Review Request: golang-github-shurcooL-sanitized_anchor_name - Package sanitized_anchor_name provides a func to create sanitized anchor names

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1196551

Bug ID: 1196551
   Summary: Review Request:
golang-github-shurcooL-sanitized_anchor_name - Package
sanitized_anchor_name provides a func to create
sanitized anchor names
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: jchal...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
https://jchaloup.fedorapeople.org/reviews/golang-github-shurcooL-sanitized_anchor_name/golang-github-shurcooL-sanitized_anchor_name.spec

SRPM URL:
https://jchaloup.fedorapeople.org/reviews/golang-github-shurcooL-sanitized_anchor_name/golang-github-shurcooL-sanitized_anchor_name-0-0.1.git8e87604.fc20.src.rpm

Description: Package sanitized_anchor_name provides a func to create sanitized
anchor names

Fedora Account System Username: jchaloup

Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=9080193

$ rpmlint
golang-github-shurcooL-sanitized_anchor_name-0-0.1.git8e87604.fc20.src.rpm
golang-github-shurcooL-sanitized_anchor_name-devel-0-0.1.git8e87604.fc20.noarch.rpm
golang-github-shurcooL-sanitized_anchor_name.spec
golang-github-shurcooL-sanitized_anchor_name.src: W: spelling-error
Summary(en_US) func -> fun, fund, funk
golang-github-shurcooL-sanitized_anchor_name.src: W: spelling-error
%description -l en_US func -> fun, fund, funk
golang-github-shurcooL-sanitized_anchor_name-devel.noarch: W: spelling-error
Summary(en_US) func -> fun, fund, funk
golang-github-shurcooL-sanitized_anchor_name-devel.noarch: W: spelling-error
%description -l en_US func -> fun, fund, funk
2 packages and 1 specfiles checked; 0 errors, 4 warnings.

Upstream: https://github.com/shurcooL/sanitized_anchor_name

Notes: Licence in README.md, thanks Dmitri for the license.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1119117] Review Request: perl-Algorithm-Cron - Abstract implementation of the cron(8) scheduling algorithm

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1119117



--- Comment #3 from Jitka Plesnikova  ---
Hi Christopher,

As per policy, unless you can respond in a week, I'll close this as a stalled
review.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1119148] Review Request: perl-Daemon-Control - Create init scripts in Perl

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1119148



--- Comment #5 from Jitka Plesnikova  ---
Hi Christopher,

As per policy, unless you can respond in a week, I'll close this as a stalled
review.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1195557] Review Request: ghc-Decimal - Decimal numbers with variable precision

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1195557

Jens Petersen  changed:

   What|Removed |Added

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



--- Comment #2 from Jens Petersen  ---
Looks good to me.

- BSD license
- standard Haskell packaging
- trivial cblrpm diff output.

Package is APPROVED.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1195555] Review Request: ghc-tabular - Two-dimensional data tables with rendering functions

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=119

Jens Petersen  changed:

   What|Removed |Added

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



--- Comment #4 from Jens Petersen  ---
Your spec file and srpm are out of sync. :(

|> mv %{buildroot}%{_datadir}/%{pkg_name}-%{version}
%{buildroot}%{_datadir}/%{name}

There is already:

%doc example

so no need to duplicate the samples in datadir, so please remove
datadir/example.

Otherwise this looks good to me.

- License is BSD.
- Standard Haskell packaging.

Package is APPROVED, but please fix above when importing.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1189385] Review Request: - nodejs-line-reader-Asynchronous line-by-line file reader

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1189385



--- Comment #1 from anish  ---
Spec URL: https://anishpatil.fedorapeople.org/nodejs-line-reader.spec
SRPM URL:
https://anishpatil.fedorapeople.org/nodejs-line-reader-0.2.4-2.fc21.src.rpm
Description: Nodejs line reader is asynchronous line-by-line file reader
Fedora Account System Username:anishpatil

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1189375] Review Request:nodejs-btoa - conversion from base64-encoded to ascii

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1189375



--- Comment #1 from anish  ---
Spec URL: https://anishpatil.fedorapeople.org/nodejs-btoa.spec
SRPM URL: https://anishpatil.fedorapeople.org/nodejs-btoa-1.1.2-2.fc21.src.rpm
Description: It turns binary data to base64-encoded ascii
Fedora Account System Username:anishpatil

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1195555] Review Request: ghc-tabular - Two-dimensional data tables with rendering functions

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=119



--- Comment #3 from Jens Petersen  ---
cblrpm diff output:

--- ghc-tabular.spec2015-02-24 13:34:33.0 +0900
+++ ghc-tabular.spec.cblrpm2015-02-26 17:30:37.399281724 +0900
@@ -25,6 +25,17 @@
 comes with some default rendering functions for turning those tables into
ASCII
 art, simple text with an arbitrary delimiter, CSV, HTML or LaTeX.

+Below is an example of the kind of output this library produces. The tabular
+package can group rows and columns, each group having one of three separators
+(no line, single line, double line) between its members.
+
+> || memtest 1 | memtest 2 || time test | time test 2 >
+++===+===++=+ > A 1 || hog |
+terrible || slow | slower > A 2 || pig | not bad || fast | slowest >
+++---+---++-+ > B 1 || good |
awful
+|| intolerable | bearable > B 2 || better | no chance || crawling | amazing >
B
+3 || meh | well... || worst ever | ok.
+

 %package devel
 Summary:Haskell %{pkg_name} library development files
@@ -48,7 +59,6 @@

 %install
 %ghc_lib_install
-mv %{buildroot}%{_datadir}/%{pkg_name}-%{version}
%{buildroot}%{_datadir}/%{name}


 %post devel
@@ -65,10 +75,8 @@

 %files devel -f %{name}-devel.files
 %doc example
-%{_datadir}/%{name}


 %changelog
-* Tue Feb 24 2015 Fedora Haskell SIG  -
0.2.2.6-1
-- spec file generated by cabal-rpm-0.9.3
-- move examples to more Fedora-acceptable location
+* Thu Feb 26 2015 Fedora Haskell SIG  -
0.2.2.6-1
+- spec file generated by cabal-rpm-0.9.3.50

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1195553] Review Request: ghc-control-monad-free - Free monads and monad transformers

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1195553

Jens Petersen  changed:

   What|Removed |Added

 CC||maths...@gmail.com
  Flags||needinfo?(maths...@gmail.co
   ||m)



--- Comment #2 from Jens Petersen  ---
Builds fine in F22+ with ghc-7.8.4 - fails to build under F21 ghc-7.6.3.

License is Public Domain so no license file.

However according to http://packdeps.haskellers.com/reverse/control-monad-free
wizards needs 0.5.* of this package, so 0.6 seems too new.

It might be nice to ask upstream politely if they wouldn't like
to put the source under a proper FOSS license, like BSD.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1045676] Review Request: sunwait - Calculate sunrise, sunset, twilight

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1045676



--- Comment #9 from Eric Smith  ---
Includes changes for Ralf Corsepius' comment 8 above.  Thanks!

Spec URL: http://fedorapeople.org/~brouhaha/sunwait/sunwait.spec
SRPM URL:
http://fedorapeople.org/~brouhaha/sunwait/sunwait-0.1-0.2.20041208.fc21.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1196366] Review Request: openstack-designateclient - Client library for OpenStack Designate DNS API

2015-02-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1196366

Haïkel Guémar  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||karlthe...@gmail.com
 Blocks||177841 (FE-NEEDSPONSOR)
   Assignee|nob...@fedoraproject.org|karlthe...@gmail.com




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=177841
[Bug 177841] Tracker: Review requests from new Fedora packagers who need a
sponsor
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

<    1   2