[Bug 713461] Review Request: tomahawk - The Social Music Player

[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=713461

Magnus Tuominen magnus.tuomi...@gmail.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #20 from Magnus Tuominen magnus.tuomi...@gmail.com 2012-01-07 
03:18:40 EST ---
No others, approving.

-- 
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

[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

Magnus Tuominen magnus.tuomi...@gmail.com changed:

   What|Removed |Added

 CC||magnus.tuomi...@gmail.com

--- Comment #33 from Magnus Tuominen magnus.tuomi...@gmail.com 2012-01-07 
04:14:03 EST ---
tomahawk seems to use the bundled simplesasl to connect to jabber and google,
need to split it to its own 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 766083] Fedora 17 feature Derelict

[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=766083

--- Comment #8 from Matthieu Saulnier casper.le.fan...@gmail.com 2012-01-07 
04:31:42 EST ---
Package Review
==

Key:
- = N/A
x = Pass
! = Fail
? = Not evaluated


 C/C++ 
[x]: MUST Package does not contain any libtool archives (.la)
[x]: MUST Package does not contain kernel modules.
[x]: MUST Package contains no static executables.
[x]: MUST Rpath absent or only used for internal libs.
[x]: MUST Package is not relocatable.
[x]: MUST Static libraries in -static subpackage, if present.


 Generic 
[x]: MUST Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[x]: MUST Package successfully compiles and builds into binary rpms on at
 least one supported architecture.
[x]: MUST All build dependencies are listed in BuildRequires, except for any
 that are listed in the exceptions section of Packaging Guidelines.
[x]: MUST Buildroot is not present
 Note: Unless packager wants to package for EPEL5 this is fine
[x]: MUST Package contains no bundled libraries.
[x]: MUST Changelog in prescribed format.
[x]: MUST Package has no %clean section with rm -rf %{buildroot} (or
 $RPM_BUILD_ROOT)
 Note: Clean would be needed if support for EPEL is required
[x]: MUST Sources contain only permissible code or content.
[-]: MUST Each %files section contains %defattr if rpm  4.4
 Note: defattr macros not found. They would be needed for EPEL5
[x]: MUST Macros in Summary, %description expandable at SRPM build time.
[x]: MUST Package requires other packages for directories it uses.
[x]: MUST Package uses nothing in %doc for runtime.
[x]: MUST Package is not known to require ExcludeArch.
[x]: MUST Permissions on files are set properly.
[x]: MUST Package does not contain duplicates in %files.
[x]: MUST Spec file lacks Packager, Vendor, PreReq tags.
[x]: MUST Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
 beginning of %install.
 Note: rm -rf would be needed if support for EPEL5 is required
[-]: 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 is included in %doc.
[x]: MUST License field in the package spec file matches the actual license.
[x]: MUST License file installed when any subpackage combination is installed.
[x]: MUST Package consistently uses macros (instead of hard-coded directory
 names).
[x]: MUST Package meets the Packaging Guidelines.
[x]: MUST Package is named according to the Package Naming Guidelines.
[x]: MUST Package does not generates any conflict.
[x]: MUST Package obeys FHS, except libexecdir and /usr/target.
[x]: MUST Package must own all directories that it creates.
[x]: MUST Package does not own files or directories owned by other packages.
[x]: MUST Package installs properly.
[!]: MUST Requires correct, justified where necessary.
See rpmlint output
[!]: MUST Rpmlint output is silent.

rpmlint derelict-Ogg-devel-2-17.2006svn593.fc17.i686.rpm

derelict-Ogg-devel.i686: W: spelling-error Summary(fr) developing -
développante
derelict-Ogg-devel.i686: W: spelling-error %description -l en_US derelictOgg -
derelict
derelict-Ogg-devel.i686: W: no-documentation
1 packages and 0 specfiles checked; 0 errors, 3 warnings.


rpmlint derelict-PA-devel-2-17.2006svn593.fc17.i686.rpm

derelict-PA-devel.i686: W: spelling-error Summary(fr) developing -
développante
derelict-PA-devel.i686: W: spelling-error %description -l en_US derelictPA -
derelict Pa, derelict PA, derelict-PA
derelict-PA-devel.i686: W: no-documentation
1 packages and 0 specfiles checked; 0 errors, 3 warnings.


rpmlint derelict-2-17.2006svn593.fc17.src.rpm

derelict.src: W: spelling-error Summary(fr) It - T, Itô, Dt
derelict.src: W: spelling-error Summary(fr) is - si, us, os
derelict.src: W: spelling-error Summary(fr) of - off, if, or
derelict.src: W: spelling-error Summary(fr) bindings - buildings
derelict.src: W: spelling-error Summary(fr) to - tau, go, t
derelict.src: W: spelling-error Summary(fr) shared - daredare, sed
derelict.src: W: spelling-error Summary(fr) libraries - libraires, librairies,
librairie
derelict.src: W: spelling-error %description -l en_US runtime - run time,
run-time, rudiment
derelict.src: W: invalid-url Source0: derelict-2006svn593.tar.xz
1 packages and 0 specfiles checked; 0 errors, 9 warnings.


rpmlint derelict-ODE-devel-2-17.2006svn593.fc17.i686.rpm

derelict-ODE-devel.i686: W: spelling-error Summary(fr) developing -
développante
derelict-ODE-devel.i686: W: spelling-error %description -l en_US derelictODE -
derelict Ode, derelict-ode, dereliction
derelict-ODE-devel.i686: W: spelling-error %description -l fr derelic -
Rellich

[Bug 771123] Review Request: fex - Field split/extraction like cut/awk

[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=771123

Mattia Verga mattia.ve...@tiscali.it changed:

   What|Removed |Added

   Flag||needinfo?(mmckinst@nexcess.
   ||net)

--- Comment #2 from Mattia Verga mattia.ve...@tiscali.it 2012-01-07 05:34:11 
EST ---
- rpmlint checks return:
SPECS/fex.spec: W: invalid-url Source0:
http://semicomplete.googlecode.com/files/fex-1.20100416.2814.tar.gz HTTP Error
404: Not Found
fex.x86_64: W: spelling-error Summary(en_US) awk - awl, aw, wk
fex.x86_64: W: spelling-error %description -l en_US tokenizeing - betokening
fex.x86_64: W: no-manual-page-for-binary fex
fex.src: W: spelling-error Summary(en_US) awk - awl, aw, wk
fex.src: W: spelling-error %description -l en_US tokenizeing - betokening
fex.src: W: invalid-url Source0:
http://semicomplete.googlecode.com/files/fex-1.20100416.2814.tar.gz HTTP Error
404: Not Found
3 packages and 1 specfiles checked; 0 errors, 7 warnings.

URL is valid (not sure why it says different) and all other warnings can be
ignored except the spelling error in %description.

- naming guidelines and .spec file name: OK
+ packaging guidelines: SEE NOTE 1
- license: OK (BSD), in %doc
- spec file legible, in am. english: OK
- source matches upstream
- package compiles on rawhide (i686/x86_64): OK
- Build Requires: OK
- no locales
- no libraries
- not relocatable
- no directories created
- no duplicate files
- file permissions: OK
+ macros: SEE NOTE 1
- code/content: OK
- no need for -docs
- nothing in %doc affects runtime
- no headers / no need for -devel
- no GUI and no .desktop
- ownership: OK
- filenames: OK

MUST FIX:
--
1: The BuildRoot tag is required only for EPEL. 
   If you think to build this package also for EPEL you must add the %clean
section.
   See http://fedoraproject.org/wiki/Packaging:Guidelines#BuildRoot_tag
2: Trivial: fix spelling in %description

-- 
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 766083] Fedora 17 feature Derelict

[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=766083

--- Comment #9 from MERCIER Jonathan bioinfornat...@gmail.com 2012-01-07 
05:53:22 EST ---
In first, thanks for your great job
I have do some minor change who fix:
- E: devel-dependency ldc-druntime-devel
- /usr/share/doc/derelict-2/styles.css

About derelict.i686: E: no-binary
This error given by rpmlint is not really an error because if i set the main
package as noarch all subpackage will be noarch and this is wrong, i.e:
http://lists.fedoraproject.org/pipermail/devel/2012-January/160806.html

About derelict-geany-tags.i686: E: script-without-shebang
This file has exactly the same right as other tags in /usr/share/geany/tags/

http://bioinfornatics.fedorapeople.org/derelict.spec
http://bioinfornatics.fedorapeople.org/derelict-2-18.2006svn593.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 674008] Review Request: openrave - Open Robotics Automation Virtual Environment

[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=674008

--- Comment #77 from Tim Niemueller t...@niemueller.de 2012-01-07 06:53:08 
EST ---
I'll build the new version today or tomorrow as time permits.

-- 
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 753597] Review Request: yazpp - C++ API for YAZ

[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=753597

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

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|martin.giesek...@uos.de
   Flag||fedora-review?

--- Comment #5 from Martin Gieseking martin.giesek...@uos.de 2012-01-07 
07:28:58 EST ---
The package looks almost fine. There are just a few minor things that need some
attention:

- Drop DESTDIR=$RPM_BUILD_ROOT from the make statement in %build. As far as I 
  see, it's not necessary to build the package properly.

- Drop %post devel -p /sbin/ldconfig and %postun devel -p /sbin/ldconfig.
  ldconfig doesn't need to be called for symlinks.

- You can remove rm -rf ${RPM_BUILD_ROOT} from the %install section. The 
  %clean section is redundant as well.

- I recommend not to package the .3 man pages generated by doxygen. Instead, 
  add the html variant. Since you create a -docs subpackage, move the doxygen 
  docs there:
+ drop the whole %doc line from %files devel
+ add %doc dox/html/ to %files doc

- add the following lines to the %install section:
mkdir tmp
mv ${RPM_BUILD_ROOT}/%{_docdir}/yazpp/ tmp
  and the line in %files docs should look like this: 
%doc tmp/* dox/html/
  This way you ensure that all the doc files go to the same (and correct) 
  directory.


$ rpmlint yazpp-*
yazpp.src:48: W: rpm-buildroot-usage %build make %{?_smp_mflags}
DESTDIR=${RPM_BUILD_ROOT} all dox
yazpp-utils.x86_64: W: no-manual-page-for-binary zclient
yazpp-utils.x86_64: W: no-manual-page-for-binary zlint
6 packages and 0 specfiles checked; 0 errors, 3 warnings.


-
key:

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

[+] 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.
[+] MUST: The License field in the package spec file must match the actual
license.
[+] MUST: The 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.
$ md5sum yazpp-1.2.7.tar.gz*
587f778f34b9b16de47ec26a2a3d1927  yazpp-1.2.7.tar.gz
587f778f34b9b16de47ec26a2a3d1927  yazpp-1.2.7.tar.gz.upstream

[+] MUST: The package MUST successfully compile and build into binary rpms on
at least one primary architecture.
[.] MUST: If the package does not successfully compile, build or work on an
architecture, ...
[+] MUST: All build dependencies must be listed in BuildRequires.
[+] MUST: When compiling C, C++, or Fortran files, %{optflags} must be applied.
[.] MUST: The spec file MUST handle locales properly.
[X] MUST: Packages storing shared library files (not just symlinks) must call
ldconfig in %post and %postun.
- drop the calls of ldconfig for the devel package

[+] MUST: Packages must NOT bundle copies of system libraries.
[.] MUST: If the package is designed to be relocatable, ...
[X] MUST: A package must own all directories that it creates. 
- %{docdir}/yazpp/ is unowned

[X] MUST: A Fedora package must not list a file more than once in %files.
- doc/yazpp-config.1* is listed twice

[+] MUST: Permissions on files must be set properly.
[+] MUST: Each package must consistently use macros.
[+] MUST: The package must contain code, or permissable content.
[+] MUST: Large documentation files must go in a -doc subpackage.
[+] MUST: Files in %doc 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, then library files
that end in .so (without suffix) must go in a -devel package.
[+] MUST: devel packages must require the base package using a fully versioned
dependency.
[+] MUST: Packages must NOT contain any .la libtool archives.
[.] 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. If you feel that your packaged GUI application does not need
a .desktop file, you must put a comment in the spec file with your explanation.
[+] MUST: Packages must not own files or directories already owned by other
packages.
[+] MUST: All filenames in rpm packages must be valid UTF-8.

EPEL = 5 only:
[X] MUST: The spec file 

[Bug 458975] Review Request: Pivy - Coin binding for Python

[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=458975

--- Comment #18 from Richard Shaw hobbes1...@gmail.com 2012-01-07 07:36:26 
EST ---
FreeCAD was the reason I found this bug :)

I've already got local builds of freecad working (I contributed quite a bit to
their cmake build) but there's still a few dependencies I need.

I've already gotten OpenCASCADE (the community edition version) in RPM Fusion
which has to go in non-free because of their quirky license.

I have a review request for SMESH which freecad bundles:
https://bugzilla.rpmfusion.org/show_bug.cgi?id=2112

The last main dependency for Pivy is Coin3:
https://bugzilla.redhat.com/show_bug.cgi?id=665733

After those are taken care of I think I can submit the review request for
FreeCAD.

-- 
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

[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

--- Comment #34 from Rex Dieter rdie...@math.unl.edu 2012-01-07 07:51:48 EST 
---
maybe, let's get review blockers handled first, then we can work through the
runtime details between jreen/tomahawk upstreams later.

-- 
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 713461] Review Request: tomahawk - The Social Music Player

[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=713461

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

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #21 from Rex Dieter rdie...@math.unl.edu 2012-01-07 07:50:53 EST 
---
thanks!

New Package SCM Request
===
Package Name: tomahawk
Short Description: The Social Music Player 
Owners: rdieter
Branches: 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 752454] Review Request: qca-cyrus-sasl - Cyrus-SASL plugin for the Qt Cryptographic Architecture v2

[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=752454

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

   What|Removed |Added

 Blocks||731456(jreen)

-- 
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

[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

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

   What|Removed |Added

 Depends on||752454(qca-cyrus-sasl)

--- Comment #35 from Rex Dieter rdie...@math.unl.edu 2012-01-07 08:05:25 EST 
---
In the meantime, I've tentatively marked the qca-cyrus-sasl review as a
dependency here, in case it truly is a viable alternative to using simplesasl

I'm still curious how libnice comes into play, I cannot find any reference to
it in the sources.

-- 
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

[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 #12 from Matthieu Saulnier casper.le.fan...@gmail.com 2012-01-07 
08:19:42 EST ---
Hello,
this is the latest release:
Spec URL: http://fantom.fedorapeople.org/bvi.spec
SRPM URL: http://fantom.fedorapeople.org/bvi-1.3.2-6.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 731456] Review Request: jreen - Qt XMPP Library

[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

--- Comment #36 from Rex Dieter rdie...@math.unl.edu 2012-01-07 08:27:16 EST 
---
I can confirm that logging into a google account works in tomahawk after
instalilng qca-cyrus-sasl ... I think I'll add the dep here, once it's review
is done and it's ready and available.

-- 
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

[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

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

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #13 from Mohamed El Morabity pikachu.2...@gmail.com 2012-01-07 
08:32:13 EST ---
APPROVED.

-- 
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 728302] Review Request: pjproject - Libraries written in C language for building embedded/non-embedded VoIP applications

[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=728302

--- Comment #15 from Rex Dieter rdie...@math.unl.edu 2012-01-07 09:48:03 EST 
---
some more now,

6.  Licensing.   using licensecheck too, seems all files include the gpl 'or
later' clause, so we can go with
License: GPLv2+

7. use the included configure script (esp to control build options, see 8). 
unless there's some good reasons not to, and if so, please document them in a
.spec comment

8.  bundled libraries in third_party, to better ensure they're not used, add
build options:
--with-external-speex
--with-external-gsm
--with-external-pa
will require additional BuildRequires for each of these, like
BuildRequires: speex-devel
BuildRequires: gsm-devel
BuildRequires: portaudio-devel
respectively

(there's other stuff in there that requires more scrutiny, but this is a good
first effort)

9. document in %build section comment that %{?_smp_mflags} cannot be used

10. drop/omit empty main pkg, ie, remove 
%files
and only do
%files devel
...

-- 
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 766154] Review Request: lv2-kn0ck0ut - An LV2 spectral subtraction plugin (linux audio)

[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=766154

--- Comment #6 from Brendan Jones brendan.jones...@gmail.com 2012-01-07 
10:23:50 EST ---
Hi Peter,

this is from upstream:

Hi Brendan,

As the readme says,

kn0ck0ut-LV2 is released under the terms of the GPL version 3. 
(this is done so with the permission of st3pan0va).

And then below that

Below Reads the Original Readme

I left the original readme at the bottom of the readme for historical purposes
(and to preserve acknowledgments).  The original author gave me explicit
permission to release under the GPL.  I've pushed changes to the repository
which further clarify things under the tag 1.01

Jeremy

-- 
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 769029] Review Request: oxygen-gtk3 - Oxygen GTK+3 theme

[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=769029

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

   What|Removed |Added

 CC||rdie...@math.unl.edu
 AssignedTo|nob...@fedoraproject.org|rdie...@math.unl.edu
   Flag||fedora-review?

--- Comment #6 from Rex Dieter rdie...@math.unl.edu 2012-01-07 10:29:55 EST 
---
I guess I can formally review this.

-- 
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 769029] Review Request: oxygen-gtk3 - Oxygen GTK+3 theme

[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=769029

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

   What|Removed |Added

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

--- Comment #7 from Rex Dieter rdie...@math.unl.edu 2012-01-07 10:36:50 EST 
---
naming: ok

license: ok

sources: ok (snapshot for now)

macros: ok

scriptlets: n/a

otherwise, fairly simple and close to existing oxygen-gtk2 packaging.  Looks
good.


APPROVED.

-- 
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

[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

Matthieu Saulnier casper.le.fan...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #14 from Matthieu Saulnier casper.le.fan...@gmail.com 2012-01-07 
10:50:25 EST ---
New Package SCM Request
===
Package Name: bvi
Short Description: Display-oriented editor for binary files
Owners: fantom
Branches: f15 f16 el6
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 688886] Review Request: kflickr - Standalone Flickr Uploader

[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=66

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

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE
Last Closed||2012-01-07 10:54:19

-- 
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 619437] Review Request: bm - Rpm package building helper

[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=619437

--- Comment #13 from Rex Dieter rdie...@math.unl.edu 2012-01-07 10:56:51 EST 
---
OH boy, I suck, forgot to follow up with anything useful here.  will try to do
so this weekend.

-- 
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 688886] Review Request: kflickr - Standalone Flickr Uploader

[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=66

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

   What|Removed |Added

 Status|CLOSED  |ASSIGNED
 Resolution|RAWHIDE |
   Keywords||Reopened

--- Comment #16 from Rex Dieter rdie...@math.unl.edu 2012-01-07 10:55:43 EST 
---
Err... ping Jan, I don't see this ever got imported.  Was there a problem or
something I can help with?

-- 
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 688886] Review Request: kflickr - Standalone Flickr Uploader

[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=66

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

   What|Removed |Added

   Flag||needinfo?(jan.klepek@gmail.
   ||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 766154] Review Request: lv2-kn0ck0ut - An LV2 spectral subtraction plugin (linux audio)

[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=766154

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

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #7 from Peter Lemenkov lemen...@gmail.com 2012-01-07 11:03:13 EST 
---
Ah, ok. And here is a commit with new licensing terms:

https://github.com/jeremysalwen/kn0ck0ut-LV2/commit/60421a3

Ok, here is my formal 

REVIEW:

Legend: + = PASSED, - = FAILED, 0 = Not Applicable

+ rpmlint is silent

work ~/temp: rpmlint *
lv2-kn0ck0ut.src: I: enchant-dictionary-not-found en_US
lv2-kn0ck0ut.src: W: invalid-url Source0: lv2-kn0ck0ut-1.1-gitd03e8db0.tar.bz2

^^^ that's expected from GitHub's repos.

3 packages and 0 specfiles checked; 0 errors, 1 warnings.
work ~/temp: 

+ The package is named according to the  Package Naming Guidelines.
+ The spec file name matches the base package %{name}, in the format
%{name}.spec.
+ The package meets the Packaging Guidelines.
+ The package is licensed with a Fedora approved license and meets the
Licensing Guidelines.
+ The License field in the package spec file matches the actual license (GPLv3
or later, as stated in the sources).
+ The file, containing the text of the license(s) for the package, is included
in %doc.
+ The spec file is written in American English.
+ The spec file for the package is legible.
+ The sources used to build the package, match the upstream source, as provided
in the spec URL. Unfortunately it's almost impossible to create bit-exact
tarball from SCM (different usernames, different timestamps) so I used diff -ru
to compare.
+ The package successfully compiles and builds into binary rpms on at least one
primary architecture. See koji link above.
+ All build dependencies are listed in BuildRequires.
0 No need to handle locales.
0 No shared library files in some of the dynamic linker's default paths.
+ The package does NOT bundle copies of system libraries.
0 The package is not designed to be relocatable.
+ The package owns all directories that it creates.
+ The package does not list a file more than once in the spec file's %files
listings.
+ Permissions on files are set properly.
0 The package DOESN'T have a %clean section, so it won't build cleanly on
systems with old rpm (EL-4 and EL-5, not sure about EL-6). Beware.
+ The package consistently uses macros.
+ The package contains code, or permissible content.
0 No extremely large documentation files.
+ Anything, the package includes as %doc, does not affect the runtime of the
application.
0 No header files.
0 No static libraries.
0 No pkgconfig(.pc) files.
0 The package doesn't contain library files without a suffix (e.g. libfoo.so).
0 No devel sub-package.
+ The package does NOT contain any .la libtool archives.
0 Not a GUI application.
+ The package does not own files or directories already owned by other
packages.
+ At the beginning of %install, the package runs rm -rf %{buildroot} (or
$RPM_BUILD_ROOT).
+ All filenames in rpm packages are valid UTF-8.


APPROVED.

-- 
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

[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 #9 from Willington Vega wv...@wvega.com 2012-01-07 11:14:17 EST 
---
Hello Mohamed,

Thank you for your feedback. I updated the spec file following your
recommendations. Here are the links to the SPEC and the SRPM files

http://files.wvega.com/fedora/inkscape-textext/inkscape-textext.spec
http://files.wvega.com/fedora/inkscape-textext/inkscape-textext-0.4.4-4.20091127hg9260a2461344.fc15.src.rpm

previous spec file can be found at:

http://files.wvega.com/fedora/inkscape-textext/inkscape-textext-0.4.4-3.spec


rpmlint output:

$ rpmlint SPECS/inkscape-textext.spec 
0 packages and 1 specfiles checked; 0 errors, 0 warnings.

$ rpmlint
RPMS/noarch/inkscape-textext-0.4.4-4.20091127hg9260a2461344.fc15.noarch.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

$ rpmlint SRPMS/inkscape-textext-0.4.4-4.20091127hg9260a2461344.fc15.src.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.


and Koji scratch build:

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

-- 
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 713461] Review Request: tomahawk - The Social Music Player

[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=713461

--- Comment #22 from Jon Ciesla limburg...@gmail.com 2012-01-07 11:17:11 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 765652] Review Request: bvi - Display-oriented editor for binary files

[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 #15 from Jon Ciesla limburg...@gmail.com 2012-01-07 11:18: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 688886] Review Request: kflickr - Standalone Flickr Uploader

[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=66

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

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE
   Flag|needinfo?(jan.klepek@gmail. |
   |com)|
Last Closed|2012-01-07 10:54:19 |2012-01-07 11:28:08

--- Comment #17 from Rex Dieter rdie...@math.unl.edu 2012-01-07 11:28:08 EST 
---
nevermind, it was imported and built, but only for rawhide/f17.  so, we can
close this.

I'd love a f16 build though... :)

-- 
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 752454] Review Request: qca-cyrus-sasl - Cyrus-SASL plugin for the Qt Cryptographic Architecture v2

[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=752454

Magnus Tuominen magnus.tuomi...@gmail.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||magnus.tuomi...@gmail.com
 AssignedTo|nob...@fedoraproject.org|magnus.tuomi...@gmail.com
   Flag||fedora-review?

--- Comment #1 from Magnus Tuominen magnus.tuomi...@gmail.com 2012-01-07 
11:29:28 EST ---
Review coming soon.

-- 
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 514105] Review Request: courier-imap - The Courier IMAP server

[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=514105

--- Comment #20 from Rex Dieter rdie...@math.unl.edu 2012-01-07 11:42:32 EST 
---
ping reporter, been another couple of months... (sorry, for my own delays)

-- 
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

[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

--- Comment #37 from Ruslan Nigmatullin euroeles...@yandex.ru 2012-01-07 
11:41:19 EST ---
If it's still does matter official Jreen's page is http://qutim.org/jreen,
sources can be downloaded from http://qutim.org/dwnl/18/libjreen-1.0.1.tar.bz2
.
Unfortunataly links at github don't work, I don't know why.

-- 
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

[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

--- Comment #38 from Rex Dieter rdie...@math.unl.edu 2012-01-07 11:57:09 EST 
---
OK, I'll use that URL.  thanks!

http://rdieter.fedorapeople.org/rpms/tomahawk/jreen.spec
http://rdieter.fedorapeople.org/rpms/tomahawk/jreen-1.0.1-3.fc16.src.rpm

%changelog
* Sat Jan 07 2012 Rex Dieter rdie...@fedoraproject.org 1.0.1-3
- update URL, Source0

-- 
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 713461] Review Request: tomahawk - The Social Music Player

[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=713461

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

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE
Last Closed||2012-01-07 12:05:41

--- Comment #23 from Rex Dieter rdie...@math.unl.edu 2012-01-07 12:05:41 EST 
---
imported, thanks all!

(waiting for jreen before doing any builds)

-- 
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 752454] Review Request: qca-cyrus-sasl - Cyrus-SASL plugin for the Qt Cryptographic Architecture v2

[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=752454

Magnus Tuominen magnus.tuomi...@gmail.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #2 from Magnus Tuominen magnus.tuomi...@gmail.com 2012-01-07 
12:21:48 EST ---
review, looking good:

MUST: rpmlint must be run on the source rpm and all binary rpms the build
produces. The output should be posted in the review.
rpmlint -iv qca-cyrus-sasl.spec 
qca-cyrus-sasl.spec:34: W: configure-without-libdir-spec
A configure script is run without specifying the libdir. configure options
must be augmented with something like --libdir=%{_libdir} whenever the script
supports it.

qca-cyrus-sasl.spec: I: checking-url
http://delta.affinix.com/download/qca/2.0/plugins/qca-cyrus-sasl-2.0.0-beta3.tar.bz2
(timeout 10 seconds)
0 packages and 1 specfiles checked; 0 errors, 1 warnings.

rpmlint ../RPMS/x86_64/qca-cyrus-sasl-*.rpm
qca-cyrus-sasl.x86_64: E: incorrect-fsf-address
/usr/share/doc/qca-cyrus-sasl-2.0.0/COPYING
2 packages and 0 specfiles checked; 1 errors, 0 warnings.

rpmlint -iv ../SRPMS/qca-cyrus-sasl-2.0.0-0.2.beta3.fc16.src.rpm 
qca-cyrus-sasl.src: I: checking
qca-cyrus-sasl.src: I: checking-url http://delta.affinix.com/qca/ (timeout 10
seconds)
qca-cyrus-sasl.src:34: W: configure-without-libdir-spec
A configure script is run without specifying the libdir. configure options
must be augmented with something like --libdir=%{_libdir} whenever the script
supports it.

qca-cyrus-sasl.src: I: checking-url
http://delta.affinix.com/download/qca/2.0/plugins/qca-cyrus-sasl-2.0.0-beta3.tar.bz2
(timeout 10 seconds)
1 packages and 0 specfiles checked; 0 errors, 1 warnings.

OK
MUST: The package must be named according to the Package Naming Guidelines.
OK
MUST: The spec file name must match the base package %{name}, in the format
%{name}.spec unless your package has an exemption.
OK
MUST: The package must meet the Packaging Guidelines.
OK
MUST: The package must be licensed with a Fedora approved license and meet the
Licensing Guidelines .
OK
MUST: The License field in the package spec file must match the actual license.
OK
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.
OK
MUST: The spec file must be written in American English.
OK
MUST: The spec file for the package MUST be legible.
OK
MUST: The sources used to build the package must match the upstream source, as
provided in the spec URL. Reviewers should use md5sum for this task. If no
upstream URL can be specified for this package, please see the Source URL
Guidelines for how to deal with this.
src.rpm: db51df751478f60416659809e11990fd
upstream: db51df751478f60416659809e11990fd
MUST: The package MUST successfully compile and build into binary rpms on at
least one primary architecture.
OK
MUST: If the package does not successfully compile, build or work on an
architecture, then those architectures should be listed in the spec in
ExcludeArch. Each architecture listed in ExcludeArch MUST have a bug filed in
bugzilla, describing the reason that the package does not compile/build/work on
that architecture. The bug number MUST be placed in a comment, next to the
corresponding ExcludeArch line.
OK
MUST: All build dependencies must be listed in BuildRequires, except for any
that are listed in the exceptions section of the Packaging Guidelines ;
inclusion of those as BuildRequires is optional. Apply common sense.
OK
MUST: The spec file MUST handle locales properly. This is done by using the
%find_lang macro. Using %{_datadir}/locale/* is strictly forbidden.
OK
MUST: Every binary RPM package (or subpackage) which stores shared library
files (not just symlinks) in any of the dynamic linker's default paths, must
call ldconfig in %post and %postun.
OK
MUST: Packages must NOT bundle copies of system libraries.
OK
MUST: If the package is designed to be relocatable, the packager must state
this fact in the request for review, along with the rationalization for
relocation of that specific package. Without this, use of Prefix: /usr is
considered a blocker.
OK
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.
OK
MUST: A Fedora package must not list a file more than once in the spec file's
%files listings. (Notable exception: license texts in specific situations)
OK
MUST: Permissions on files must be set properly. Executables should be set with
executable permissions, for example. Every %files section must include a
%defattr(...) line.
OK
MUST: Each package must consistently use macros.
OK
MUST: 

[Bug 769029] Review Request: oxygen-gtk3 - Oxygen GTK+3 theme

[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=769029

nucleo alekc...@googlemail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #8 from nucleo alekc...@googlemail.com 2012-01-07 12:47:33 EST ---
Thanks.
There was no any commits to gtk3 branch, so no need to do new snapshot.
I will fix macro-in-comment when importing this package in git.

New Package SCM Request
===
Package Name: oxygen-gtk3
Short Description: Oxygen GTK+3 theme
Owners: rdieter nucleo
Branches: f15 f16

-- 
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

[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

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

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

-- 
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

[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 #16 from Fedora Update System upda...@fedoraproject.org 
2012-01-07 13:29:50 EST ---
bvi-1.3.2-6.fc16 has been submitted as an update for Fedora 16.
https://admin.fedoraproject.org/updates/bvi-1.3.2-6.fc16

-- 
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

[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

--- Comment #39 from Ruslan Nigmatullin euroeles...@yandex.ru 2012-01-07 
13:27:05 EST ---
Created attachment 551376
  -- https://bugzilla.redhat.com/attachment.cgi?id=551376
Fixes compilation error with gcc-4.7

 4) Build of package failed on rawhide:
 http://koji.fedoraproject.org/koji/taskinfo?taskID=3625703

This patch fixes compilation error with gcc-4.7

-- 
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

[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 #17 from Fedora Update System upda...@fedoraproject.org 
2012-01-07 13:31:37 EST ---
bvi-1.3.2-6.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/bvi-1.3.2-6.fc15

-- 
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

[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 #18 from Fedora Update System upda...@fedoraproject.org 
2012-01-07 13:37:13 EST ---
bvi-1.3.2-6.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/bvi-1.3.2-6.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 731456] Review Request: jreen - Qt XMPP Library

[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

Pavel Alexeev (aka Pahan-Hubbitus) pa...@hubbitus.info changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #40 from Pavel Alexeev (aka Pahan-Hubbitus) pa...@hubbitus.info 
2012-01-07 13:43:42 EST ---
$ rpmlint *
jreen.src:13: W: macro-in-comment %{version}
jreen.spec:13: W: macro-in-comment %{version}
1 packages and 1 specfiles checked; 0 errors, 2 warnings.

It is trivial to fix, please do it.

Libnice indeed does not required in current version, only in further, so its
ok.

So, there left only one stop issue - it does not build on gcc 4.7. I speak with
Ruslan and they provide patch, please try it. In any case, Rex, you can't
import it until it will be resolved.
So, package APPROVED.

-- 
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 761319] Fedora 17 feature GTKD

[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=761319

--- Comment #4 from MERCIER Jonathan bioinfornat...@gmail.com 2012-01-07 
13:46:28 EST ---
http://bioinfornatics.fedorapeople.org/gtkd-1.5.1-7.20111024svn906.fc16.src.rpm
http://bioinfornatics.fedorapeople.org/gtkd.spec

 remove %defattr and Group section
 generate devhelp book in  %build section
 Add tag for geany

-- 
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 752454] Review Request: qca-cyrus-sasl - Cyrus-SASL plugin for the Qt Cryptographic Architecture

[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=752454

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

   What|Removed |Added

Summary|Review Request: |Review Request:
   |qca-cyrus-sasl - Cyrus-SASL |qca-cyrus-sasl - Cyrus-SASL
   |plugin for the Qt   |plugin for the Qt
   |Cryptographic Architecture  |Cryptographic Architecture
   |v2  |
   Flag||fedora-cvs?

--- Comment #3 from Rex Dieter rdie...@math.unl.edu 2012-01-07 13:52:39 EST 
---
Thanks!

New Package SCM Request
===
Package Name: qca-cyrus-sasl
Short Description: Curus-SASL plugin for the Qt Cryptographic Architecture
Owners: rdieter
Branches: f15 f16

-- 
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 737286] Review Request: salt - A parallel remote execution system

[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=737286

--- Comment #9 from Clint Savage her...@gmail.com 2012-01-07 13:54:24 EST ---
Okay, systemd support for Fedora15+ has been added to the rpm now. Thanks to
upstream.

SPEC: http://herlo.fedorapeople.org/rpms/salt.spec
SRPM: http://herlo.fedorapeople.org/rpms/salt-0.9.4-3.fc16.src.rpm

Tom, if you could review it ASAP, that'd rock!

Clint

-- 
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

[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

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

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #41 from Rex Dieter rdie...@math.unl.edu 2012-01-07 14:08:43 EST 
---
Thanks!  (for review, and gcc47 patch)

New Package SCM Request
===
Package Name: jreen
Short Description: Qt XMPP Library
Owners: rdieter
Branches: 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 728302] Review Request: pjproject - Libraries written in C language for building embedded/non-embedded VoIP applications

[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=728302

--- Comment #16 from Mario Santagiuliana fed...@marionline.it 2012-01-07 
14:15:51 EST ---
5) should be ok now...I update the patch;

6) done;

7) 8) done;

9) if I add %{?_smp_mflags} the package is not build...Done;

10) done.

SPEC: http://marionline.fedorapeople.org/packages/SPECS/pjproject.spec
SRPM:
http://marionline.fedorapeople.org/packages/SRPMS/pjproject-1.10-6.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 772406] Review Request: cpulimit - CPU Usage Limiter for Linux

[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=772406

Christos Triantafyllidis christos.triantafylli...@gmail.com changed:

   What|Removed |Added

 Blocks||177841(FE-NEEDSPONSOR)

-- 
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 772406] New: Review Request: cpulimit - CPU Usage Limiter for Linux

[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: cpulimit - CPU Usage Limiter for Linux

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

   Summary: Review Request: cpulimit - CPU Usage Limiter for Linux
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: christos.triantafylli...@gmail.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: https://raw.github.com/ctria/cpulimit-spec/master/cpulimit.spec
EL5 SRPM URL:
https://github.com/ctria/cpulimit-spec/raw/master/cpulimit-1.1-1.el5.src.rpm
EL6 SRPM URL:
https://github.com/ctria/cpulimit-spec/raw/master/cpulimit-1.1-1.el6.src.rpm
Fedora 16 SRPM URL:
https://github.com/ctria/cpulimit-spec/raw/master/cpulimit-1.1-1.fc16.src.rpm
Fedora 17 SRPM URL:
https://github.com/ctria/cpulimit-spec/raw/master/cpulimit-1.1-1.fc17.src.rpm

Description: cpulimit is a simple program which attempts to limit the CPU usage
of a process
(expressed in percentage, not in CPU time). This is useful to control batch
jobs, when you don't want them to eat too much CPU. It does not act on the nice
value or other scheduling priority stuff, but on the real CPU usage. Also, it
is able to adapt itself to the overall system load, dynamically and quickly.

Scratch builds at Fedora's koji:
Fedora 17 (rawhide):
http://koji.fedoraproject.org/koji/taskinfo?taskID=3627881
Fedora 16:
http://koji.fedoraproject.org/koji/taskinfo?taskID=3627890
EPEL 6:
http://koji.fedoraproject.org/koji/taskinfo?taskID=3627893
EPEL 5:
http://koji.fedoraproject.org/koji/taskinfo?taskID=3627924

-- 
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 772406] Review Request: cpulimit - CPU Usage Limiter for Linux

[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=772406

--- Comment #1 from Christos Triantafyllidis 
christos.triantafylli...@gmail.com 2012-01-07 14:37:45 EST ---
I see that i'm not the first who tries to package cpulimit for Fedora/EPEL,
this looks like the 3rd attempt. I hope that the package is not cursed :).

$ rpmlint cpulimit-1.1-1.el5.src.rpm cpulimit-1.1-1.el6.src.rpm
cpulimit-1.1-1.fc16.src.rpm cpulimit-1.1-1.fc17.src.rpm cpulimit.spec
4 packages and 1 specfiles checked; 0 errors, 0 warnings.

Christos

-- 
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 728302] Review Request: pjproject - Libraries written in C language for building embedded/non-embedded VoIP applications

[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=728302

--- Comment #17 from Rex Dieter rdie...@math.unl.edu 2012-01-07 15:07:17 EST 
---
Excellent!

I'm reviewing the remaining third_party bits.


in the meantime, one piece of advice, I see stuff like
%files devel
%dir %{_includedir}/pj++/
%{_includedir}/pj++/*.hpp

you can simplify these 2 down to 1 line:
%{_includedir}/pj++/

(listing a directory will own it, and recursively everything under it)
which, for me at least, is simpler and easier to read.

-- 
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

[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

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

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #19 from Fedora Update System upda...@fedoraproject.org 
2012-01-07 15:37:43 EST ---
bvi-1.3.2-6.el6 has been pushed to the Fedora EPEL 6 testing 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 728302] Review Request: pjproject - Libraries written in C language for building embedded/non-embedded VoIP applications

[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=728302

--- Comment #18 from Rex Dieter rdie...@math.unl.edu 2012-01-07 15:42:13 EST 
---
OK, found 2 bundled libraries that require clarification
third_party/resample  = libresample
third_party/srtp  = libsrtp

both of these are included in fedora already, and it's not clear
1.  why it's bundled, and distributing it's own copies of these libraries
2.  why external/system copies cannot be used.

See also,
https://fedoraproject.org/wiki/Packaging:Treatment_Of_Bundled_Libraries

if we are to justify an attempt to use the bundled libraries in this fedora
package, could you contact pjproject upstream to provide the answers to these:
https://fedoraproject.org/wiki/Packaging:No_Bundled_Libraries#Standard_questions

-- 
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 728302] Review Request: pjproject - Libraries written in C language for building embedded/non-embedded VoIP applications

[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=728302

--- Comment #19 from nucleo alekc...@googlemail.com 2012-01-07 15:58:52 EST 
---
There is also bundled iLBC codec.
See about iLBC
http://lists.fedoraproject.org/pipermail/legal/2011-November/001755.html
and http://lists.fedoraproject.org/pipermail/legal/2011-November/001756.html

What about building shared pjproject library?

-- 
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 772413] New: Review Request: test

[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: test

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

   Summary: Review Request: test
   Product: Fedora
   Version: rawhide
  Platform: Unspecified
OS/Version: Unspecified
Status: NEW
  Severity: unspecified
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: pin...@pingoured.fr
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


Test from pingou

-- 
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 772412] New: Review Request: test

[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: test

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

   Summary: Review Request: test
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: pin...@pingoured.fr
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


Test from pingou

-- 
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 772412] Review Request: test

[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=772412

Pierre-YvesChibon pin...@pingoured.fr changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||NOTABUG
Last Closed||2012-01-07 16:18:52

-- 
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 772413] Review Request: test

[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=772413

Pierre-YvesChibon pin...@pingoured.fr changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||NOTABUG
Last Closed||2012-01-07 16:31: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 728302] Review Request: pjproject - Libraries written in C language for building embedded/non-embedded VoIP applications

[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=728302

--- Comment #20 from Rex Dieter rdie...@math.unl.edu 2012-01-07 16:38:13 EST 
---
OK, ilbc will need to be stripped out (somehow), until the legal issues are
sorted out.

shared library... well, that's upstream's call, whether they are willing to
support an ABI or not.

-- 
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 772415] New: Review Request: trac-mastertickets-plugin - Add support for ticket dependencies to Trac

[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: trac-mastertickets-plugin - Add support for ticket 
dependencies to Trac

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

   Summary: Review Request: trac-mastertickets-plugin - Add
support for ticket dependencies to Trac
   Product: Fedora
   Version: rawhide
  Platform: Unspecified
OS/Version: Unspecified
Status: NEW
  Severity: unspecified
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: pin...@pingoured.fr
 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://pingou.fedoraproject.org///home/pierrey/rpmbuild/SPECS/trac-mastertickets-plugin.spec
SRPM URL:
http://pingou.fedoraproject.org///home/pierrey/rpmbuild/SRPMS/trac-mastertickets-plugin-3.0.2-2.20111215.git43a7537.el6.src.rpm

Description:
This plugin adds blocks and blocked by fields to each ticket, enabling
you to express dependencies between tickets. It also provides a
graphviz-based dependency-graph feature for those tickets having
dependencies specified, allowing you to visually understand the
dependency tree. The dependency graph is viewable by clicking 'depgraph' in
the context (in the upper right corner) menu when viewing a ticket that
blocks or is blocked by another ticket.

-- 
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 772415] Review Request: trac-mastertickets-plugin - Add support for ticket dependencies to Trac

[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=772415

Pierre-YvesChibon pin...@pingoured.fr changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||NOTABUG
Last Closed||2012-01-07 17:08:27

-- 
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 772417] Review Request: trac-mastertickets-plugin - Add support for ticket dependencies to Trac

[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=772417

--- Comment #1 from Pierre-YvesChibon pin...@pingoured.fr 2012-01-07 17:15:03 
EST ---
This package built on koji: 
http://koji.fedoraproject.org/koji/taskinfo?taskID=3628135

-- 
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 772416] Review Request: trac-mastertickets-plugin - Add support for ticket dependencies to Trac

[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=772416

Pierre-YvesChibon pin...@pingoured.fr changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||NOTABUG
Last Closed||2012-01-07 17:12:34

-- 
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 772416] New: Review Request: trac-mastertickets-plugin - Add support for ticket dependencies to Trac

[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: trac-mastertickets-plugin - Add support for ticket 
dependencies to Trac

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

   Summary: Review Request: trac-mastertickets-plugin - Add
support for ticket dependencies to Trac
   Product: Fedora
   Version: rawhide
  Platform: Unspecified
OS/Version: Unspecified
Status: NEW
  Severity: unspecified
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: pin...@pingoured.fr
 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://pingou.fedoraproject.org//home
SRPM URL: http://pingou.fedoraproject.org//home

Description:
This plugin adds blocks and blocked by fields to each ticket, enabling
you to express dependencies between tickets. It also provides a
graphviz-based dependency-graph feature for those tickets having
dependencies specified, allowing you to visually understand the
dependency tree. The dependency graph is viewable by clicking 'depgraph' in
the context (in the upper right corner) menu when viewing a ticket that
blocks or is blocked by another ticket.

-- 
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 772417] New: Review Request: trac-mastertickets-plugin - Add support for ticket dependencies to Trac

[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: trac-mastertickets-plugin - Add support for ticket 
dependencies to Trac

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

   Summary: Review Request: trac-mastertickets-plugin - Add
support for ticket dependencies to Trac
   Product: Fedora
   Version: rawhide
  Platform: Unspecified
OS/Version: Unspecified
Status: NEW
  Severity: unspecified
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: pin...@pingoured.fr
 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://pingou.fedoraproject.org//trac-mastertickets-plugin.spec
SRPM URL:
http://pingou.fedoraproject.org//trac-mastertickets-plugin-3.0.2-2.20111215.git43a7537.el6.src.rpm

Description:
This plugin adds blocks and blocked by fields to each ticket, enabling
you to express dependencies between tickets. It also provides a
graphviz-based dependency-graph feature for those tickets having
dependencies specified, allowing you to visually understand the
dependency tree. The dependency graph is viewable by clicking 'depgraph' in
the context (in the upper right corner) menu when viewing a ticket that
blocks or is blocked by another ticket.

-- 
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 772419] Review Request: trac-mastertickets-plugin - Add support for ticket dependencies to Trac

[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=772419

--- Comment #1 from Pierre-YvesChibon pin...@pingoured.fr 2012-01-07 17:19:15 
EST ---
This package built on koji: 
http://koji.fedoraproject.org/koji/taskinfo?taskID=3628139

-- 
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 772419] New: Review Request: trac-mastertickets-plugin - Add support for ticket dependencies to Trac

[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: trac-mastertickets-plugin - Add support for ticket 
dependencies to Trac

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

   Summary: Review Request: trac-mastertickets-plugin - Add
support for ticket dependencies to Trac
   Product: Fedora
   Version: rawhide
  Platform: Unspecified
OS/Version: Unspecified
Status: NEW
  Severity: unspecified
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: pin...@pingoured.fr
 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://pingou.fedorapeople.org//trac-mastertickets-plugin.spec
SRPM URL:
http://pingou.fedorapeople.org//trac-mastertickets-plugin-3.0.2-2.20111215.git43a7537.el6.src.rpm

Description:
This plugin adds blocks and blocked by fields to each ticket, enabling
you to express dependencies between tickets. It also provides a
graphviz-based dependency-graph feature for those tickets having
dependencies specified, allowing you to visually understand the
dependency tree. The dependency graph is viewable by clicking 'depgraph' in
the context (in the upper right corner) menu when viewing a ticket that
blocks or is blocked by another ticket.

-- 
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 772417] Review Request: trac-mastertickets-plugin - Add support for ticket dependencies to Trac

[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=772417

Pierre-YvesChibon pin...@pingoured.fr changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||NOTABUG
Last Closed||2012-01-07 17:16: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

[Bug 772420] Review Request: trac-mastertickets-plugin - Add support for ticket dependencies to Trac

[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=772420

--- Comment #1 from Pierre-YvesChibon pin...@pingoured.fr 2012-01-07 17:22:42 
EST ---
This package built on koji: 
http://koji.fedoraproject.org/koji/taskinfo?taskID=3628141

-- 
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 772420] Review Request: trac-mastertickets-plugin - Add support for ticket dependencies to Trac

[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=772420

Pierre-YvesChibon pin...@pingoured.fr changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||NOTABUG
Last Closed||2012-01-07 17:23:22

-- 
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 772421] Review Request: trac-mastertickets-plugin - Add support for ticket dependencies to Trac

[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=772421

Pierre-YvesChibon pin...@pingoured.fr changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||NOTABUG
Last Closed||2012-01-07 17:23:51

-- 
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 772419] Review Request: trac-mastertickets-plugin - Add support for ticket dependencies to Trac

[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=772419

Pierre-YvesChibon pin...@pingoured.fr changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||NOTABUG
Last Closed||2012-01-07 17:21:24

-- 
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 772420] New: Review Request: trac-mastertickets-plugin - Add support for ticket dependencies to Trac

[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: trac-mastertickets-plugin - Add support for ticket 
dependencies to Trac

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

   Summary: Review Request: trac-mastertickets-plugin - Add
support for ticket dependencies to Trac
   Product: Fedora
   Version: rawhide
  Platform: Unspecified
OS/Version: Unspecified
Status: NEW
  Severity: unspecified
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: pin...@pingoured.fr
 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://pingou.fedorapeople.org//trac-mastertickets-plugin.spec
SRPM URL:
http://pingou.fedorapeople.org//trac-mastertickets-plugin-3.0.2-2.20111215.git43a7537.el6.src.rpm

Description:
This plugin adds blocks and blocked by fields to each ticket, enabling
you to express dependencies between tickets. It also provides a
graphviz-based dependency-graph feature for those tickets having
dependencies specified, allowing you to visually understand the
dependency tree. The dependency graph is viewable by clicking 'depgraph' in
the context (in the upper right corner) menu when viewing a ticket that
blocks or is blocked by another ticket.

-- 
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 772421] New: Review Request: trac-mastertickets-plugin - Add support for ticket dependencies to Trac

[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: trac-mastertickets-plugin - Add support for ticket 
dependencies to Trac

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

   Summary: Review Request: trac-mastertickets-plugin - Add
support for ticket dependencies to Trac
   Product: Fedora
   Version: rawhide
  Platform: Unspecified
OS/Version: Unspecified
Status: NEW
  Severity: unspecified
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: pin...@pingoured.fr
 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://pingou.fedorapeople.org//trac-mastertickets-plugin.spec
SRPM URL:
http://pingou.fedorapeople.org//trac-mastertickets-plugin-3.0.2-2.20111215.git43a7537.el6.src.rpm

Description:
This plugin adds blocks and blocked by fields to each ticket, enabling
you to express dependencies between tickets. It also provides a
graphviz-based dependency-graph feature for those tickets having
dependencies specified, allowing you to visually understand the
dependency tree. The dependency graph is viewable by clicking 'depgraph' in
the context (in the upper right corner) menu when viewing a ticket that
blocks or is blocked by another ticket.

-- 
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 772422] Review Request: trac-mastertickets-plugin - Add support for ticket dependencies to Trac

[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=772422

Pierre-YvesChibon pin...@pingoured.fr changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||NOTABUG
Last Closed||2012-01-07 17:28:36

-- 
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 772422] New: Review Request: trac-mastertickets-plugin - Add support for ticket dependencies to Trac

[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: trac-mastertickets-plugin - Add support for ticket 
dependencies to Trac

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

   Summary: Review Request: trac-mastertickets-plugin - Add
support for ticket dependencies to Trac
   Product: Fedora
   Version: rawhide
  Platform: Unspecified
OS/Version: Unspecified
Status: NEW
  Severity: unspecified
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: pin...@pingoured.fr
 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://pingou.fedorapeople.org//trac-mastertickets-plugin.spec
SRPM URL:
http://pingou.fedorapeople.org//trac-mastertickets-plugin-3.0.2-2.20111215.git43a7537.el6.src.rpm

Description:
This plugin adds blocks and blocked by fields to each ticket, enabling
you to express dependencies between tickets. It also provides a
graphviz-based dependency-graph feature for those tickets having
dependencies specified, allowing you to visually understand the
dependency tree. The dependency graph is viewable by clicking 'depgraph' in
the context (in the upper right corner) menu when viewing a ticket that
blocks or is blocked by another ticket.

-- 
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 772424] Rename Request: trac-mastertickets-plugin - Add support for ticket dependencies to Trac

[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=772424

Pierre-YvesChibon pin...@pingoured.fr changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||NOTABUG
Last Closed||2012-01-07 17:49:05

-- 
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 772423] Rename Request: trac-mastertickets-plugin - Add support for ticket dependencies to Trac

[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=772423

Pierre-YvesChibon pin...@pingoured.fr changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||NOTABUG
Last Closed||2012-01-07 17:47:56

-- 
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 772423] New: Rename Request: trac-mastertickets-plugin - Add support for ticket dependencies to Trac

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Rename Request: trac-mastertickets-plugin - Add support for ticket 
dependencies to Trac

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

   Summary: Rename Request: trac-mastertickets-plugin - Add
support for ticket dependencies to Trac
   Product: Fedora
   Version: rawhide
  Platform: Unspecified
OS/Version: Unspecified
Status: NEW
  Severity: unspecified
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: pin...@pingoured.fr
 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://pingou.fedorapeople.org//trac-mastertickets-plugin.spec
SRPM URL:
http://pingou.fedorapeople.org//trac-mastertickets-plugin-3.0.2-2.20111215.git43a7537.el6.src.rpm

Description:
This plugin adds blocks and blocked by fields to each ticket, enabling
you to express dependencies between tickets. It also provides a
graphviz-based dependency-graph feature for those tickets having
dependencies specified, allowing you to visually understand the
dependency tree. The dependency graph is viewable by clicking 'depgraph' in
the context (in the upper right corner) menu when viewing a ticket that
blocks or is blocked by another ticket.

-- 
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 772424] New: Rename Request: trac-mastertickets-plugin - Add support for ticket dependencies to Trac

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Rename Request: trac-mastertickets-plugin - Add support for ticket 
dependencies to Trac

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

   Summary: Rename Request: trac-mastertickets-plugin - Add
support for ticket dependencies to Trac
   Product: Fedora
   Version: rawhide
  Platform: Unspecified
OS/Version: Unspecified
Status: NEW
  Severity: unspecified
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: pin...@pingoured.fr
 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://pingou.fedorapeople.org//trac-mastertickets-plugin.spec
SRPM URL:
http://pingou.fedorapeople.org//trac-mastertickets-plugin-3.0.2-2.20111215.git43a7537.el6.src.rpm

Description:
This plugin adds blocks and blocked by fields to each ticket, enabling
you to express dependencies between tickets. It also provides a
graphviz-based dependency-graph feature for those tickets having
dependencies specified, allowing you to visually understand the
dependency tree. The dependency graph is viewable by clicking 'depgraph' in
the context (in the upper right corner) menu when viewing a ticket that
blocks or is blocked by another ticket.


 This is a Rename 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 772425] Rename Request: trac-mastertickets-plugin - Add support for ticket dependencies to Trac

[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=772425

Pierre-YvesChibon pin...@pingoured.fr changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||NOTABUG
Last Closed||2012-01-07 17:54:22

-- 
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 772425] New: Rename Request: trac-mastertickets-plugin - Add support for ticket dependencies to Trac

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Rename Request: trac-mastertickets-plugin - Add support for ticket 
dependencies to Trac

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

   Summary: Rename Request: trac-mastertickets-plugin - Add
support for ticket dependencies to Trac
   Product: Fedora
   Version: rawhide
  Platform: Unspecified
OS/Version: Unspecified
Status: NEW
  Severity: unspecified
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: pin...@pingoured.fr
 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://pingou.fedorapeople.org//trac-mastertickets-plugin.spec
SRPM URL:
http://pingou.fedorapeople.org//trac-mastertickets-plugin-3.0.2-2.20111215.git43a7537.el6.src.rpm

Description:
This plugin adds blocks and blocked by fields to each ticket, enabling
you to express dependencies between tickets. It also provides a
graphviz-based dependency-graph feature for those tickets having
dependencies specified, allowing you to visually understand the
dependency tree. The dependency graph is viewable by clicking 'depgraph' in
the context (in the upper right corner) menu when viewing a ticket that
blocks or is blocked by another ticket.


 This is a Rename request for the former package test

-- 
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 768100] Review Request: trac-mastertickets-plugin - Add support for ticket dependencies to Trac

[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=768100

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

   What|Removed |Added

   Fixed In Version|trac-mastertickets-plugin-3 |trac-mastertickets-plugin-3
   |.0.2-2.20111215.git43a7537. |.0.2-2.20111215.git43a7537.
   |el6 |fc16

--- Comment #10 from Fedora Update System upda...@fedoraproject.org 
2012-01-07 18:00:37 EST ---
trac-mastertickets-plugin-3.0.2-2.20111215.git43a7537.fc16 has been pushed to
the Fedora 16 stable repository.  If problems still persist, please make note
of it in this bug report.

-- 
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 768286] Review Request: libskk - Library to deal with Japanese kana-to-kanji conversion method

[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=768286

--- Comment #7 from Fedora Update System upda...@fedoraproject.org 2012-01-07 
18:03:02 EST ---
libskk-0.0.5-1.fc16 has been pushed to the Fedora 16 stable 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 746976] Review Request: python-passlib - Comprehensive password hashing framework supporting over 20 schemes

[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=746976

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

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||python-passlib-1.5.3-1.fc16
 Resolution||ERRATA
Last Closed||2012-01-07 18:07:28

--- Comment #10 from Fedora Update System upda...@fedoraproject.org 
2012-01-07 18:07:28 EST ---
python-passlib-1.5.3-1.fc16 has been pushed to the Fedora 16 stable 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 734014] Review Request: nafees-pakistani-web-naskh-fonts nafees-pakistani-web-naskh-fonts for writing Urdu

[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=734014

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

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||nafees-pakistani-web-naskh-
   ||fonts-2.0-2.fc16
 Resolution||ERRATA
Last Closed||2012-01-07 18:08:41

--- Comment #13 from Fedora Update System upda...@fedoraproject.org 
2012-01-07 18:08:41 EST ---
nafees-pakistani-web-naskh-fonts-2.0-2.fc16 has been pushed to the Fedora 16
stable repository.  If problems still persist, please make note of it in this
bug report.

-- 
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 768286] Review Request: libskk - Library to deal with Japanese kana-to-kanji conversion method

[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=768286

--- Comment #8 from Fedora Update System upda...@fedoraproject.org 2012-01-07 
18:10:05 EST ---
libskk-0.0.5-1.fc15 has been pushed to the Fedora 15 stable 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 737286] Review Request: salt - A parallel remote execution system

[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=737286

Thomas Spura toms...@fedoraproject.org changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|toms...@fedoraproject.org
   Flag||fedora-review?

--- Comment #10 from Thomas Spura toms...@fedoraproject.org 2012-01-07 
19:12:55 EST ---
(In reply to comment #9)
 Okay, systemd support for Fedora15+ has been added to the rpm now. Thanks to
 upstream.

Hurray, I was just waiting on that :P

Review:
- name ok
- noarch ok
- macros everywhere
- no libs
- no *.la
- %files see below (%doc question and more explicit in python_sitelib)
- services are not %configed

NEEDSWORK:
- R: python(abi) is not needed, as it's added automatically anyway (even with
version depending on 2.7)
- master and minion R the main package, please to that with %{version}
- It would be nice to install systemd scripts in rhel 7 (I bet rhel7 will have
it ;))
- description of master and minion contain Summary. Please delete that and
write e.g. what the packages are needed for (so they have more information than
the main package description ;))
- Please be more explicit in python_sitelib, so you know, when the egg fails to
build:
  %{python_sitelib}/salt-%{version}-py?.?.egg-info
  %{python_sitelib}/salt/
- systemd scripts should be 644 and not 775
- sysV ones should be 755 and not 775
- Shouldn't group be System Environment/Daemons, System/Utilities is not a
standard group.
- Missing R MySQL-python in modules/mysql.py
PyXML in modules/virt.py
libvirt-python in modules/virt.py
yum in modules/yumpkg.py
- add LICENSE as %doc
- ask upstream to add headers
- missing BR on systemd-units (and then you can omit defining _unitdir)

COMMENTS:
- saltstack as URL makes more sense to me than github, but it's a matter of
taste... (But at least point to saltstack's github ;))
- why are the man pages marked as %doc? Didn't see that before IIRC.

-- 
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 718502] Review Request: Temperature.app - Window Maker applet which fetches local temperature information

[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=718502

--- Comment #11 from Mario Blättermann mario.blaetterm...@gmail.com 
2012-01-07 20:03:15 EST ---
The issues are fixed now. Koji scratch build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=3628228

Spec URL: http://mariobl.fedorapeople.org/Review/SPECS/Temperature.app.spec
SRPM URL:
http://mariobl.fedorapeople.org/Review/SRPMS/Temperature.app-1.5-8.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 772432] New: Review Request: gnome-applet-sensors - Gnome panel applet for hardware sensors

[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: gnome-applet-sensors - Gnome panel applet for hardware 
sensors

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

   Summary: Review Request: gnome-applet-sensors - Gnome panel
applet for hardware sensors
   Product: Fedora
   Version: rawhide
  Platform: Unspecified
OS/Version: Unspecified
Status: NEW
  Severity: unspecified
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: redhat-bugzi...@linuxnetz.de
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, den...@ausil.us,
huzai...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---
Regression: ---
Mount Type: ---
 Documentation: ---


Spec URL: http://labs.linuxnetz.de/bugzilla/gnome-applet-sensors.spec
SRPM URL:
http://labs.linuxnetz.de/bugzilla/gnome-applet-sensors-3.0.0-2.src.rpm
Description:
GNOME Sensors Applet is an applet for the GNOME Panel to display readings
from hardware sensors, including CPU temperature, fan speeds and voltage
readings under Linux. Includes a simple, yet highly customizable display
and intuitive user-interface. Alarms can be set for each sensor to notify
the user once a certain high or low value has been reached, and can be
configured to execute a given command at specific repeated intervals. It
is also GNOME HIG v2.0 compliant.


Please note this is a re-review request as required by Fedora policies at
https://fedorahosted.org/rel-eng/ticket/4999. Oh and I am not the previous
gnome-applet-sensors package maintainer, however I would like to help him.
If he maybe does not want to take care about this package anymore, I'm also
willing to take the package. For the time being, I'll assume he would like
to take care of his package further on.

-- 
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 772432] Review Request: gnome-applet-sensors - GNOME panel applet for hardware sensors

[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=772432

Robert Scheck redhat-bugzi...@linuxnetz.de changed:

   What|Removed |Added

Summary|Review Request: |Review Request:
   |gnome-applet-sensors -  |gnome-applet-sensors -
   |Gnome panel applet for  |GNOME panel applet for
   |hardware sensors|hardware sensors

-- 
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 772432] Review Request: gnome-applet-sensors - GNOME panel applet for hardware sensors

[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=772432

Robert Scheck red...@linuxnetz.de changed:

   What|Removed |Added

 Blocks||694549

-- 
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 772432] Review Request: gnome-applet-sensors - GNOME panel applet for hardware sensors

[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=772432

Robert Scheck red...@linuxnetz.de changed:

   What|Removed |Added

 Blocks||691085

-- 
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 772432] Review Request: gnome-applet-sensors - GNOME panel applet for hardware sensors

[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=772432

Robert Scheck red...@linuxnetz.de changed:

   What|Removed |Added

 Blocks||716008

-- 
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

[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 #7 from Volker Fröhlich volke...@gmx.at 2012-01-07 23:19:52 EST 
---
SeqAn must still be un-bundled from tophat to finish the 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 754023] Review Request: Mumpot - GTK mapping application

[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=754023

--- Comment #2 from Volker Fröhlich volke...@gmx.at 2012-01-07 23:47:50 EST 
---
Spec URL: http://www.geofrogger.net/review/mumpot.spec
SRPM URL: http://www.geofrogger.net/review/mumpot-0.6-2.fc15.src.rpm

I hope, I have addressed all issues.

* Sun Nov 20 2011 Volker Fröhlich volke...@gmx.at - 0.6-2
- Rename patch1 backup file
- BR desktop-file-utils and validate desktop file
- Generalize files section
- Solve unsigned int to digit formatting problem (Patch #3)

-- 
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 737286] Review Request: salt - A parallel remote execution system

[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=737286

--- Comment #11 from Clint Savage her...@gmail.com 2012-01-08 00:22:31 EST ---
I think ti have accommodated all of those requests. Let me know if I missed
anything.

I also added systemctl reloads in %post and %postun.

SPEC: http://herlo.fedorapeople.org/rpms/salt.spec
SRPM: http://herlo.fedorapeople.org/rpms/salt-0.9.4-4.fc16.src.rpm

Clint

-- 
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 771137] Review Request: decibel-audio-player - straightforward music player

[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=771137

Volker Fröhlich volke...@gmx.at changed:

   What|Removed |Added

 CC||volke...@gmx.at

--- Comment #1 from Volker Fröhlich volke...@gmx.at 2012-01-08 01:47:08 EST 
---
Just some comments:

- Drop defattr, it is no longer necessary
- If you don't go for EPEL 5, you can drop the buildroot definition, the clean
section and the rm in the install section
- Consider using a loop to create directories and install the icons


%{_datadir}/%{name}

could replace

%dir %{_datadir}/%{name}
%{_datadir}/%{name}/pix
%{_datadir}/%{name}/res
%{_datadir}/%{name}/src

as far as I can see


- The RPM changelog is intended for changes in packaging, not within the
software
- Man pages should not be marked as documentation
- Was vendor used in desktop files in older versions of this package? If not,
it should not be used.
- Consider harmonizing the style of your sed invocations

I'm not sure about those shell scripts that act as starters. Are they really
necessary?

-- 
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 772243] Review Request: rds-tools - utilities for testing rds protocol networking

[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=772243

Volker Fröhlich volke...@gmx.at changed:

   What|Removed |Added

 CC||volke...@gmx.at

--- Comment #1 from Volker Fröhlich volke...@gmx.at 2012-01-08 02:08:38 EST 
---
Just some comments:

- License seems to be GPLv2 or BSD -- not GPLv2+ or BSD
- You can use the name macro in Source0
- Comment on the patches in the spec file, if possible
- FSF address is wrong in rds-sample.c
- Please harmonize the use of RPM_BUILD_ROOT and buildroot macro
- If you don't go for EPEL 5, you can delete the buildroot definition, the
clean section and the rm in the install section
- defattr is no longer necessary
- Remove README from the documentation as it holds no valuable information
- The optflags are not used when compiling
- You can patch the permissions for the executables in rds-tools-make.patch
instead of changing them in the install section
- What is that rds.conf file about?

-- 
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 772243] Review Request: rds-tools - utilities for testing rds protocol networking

[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=772243

--- Comment #2 from Volker Fröhlich volke...@gmx.at 2012-01-08 02:13:41 EST 
---
I forgot: There are two consecutive spaces in your description. This seems to
be the case for some of your other packages too.

By the way, review requests should follow this guideline:
http://fedoraproject.org/wiki/New_package_process_for_existing_contributors

That is, giving the URL to spec file and SRPM in the text.

You can also add the rpmlint output and a link to a Koji scratch build, what
many reviewers appreciate.

-- 
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