[Bug 771297] New: Review Request: rubygem-bluecloth - A Ruby implementation of John Gruber's Markdown

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: rubygem-bluecloth - A Ruby implementation of John 
Gruber's Markdown

https://bugzilla.redhat.com/show_bug.cgi?id=771297

   Summary: Review Request: rubygem-bluecloth - A Ruby
implementation of John Gruber's Markdown
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: mfoj...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


Spec URL: http://omicron.mifo.sk/rubygem-bluecloth.spec
SRPM URL: http://omicron.mifo.sk/rubygem-bluecloth-2.2.0-1.fc14.src.rpm
Description:

BlueCloth is a complete rewrite using David Parsons Discount
library, a C implementation of Markdown.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 771171] Review Request: django-tastypie - A webservice API framework for Django

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=771171

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

   What|Removed |Added

Summary|Review Request: |Review Request:
   |django-tastypie |django-tastypie - A
   ||webservice API framework
   ||for Django

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 771311] New: Review Request: rubygem-gli - Git-Like Interface Command Line Parser for Ruby

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: rubygem-gli - Git-Like Interface Command Line Parser 
for Ruby

https://bugzilla.redhat.com/show_bug.cgi?id=771311

   Summary: Review Request: rubygem-gli - Git-Like Interface
Command Line Parser for Ruby
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: mfoj...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


Spec URL: http://omicron.mifo.sk/rubygem-gli.spec
SRPM URL: http://omicron.mifo.sk/rubygem-gli-1.4.0-1.fc14.src.rpm
Description:

DSL you can use to create a command line interface like git, gem or svn, in
that the first argument is a command, and there are global and command specific
flags.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 771314] New: Review Request: rubygem-parslet - Parser construction library with great error reporting in Ruby

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: rubygem-parslet - Parser construction library with 
great error reporting in Ruby

https://bugzilla.redhat.com/show_bug.cgi?id=771314

   Summary: Review Request: rubygem-parslet - Parser construction
library with great error reporting in Ruby
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: mfoj...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


Spec URL: http://omicron.mifo.sk/rubygem-parslet.spec
SRPM URL: http://omicron.mifo.sk/rubygem-gli-1.4.0-1.fc14.src.rpm

Description:

Parslet makes developing complex parsers easy. It does so by providing the best
error reporting possible and not generating reams of code for you to debug

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 676129] Review Request: qconf - Allows you to have a nice configure script for your qmake-based project

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=676129

--- Comment #42 from Fedora Update System upda...@fedoraproject.org 
2012-01-03 05:11:13 EST ---
qconf-1.4-4.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/qconf-1.4-4.el6

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 771316] New: Review Request: rubygem-blankslate - Ruby gem which provides an abstract base class with no predefined methods

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: rubygem-blankslate - Ruby gem which provides an 
abstract base class with no predefined methods

https://bugzilla.redhat.com/show_bug.cgi?id=771316

   Summary: Review Request: rubygem-blankslate - Ruby gem which
provides an abstract base class with no predefined
methods
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: mfoj...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


Spec URL: http://omicron.mifo.sk/rubygem-blankslate.spec
SRPM URL: http://omicron.mifo.sk/rubygem-blankslate-2.1.2.4-1.fc14.src.rpm

Description:

BlankSlate provides an abstract base class with no predefined
methods. BlankSlate is useful as a base class when writing classes that
depend upon method_missing.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 771311] Review Request: rubygem-gli - Git-Like Interface Command Line Parser for Ruby

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=771311

Michal Fojtik mfoj...@redhat.com changed:

   What|Removed |Added

 Blocks||771318

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 771297] Review Request: rubygem-bluecloth - A Ruby implementation of John Gruber's Markdown

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=771297

Michal Fojtik mfoj...@redhat.com changed:

   What|Removed |Added

 Blocks||771318

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 771297] Review Request: rubygem-bluecloth - A Ruby implementation of John Gruber's Markdown

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=771297

--- Comment #1 from Michal Fojtik mfoj...@redhat.com 2012-01-03 05:22:26 EST 
---
Revision -2:

Spec URL: http://omicron.mifo.sk/rubygem-bluecloth.spec
SRPM URL: http://omicron.mifo.sk/rubygem-bluecloth-2.2.0-2.fc14.src.rpm

Changelog:

- Fixed wrong binary location

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 771318] Review Request: rubygem-showoff - Ruby web app that reads simple configuration files for a presentation

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=771318

Michal Fojtik mfoj...@redhat.com changed:

   What|Removed |Added

 Depends on||771314, 771311, 771297

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 771314] Review Request: rubygem-parslet - Parser construction library with great error reporting in Ruby

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=771314

Michal Fojtik mfoj...@redhat.com changed:

   What|Removed |Added

 Blocks||771318

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 771318] New: Review Request: rubygem-showoff - Ruby web app that reads simple configuration files for a presentation

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: rubygem-showoff - Ruby web app that reads simple 
configuration files for a presentation

https://bugzilla.redhat.com/show_bug.cgi?id=771318

   Summary: Review Request: rubygem-showoff - Ruby web app that
reads simple configuration files for a presentation
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: mfoj...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


Spec URL: http://omicron.mifo.sk/rubygem-showoff.spec
SRPM URL: http://omicron.mifo.sk/rubygem-rubygem-showoff-0.7.0-2.fc14.src.rpm

Description:

ShowOff is a Sinatra web app that reads simple configuration files for a
presentation. It is sort of like a Keynote web app engine - think S5 +
Slidedown.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 771314] Review Request: rubygem-parslet - Parser construction library with great error reporting in Ruby

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=771314

Michal Fojtik mfoj...@redhat.com changed:

   What|Removed |Added

 Depends on||771316

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 771316] Review Request: rubygem-blankslate - Ruby gem which provides an abstract base class with no predefined methods

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=771316

Michal Fojtik mfoj...@redhat.com changed:

   What|Removed |Added

 Blocks||771314

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 771237] Review Request: rubygem-showoff - Ruby web app that reads simple configuration files for a presentation

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=771237

Michal Fojtik mfoj...@redhat.com changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||WONTFIX
Last Closed||2012-01-03 06:30:08

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 765652] Review Request: bvi - Display-oriented editor for binary files

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=765652

--- Comment #11 from Mohamed El Morabity pikachu.2...@gmail.com 2012-01-03 
06:26:05 EST ---
Here is the review:

 +:ok, =:needs attention, -:needs fixing

MUST Items:

[+] MUST: rpmlint must be run on every package.
bvi.src: W: spelling-error %description -l en_US bmore - bore, more, b more
bvi.x86_64: W: manual-page-warning /usr/share/man/man1/bvi.1.gz 2: warning:
macro `''' not defined
bvi.x86_64: W: manual-page-warning /usr/share/man/man1/bvi.1.gz 36: warning:
macro `--' not defined
bvi.x86_64: W: manual-page-warning /usr/share/man/man1/bvi.1.gz 48: warning:
macro `M' not defined
bvi.x86_64: W: manual-page-warning /usr/share/man/man1/bvi.1.gz 48: warning:
macro `S' not defined
bvi.x86_64: W: manual-page-warning /usr/share/man/man1/bvi.1.gz 48: warning:
macro `N' not defined
bvi.x86_64: W: manual-page-warning /usr/share/man/man1/bvi.1.gz 48: warning:
macro `T' not defined
bvi.x86_64: W: manual-page-warning /usr/share/man/man1/bmore.1.gz 56: warning:
macro `L'' not defined
bvi.x86_64: W: manual-page-warning /usr/share/man/man1/bmore.1.gz 56: warning:
macro `R'' not defined
bvi.x86_64: E: incorrect-fsf-address /usr/share/doc/bvi-1.3.2/COPYING
bvi.x86_64: W: no-manual-page-for-binary bview
bvi.x86_64: W: no-manual-page-for-binary bvedit
3 packages and 0 specfiles checked; 1 errors, 11 warnings.
The warnings can be safely ignored

[+] MUST: The package must be named according to the Package Naming Guidelines.

[+] MUST: The spec file name must match the base package %{name}

[+] MUST: The package must meet the Packaging Guidelines.

[+] MUST: The package must be licensed with a Fedora approved license and meet
the Licensing Guidelines.

[-] MUST: The License field in the package spec file must match the actual
license.
All the sources files specify in their header comment a GPLv2 or more 
license.

[+] MUST: If (and only if) the source package includes the text of the
license(s) in its own file, then that file, containing the text of the
license(s) for the package must be included in %doc.

[+] MUST: The spec file must be written in American English.

[+] MUST: The spec file for the package MUST be legible.

[+] MUST: The sources used to build the package must match the upstream source,
as provided in the spec URL.
MD5 sum=4257305ffb27177a6d5208b2df4ca92d OK

[+] MUST: The package must successfully compile and build into binary rpms on
at least one supported architecture.

[+] MUST: If the package does not successfully compile, build or work on an
architecture, then those architectures should be listed in the spec in
ExcludeArch.

[+] MUST: All build dependencies must be listed in BuildRequires

[+] MUST: The spec file MUST handle locales properly. This is done by using the
%find_lang macro.

[+] MUST: Every binary RPM package which stores shared library files (not just
symlinks) in any of the dynamic linker's default paths, must call ldconfig in
%post and %postun.

[+] MUST: If the package is designed to be relocatable, the packager must state
this fact in the request for review

[+] MUST: A package must own all directories that it creates. If it does not
create a directory that it uses, then it should require a package which does
create that directory.

[+] MUST: A package must not contain any duplicate files in the %files listing.

[+] MUST: Permissions on files must be set properly. Executables should be set
with executable permissions, for example.

[+] MUST: Each package must consistently use macros, as described in the macros
section of Packaging Guidelines.

[+] MUST: The package must contain code, or permissible content. This is
described in detail in the code vs. content section of Packaging Guidelines.

[+] MUST: Large documentation files should go in a doc subpackage.

[+] MUST: If a package includes something as %doc, it must not affect the
runtime of the application.

[+] MUST: Header files must be in a -devel package.

[+] MUST: Static libraries must be in a -static package.

[+] MUST: If a package contains library files with a suffix (e.g.
libfoo.so.1.1), then library files that end in .so (without suffix) must go in
a -devel package.

[+] MUST: In the vast majority of cases, devel packages must require the base
package using a fully versioned dependency: Requires: %{name} =
%{version}-%{release} 

[+] MUST: Packages must NOT contain any .la libtool archives, these should be
removed in the spec.

[+] MUST: Packages containing GUI applications must include a %{name}.desktop
file, and that file must be properly installed with desktop-file-install in the
%install section.

[+] MUST: Packages must not own files or directories already owned by other
packages.

[+] MUST: All filenames in rpm packages must be valid UTF-8.

Once the License tag fixed to GPLv2+, I will approve your package.

-- 
Configure bugmail: 

[Bug 770371] Review Request: libva-intel-driver - HW video decode support for Intel integrated graphics

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=770371

--- Comment #5 from Nicolas Chauvet (kwizart) kwiz...@gmail.com 2012-01-03 
06:37:09 EST ---
SRPM:
http://rpms.kwizart.net/fedora/reviews/libva-intel-driver/libva-intel-driver-1.0.15-2.fc16.src.rpm
SPEC:
http://rpms.kwizart.net/fedora/reviews/libva-intel-driver/libva-intel-driver.spec
Summary: HW video decode support for Intel integrated graphics

Update package with few clean-ups as there is only a dlopened shared object (so
no ldconfig).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 766932] Review Request: icfg - utiltiy for scriptable editing of network interface files

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=766932

Neil Horman nhor...@redhat.com changed:

   What|Removed |Added

   Flag|needinfo?(nhor...@redhat.co |
   |m)  |

--- Comment #15 from Neil Horman nhor...@redhat.com 2012-01-03 06:42:45 EST 
---
New Package SCM Request
===
Package Name: icfg
Short Description: sysv network configuration file command line editor
Owners: nhorman
Branches: rawhide, fc16
InitialCC: nhor...@redhat.com, agosp...@redhat.com

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 771318] Review Request: rubygem-showoff - Ruby web app that reads simple configuration files for a presentation

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=771318

--- Comment #1 from Michal Fojtik mfoj...@redhat.com 2012-01-03 06:36:26 EST 
---
Sorry for wrong links (again ;-)

Spec URL: http://omicron.mifo.sk/rubygem-showoff.spec
SRPM URL: http://omicron.mifo.sk/rubygem-showoff-0.7.0-2.fc14.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 771314] Review Request: rubygem-parslet - Parser construction library with great error reporting in Ruby

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=771314

--- Comment #1 from Michal Fojtik mfoj...@redhat.com 2012-01-03 06:35:45 EST 
---
Sorry for wrong links:

Spec URL: http://omicron.mifo.sk/rubygem-parslet.spec
SRPM URL: http://omicron.mifo.sk/rubygem-parslet-1.2.3-1.fc14.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 770721] Review Request: substance - Swing look-and-feel library

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=770721

--- Comment #5 from Mohamed El Morabity pikachu.2...@gmail.com 2012-01-03 
06:49:01 EST ---
I'm unable to reproduce your bug. I think the source archive you downloaded is
corrupted. Delete it and try to retrieve it again using this command:
   $ spectool -g substance.spec -C ../SOURCES/

By the way, be very careful with the formatting of the changelog; be sure to
use for each entry  the following pattern:
* DATE AUTHOR EMAIL VERSION-RELEASE
 - COMMENT1
 - COMMENT2
 - ...
Don't use also macros in changelog, unless you escape them using %:
- added %{_javadir]/* to files section →- added %%{_javadir}/* to files
section

You'd better put the Git commit ID in a macro, in case substance is updated:
   # Git commit ID corresponding to the current version of substance
   %global commit  58d3873

   Name:  substance
   Version: 6.0
   ...
   Source0:
https://github.com/kirillcool/%{name}/zipball/master/kirillcool-%{name}-%{commit}.zip
   ...
   %prep
   %setup -q -n kirillcool-%{name}-%{commit}
   ...

The most important thing is that substance embeds dependencies libs, as for
TV-Browser. You must delete them before building and rely on those provided by
the Fedora repos. To help you, here is the list of the embedded jar files and
the corresponding Fedora package providing it, if available:
   trident.jar → not available, must be packaged
(http://kenai.com/projects/trident/pages/Home)
   forms-1.2.0.jar → jgoodies-forms
   laf-plugin.jar → laf-plugin
   asm-all-2.2.2.jar → objectweb-asm
   laf-widget.jar → not available, must be packaged
(http://java.net/projects/laf-widget/s)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 771070] Review Request: nwipe - Securely erase disks using a variety of recognized methods

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=771070

--- Comment #9 from Ivan Romanov dr...@land.ru 2012-01-03 06:54:49 EST ---
Your package hasn't Group field. You must add it.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 771233] Review Request: rubygem-rack-protection - Ruby gem that protects against typical web attacks

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=771233

Michal Fojtik mfoj...@redhat.com changed:

   What|Removed |Added

 Blocks||691731

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 721179] Review Request: rubygem-extlib - Support library for DataMapper and Merb

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=721179

--- Comment #29 from Vít Ondruch vondr...@redhat.com 2012-01-03 06:57:29 EST 
---
Hi Shawn,

* There is still unnecessary noise in rpmlint ouput:

./rubygem-extlib.spec:2: W: macro-in-comment %global
./rubygem-extlib.spec:2: W: macro-in-comment %(ruby
./rubygem-extlib.spec:81: W: macro-in-comment %exclude
./rubygem-extlib.spec:81: W: macro-in-comment %{geminstdir}
./rubygem-extlib.spec:91: W: macro-in-comment %exclude
./rubygem-extlib.spec:91: W: macro-in-comment %{geminstdir}
rubygem-extlib.src: W: strange-permission extlib-0.9.15.gem 0666L

* I am not sure why are still defining the ruby_sitearch macro. I would suggest
to remove lines 2-3 (sed '2,3d' rubygem-extlib.spec). Note that this would fix
also one of the rpmlint complaints.

* The Source0 URL is obsolete. You should use
http://rubygems.org/gems/%{gemname}-%{version}.gem; instead.

* Be careful with the VERSION file. Some gems may require it for proper
functionality. Nevertheless in this case, it is not mandatory, so not and
issue.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 771295] Review Request: rubygem-delorean - lets you travel in time with Ruby by mocking Time.now

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=771295

Vít Ondruch vondr...@redhat.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||vondr...@redhat.com
 AssignedTo|nob...@fedoraproject.org|vondr...@redhat.com

--- Comment #1 from Vít Ondruch vondr...@redhat.com 2012-01-03 07:06:05 EST 
---
I'll take it for a review.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 756772] Review Request: libosinfo - A library for managing OS information for virtualization

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=756772

--- Comment #7 from Daniel Berrange berra...@redhat.com 2012-01-03 07:15:17 
EST ---
1. The 'extra_release' tag enables creation of easy one-off test builds with
unique nevr. This is common across most virt related RPMs in Fedora.

2. manpages are an RFE for a future upstream release

3. Yes, this isn't even suitable for EPEL6, due to requirement for gobject
introspection support

4. Yes, we can likely use %global with no illeffects

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 759057] Review Request: SOIL - Simple OpenGL Image Library

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=759057

--- Comment #5 from Mohamed El Morabity pikachu.2...@gmail.com 2012-01-03 
07:19:28 EST ---
Notice also these rpmlint warnings:
   SOIL-devel.x86_64: W: spurious-executable-perm /usr/include/SOIL/SOIL.h
   SOIL-devel.x86_64: E: wrong-script-end-of-line-encoding
/usr/include/SOIL/SOIL.h
   SOIL-devel.x86_64: W: spurious-executable-perm /usr/include/SOIL/image_DXT.h
   SOIL-devel.x86_64: E: wrong-script-end-of-line-encoding
/usr/include/SOIL/image_DXT.h
   SOIL-devel.x86_64: W: spurious-executable-perm
/usr/include/SOIL/stbi_DDS_aug.h
   SOIL-devel.x86_64: E: wrong-script-end-of-line-encoding
/usr/include/SOIL/stbi_DDS_aug.h
   SOIL-devel.x86_64: W: spurious-executable-perm
/usr/include/SOIL/image_helper.h
   SOIL-devel.x86_64: E: wrong-script-end-of-line-encoding
/usr/include/SOIL/image_helper.h
   SOIL-devel.x86_64: W: spurious-executable-perm
/usr/include/SOIL/stbi_DDS_aug_c.h
   SOIL-devel.x86_64: E: wrong-script-end-of-line-encoding
/usr/include/SOIL/stbi_DDS_aug_c.h
   SOIL-devel.x86_64: W: spurious-executable-perm
/usr/include/SOIL/stb_image_aug.h
   SOIL-devel.x86_64: E: wrong-script-end-of-line-encoding
/usr/include/SOIL/stb_image_aug.h

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 771233] Review Request: rubygem-rack-protection - Ruby gem that protects against typical web attacks

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=771233

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

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||bkab...@redhat.com
 AssignedTo|nob...@fedoraproject.org|bkab...@redhat.com
   Flag||fedora-review?

--- Comment #2 from Bohuslav Kabrda bkab...@redhat.com 2012-01-03 07:15:45 
EST ---
I'm taking this one.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 759057] Review Request: SOIL - Simple OpenGL Image Library

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=759057

Mohamed El Morabity pikachu.2...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-review?

--- Comment #4 from Mohamed El Morabity pikachu.2...@gmail.com 2012-01-03 
07:18:01 EST ---
The package looks quite good. Just one comment anyway: I think you could get
rid of the sed patching, since all the Makefile parameters you modify can be
overridden, as below:
  %build
  pushd src
  make %{?_smp_mflags} CFLAGS=$RPM_OPT_FLAGS -c -fPIC -f
../projects/makefile/alternate\ Makefile.txt
  popd

  %install
  rm -rf $RPM_BUILD_ROOT
  pushd src
  make install -f ../projects/makefile/alternate\ Makefile.txt install
DESTDIR=$RPM_BUILD_ROOT INSTALL_FILE=install -p INSTALL_DIR=install -dp
LIBDIR=%{_libdir} INCLUDEDIR=%{_includedir}/SOIL
  popd

The less you patch, the better it is ^^.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 752311] Review Request: gitso - A front-end to reverse VNC connections

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=752311

--- Comment #4 from Mohamed El Morabity pikachu.2...@gmail.com 2012-01-03 
07:25:23 EST ---
Your package doesn't provide any binary. It should be noarch.

Notice also the following rpmlint warnings:
   gitso.x86_64: E: non-executable-script /usr/share/gitso/ArgsParser.py 0644L
/usr/bin/env
   gitso.x86_64: E: non-executable-script /usr/share/gitso/Gitso.py 0644L
/usr/bin/env
   gitso.x86_64: E: non-executable-script /usr/share/gitso/ConnectionWindow.py
0644L /usr/bin/env
   gitso.x86_64: E: non-executable-script /usr/share/gitso/NATPMP.py 0644L
/usr/bin/env
   gitso.x86_64: E: non-executable-script /usr/share/gitso/Processes.py 0644L
/usr/bin/env
   gitso.x86_64: E: non-executable-script /usr/share/gitso/GitsoThread.py 0644L
/usr/bin/env
You must remove the shebang at the top of each of these files, as described
here:
  
http://fedoraproject.org/wiki/Packaging_tricks#Remove_shebang_from_Python_libraries

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 770755] Review Request: gnome-shell-extension-calc - A simple calculator in the search overview

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=770755

Mohamed El Morabity pikachu.2...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-review?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 769096] Review Request: gnome-shell-extension-windowoverlay-icons - Easily discover which application to select by viewing the app icons in the windows overview.

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=769096

Mohamed El Morabity pikachu.2...@gmail.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Flag||fedora-review?

--- Comment #9 from Mohamed El Morabity pikachu.2...@gmail.com 2012-01-03 
07:56:27 EST ---
That looks perfect :).

I've just thinking about something: since the git repository contains in fact
two extensions (windowoverlay-icons and historymanager-prefix-search), would
you be OK to package both? You'll just have to rename your spec file to 
gnome-shell-extensions-sustmi and create a subpackage for each extension.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 771295] Review Request: rubygem-delorean - lets you travel in time with Ruby by mocking Time.now

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=771295

Vít Ondruch vondr...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-review+

--- Comment #2 from Vít Ondruch vondr...@redhat.com 2012-01-03 07:55:25 EST 
---
* Test suite
  - Please consider execution of the test suite in the %check section? It seems
there are specs available upstream.

* rpmlint output
  - rpmlint complains about permissions of .spec file:

rubygem-delorean.src: W: strange-permission rubygem-delorean.spec 0666L
A file that you listed to include in your package has strange permissions.
Usually, a file should have 0644 permissions.

Otherwise, I have no other comments, so I APPROVE the package. Please fix the
rpmlint before import of the package and consider execution of the test suite.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 771233] Review Request: rubygem-rack-protection - Ruby gem that protects against typical web attacks

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=771233

--- Comment #3 from Bohuslav Kabrda bkab...@redhat.com 2012-01-03 07:56:05 
EST ---
- The package FTBTS [1]. Please do a Koji build before every review to see if
it passes.
-- You need to BR: rubygem(rspec-core) instead of rubygem(rspec).
-- You also need to BR: rubygem(rack-test) for running the specs.
- Could you specify what is the advantage of running the specs with ruby -S
rspec spec, when rspec spec suffices? This is not a blocker, but why not
make things as simple as possible?
- Do you intend to place the package into EPEL, too? If not, please remove the
unnecessary BuildRoot tag (see [2] for more info).
- Please be consistent in usage of macros for shell commands and the commands
themselves. For example, you use both %{__mkdir_p} macro and mkdir -p
command. So decide whether you want to use macros or commands and don't mix the
two.
- Consider excluding the cached gem, as it is not an unnecessary payload, not
needed for RPM package.
- Mark %{geminstdir}/License with %doc.
- Move %{geminstdir}/README.md to the -doc subpackage, if it is not needed for
runtime (which I believe it isn't) and mark it with %doc.
- Also, mark %{gemdir}/doc/%{gemname}-%{version} with %doc.


[1] http://koji.fedoraproject.org/koji/taskinfo?taskID=3615389
[2] https://fedoraproject.org/wiki/Packaging:Guidelines#BuildRoot_tag

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 771295] Review Request: rubygem-delorean - lets you travel in time with Ruby by mocking Time.now

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=771295

--- Comment #3 from Bohuslav Kabrda bkab...@redhat.com 2012-01-03 08:05:08 
EST ---
Thank you for your review. I'll fix these things before making a SCM request.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 708402] Review Request: tophat - A spliced read mapper for RNA-Seq

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=708402

--- Comment #6 from Adam Huffman bl...@verdurin.com 2012-01-03 08:20:06 EST 
---
Update to 1.3.3:

http://verdurin.fedorapeople.org/reviews/tophat/tophat.spec

http://verdurin.fedorapeople.org/reviews/tophat/tophat-1.3.3-1.fc16.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 699316] Review Request: inkscape-textext - Textext is an extension for Inkscape that allows adding LaTeX-generated editable text objects to the SVG drawings

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=699316

--- Comment #8 from Mohamed El Morabity pikachu.2...@gmail.com 2012-01-03 
08:18:44 EST ---
Just one suggestion: to identify in a unique way the version/commit, you should
mention the commit hash in the release:
Release: 3.%{?hgdate}hg%{hghash}%{?dist}

For consistency, you should use also the install command to install the
textext.py file.

The development version is OK for me, since it includes many fixes. And the
package works very well in Inkscape :)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 722640] Review Request: R-qcc - SQC package for R

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=722640

--- Comment #18 from Jon Ciesla limburg...@gmail.com 2012-01-03 08:23:38 EST 
---
Git done (by process-git-requests).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 742388] Review Request: polybori - Framework for Boolean Rings

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=742388

--- Comment #13 from Jon Ciesla limburg...@gmail.com 2012-01-03 08:24:48 EST 
---
Git done (by process-git-requests).

Brendan, please take ownership of review BZs, thanks!

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 749885] Review Request: iris - A library for working with the XMPP/Jabber protocol

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=749885

--- Comment #36 from Jon Ciesla limburg...@gmail.com 2012-01-03 08:25:52 EST 
---
Git done (by process-git-requests).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 768976] Review Request: perl-Dancer-Session-Cookie - Encrypted cookie-based session back-end for Dancer

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=768976

--- Comment #3 from Jon Ciesla limburg...@gmail.com 2012-01-03 08:29:40 EST 
---
Git done (by process-git-requests).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 755275] Review Request: CSSLint - Detecting potential problems in CSS code

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=755275

--- Comment #8 from Jon Ciesla limburg...@gmail.com 2012-01-03 08:26:58 EST 
---
No valid FAS accounts listed, summary name and SCM request names don't
match.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 766932] Review Request: icfg - utiltiy for scriptable editing of network interface files

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=766932

--- Comment #16 from Jon Ciesla limburg...@gmail.com 2012-01-03 08:28:51 EST 
---
nhorman is not a valid FAS account.
Use f16, not fc16.
rawhide==devel, do not include, it's created by default.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 769444] Review Request: ruby-korundum - Ruby bindings for KDE libraries

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=769444

--- Comment #9 from Jon Ciesla limburg...@gmail.com 2012-01-03 08:31:15 EST 
---
Git done (by process-git-requests).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 771318] Review Request: rubygem-showoff - Ruby web app that reads simple configuration files for a presentation

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=771318

--- Comment #2 from Michal Fojtik mfoj...@redhat.com 2012-01-03 08:31:00 EST 
---
Revision -3:

Spec URL: http://omicron.mifo.sk/rubygem-showoff.spec
SRPM URL: http://omicron.mifo.sk/rubygem-showoff-0.7.0-3.fc14.src.rpm

* Mon Jan 03 2012 Michal Fojtik mfoj...@redhat.com - 0.7.0-3

- Removed rspec dependency
- Fixed licenses for some javascript files
- Added RMagick dependency for handling image resizing

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 771295] Review Request: rubygem-delorean - lets you travel in time with Ruby by mocking Time.now

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=771295

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

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #4 from Bohuslav Kabrda bkab...@redhat.com 2012-01-03 08:41:21 
EST ---
New Package SCM Request
===
Package Name: rubygem-delorean
Short Description: Delorean lets you travel in time with Ruby by mocking
Time.now
Owners: bkabrda
Branches: 
InitialCC:

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 771318] Review Request: showoff - Ruby web app that reads simple configuration files for a presentation

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=771318

Michal Fojtik mfoj...@redhat.com changed:

   What|Removed |Added

Summary|Review Request: |Review Request: showoff -
   |rubygem-showoff - Ruby web  |Ruby web app that reads
   |app that reads simple   |simple configuration files
   |configuration files for a   |for a presentation
   |presentation|

--- Comment #3 from Michal Fojtik mfoj...@redhat.com 2012-01-03 08:46:44 EST 
---
Revision -4:

Spec URL: http://omicron.mifo.sk/showoff.spec
SRPM URL: http://omicron.mifo.sk/showoff-0.7.0-4.fc14.src.rpm

* Mon Jan 03 2012 Michal Fojtik mfoj...@redhat.com - 0.7.0-4

- Switched to application package type instead of gem

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 771295] Review Request: rubygem-delorean - lets you travel in time with Ruby by mocking Time.now

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=771295

--- Comment #5 from Jon Ciesla limburg...@gmail.com 2012-01-03 08:49:05 EST 
---
Git done (by process-git-requests).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 766932] Review Request: icfg - utiltiy for scriptable editing of network interface files

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=766932

--- Comment #17 from Neil Horman nhor...@redhat.com 2012-01-03 08:46:10 EST 
---
Yes, nhorman is my fedora account username:
https://admin.fedoraproject.org/accounts/user/view/nhorman

I'm not sure what else you want, its worked perfectly well in the past:
https://bugzilla.redhat.com/show_bug.cgi?id=491240


New Package SCM Request
===
Package Name: icfg
Short Description: sysv network configuration file command line editor
Owners: nhorman
Branches: rawhide, f16
InitialCC: nhor...@redhat.com, agosp...@redhat.com

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 769444] Review Request: ruby-korundum - Ruby bindings for KDE libraries

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=769444

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

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE
Last Closed||2012-01-03 08:59:46

--- Comment #10 from Rex Dieter rdie...@math.unl.edu 2012-01-03 08:59:46 EST 
---
thanks!

imported.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 731456] Review Request: jreen - Qt XMPP Library

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=731456

Bug 731456 depends on bug 749885, which changed state.

Bug 749885 Summary: Review Request: iris - A library for working with the 
XMPP/Jabber protocol
https://bugzilla.redhat.com/show_bug.cgi?id=749885

   What|Old Value   |New Value

 Status|NEW |ASSIGNED
 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 749885] Review Request: iris - A library for working with the XMPP/Jabber protocol

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=749885

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

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE
Last Closed||2012-01-03 09:02:47

--- Comment #37 from Rex Dieter rdie...@math.unl.edu 2012-01-03 09:02:47 EST 
---
imported.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 771318] Review Request: showoff - Ruby web app that reads simple configuration files for a presentation

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=771318

--- Comment #4 from Michal Fojtik mfoj...@redhat.com 2012-01-03 09:10:21 EST 
---
Revision -5:

Spec URL: http://omicron.mifo.sk/showoff.spec
SRPM URL: http://omicron.mifo.sk/showoff-0.7.0-5.fc14.src.rpm

* Mon Jan 03 2012 Michal Fojtik mfoj...@redhat.com - 0.7.0-5
- Changed package group to Applications/Publishing

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 768976] Review Request: perl-Dancer-Session-Cookie - Encrypted cookie-based session back-end for Dancer

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=768976

Petr Pisar ppi...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
   Fixed In Version||perl-Dancer-Session-Cookie-
   ||0.15-1.fc17
 Resolution||RAWHIDE
Last Closed||2012-01-03 09:13:53

--- Comment #4 from Petr Pisar ppi...@redhat.com 2012-01-03 09:13:53 EST ---
Thank you for the review and the repository.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 771295] Review Request: rubygem-delorean - lets you travel in time with Ruby by mocking Time.now

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=771295

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

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE
Last Closed||2012-01-03 09:20:57

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 766943] Review Request: jsilver - A pure-Java implementation of Clearsilver

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=766943

Andy Grimm agr...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #3 from Andy Grimm agr...@gmail.com 2012-01-03 09:46:59 EST ---
New Package SCM Request
===
Package Name: jsilver
Short Description: A pure-Java implementation of Clearsilver
Owners: arg
Branches:
InitialCC:

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 756772] Review Request: libosinfo - A library for managing OS information for virtualization

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=756772

--- Comment #8 from Daniel Berrange berra...@redhat.com 2012-01-03 09:57:38 
EST ---
On closer examination I just removed the global macros, since there is no value
in being able to disable introspection, and attempting to disable it would
break the build of the -vala sub-RPM anyway.

Updated SPEC file at same URL as before

http://berrange.fedorapeople.org/review/libosinfo/libosinfo-0.0.4-2.fc17.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 754583] Review Request: dnssec-trigger - Update/reconfigure DNSSEC resolving

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=754583

--- Comment #7 from Paul Wouters p...@nohats.ca 2012-01-03 10:20:39 EST ---

Maybe it got fixed the issue meanwhile? I cannot reproduce it. Please try
version 0.9

Spec URL: ftp://ftp.xelerance.com/dnssec-trigger/dnssec-trigger.spec
SRPM URL:
ftp://ftp.xelerance.com/dnssec-trigger/dnssec-trigger-0.9-1.fc16.src.rpm


[paul@bofh mock]$ ls -l default.cfg 
lrwxrwxrwx 1 root root 25 Dec 18 15:28 default.cfg - fedora-rawhide-x86_64.cfg
[paul@bofh mock]$ mock rebuild ~/SRPMS/dnssec-trigger-0.9-1.fc16.src.rpm 
INFO: mock.py version 1.1.18 starting...
State Changed: init plugins
INFO: selinux disabled
State Changed: start
INFO: Start(/home/paul/SRPMS/dnssec-trigger-0.9-1.fc16.src.rpm) 
Config(fedora-rawhide-x86_64)
State Changed: lock buildroot
State Changed: clean
INFO: chroot (/var/lib/mock/fedora-rawhide-x86_64) unlocked and deleted
State Changed: unlock buildroot
State Changed: init
State Changed: lock buildroot
Mock Version: 1.1.18
INFO: Mock Version: 1.1.18
INFO: calling preinit hooks
INFO: enabled root cache
INFO: root cache aged out! cache will be rebuilt
INFO: enabled yum cache
State Changed: cleaning yum metadata
INFO: enabled ccache
State Changed: running yum
State Changed: creating cache
State Changed: unlock buildroot
INFO: Installed packages:
State Changed: setup
State Changed: build
INFO: Done(/home/paul/SRPMS/dnssec-trigger-0.9-1.fc16.src.rpm) Config(default)
7 minutes 49 seconds
INFO: Results and/or logs in: /var/lib/mock/fedora-rawhide-x86_64/result
State Changed: end

I'm currently rebuilding with mock on i386 to see if that makes a difference...

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 766943] Review Request: jsilver - A pure-Java implementation of Clearsilver

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=766943

--- Comment #4 from Jon Ciesla limburg...@gmail.com 2012-01-03 10:22:53 EST 
---
Git done (by process-git-requests).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 766932] Review Request: icfg - utiltiy for scriptable editing of network interface files

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=766932

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

   What|Removed |Added

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

--- Comment #18 from Jon Ciesla limburg...@gmail.com 2012-01-03 10:25:40 EST 
---
Complete.

Use FAS for initialCC, not email.  agospoda isn't a valid FAS account.  Also,
don't include rawhide.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 479598] Review Request: aopalliance - AOP offers a better solution to many problems than do existing technologies

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=479598

Tom spot Callaway tcall...@redhat.com changed:

   What|Removed |Added

 Blocks|182235(FE-Legal)|

--- Comment #30 from Tom spot Callaway tcall...@redhat.com 2012-01-03 
10:29:26 EST ---
Given Fedora's revised position on Public Domain code, this package is now
considered legally permissible in Fedora. I am dropping the FE-Legal blocker,
but I am not reopening this old bug. If Sandro wants to reopen this request, he
can do so, or a new review bug can be opened by an interested maintainer.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 754583] Review Request: dnssec-trigger - Update/reconfigure DNSSEC resolving

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=754583

--- Comment #8 from Paul Wouters p...@nohats.ca 2012-01-03 10:34:48 EST ---
mock -r fedora-rawhide-i386 ~/SRPMS/dnssec-trigger-0.9-1.fc16.src.rpm
State Changed: creating cache
State Changed: unlock buildroot
INFO: Installed packages:
State Changed: setup
State Changed: build
INFO: Done(/home/paul/SRPMS/dnssec-trigger-0.9-1.fc16.src.rpm)
Config(fedora-rawhide-i386) 13 minutes 33 seconds
INFO: Results and/or logs in: /var/lib/mock/fedora-rawhide-i386/result
State Changed: end
[paul@bofh mock]$ cd /var/lib/mock/fedora-rawhide-i386/result
[paul@bofh result]$ ls
build.log
dnssec-trigger-0.9-1.fc17.i686.rpm
dnssec-trigger-0.9-1.fc17.src.rpm
dnssec-trigger-debuginfo-0.9-1.fc17.i686.rpm
root.log
state.log

Works for me. Please retest ?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 767556] Review Request: scl-utils - Utilities for alternative packaging

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=767556

Jindrich Novy jn...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE
Last Closed||2012-01-03 10:31:23

--- Comment #6 from Jindrich Novy jn...@redhat.com 2012-01-03 10:31:23 EST ---
scl-utils are now included in rawhide/f15/f16/el5/el6 and new package updates
are relased.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 749752] Review Request: dmg2img - Uncompress the Apple compressed disk image files

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=749752

--- Comment #4 from Richard Shaw hobbes1...@gmail.com 2012-01-03 10:36:04 EST 
---
Ping!

Is the requester still interested in this review request?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 742388] Review Request: polybori - Framework for Boolean Rings

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=742388

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

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||RAWHIDE
Last Closed||2012-01-03 10:57:45

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 771233] Review Request: rubygem-rack-protection - Ruby gem that protects against typical web attacks

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=771233

--- Comment #4 from Michal Fojtik mfoj...@redhat.com 2012-01-03 11:09:52 EST 
---
(In reply to comment #3)

Hi Bohuslav

First thanks for reviewing this package!

 - The package FTBTS [1]. Please do a Koji build before every review to see if
 it passes.
 -- You need to BR: rubygem(rspec-core) instead of rubygem(rspec).
 -- You also need to BR: rubygem(rack-test) for running the specs.

All done.

 - Could you specify what is the advantage of running the specs with ruby -S
 rspec spec, when rspec spec suffices? This is not a blocker, but why not
 make things as simple as possible?

We used it in thin package. However this form sounds better and shorter for me.
Fixed in -2.

 - Do you intend to place the package into EPEL, too? If not, please remove the
 unnecessary BuildRoot tag (see [2] for more info).

This package will be imported to EPEL. I'll exclude BuildRoot from Fedora
packages
before import.

 - Please be consistent in usage of macros for shell commands and the commands
 themselves. For example, you use both %{__mkdir_p} macro and mkdir -p
 command. So decide whether you want to use macros or commands and don't mix 
 the
 two.

Done.

 - Consider excluding the cached gem, as it is not an unnecessary payload, not
 needed for RPM package.

Done.

 - Mark %{geminstdir}/License with %doc.

Done,

 - Move %{geminstdir}/README.md to the -doc subpackage, if it is not needed for
 runtime (which I believe it isn't) and mark it with %doc.
 - Also, mark %{gemdir}/doc/%{gemname}-%{version} with %doc.

Actually I don't think marking -doc subpackage files with %doc is necessary.
Could you point me to a guideline where this is required?

=

koji:
http://koji.fedoraproject.org/koji/taskinfo?taskID=3615827

rpmlint:
rubygem-rack-protection.noarch: I: enchant-dictionary-not-found en_US
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

* Mon Jan 03 2012 Michal Fojtik mfoj...@redhat.com - 1.2.0-2
- Fixed BR
- Marked documentation file with doc tag
- Changed the way how to run rspec tests

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 771233] Review Request: rubygem-rack-protection - Ruby gem that protects against typical web attacks

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=771233

--- Comment #5 from Michal Fojtik mfoj...@redhat.com 2012-01-03 11:11:11 EST 
---
Revision -2:

Spec: http://omicron.mifo.sk/rubygem-rack-protection.spec
SRPM: http://omicron.mifo.sk/rubygem-rack-protection-1.2.0-1.fc14.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 766932] Review Request: icfg - utiltiy for scriptable editing of network interface files

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=766932

--- Comment #19 from Andy Gospodarek agosp...@redhat.com 2012-01-03 11:11:29 
EST ---
(In reply to comment #18)
 Use FAS for initialCC, not email.  agospoda isn't a valid FAS account.

The proper FAS account for 'agosp...@redhat.com' is gospo.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 766932] Review Request: icfg - utiltiy for scriptable editing of network interface files

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=766932

--- Comment #20 from Itamar Reis Peixoto ita...@ispbrasil.com.br 2012-01-03 
11:18:45 EST ---
(In reply to comment #19)
 (In reply to comment #18)
  Use FAS for initialCC, not email.  agospoda isn't a valid FAS account.
 
 The proper FAS account for 'agosp...@redhat.com' is gospo.

now is too late, request commit access here - 

https://admin.fedoraproject.org/pkgdb/acls/name/icfg

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 771297] Review Request: rubygem-bluecloth - A Ruby implementation of John Gruber's Markdown

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=771297

--- Comment #2 from Michal Fojtik mfoj...@redhat.com 2012-01-03 11:16:45 EST 
---
Revision -3:

Spec URL: http://omicron.mifo.sk/rubygem-bluecloth.spec
SRPM URL: http://omicron.mifo.sk/rubygem-bluecloth-2.2.0-3.fc14.src.rpm

Koji:

http://koji.fedoraproject.org/koji/taskinfo?taskID=3615839

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 771311] Review Request: rubygem-gli - Git-Like Interface Command Line Parser for Ruby

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=771311

--- Comment #1 from Michal Fojtik mfoj...@redhat.com 2012-01-03 11:25:05 EST 
---
Koji:

http://koji.fedoraproject.org/koji/taskinfo?taskID=3615865

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 771311] Review Request: rubygem-gli - Git-Like Interface Command Line Parser for Ruby

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=771311

--- Comment #2 from Michal Fojtik mfoj...@redhat.com 2012-01-03 11:31:28 EST 
---
Sorry for wrong Koji link:

http://koji.fedoraproject.org/koji/taskinfo?taskID=3615876

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 768500] Review Request: subscription-manager - Tools and libraries for subscription and repository management.

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=768500

--- Comment #5 from Devan Goodwin dgood...@redhat.com 2012-01-03 11:33:53 EST 
---
New Package SCM Request
===
Package Name: subscription-manager
Short Description: Tools and libraries for subscription and repository
management.
Owners: dgoodwin jbowes
Branches: f15 f16
InitialCC:

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 768500] Review Request: subscription-manager - Tools and libraries for subscription and repository management.

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=768500

Devan Goodwin dgood...@rm-rf.ca changed:

   What|Removed |Added

   Flag||fedora-cvs?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 771316] Review Request: rubygem-blankslate - Ruby gem which provides an abstract base class with no predefined methods

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=771316

--- Comment #1 from Michal Fojtik mfoj...@redhat.com 2012-01-03 11:38:19 EST 
---
Koji build:

http://koji.fedoraproject.org/koji/taskinfo?taskID=3615894

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 768500] Review Request: subscription-manager - Tools and libraries for subscription and repository management.

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=768500

Devan Goodwin dgood...@rm-rf.ca changed:

   What|Removed |Added

 CC||dgood...@rm-rf.ca
 AssignedTo|dgood...@redhat.com |dgood...@rm-rf.ca

--- Comment #6 from Devan Goodwin dgood...@rm-rf.ca 2012-01-03 11:38:49 EST 
---
Filed this on my work account, suspect that is why I can't set fedora-cvs flag.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 768500] Review Request: subscription-manager - Tools and libraries for subscription and repository management.

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=768500

--- Comment #7 from Jon Ciesla limburg...@gmail.com 2012-01-03 11:50:22 EST 
---
Git done (by process-git-requests).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 757657] Review Request: perl-Wx-Scintilla - Scintilla source code editing component for wxWidgets

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=757657

--- Comment #3 from Petr Pisar ppi...@redhat.com 2012-01-03 11:51:42 EST ---
Spec file changes:
--- perl-Wx-Scintilla.spec 2011-11-28 11:00:50.0 +0100
+++ perl-Wx-Scintilla.spec.1 2012-01-02 16:58:24.0 +0100
@@ -4,17 +4,33 @@
 Version:0.34
 Release:1%{?dist}
 Summary:Scintilla source code editing component for wxWidgets
-License:GPL+ or Artistic
+# details of licenses:
+# wx-scintilla is bundle of scintilla under wxwindows
+# wx-scintilla/src/scintilla/src is copy of files from
scintilla/{lexers,lexlib,src}
+# wx-scintilla/src/scintilla/include contains upstream scintilla/include
without *.py file
+# wx-scintilla/src/scintilla/License.txt is MIT
+# wx-scintilla/include files are marked as wxWindows license
+# According to email with upstream the bundle is necessary, because every
project needs
+# modify scintilla for their own use.
+License:(GPL+ or Artistic) and MIT and wxWidgets
 Group:  Development/Libraries
 URL:http://search.cpan.org/dist/Wx-Scintilla/
 Source0:   
http://www.cpan.org/authors/id/A/AZ/AZAWAWI/Wx-Scintilla-%{version}.tar.gz
 BuildRequires:  perl(Alien::wxWidgets)
+BuildRequires:  perl(Carp)
+BuildRequires:  perl(Exporter)
 BuildRequires:  perl(ExtUtils::XSpp) = 0.1602
 BuildRequires:  perl(Module::Build)
 BuildRequires:  perl(Test::More)
 BuildRequires:  perl(Wx)
+BuildRequires:  perl(XSLoader)
 # tests BR
 BuildRequires:  wxGTK-devel, gtk2-devel
+BuildRequires:  perl(base)
+BuildRequires:  perl(Wx::App)
+BuildRequires:  perl(Wx::Frame)
+BuildRequires:  perl(Wx::Event)
+BuildRequires:  perl(lib)
 %if %{use_x11_tests}
 # X11 tests:
 BuildRequires:  xorg-x11-server-Xvfb
@@ -26,18 +42,26 @@

 %description
 While we already have a good scintilla editor component support via
-Wx::StyledTextCtrl in Perl, we already suffer from an older scintilla
-package and thus lagging Perl support in the popular Wx Scintilla
-component. wxWidgets http://wxwidgets.org has a *very slow* release
-timeline. Scintilla is a contributed project which means it will not be the
-latest by the time a new wxWidgets distribution is released. And on the
-scintilla front, the Perl 5 lexer is not 100% bug free even and we do not
-have any kind of Perl 6 support in Scintilla.
+Wx::StyledTextCtrl in Perl, we already suffer from an older scintilla package
+and thus lagging Perl support in the popular Wx Scintilla component. wxWidgets
+Lhttp://wxwidgets.org has a *very slow* release timeline. Scintilla is a
+contributed project which means it will not be the latest by the time a new
+wxWidgets distribution is released. And on the scintilla front, the Perl 5
lexer
+is not 100% bug free even and we do not have any kind of Perl 6 support in
+Scintilla.
+
+The ambitious goal of this project is to provide fresh Perl 5 and maybe 6
+support in LWx while preserving compatibility with Wx::StyledTextCtrl
+and continually contribute it back to Scintilla project.
+
+%{?perl_default_filter}

 %prep
 %setup -q -n Wx-Scintilla-%{version}
+rm -rf ./Wx-Scintilla-%{version}/wx-scintilla

 %build
+export LDFLAGS=--enable-new-dtags
 %{__perl} Build.PL installdirs=vendor optimize=$RPM_OPT_FLAGS
 ./Build

@@ -59,7 +83,7 @@
 %endif

 %files
-%doc Changes META.json README Scintilla_README 
+%doc Changes README Scintilla_README 
 %{perl_vendorarch}/auto/*
 %{perl_vendorarch}/Wx*
 %{_mandir}/man3/*


 FIX: License tag does not cover bundled Scintilla sources
 (wx-scintilla/src/scintilla/).
-License:GPL+ or Artistic
+# details of licenses:
+# wx-scintilla is bundle of scintilla under wxwindows
+# wx-scintilla/src/scintilla/src is copy of files from
scintilla/{lexers,lexlib,src}
+# wx-scintilla/src/scintilla/include contains upstream scintilla/include
without *.py file
+# wx-scintilla/src/scintilla/License.txt is MIT
+# wx-scintilla/include files are marked as wxWindows license
+# According to email with upstream the bundle is necessary, because every
project needs
+# modify scintilla for their own use.
+License:(GPL+ or Artistic) and MIT and wxWidgets

FIX: wx-scintilla/src/scintilla/License.txt looks like MIT but it is not exact
copy of any wordings from https://fedoraproject.org/wiki/Licensing/MIT.
Please get confirmation by Fedora legal department. Scintiall variant should
appear on the web page.

  FIX: Do not bundle Scintilla (wx-scintilla/src/scintilla/).
 Scintilla must be bundled.
FIX: Then you need exception from Fedora Packaging Committee
(https://fedoraproject.org/wiki/Packaging:No_Bundled_Libraries#Exceptions).

 TODO: Augment description with other paragraphs from lib/Wx/Scintilla.pm POD.
 The first one does not define what this package delivers.
 %description
 While we already have a good scintilla editor component support via
-Wx::StyledTextCtrl in Perl, we already suffer 

[Bug 770371] Review Request: libva-intel-driver - HW video decode support for Intel integrated graphics

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=770371

Tom spot Callaway tcall...@redhat.com changed:

   What|Removed |Added

 CC||tcall...@redhat.com

--- Comment #6 from Tom spot Callaway tcall...@redhat.com 2012-01-03 
12:28:53 EST ---
Just to be clear here, this package is just the same mess that we said was not
okay before, just in a separate package?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 759059] Review Request: SFML - Simple and Fast Multimedia Library

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=759059

Leif Walsh leif.wa...@gmail.com changed:

   What|Removed |Added

 CC||leif.wa...@gmail.com

--- Comment #4 from Leif Walsh leif.wa...@gmail.com 2012-01-03 13:08:27 EST 
---
I am trying to build dolphin, which requires sfml-devel.  I could only find the
binary on your server, so I tried to build from spec, but I also could not find
the two patches the spec references, so I commented them out, and got this
failure:

g++ -o SocketUDP.o -c SocketUDP.cpp -I../../../include -I../../ -O2 -g -fPIC
Linux/WindowImplX11.cpp:30:38: fatal error: SFML/Window/glext/glxext.h: No such
file or directory
compilation terminated.

Can you provide the patches if they fix this, or help me debug this failure?  I
don't even see the SFML/Window/glext directory.

Also, your binary packages assume the existence of a user julas.  This seems
to be an error.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 759059] Review Request: SFML - Simple and Fast Multimedia Library

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=759059

--- Comment #5 from Leif Walsh leif.wa...@gmail.com 2012-01-03 13:12:54 EST 
---
It seems I just don't know what a source RPM is.  I'm new to RPM, sorry. 
Investigating on my own, will come back if I don't get it right.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 721179] Review Request: rubygem-extlib - Support library for DataMapper and Merb

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=721179

--- Comment #30 from Shawn Starr shawn.st...@rogers.com 2012-01-03 13:24:01 
EST ---
Good idea on rpmlint for .spec file itself:

1) Fixed noise in comments
2) Removed sitearch macro altogether, this rubygem has no DSOs
3) Replace SOURCE0 URL with new location - URL also is valid.
4) Don't delete VERSION and keep it with the package for consistency mark as a
%doc file - /usr/lib/ruby/gems/1.8/gems/extlib-0.9.15/VERSION


Spec URL: http://www.sh0n.net/spstarr/fedora/rubygem-extlib/rubygem-extlib.spec
SRPM URL:
http://www.sh0n.net/spstarr/fedora/rubygem-extlib/rubygem-extlib-0.9.15-3.fc17.src.rpm

rpmlint: PASS (with usual ruby doc warnings) RPMs
rpmlint: PASS .spec file

mock build: PASS

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 756491] Review Request: python-libcloud - Python library that abstracts away differences among multiple cloud provider APIs.

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=756491

Daniel Bruno dbr...@fedoraproject.org changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #5 from Daniel Bruno dbr...@fedoraproject.org 2012-01-03 13:27:04 
EST ---
New Package SCM Request
===
Package Name: python-libcloud
Short Description: A Python library to address multiple cloud provider APIs
Owners: dbruno
Branches: f15 f16 f17
InitialCC:

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 756491] Review Request: python-libcloud - Python library that abstracts away differences among multiple cloud provider APIs.

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=756491

--- Comment #6 from Jon Ciesla limburg...@gmail.com 2012-01-03 13:31:57 EST 
---
Git done (by process-git-requests).

f17==devel, no need to explicitly request.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 759057] Review Request: SOIL - Simple OpenGL Image Library

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=759057

--- Comment #6 from Julian Sikorski beleg...@gmail.com 2012-01-03 13:43:51 
EST ---
I am not sure how you got these warnings:
[julas@snowball2 SPECS]$ rpm -q SOIL-devel
SOIL-devel-1.07-3.20080706.fc16.x86_64
[julas@snowball2 SPECS]$ rpmlint SOIL-devel
SOIL-devel.x86_64: W: no-documentation
1 packages and 0 specfiles checked; 0 errors, 1 warnings.
[julas@snowball2 SPECS]$ ls -l /usr/include/SOIL
razem 64
-rw-r--r--. 1 root root  3212 2008-07-07  image_DXT.h
-rw-r--r--. 1 root root  2287 2008-07-07  image_helper.h
-rw-r--r--. 1 root root 15545 2008-07-07  SOIL.h
-rw-r--r--. 1 root root 15447 2008-07-07  stbi_DDS_aug_c.h
-rw-r--r--. 1 root root   797 2008-07-07  stbi_DDS_aug.h
-rw-r--r--. 1 root root 16945 2008-07-07  stb_image_aug.h

I think it is a side-effect of your sed replacement, -m 644 got missing
somehow.
Are you 100 % positive that overriding the makefile variables is better than
sed? A proper patch would be the best, but it is impossible for macros encoding
arch-dependent paths. To me it seems to be the matter of personal preference -
I find the sed overrides more legible since they change one thing at a time.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 759059] Review Request: SFML - Simple and Fast Multimedia Library

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=759059

--- Comment #6 from Julian Sikorski beleg...@gmail.com 2012-01-03 13:46:08 
EST ---
Just ignore the julas user/group - it is the leftover from the system I created
the srpm on.
A simple
$ rpmbuild --rebuild SFML-1.6-2.fc16.src.rpm
should do the trick, provided you build and install SOIL (blocks this bug)
first.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 752311] Review Request: gitso - A front-end to reverse VNC connections

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=752311

--- Comment #5 from Cédric OLIVIER cedric.oliv...@free.fr 2012-01-03 13:59:03 
EST ---
followed your observations I made the change :

SRPM URL :
http://cedric.olivier.free.fr/rpms/gitso-0.6-4/gitso-0.6-4.fc16.src.rpm
Spec URL : http://cedric.olivier.free.fr/rpms/gitso-0.6-4/gitso.spec

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 769794] new package: rpm2targz

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=769794

Bill Nottingham nott...@redhat.com changed:

   What|Removed |Added

  Component|distribution|Package Review
 CC||nott...@redhat.com,
   ||package-review@lists.fedora
   ||project.org
 AssignedTo|nott...@redhat.com  |nob...@fedoraproject.org
  QAContact|nott...@redhat.com  |extras...@fedoraproject.org

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 759059] Review Request: SFML - Simple and Fast Multimedia Library

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=759059

--- Comment #7 from Leif Walsh leif.wa...@gmail.com 2012-01-03 14:16:34 EST 
---
Got it working, thanks!

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 752311] Review Request: gitso - A front-end to reverse VNC connections

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=752311

--- Comment #6 from Mohamed El Morabity pikachu.2...@gmail.com 2012-01-03 
14:20:55 EST ---
Two last comments and I will approve your package:
- add a %build section, even if it's empty
- please change %{_mandir}/man1/%{name}.1.gz to %{_mandir}/man1/gnac.1* so the
extension .gz is not hardcoded, in case the man pages compressing format
changes.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 771462] New: Review Request: Wnotes - Graphical text notes for X Window System display

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: Wnotes - Graphical text notes for X Window System 
display

https://bugzilla.redhat.com/show_bug.cgi?id=771462

   Summary: Review Request: Wnotes - Graphical text notes for X
Window System display
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: sou...@dgplug.org
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


Spec URL: http://soumya.fedorapeople.org/wnotes/wnotes.spec
SRPM URL: http://soumya.fedorapeople.org/wnotes/wnotes-1.2-1.fc16.src.rpm
Description: Wnotes are Post-It (tm) type notes for Linux and other systems
with X Window System desktops. They use the Xlib libraries and are self
contained, and should work well with many different GUI desktops.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 752311] Review Request: gitso - A front-end to reverse VNC connections

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=752311

--- Comment #7 from Cédric OLIVIER cedric.oliv...@free.fr 2012-01-03 15:34:11 
EST ---
Thanks a lot, new spec and srpms :

SRPM URL :
http://cedric.olivier.free.fr/rpms/gitso-0.6-5/gitso-0.6-5.fc16.src.rpm
Spec URL : http://cedric.olivier.free.fr/rpms/gitso-0.6-5/gitso.spec

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 770721] Review Request: substance - Swing look-and-feel library

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=770721

--- Comment #6 from Sven s.bau...@gmx.net 2012-01-03 15:53:34 EST ---
Thanks for you help, this really helped me a bit forward.

As you already said, substance depends on some other embedded jar files, that I
need to rebuild, so I'm currently working on a Review Request for trident and
after this for laf.widget.

I also updated the SPEC and look forward for the other jar files to get
packaged ;).

Greetings
Sven

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 450616] opensm package

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=450616

Doug Ledford dledf...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE
Last Closed||2012-01-03 15:59:58

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 450482] libibumad package

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=450482

Doug Ledford dledf...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE
Last Closed||2012-01-03 15:59:40

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 450616] opensm package

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=450616

Bug 450616 depends on bug 450483, which changed state.

Bug 450483 Summary: libibmad package
https://bugzilla.redhat.com/show_bug.cgi?id=450483

   What|Old Value   |New Value

 Resolution||RAWHIDE
 Status|ASSIGNED|CLOSED

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 450483] libibmad package

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=450483

Doug Ledford dledf...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE
Last Closed||2012-01-03 15:59:49

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 450483] libibmad package

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=450483

Bug 450483 depends on bug 450482, which changed state.

Bug 450482 Summary: libibumad package
https://bugzilla.redhat.com/show_bug.cgi?id=450482

   What|Old Value   |New Value

 Resolution||RAWHIDE
 Status|ASSIGNED|CLOSED

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 543689] Package review: libipathverbs - Driver library for libibverbs

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=543689

Doug Ledford dledf...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE
Last Closed||2012-01-03 16:00:17

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 543685] Package review: libnes - Driver library for libibverbs

2012-01-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=543685

Doug Ledford dledf...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE
Last Closed||2012-01-03 16:00:09

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

  1   2   >