[Bug 1029197] Rename Request: gshhg-gmt-nc4 - Global Self-consistent Hierarchical High-resolution Geography (GSHHG)

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1029197

Christopher Meng  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||cicku...@gmail.com
   Assignee|nob...@fedoraproject.org|cicku...@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 1025102] Review Request: masscan - Port scanner

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1025102



--- Comment #14 from Christopher Meng  ---
Remember to use standard make install way in the future update:

https://github.com/robertdavidgraham/masscan/pull/48

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

[Bug 797330] Review request: xsensors - An X11 interface to lm_sensors

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=797330



--- Comment #20 from Christopher Meng  ---
I don't know how you got 0.71 version? I can't find this version anywhere.

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

[Bug 852213] Review Request: iii - A server for Eye-Fi SD cards

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=852213

Jeffrey Walton  changed:

   What|Removed |Added

 CC||noloa...@gmail.com



--- Comment #1 from Jeffrey Walton  ---
No Wifi drivers on Fedora 19 or Fedora 20 either. And the RPMs from
http://avi.alkalay.net/software/iii/ are for F17.

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

[Bug 852213] Review Request: iii - A server for Eye-Fi SD cards

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=852213

Christopher Meng  changed:

   What|Removed |Added

 CC||cicku...@gmail.com



--- Comment #2 from Christopher Meng  ---
Avi, please tell us your FAS account name.

Also, please use git snapshot and build package again.

Thanks.

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

[Bug 1046807] Review Request: rubygem-activerecord-nulldb-adapter - The Null Object pattern as applied to ActiveRecord database adapters

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1046807

Achilleas Pipinellis  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||axill...@archlinux.gr
   Assignee|nob...@fedoraproject.org|axill...@archlinux.gr
  Flags||fedora-review?



--- Comment #1 from Achilleas Pipinellis  ---
Taken.

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

[Bug 1040180] Review Request: rubygem-virtus - Attributes on Steroids for Plain Old Ruby Objects

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1040180



--- Comment #4 from Achilleas Pipinellis  ---
Thanks for informing me. If that's the case I'll take it as well, seems an easy
one.

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

[Bug 1029197] Rename Request: gshhg-gmt-nc4 - Global Self-consistent Hierarchical High-resolution Geography (GSHHG)

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1029197

Christopher Meng  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #2 from Christopher Meng  ---
Obsoletes and Provides are OK.

-
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 1039814] Review Request: criu (crtools) - Tool for Checkpoint/Restore in User-space

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1039814

Christopher Meng  changed:

   What|Removed |Added

 CC||ava...@gmail.com
  Flags||needinfo?(ava...@gmail.com)



--- Comment #7 from Christopher Meng  ---
Please retire crtools correctly ASAP and block it in Koji referred from:

https://fedoraproject.org/wiki/How_to_remove_a_package_at_end_of_life

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

[Bug 1047647] New: Review Request: libchardet - Mozilla's Universal Charset Detector C/C++ API

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1047647

Bug ID: 1047647
   Summary: Review Request: libchardet - Mozilla's Universal
Charset Detector C/C++ API
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Assignee: nob...@fedoraproject.org
  Reporter: theben...@gmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: http://breed808.com/rpmfusion-submission/libchardet.spec
SRPM URL:
http://breed808.com/rpmfusion-submission/libchardet-1.0.2-1.fc20.src.rpm
Description: Mozilla's Universal Charset Detector C/C++ API
Fedora Account System Username: breed808

rpmlint -i libchardet-1.0.2-1.fc20.src.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.
rpmlint -i libchardet-1.0.2-1.fc20.x86_64.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

rpmlint -i libchardet-devel-1.0.2-1.fc20.x86_64.rpm
libchardet-devel.x86_64: W: no-manual-page-for-binary chardet-config
Each executable in standard binary directories should have a man page.

1 packages and 0 specfiles checked; 0 errors, 1 warnings.

Hello! This is my first time submitting a package for review, so please let me
know if I've made any mistakes.

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

[Bug 1047647] Review Request: libchardet - Mozilla's Universal Charset Detector C/C++ API

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1047647

Ben Reedy  changed:

   What|Removed |Added

 CC||theben...@gmail.com
 Blocks||177841 (FE-NEEDSPONSOR)




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

[Bug 852213] Review Request: iii - A server for Eye-Fi SD cards

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=852213



--- Comment #3 from Avi Alkalay  ---
There's no need for wifi drivers. This is only the server part of the EyeFi
protocol.

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

[Bug 1047596] Review Request: php-channel-phpseclib - Adds the phpseclib channel to PEAR

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1047596

Shawn Iwinski  changed:

   What|Removed |Added

 CC||shawn.iwin...@gmail.com
   Assignee|nob...@fedoraproject.org|fed...@famillecollet.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 1047647] Review Request: libchardet - Mozilla's Universal Charset Detector C/C++ API

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1047647



--- Comment #1 from MartinKG  ---
Please insert each time you make a change an entry in the changelog and
increase also the Release number. + post the %changelog.
https://fedoraproject.org/wiki/Packaging:Guidelines#Changelogs

necessary changes in the spec file:
Release:2%{dist}

* Wed Jan 01 2014 Ben Reedy  - 1.0.2-2
- Added predefined macros to configure script in place of set paths

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

[Bug 1029197] Rename Request: gshhg-gmt-nc4 - Global Self-consistent Hierarchical High-resolution Geography (GSHHG)

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1029197

Orion Poplawski  changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #3 from Orion Poplawski  ---
Thanks for the review!

New Package SCM Request
===
Package Name: gshhg-gmt-nc4
Short Description: Global Self-consistent Hierarchical High-resolution
Geography (GSHHG)
Owners: orion
Branches: 
InitialCC:

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

[Bug 1047655] New: Review Request: rubygem-cairo-gobject - Ruby binding of cairo-gobject

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1047655

Bug ID: 1047655
   Summary: Review Request: rubygem-cairo-gobject - Ruby binding
of cairo-gobject
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: mtas...@fedoraproject.org
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
http://mtasaka.fedorapeople.org/Review_request/ruby-gnome2-suite/rubygem-cairo-gobject.spec
SRPM URL:
http://mtasaka.fedorapeople.org/Review_request/ruby-gnome2-suite/rubygem-cairo-gobject-2.1.0-1.fc.src.rpm
Description: 
Ruby/CairoGObject is a Ruby binding of cairo-gobject.

Fedora Account System Username: mtasaka

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

[Bug 1046813] Review Request: rubygem-cocaine - A small library for doing (command) lines

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1046813

Mamoru TASAKA  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Assignee|nob...@fedoraproject.org|mtas...@fedoraproject.org
  Flags||fedora-review?



--- Comment #1 from Mamoru TASAKA  ---
Taking.

I would appreciate it if you would review my review request (bug 1047655)

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

[Bug 1047414] Review Request: ioping - Simple disk I/O latency monitoring tool

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1047414

Orion Poplawski  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||or...@cora.nwra.com
   Assignee|nob...@fedoraproject.org|or...@cora.nwra.com
  Flags||fedora-review?



--- Comment #1 from Orion Poplawski  ---
Package Review
==

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

Issues:

- Makefile strips the binary so no debuginfo information is generated
- Not a blocker, but ask upstream to include a copy of the license

= MUST items =

C/C++:
[X]: Package does not contain kernel modules.
[X]: Package contains no static executables.
[x]: Package does not contain any libtool archives (.la)
[x]: Rpath absent or only used for internal libs.

Generic:
[X]: Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[X]: If (and only if) the source package includes the text of the license(s)
 in its own file, then that file, containing the text of the license(s)
 for the package is included in %doc.
[X]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses found:
 "GPL (v3 or later)". Detailed output of licensecheck in
 /export/home/orion/redhat/ioping-0.8/1047414-ioping/licensecheck.txt
[X]: %build honors applicable compiler flags or justifies otherwise.
[X]: Package contains no bundled libraries without FPC exception.
[X]: Changelog in prescribed format.
[X]: Sources contain only permissible code or content.
[-]: Package contains desktop file if it is a GUI application.
[-]: Development files must be in a -devel package
[X]: Package uses nothing in %doc for runtime.
[X]: Package consistently uses macros (instead of hard-coded directory names).
[X]: Package is named according to the Package Naming Guidelines.
[X]: Package does not generate any conflict.
[X]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[X]: Requires correct, justified where necessary.
[X]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[!]: Useful -debuginfo package or justification otherwise.
[X]: Package is not known to require an ExcludeArch tag.
[-]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 10240 bytes in 2 files.
[X]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least one
 supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
 Note: No rpmlint messages.
[x]: Package requires other packages for directories it uses.
[x]: Package must own all directories that it creates.
[x]: Package does not own files or directories owned by other packages.
[x]: All build dependencies are listed in BuildRequires, except for any that
 are listed in the exceptions section of Packaging Guidelines.
[x]: Package uses either %{buildroot} or $RPM_BUILD_ROOT
[x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
[x]: Each %files section contains %defattr if rpm < 4.4
[x]: Macros in Summary, %description expandable at SRPM build time.
[x]: Package does not contain duplicates in %files.
[x]: Permissions on files are set properly.
[x]: Package use %makeinstall only when make install' ' DESTDIR=... doesn't
 work.
[x]: Package is named using only allowed ASCII characters.
[x]: Package do not use a name that already exist
[x]: Package is not relocatable.
[x]: Sources used to build the package match the upstream source, as provided
 in the spec URL.
[x]: Spec file name must match the spec package %{name}, in the format
 %{name}.spec.
[x]: File names are valid UTF-8.
[x]: Packages must not store files under /srv, /opt or /usr/local

= SHOULD items =

Generic:
[x]: Packager, Vendor, PreReq, Copyright tags should not be in spec file
[!]: If the source package does not include license text(s) as a separate file
 from upstream, the packager SHOULD query upstream to include it.
[x]: Final provides and requires are sane (see attachments).
[x]: Package functions as described.
[x]: Latest version is packaged.
[x]: Package does not include license text files separate from upstream.
[-]: Description and summary sections in the package spec file contains
 translations for supported Non-English languages, if available.
[x]: Package should compile

[Bug 797330] Review request: xsensors - An X11 interface to lm_sensors

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=797330



--- Comment #21 from Jeremy Newton  ---
Hi, sorry it's been a while since I looked at this. I should do a test build
because I'm not sure if it builds in rawhide anymore.

0.71 was my own fork, which collaborates various patches I've found,  as the
project is no longer maintained.

I have it hosted on git hub:
https://github.com/Mystro256/xsensors

Let me make sure it still 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 1039814] Review Request: criu (crtools) - Tool for Checkpoint/Restore in User-space

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1039814



--- Comment #8 from Fedora Update System  ---
criu-1.0-3.fc19 has been submitted as an update for Fedora 19.
https://admin.fedoraproject.org/updates/criu-1.0-3.fc19

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

[Bug 1039814] Review Request: criu (crtools) - Tool for Checkpoint/Restore in User-space

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1039814



--- Comment #9 from Fedora Update System  ---
criu-1.0-3.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/criu-1.0-3.fc20

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

[Bug 797330] Review request: xsensors - An X11 interface to lm_sensors

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=797330



--- Comment #22 from Jeremy Newton  ---
It seems everything is working fine.

I noticed there was a couple date mistakes, so I fixed them.

Here's the current files:

SPEC:
http://dl.dropbox.com/u/42480493/xsensors.spec
SRPM:
http://dl.dropbox.com/u/42480493/xsensors-0.71-2.fc20.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 825409] Review Request: gazebo - 3D multi-robot simulator with dynamics

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=825409



--- Comment #22 from solanum  ---
Increasing the number of times Latex reruns fixes the doc problem but it is
higher then 8 which is the doxygen default for the new version (1.8.6). The old
version (1.8.5 which is in F20) defaults to 5.

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

[Bug 1047497] Review Request: otcl - MIT Object Tcl

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1047497



--- Comment #2 from Mosaab Alzoubi  ---
- Fix path of includes.

Spec URL: http://ojuba.org/oji/SPECS/otcl.spec
SRPM URL: http://ojuba.org/oji/SRPMS/otcl-1.14-2.oji.fc19.src.rpm

https://koji.fedoraproject.org/koji/taskinfo?taskID=6346370

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

[Bug 1047647] Review Request: libchardet - Mozilla's Universal Charset Detector C/C++ API

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1047647

Christopher Meng  changed:

   What|Removed |Added

 CC||cicku...@gmail.com



--- Comment #2 from Christopher Meng  ---
A dep of cmplayer, right?

I can't sponsor you, but some thoughts:

1. Group tag is not a MUST HAVE tag now, you can drop it on your own.

2. These 2 lines are not needed:

Packager:Ben Reedy 
BuildRequires:gcc-c++

3. Description and summary are the same, that's bad. Please improve.

4. Use macro:

%configure

instead of 

./configure --prefix=%{_prefix} --sysconfdir=%{_sysconfdir} --libdir=%{_libdir}
\
--mandir=%{_mandir} --disable-static

5. %files
# Libraries
%{_libdir}/%{name}.so.*
# Man pages
%{_mandir}/ko/man3/*
# We don't want the libtool archive
%exclude %{_libdir}/%{name}.la

%files devel
#Binary
%{_bindir}/chardet-config
# Header files
%{_includedir}/chardet/
# Development library
%{_libdir}/%{name}.so
# Documentation
%doc README LICENSE Changelog

Why did you add so many comments here? I don't think you need to write down
"this part below is Binary" as you are the packager, you MUST know their usage,
and we reviewer know them certainly as well.

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

[Bug 1047702] New: Review Request: tclcl - Tcl/C++ interface

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1047702

Bug ID: 1047702
   Summary: Review Request: tclcl - Tcl/C++ interface
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: moc...@hotmail.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: http://ojuba.org/oji/SPECS/tclcl.spec
SRPM URL: http://ojuba.org/oji/SRPMS/tclcl-1.20-1.oji.fc19.src.rpm
Description:
TclCL (Tcl with classes) is a Tcl/C++ interface used by Mash,
vic, vat, rtp_play, ns, and nam. It provides a layer of C++
glue over OTcl. 

Fedora Account System Username: moceap

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

[Bug 1047497] Review Request: otcl - MIT Object Tcl

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1047497

Mosaab Alzoubi  changed:

   What|Removed |Added

 Blocks||1047702 (TCLCL)




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1047702
[Bug 1047702] Review Request: tclcl - Tcl/C++ interface
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1047702] Review Request: tclcl - Tcl/C++ interface

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1047702

Mosaab Alzoubi  changed:

   What|Removed |Added

 Depends On||1047497 (OTcl)
  Alias||TCLCL




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1047497
[Bug 1047497] Review Request: otcl - MIT Object Tcl
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1044607] Review Request: QsLog - An easy to use logger that is based on Qt's QDebug class

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1044607

Susi Lehtola  changed:

   What|Removed |Added

 CC||anto.tra...@gmail.com
  Flags||needinfo?(anto.trande@gmail
   ||.com)



--- Comment #5 from Susi Lehtola  ---
Any progress?

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

[Bug 1003322] Review Request: glite-lbjp-common-server-bones - Skeleton of multi-process network server

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1003322



--- Comment #10 from Fedora Update System  ---
glite-lbjp-common-server-bones-2.3.9-1.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.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1033970] Review Request: d52 - Disassemblers for 8051, 8048, and Z80 families

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1033970

Fedora Update System  changed:

   What|Removed |Added

   Fixed In Version|d52-3.4.1-2.fc19|d52-3.4.1-2.el6



--- Comment #11 from Fedora Update System  ---
d52-3.4.1-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.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1014738] Review Request: python-astropy - A Community Python Library for Astronomy

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1014738



--- Comment #25 from Sergio Pascual  ---
Any progress with this?

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

[Bug 1046812] Review Request: jupp - Compact and feature-rich WordStar-compatible editor

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1046812

Susi Lehtola  changed:

   What|Removed |Added

 CC||susi.leht...@iki.fi



--- Comment #1 from Susi Lehtola  ---
I think the versioning is wrong. The 3.1 should be in the version field. Then
again, who knows - the tarball naming is pretty weird anyways. Please check the
package naming guidelines... and be prepared to use Epoch if the versioning
changes.

Next, the Summary is inconsistent, since the Description doesn't mention
WordStar at all. Based on the home page, I'd put something like
 Summary: An enhanced, portable fork of the JOE editor

Also, please don't use macros in the URL, because then it's not human readable
in the spec. Also, note that according to the web page the url should be
 URL: http://mirbsd.de/jupp

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

[Bug 1046812] Review Request: jupp - Compact and feature-rich WordStar-compatible editor

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1046812



--- Comment #2 from Robert Scheck  ---
(In reply to Susi Lehtola from comment #1)
> I think the versioning is wrong. The 3.1 should be in the version field.
> Then again, who knows - the tarball naming is pretty weird anyways. Please
> check the package naming guidelines... and be prepared to use Epoch if the
> versioning changes.

I was in touch with upstream before: The goal is to rename to jupp and the
versioning is correct. Personally, I also can not see any conflict with our
guidelines at https://fedoraproject.org/wiki/Packaging:NamingGuidelines

Do you see a blocker here? If so, please let me know so that we can talk to
upstream directly.

> Next, the Summary is inconsistent, since the Description doesn't mention
> WordStar at all. Based on the home page, I'd put something like
>  Summary: An enhanced, portable fork of the JOE editor

As the summary proposal came from upstream I would like to update description
instead. However changing one of the two texts is IMHO more cosmetic.

> Also, please don't use macros in the URL, because then it's not human
> readable in the spec. Also, note that according to the web page the url
> should be
>  URL: http://mirbsd.de/jupp

But you noticed that this URL redirects to https://www.mirbsd.org/jupp.htm?

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

[Bug 1047647] Review Request: libchardet - Mozilla's Universal Charset Detector C/C++ API

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1047647



--- Comment #3 from Ben Reedy  ---
spec file is in the same location
SRPM is located at
http://breed808.com/rpmfusion-submission/libchardet-1.0.2-3.fc20.src.rpm

(In reply to Christopher Meng from comment #2)

Thank you for the help, Christopher.

>1. Group tag is not a MUST HAVE tag now, you can drop it on your own.
>
>2. These 2 lines are not needed:
>
>Packager:  Ben Reedy 
>BuildRequires: gcc-c++

Removed.

> 3. Description and summary are the same, that's bad. Please improve.

I've updated the description, though it is fairly short. I've had difficulty
finding a valuable description for libchardet.

> 4. Use macro:
> 
> %configure
> 
> instead of 
> 
> ./configure --prefix=%{_prefix} --sysconfdir=%{_sysconfdir}
> --libdir=%{_libdir} \
>   --mandir=%{_mandir} --disable-static

Is it ok that I've added the '--disable-static' flag after the %configure
macro?

> Why did you add so many comments here? I don't think you need to write down
> "this part below is Binary" as you are the packager, you MUST know their
> usage, and we reviewer know them certainly as well.

Agreed, I should have removed these some time ago.


(In reply to MartinKG from comment #1)
> Please insert each time you make a change an entry in the changelog and
> increase also the Release number. + post the %changelog.
> https://fedoraproject.org/wiki/Packaging:Guidelines#Changelogs

Hi Martin, I've updated the changelog and release number.
One question: the changelog guidelines mention that if a package has been
updated but not built, the package's Release does not need to be incremented
and a new entry can be added to the changelog. Does that apply for packages
under review?
https://fedoraproject.org/wiki/Packaging:Guidelines#Repeat_the_old_version_release_with_a_new_entry

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

[Bug 1025102] Review Request: masscan - Port scanner

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1025102



--- Comment #15 from Rino Rondan  ---
Updated the description field.

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

[Bug 1025102] Review Request: masscan - Port scanner

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1025102

Rino Rondan  changed:

   What|Removed |Added

  Flags||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 1025102] Review Request: masscan - Port scanner

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1025102



--- Comment #16 from Christopher Meng  ---
Also, remember to specify LDFLAGS by %__global_ldflags in the future also.

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

[Bug 1047497] Review Request: otcl - MIT Object Tcl

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1047497

Christopher Meng  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||cicku...@gmail.com
   Assignee|nob...@fedoraproject.org|cicku...@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 1025102] Review Request: masscan - Port scanner

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1025102



--- Comment #17 from Rino Rondan  ---
New Package SCM Request
===
Package Name: masscan
Short Description: This is the fastest Internet port scanner
Owners: villadalmine
Branches: f19 f20 
InitialCC: villadalmine

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

[Bug 1017766] Review Request: google-crosextra-caladea-fonts - Sans-serif font metric-compatible with Cambria font

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1017766

Parag  changed:

   What|Removed |Added

  Flags|fedora-cvs+ |fedora-cvs?



--- Comment #28 from Parag  ---
Package Change Request
==
Package Name: google-crosextra-caladea-fonts
New Branches: el6
Owners: pnemade

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

[Bug 1017767] Review Request: google-crosextra-carlito-fonts - Sans-serif font metric-compatible with Calibri font

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1017767

Parag  changed:

   What|Removed |Added

  Flags|fedora-cvs+ |fedora-cvs?



--- Comment #22 from Parag  ---
Package Change Request
==
Package Name: google-crosextra-carlito-fonts
New Branches: el6
Owners: pnemade

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

[Bug 1047510] Review Request: php-opencloud - PHP SDK for OpenStack/Rackspace APIs

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1047510



--- Comment #5 from Adam Williamson  ---
I've been looking into making OC's loading of 3rdparty libs for apps sane
today. AFAICS, OwnCloud's autoloader doesn't handle libs in
apps/some_app/3rdparty directories. It only handles core stuff. It might be
nice to make it handle this well - something like look in
apps/some_app/3rdparty when something from apps/some_app tries to access a
function - but I don't think it's trivial, and I'm not sure they'd want to take
patches which simply added apps/some_app/3rdparty/foo to the autoloader
*unconditionally*.

right now I'm working on a patch set which would make all apps consistent in
how they do things: add some_app/3rdparty to the include_path, then
require_once whatever they need to require relative to that. But for us not to
carry patches downstream, that does require us to use the same layout in
/usr/share/php or /usr/share/pear or whatever that OC has in some_app/3rdparty
.

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

[Bug 1047510] Review Request: php-opencloud - PHP SDK for OpenStack/Rackspace APIs

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1047510



--- Comment #6 from Adam Williamson  ---
Upstream is very responsive, though, and I doubt they particularly *care* about
the directory layouts under 3rdparty/ directories. I suspect we might get
mileage out of submitting patches upstream to make *their* layout match *ours*,
when we don't want to make *our* layout match *theirs*. Especially if we can
make a good case for the change, like here, for PSR-0 compliance.

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

[Bug 1047418] Review Request: ghc-webkit - Binding to the Webkit library

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1047418

Jens Petersen  changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #4 from Jens Petersen  ---
Thanks again!  I also plan to package up the webkitgtk3 binding
once ghc-gtk3 (bug 1045963) has been reviewed.


New Package SCM Request
===
Package Name: ghc-webkit
Short Description: Binding to the Webkit library for gtk2
Owners: petersen
Branches: f20 f19 el6
InitialCC: haskell-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 875087] Review Request: pbsclusterviz - Visualise the status of PBS clusters

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=875087



--- Comment #21 from Orion Poplawski  ---
Package Review
==

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


Issues:
===
- Package contains BR: python2-devel or python3-devel
You need to have BuildRequires: python2-devel

- Please fix the FSF address in the license.

- It looks like cluster_status is a GUI application and as such should have a
.desktop file.

- cluster_status (and perhaps even gen_nodes_file) is too generic.  Please add
a pbs (or pbs_) prefix.

- Drop BuildRoot and %clean unless you want to target epel5

- You only need to define python_sitelib, etc if you want to target epel5.  If
you do, please use the form from the guidelines:

%if 0%{?rhel} && 0%{?rhel} <= 5
%{!?python_sitelib: %global python_sitelib %(%{__python} -c "from
distutils.sysconfig import get_python_lib; print(get_python_lib())")}
%{!?python_sitearch: %global python_sitearch %(%{__python} -c "from
distutils.sysconfig import get_python_lib; print(get_python_lib(1))")}
%endif

- Your changelog versions do not match the package version "0.7" vs "0.7a".

- You need to escape "%" in %changelog entries with another %, eg %%clean

= MUST items =

Generic:
[x]: Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[x]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses found:
 "GPL (v2 or later) (with incorrect FSF address)", "Unknown or generated".
 2 files have unknown license. Detailed output of licensecheck in

/export/home/orion/redhat/pbsclusterviz-0.7a/875087-pbsclusterviz/licensecheck.txt
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[!]: Package contains desktop file if it is a GUI application.
[-]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[x]: Package consistently uses macros (instead of hard-coded directory names).
[x]: Package is named according to the Package Naming Guidelines.
[!]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: Package is not known to require an ExcludeArch tag.
[-]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 40960 bytes in 6 files.
[x]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least one
 supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
 Note: There are rpmlint messages (see attachment).
[x]: If (and only if) the source package includes the text of the license(s)
 in its own file, then that file, containing the text of the license(s)
 for the package is included in %doc.
[x]: Package requires other packages for directories it uses.
[x]: Package must own all directories that it creates.
[x]: Package does not own files or directories owned by other packages.
[x]: All build dependencies are listed in BuildRequires, except for any that
 are listed in the exceptions section of Packaging Guidelines.
[x]: Package uses either %{buildroot} or $RPM_BUILD_ROOT
[x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
[x]: %config files are marked noreplace or the reason is justified.
[x]: Each %files section contains %defattr if rpm < 4.4
[x]: Macros in Summary, %description expandable at SRPM build time.
[x]: Package does not contain duplicates in %files.
[x]: Permissions on files are set properly.
[x]: Package use %makeinstall only when make install' ' DESTDIR=... doesn't
 work.
[x]: Package is named using only allowed ASCII characters.
[x]: No %config files under /usr.
[x]: Package do not use a name that already exist
[x]: Package is not relocatable.
[x]: Sources used to build the package match the upstream source, as provided
 in the spec URL.
[x]: Spec file name must match the spec package %{name}, in the format
 %{name}.spec.
[x]: File names are valid UTF-8.
[x]: Packages must not store files under /srv, /opt or /usr/local

Python:
[x]: Python eggs must not download any dependencies during the build process.
[x]: A package which is used by another package via an egg interface should
 provide egg info.
[x]: Package meets the Packaging Guidelines::Python
[x]: Binary eggs must be removed in %prep

===

[Bug 426750] Review Request: ghc-utf8-string - Support reading and writing UTF8 Strings

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=426750



--- Comment #29 from Jens Petersen  ---
Git done (by process-git-requests).

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

[Bug 426750] Review Request: ghc-utf8-string - Support reading and writing UTF8 Strings

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=426750

Jens Petersen  changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



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

[Bug 479799] Review Request: ghc-uniplate - Uniform type generic traversals

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=479799

Jens Petersen  changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



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

[Bug 479799] Review Request: ghc-uniplate - Uniform type generic traversals

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=479799



--- Comment #12 from Jens Petersen  ---
Git done (by process-git-requests).

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

[Bug 479800] Review Request: hlint - Provides Haskell Source Code Suggestions

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=479800

Jens Petersen  changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



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

[Bug 488397] Review Request: ghc-tar - Reading, writing and manipulating ".tar" archive files in Haskell

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=488397

Jens Petersen  changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



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

[Bug 479800] Review Request: hlint - Provides Haskell Source Code Suggestions

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=479800



--- Comment #56 from Jens Petersen  ---
Git done (by process-git-requests).

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

[Bug 488397] Review Request: ghc-tar - Reading, writing and manipulating ".tar" archive files in Haskell

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=488397



--- Comment #19 from Jens Petersen  ---
Git done (by process-git-requests).

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

[Bug 537971] Review Request: ghc-mmap - Haskell binding to mmap

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=537971



--- Comment #11 from Jens Petersen  ---
Git done (by process-git-requests).

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

[Bug 537971] Review Request: ghc-mmap - Haskell binding to mmap

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=537971

Jens Petersen  changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



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

[Bug 537979] Review Request: ghc-hashed-storage - Hashed file storage support

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=537979



--- Comment #25 from Jens Petersen  ---
Git done (by process-git-requests).

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

[Bug 537979] Review Request: ghc-hashed-storage - Hashed file storage support

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=537979

Jens Petersen  changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



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

[Bug 555638] Review Request: ghc-html - Haskell HTML combinator library

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=555638



--- Comment #9 from Jens Petersen  ---
Git done (by process-git-requests).

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

[Bug 555638] Review Request: ghc-html - Haskell HTML combinator library

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=555638

Jens Petersen  changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



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

[Bug 555653] Review Request: ghc-haskeline - Haskell command-line interface for user input

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=555653

Jens Petersen  changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



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

[Bug 555653] Review Request: ghc-haskeline - Haskell command-line interface for user input

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=555653



--- Comment #15 from Jens Petersen  ---
Git done (by process-git-requests).

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

[Bug 1047746] New: Review Request: ghc-html - HTML combinator library

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1047746

Bug ID: 1047746
   Summary: Review Request: ghc-html - HTML combinator library
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Assignee: nob...@fedoraproject.org
  Reporter: peter...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org




Spec URL: http://petersen.fedorapeople.org/reviews/ghc-html.spec
SRPM URL:
http://petersen.fedorapeople.org/reviews/ghc-html-1.0.1.2-28.fc18.src.rpm

Description:
This package contains a combinator library for constructing HTML documents.

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

[Bug 1047746] Review Request: ghc-html - HTML combinator library

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1047746



--- Comment #1 from Jens Petersen  ---
This package built on koji: 
http://koji.fedoraproject.org/koji/taskinfo?taskID=6347535

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

[Bug 1001407] Review Request: herbstluftwm - A manual tiling window manager

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1001407

Christopher Meng  changed:

   What|Removed |Added

 Blocks||201449 (FE-DEADREVIEW)
 CC||co...@octayn.net



--- Comment #17 from Christopher Meng  ---
*** Bug 812538 has been marked as a duplicate of this bug. ***


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=201449
[Bug 201449] FE-DEADREVIEW -- Reviews stalled due to lack of submitter
response should be blocking this bug.
-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 812538] Review Request: herbstluftwm - Tiling window manager

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=812538

Christopher Meng  changed:

   What|Removed |Added

 CC||cicku...@gmail.com
 Resolution|NOTABUG |DUPLICATE



--- Comment #6 from Christopher Meng  ---


*** This bug has been marked as a duplicate of bug 1001407 ***

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

[Bug 1001407] Review Request: herbstluftwm - A manual tiling window manager

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1001407

Christopher Meng  changed:

   What|Removed |Added

 Blocks|201449 (FE-DEADREVIEW)  |




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=201449
[Bug 201449] FE-DEADREVIEW -- Reviews stalled due to lack of submitter
response should be blocking this bug.
-- 
You are receiving this mail because:
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1047746] Review Request: ghc-html - HTML combinator library

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1047746

Christopher Meng  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||cicku...@gmail.com
   Assignee|nob...@fedoraproject.org|cicku...@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 555948] Review Request: ghc-regex-base - Haskell regex base library

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=555948



--- Comment #10 from Jens Petersen  ---
Git done (by process-git-requests).

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

[Bug 555948] Review Request: ghc-regex-base - Haskell regex base library

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=555948

Jens Petersen  changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



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

[Bug 798891] Review Request: libtranslit - transliteration library with multiple backends

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=798891



--- Comment #15 from Mike FABIAN  ---
(In reply to Pravin Satpute from comment #13)
> I was trying to build this for epel6 but unfortunately gobject-introspection
> is not available in RHEL-6.

ibus-typing-booster needs gobject-introspection as well.

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

[Bug 1047762] New: Review Request: rubygem-rugged - Ruby binding to libgit2

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1047762

Bug ID: 1047762
   Summary: Review Request: rubygem-rugged - Ruby binding to
libgit2
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: ktdre...@ktdreyer.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: http://ktdreyer.fedorapeople.org/reviews/rubygem-rugged.spec
SRPM URL: 
Description: Rugged is a Ruby bindings to the libgit2 linkable C Git library.
This is for testing and using the libgit2 library in a language that is
awesome.
Fedora Account System Username: ktdreyer

F21 scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=6347734

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

[Bug 1047655] Review Request: rubygem-cairo-gobject - Ruby binding of cairo-gobject

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1047655

Ken Dreyer  changed:

   What|Removed |Added

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



--- Comment #1 from Ken Dreyer  ---
I can take this 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 1047765] New: Review Request: maven-shared-jarsigner - Component to assist in signing Java archives

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1047765

Bug ID: 1047765
   Summary: Review Request: maven-shared-jarsigner - Component to
assist in signing Java archives
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: mizde...@redhat.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL:
http://mizdebsk.fedorapeople.org/review/maven-shared-jarsigner/maven-shared-jarsigner.spec
SRPM URL:
http://mizdebsk.fedorapeople.org/review/maven-shared-jarsigner/maven-shared-jarsigner-1.3-1.fc21.src.rpm
Description: Apache Maven Jarsigner is a component which provides utilities to
sign and verify Java archive and other files in your Maven MOJOs.
Fedora Account System Username: mizdebsk

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

[Bug 1047765] Review Request: maven-shared-jarsigner - Component to assist in signing Java archives

2014-01-01 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1047765

Mikolaj Izdebski  changed:

   What|Removed |Added

 Blocks||652183 (FE-JAVASIG),
   ||1047210



--- Comment #1 from Mikolaj Izdebski  ---
Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=6347748


Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=652183
[Bug 652183] Java SIG tracker bug
https://bugzilla.redhat.com/show_bug.cgi?id=1047210
[Bug 1047210] maven-jarsigner-plugin-1.3 is available
-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review