[Bug 513636] Review Request: php-layers-menu - Hierarchical PHP based DHTML menu system

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #2 from David Nalley   2009-07-27 02:18:38 EDT ---
(In reply to comment #1)
> 
> MUST: The package must be named according to the Package Naming Guidelines .
> 
> I have some questions about package name The name strikes me as phplayersmenu,
> which should make the package name php-phplayersmenu, but I have asked for
> clarification. 

Based on the feedback I received, this name appears acceptable.

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 512727] Review Request: perl-Async-MergePoint - Resynchronise diverged control flow

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Marcela Maslanova  changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #5 from Marcela Maslanova   2009-07-27 
01:53:38 EDT ---
I suppose you forgot set cvs flag to ?
http://fedoraproject.org/wiki/PackageMaintainers/CVSAdminProcedure

I hope you don't mind if I set it.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 506056] Review Request: zikula-module-scribite - The scribite! module for Zikula allows integration of various text editors

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Fedora Update System  changed:

   What|Removed |Added

 Status|ASSIGNED|ON_QA




--- Comment #22 from Fedora Update System   
2009-07-27 01:49:18 EDT ---
zikula-module-scribite-3.2-3.el5 has been pushed to the Fedora EPEL 5 testing
repository.  If problems still persist, please make note of it in this bug
report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update zikula-module-scribite'.  You
can provide feedback for this update here:
http://admin.fedoraproject.org/updates/EL-5/FEDORA-EPEL-2009-0122

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 508511] Review Request: python-Lightbox - Lightbox photo display widget

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #15 from Itamar Reis Peixoto   2009-07-27 
00:35:27 EDT ---
the JavaScript code comes from

http://www.lokeshdhakar.com/projects/lightbox2/

"Can I use the script in a commercial project?

Yes. It is licensed under the Creative Commons Attribution 2.5 License. A
donation would be nice, hint hint."


the python code is MIT and javascript code is Creative Commons Attribution 2.5

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 513898] Review Request: keychecker - Generate list of installed packages sorted by GPG key

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #2 from Jon Stanley   2009-07-27 00:26:29 EDT 
---
Fixed, though I think it's a matter of personal preference rather than
guidelines :)

Spec is the same
New SRPM: http://jstanley.fedorapeople.org/keychecker-0.1-2.fc10.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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 508511] Review Request: python-Lightbox - Lightbox photo display widget

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Parag AN(पराग)  changed:

   What|Removed |Added

 CC||tcall...@redhat.com
 Blocks||182235(FE-Legal)




--- Comment #14 from Parag AN(पराग)   2009-07-27 00:22:32 
EDT ---
spot,
   Can you please help here to identify correct license for this package. I
actually got mail at
https://www.redhat.com/archives/fedora-devel-list/2009-July/msg01557.html

Thanks.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 452427] Review Request: awesome - Extremely fast, small, dynamic and awesome floating and tiling window manager

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Ben Boeckel  changed:

   What|Removed |Added

 CC||maths...@gmail.com




--- Comment #60 from Ben Boeckel   2009-07-26 23:40:23 EDT 
---
Just poking in here when I remembered that awesome looked interesting to try,
but is it possible to package version 2.x until Cairo supports XCB? That would
enable you to get sponsored and the package reviewed so that when XCB is
enabled, awesome3 can come sooner. I can jump on as comaintainer then to help
with the jump if you'd like.

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 511212] Review Request: cluster-glue - reusable clustering components

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Kevin Fenzi  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|ke...@tummy.com
   Flag||fedora-review?




--- Comment #11 from Kevin Fenzi   2009-07-26 23:26:48 EDT ---
ok, that makes sense. 

So, we need to get this reviewed and in, then update heartbeat and also
review/import pacemaker. 

Conflicts are nasty in that basically we have to tell the user "hey, we can't
do what you want". :( I think we can possibly avoid the conflict by importing
this around the same time we update heartbeat to use it. I don't know if there
is a big group that wants to keep heartbeat 2.1.x in fedora, but time does
march on, and fedora should be moving with it. ;) 

Anyhow, I guess I will look at doing a review here 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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 497622] Review Request: apbs - adaptive poisson boltzmann solver

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #32 from Tim Fenn   2009-07-26 21:49:16 EDT ---
(In reply to comment #31)
> (In reply to comment #30)
> > (In reply to comment #26)
> > > apbs has support for:
> > >   --with-mpich=PATH   toplevel MPICH directory
> > >   --with-mpich2=PATH  toplevel MPICH2 directory
> > >   --with-lam=PATH toplevel LAM-MPI directory
> > >   --with-openmpi  enable OpenMPI compilation
> > > so I suggest you add these to the package in the future.
> > > 
> > 
> > Will do.
> 
> Just no hurry - wait until the two drafts become guidelines :)
> 

Right!

> > This works, but the resulting examples take ~4hrs to run on my AMD opteron 
> > 275.
> > I can cut it down to a 1-2 of the examples...
> 
> Ugh. Do they give an error code if the result is not the wanted one..?
> 
> You could probably just drop the %check phase.
> 

There is no error code, it just prints out that the result was not within the
given error tolerance.

I just took out the %check.

> 
> Package does not build:
> RPM build errors:
> File not found:
> /builddir/build/BUILDROOT/apbs-1.1.0-5.fc11.x86_64/usr/bin/ApbsClient.pyc
> File not found:
> /builddir/build/BUILDROOT/apbs-1.1.0-5.fc11.x86_64/usr/bin/ApbsClient.pyo
> 
> This is due to RPM in Fedora 11 not byte compiling files in %{_bindir}. 
> Instead
> of %exclude you could just delete the files after %install with rm -f which
> doesn't complain about possible non-existing files.  

I just removed the %exclude statement and built using F11 (since I won't be
packaging this for F10 anyway).

Spec URL: http://www.stanford.edu/~fenn/packs/apbs.spec
SRPM URL: http://www.stanford.edu/~fenn/packs/apbs-1.1.0-6.fc11.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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 513896] Review Request: pcp - performance monitoring and collection service

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Mark Goodwin  changed:

   What|Removed |Added

 CC||mgood...@redhat.com
 Blocks||177841(FE-NEEDSPONSOR)




--- Comment #1 from Mark Goodwin   2009-07-26 21:31:55 EDT 
---

Setting the "seeking a sponsor" tags.

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 513898] Review Request: keychecker - Generate list of installed packages sorted by GPG key

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Itamar Reis Peixoto  changed:

   What|Removed |Added

 CC||ita...@ispbrasil.com.br




--- Comment #1 from Itamar Reis Peixoto   2009-07-26 
21:29:28 EDT ---
Can you merge the 2 lines below into only one using install -Dp -m ?


install -dm 0755 $RPM_BUILD_ROOT/%{_bindir}
install -m 0755 key_checker.py $RPM_BUILD_ROOT/%{_bindir}/keychecker

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 513898] New: Review Request: keychecker - Generate list of installed packages sorted by GPG key

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: keychecker - Generate list of installed packages 
sorted by GPG key

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

   Summary: Review Request: keychecker - Generate list of
installed packages sorted by GPG key
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: jonstan...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://jstanley.fedorapeople.org/keychecker.spec
SRPM URL: http://jstanley.fedorapeople.org/keychecker-0.1-1.fc10.src.rpm
Description: Sort packages by GPG key they were signed 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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 513896] New: Review Request: pcp - performance monitoring and collection service

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: pcp - performance monitoring and collection service

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

   Summary: Review Request: pcp - performance monitoring and
collection service
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: mgood...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


Spec URL: ftp://oss.sgi.com/projects/pcp/download/v3/pcp.spec
SRPM URL: ftp://oss.sgi.com/projects/pcp/download/v3/pcp-3.0.0-1.fc10.src.rpm

Description:

Performance Co-Pilot (PCP) provides a framework and services to support
system-level performance monitoring and management. It presents a unifying
abstraction for all of the performance data in a system, and many tools for
interrogating, retrieving and processing that data.

PCP is a feature-rich, mature, extensible, cross-platform toolkit supporting
both live and retrospective analysis. The distributed PCP architecture makes
it especially useful for those seeking centralized monitoring of distributed
processing. PCP is an ideal "drop-in" replacement for sysstat/sar for those
engaged in supporting enterprise level production computing environments.
For more details, documentation and white papers, see the PCP home page at
http://oss.sgi.com/projects/pcp

The associated GUI package (pcp-gui) exists in a separate tree and will be
submitted in a separate review request.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 513636] Review Request: php-layers-menu - Hierarchical PHP based DHTML menu system

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #1 from David Nalley   2009-07-26 20:35:50 EDT ---
OK: rpmlint must be run on every package. The output should be posted in the
review.
[ke4...@nalleyt61 SPECS]$ rpmlint ./php-layers-menu.spec 
0 packages and 1 specfiles checked; 0 errors, 0 warnings.
[ke4...@nalleyt61 SPECS]$ rpmlint
../SRPMS/php-layers-menu-3.2.0-0.1.rc.fc11.src.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.
[ke4...@nalleyt61 SPECS]$ rpmlint
../RPMS/noarch/php-layers-menu-3.2.0-0.1.rc.fc11.noarch.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

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

I have some questions about package name The name strikes me as phplayersmenu,
which should make the package name php-phplayersmenu, but I have asked for
clarification. 

OK: The spec file name must match the base package %{name}, in the format
%{name}.spec unless your package has an exemption. 
FIX: The package must meet the Packaging Guidelines .
See below - there are some file placement issues. 
OK: The package must be licensed with a Fedora approved license and meet the
Licensing Guidelines .
LGPLv2+ 
FIX: The License field in the package spec file must match the actual license.
you've listed LGPLv2 and based on notices it appears to be LGPLv2+
OK: 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: The spec file must be written in American English.
OK: The spec file for the package MUST be legible. 
OK: 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 
[ke4...@nalleyt61 SOURCES]$ sha1sum ./phplayersmenu-3.2.0-rc.tar.gz*
7b084da1e20ab2faa9da17b6a088b81d6eb0523b  ./phplayersmenu-3.2.0-rc.tar.gz
7b084da1e20ab2faa9da17b6a088b81d6eb0523b  ./phplayersmenu-3.2.0-rc.tar.gz.1


OK: The package MUST successfully compile and build into binary rpms on at
least one primary architecture. 
NA: 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: 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.
NA: The spec file MUST handle locales properly. This is done by using the
%find_lang macro. Using %{_datadir}/locale/* is strictly forbidden.
NA: 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. 
NA: 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: 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: A Fedora package must not list a file more than once in the spec file's
%files listings. 
OK: 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: Each package must have a %clean section, which contains rm -rf %{buildroot}
(or $RPM_BUILD_ROOT). 
OK: Each package must consistently use macros. 
OK: The package must contain code, or permissable content. 
NA: Large documentation files must go in a -doc subpackage. (The definition of
large is left up to the packager's best judgement, but is not restricted to
size. Large can refer to either size or quantity). 
OK: If a package includes something as %doc, it must not affect the runtime of
the application. To summarize: If it is in %doc, the program must run properly
if it is not present. 
NA: Header files must be in a -devel package. 
NA: Static libraries must be in a -static package. 
NA: Packages containing pkgconfig(.pc) files must 'Requires: pkgconfig' (for
directory ownership and usability). 
NA: If a package contains library files with a suffix (e.g. libfoo.so.1.1),
then library files that end in .so (without suffix) must go in a -devel
package. 
NA: In the vast majority of cases, devel packages must require 

[Bug 513636] Review Request: php-layers-menu - Hierarchical PHP based DHTML menu system

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


David Nalley  changed:

   What|Removed |Added

 Blocks||493237




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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 513892] New: Review Request: bilbo - Blogging client, focused on simplicity and usability

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: bilbo - Blogging client, focused on simplicity and 
usability

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

   Summary: Review Request: bilbo - Blogging client, focused on
simplicity and usability
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: maths...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://benboeckel.net/packaging/bilbo/bilbo.spec
SRPM URL: http://benboeckel.net/packaging/bilbo/bilbo-0.9-1.fc11.src.rpm
Description:
Blogging application for Blogger1.0, MetaWeblog, MovableType, and Google
GData with WYSIWYG and HTML editing 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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 491992] Review Request: mod_falcon - An embedded Falcon interpreter for the Apache HTTP Server

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #6 from Ben Boeckel   2009-07-26 20:18:46 EDT 
---
Sorry,
http://benboeckel.net/packaging/mod_falcon/mod_falcon-0.9.4-2.fc10.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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 473835] Review Request: autoarchive - Simple backup tool

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Julian Aloofi  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #7 from Julian Aloofi   2009-07-26 
19:12:24 EDT ---
(In reply to comment #6)
> > Requires: lzma is not enough IMHO. The package seems to need also tar, gzip 
> > and
> > bzip2.  
> 
> Yes, you are right, the 'archiver class' shows points that let indicate that
> all this tools are needed.

You are still using the old spec file in the Spec URL link, but the one in the
SRPM is good.
By the way, tar gzip and bzip2 are in the Base group anyway, you can't install
Fedora without them. I wonder why these packages are not in the exception list
in the Guidelines.

I can't see any further problems with the 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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 451773] Review Request: window-picker-applet - an alternative window chooser for small screens

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Bastien Nocera  changed:

   What|Removed |Added

 CC||michel.syl...@gmail.com




--- Comment #9 from Bastien Nocera   2009-07-26 18:58:31 
EDT ---
*** Bug 498200 has been marked as a duplicate of this bug. ***

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 498200] Review Request: window-picker-applet - Window picker applet for GNOME

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Bastien Nocera  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 CC||bnoc...@redhat.com
 Resolution||DUPLICATE




--- Comment #2 from Bastien Nocera   2009-07-26 18:58:31 
EDT ---


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

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 513307] Review Request: Netdisco - A web-based network management tool

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Silas Sewell  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #4 from Silas Sewell   2009-07-26 19:00:18 EDT ---
Looks good.

Doesn't really matter for the first release, but you'll want to append to the
changelog as opposed to overwriting. 

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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 509533] Review Request: sap - A small CLI audio player

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #29 from Fedora Update System   
2009-07-26 18:52:27 EDT ---
sap-0.4.4-7.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/sap-0.4.4-7.fc11

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 513733] Review Request: entertainer - A simple media center

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #23 from Julian Aloofi   2009-07-26 
18:40:59 EDT ---
I updated the package, here are the new files:
SPEC: http://julian.fedorapeople.org/entertainer/entertainer.spec
SRPM:
http://julian.fedorapeople.org/entertainer/entertainer-0.4.2-5.fc11.src.rpm

As I can't reproduce Rahuls problems, I guess this should be good to go.
Rahul, if you have the time and want to see it fixed, could you report it
upstream yourself? I won't be very useful if upstream asks me to try a patch or
something like that.

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 468516] Review Request: verilator - A fast simulator of synthesizable Verilog HDL

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #50 from Fedora Update System   
2009-07-26 18:35:30 EDT ---
verilator-3.712-1.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/verilator-3.712-1.fc10

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 468516] Review Request: verilator - A fast simulator of synthesizable Verilog HDL

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #51 from Fedora Update System   
2009-07-26 18:35:36 EDT ---
verilator-3.712-1.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/verilator-3.712-1.fc11

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 468516] Review Request: verilator - A fast simulator of synthesizable Verilog HDL

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #49 from Chitlesh GOORAH   2009-07-26 18:10:08 
EDT ---
Give me a few days. I'm in holidays. I'll push it to EL-5 when I return.

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 468516] Review Request: verilator - A fast simulator of synthesizable Verilog HDL

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #48 from Lane   2009-07-26 18:03:40 EDT ---
Chitlesh,

When I build the EL-5 release of verilator using koji it fails with this
message:

DEBUG util.py:280:  Executing command: /usr/bin/yum --installroot
/var/lib/mock/dist-5E-epel-build-527553-80064/root/  resolvedep 
'perl-SystemPerl-devel' 'flex' 'bison' 'perl'
DEBUG util.py:256:  No Package Found for perl-SystemPerl-devel

Is SystemPerl not available for EL-5?

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 510979] Review Request: phpSmug - PHP wrapper for the SmugMug API

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #12 from David Nalley   2009-07-26 18:01:49 EDT ---
none from me

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 513733] Review Request: entertainer - A simple media center

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #22 from Julian Aloofi   2009-07-26 
17:50:58 EDT ---
I succesfully installed and ran it in a virtual machine (Windows host). The
system was unpatched (apart from the dependencies). Worked fine.
Considering that I think we can open a cvs branch, but please wait until I make
another release (coming in a couple of hours), I need to include two other
.desktop files for the preferences manager and the collection manager, because
they're not integrated into the media center. I'll also push them upstream.

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 513731] Review request: berry4all - Utilities to tether Blackberries to Linux. Includes pre-built settings for many major cellular carriers.

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Jason Tibbitts  changed:

   What|Removed |Added

 Blocks||177841(FE-NEEDSPONSOR)




--- Comment #7 from Jason Tibbitts   2009-07-26 17:49:40 EDT 
---
It doesn't seem that you're in the packager group, so I've indicated that you
need a sponsor.  It might help to read through
http://fedoraproject.org/wiki/PackageMaintainers/Join if you haven't already.

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 513731] Review request: berry4all - Utilities to tether Blackberries to Linux. Includes pre-built settings for many major cellular carriers.

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Jason Tibbitts  changed:

   What|Removed |Added

Summary|REVIEW:  Berry4all  |Review request: berry4all -
   ||Utilities to tether
   ||Blackberries to Linux.
   ||Includes pre-built settings
   ||for many major cellular
   ||carriers.




--- Comment #6 from Jason Tibbitts   2009-07-26 17:47:16 EDT 
---
We have scripts that parse these tickets so I've fixed up the Summary.  It also
helps things along if you build the package, run rpmlint on the resulting *rpm
files, and address as much of the resulting errors.  Here's what I get; many of
these should be trivial to fix:

berry4all.x86_64: E: explicit-lib-dependency libusb
berry4all.x86_64: W: summary-ended-with-dot Utilities to tether Blackberries to
Linux.  Includes pre-built settings for many major cellular carriers.
berry4all.x86_64: E: summary-too-long Utilities to tether Blackberries to
Linux.  Includes pre-built settings for many major cellular carriers.
berry4all.x86_64: E: description-line-too-long This package contains utilities
for using Blackberries with Linux, including the ability to tether a blackberry
and use it as a modem.  It includes several pre-built settings for use with
many major cellular providers.
berry4all.x86_64: W: non-standard-group System Tools
berry4all.x86_64: W: invalid-license Artistic License 2.0
berry4all.x86_64: E: no-binary 
berry4all.x86_64: W: only-non-binary-in-usr-lib 
berry4all.x86_64: W: no-documentation 
berry4all.x86_64: E: non-executable-script /usr/lib64/bbtether/bbbackup.py 0644
/usr/bin/python
berry4all.x86_64: E: non-executable-script /usr/lib64/bbtether/bbtether.py 0644
/usr/bin/python
berry4all.x86_64: E: non-executable-script /usr/lib64/bbtether/bb_backup.py
0644 /usr/bin/python
berry4all.x86_64: E: script-without-shebang /usr/bin/bbtether
berry4all.x86_64: E: non-executable-script /usr/lib64/bbtether/bb_tether.py
0644 /usr/bin/python
berry4all-debuginfo.x86_64: W: invalid-license Artistic License 2.0
berry4all-debuginfo.x86_64: E: empty-debuginfo-package

Are you sure this package shouldn't be noarch?  I don't see anything being
compiled and I see no arch-specific content.

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 512263] Review Request: audtty - A ncurses based terminal client for the Audacious

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #9 from Fedora Update System   
2009-07-26 17:41:08 EDT ---
audtty-0.1.11-3.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/audtty-0.1.11-3.fc10

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 512263] Review Request: audtty - A ncurses based terminal client for the Audacious

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #8 from Fedora Update System   
2009-07-26 17:41:03 EDT ---
audtty-0.1.11-3.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/audtty-0.1.11-3.fc11

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 513878] MD5 sum mismach error when installing SRPM packages gotten from mirrors.

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Jason Tibbitts  changed:

   What|Removed |Added

 CC||jkeat...@redhat.com,
   ||ka...@redhat.com
  Component|Package Review  |distribution
 AssignedTo|nob...@fedoraproject.org|nott...@redhat.com
  QAContact|extras...@fedoraproject.org |nott...@redhat.com




--- Comment #1 from Jason Tibbitts   2009-07-26 17:36:48 EDT 
---
This is most certainly not a package review ticket.  Reassigning to
distribution.

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 513878] MD5 sum mismach error when installing SRPM packages gotten from mirrors.

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Jason Tibbitts  changed:

   What|Removed |Added

 CC|fedora-package-rev...@redha |
   |t.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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 513307] Review Request: Netdisco - A web-based network management tool

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #3 from Walter Gould   2009-07-26 17:30:24 
EDT ---
Ok, fixed the blocks.

* You should change /usr/share/ to %_datadir in the %files section -- DONE
* Most of the explicit %attr aren't required in %files -- DONE 
* Patches are suppose to have comments: Although it might be easier just to
change those patches to seds. -- DONE; On this one, I fixed the files w/o
applying patches. 
* I know nothing about the vendor mips packaged in netdisco-mips, I'm assuming
you know if they're allowed to be packaged in Fedora or not. -- DONE; These are
MIB (Management Information Base) files that network vendors offer to the
public that allow their devices (switches/routers) to be monitored.  Since they
are offered freely to the public, there shouldn't be a problem in including
them in this package.   

Below are the new spec file and src rpm.

http://auburn.edu/~gouldwp/netdisco/rpms/netdisco-0.95-2.fc11.src.rpm
http://auburn.edu/~gouldwp/netdisco/rpms/netdisco.spec

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 473835] Review Request: autoarchive - Simple backup tool

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #6 from Fabian Affolter   2009-07-26 
17:16:05 EDT ---
Thank you guys. 

(In reply to comment #4)
> 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. NEEDSWORK
> -Please add COPYING to the doc section

After the review of Gratien I removed COPYING.  But the fixed version came back
with 0.1.2.  COPYING added

(In reply to comment #5)
> (In reply to comment #4)
> > Notes:
> > -Your package should BuildRequire: python-devel instead of python  
> 
> Actually, python-setuptools-devel pulls in python-devel, so that's not a
> problem. But it's better to BR it exclusively.

For newer Fedora releases that's true.  For releases < F-11 the guidelines told
another story.  Changed.

> (The ./ in front of setup.py is unnecessary when run via python.)

Removed

> I suggest using autoarchive instead of %{name} in the %files section for
> consistency.

Changed

> Requires: lzma is not enough IMHO. The package seems to need also tar, gzip 
> and
> bzip2.  

Yes, you are right, the 'archiver class' shows points that let indicate that
all this tools are needed.

Here are the updated files:

Spec URL: http://fab.fedorapeople.org/packages/SRPMS/autoarchive.spec
SRPM URL:
http://fab.fedorapeople.org/packages/SRPMS/autoarchive-0.1.2-2.fc11.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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 511212] Review Request: cluster-glue - reusable clustering components

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #10 from Andrew Beekhof   2009-07-26 17:13:13 
EDT ---
(In reply to comment #9)
> ok. Finally found some time to look at this a bit. ;) 

Much appreciated

> So, my understanding is that this is a newerversion/fork of heartbeat thats
> stripped down and only has the parts pacemaker needs/wants in it. Is that
> correct or am I misunderstanding? :) 

We're in the process of splitting the rest of the codebase.
Its essentially done but just needs the sort of packaging cleanup that we're
doing here before it will be made available for general consumption.  I've also
been on vacation a bit which slowed things down.

The idea is to split the code-base into pieces based on their usage and update
frequency.  So essentially: 
  heartbeat 2.1 + features + bugfixes = cluster-glue + cluster-agents +
heartbeat 3.0 + pacemaker 1.0

or more recently (and relevantly for those not using the crm):
  heartbeat 2.99 + bugfixes = cluster-glue + cluster-agents + heartbeat 3.0

This was mentioned on the mailing list late last month:
   http://www.gossamer-threads.com/lists/linuxha/dev/56290

> 
> First, Fedora really would prefer to avoid Conflicts if at all possible. 
> (See https://fedoraproject.org/wiki/Packaging/Conflicts). 
> So, is there some way we can avoid that here? 

The Conflicts was only intended to be temporary.

I didn't want to force a synchronized update to the heartbeat package just to
get Pacemaker (and therefor cluster-glue) in.

The intention from upstream is that heartbeat 3.0 (which would be the messaging
and membership layer as well as the v1 resource manager) would build against
cluster-glue. 

If/when the cluster-glue package gets approved, I would then submit an update
to the Fedora's heartbeat package and we could then remove the Conflicts
directive.

On the other hand, if Fedora wanted to preserve heartbeat in its current
pre-split state, I figured the Conflicts would (safely) allow that.

> 
> - Can this package just rename the dirs/files that conflict with 
> 'cluster-glue'
> or something and that way they could both be installed and pacemaker could 
> have
> a compile time option to use whichever one. (Or autodetect it based on the
> files found, etc). 

In theory this could be possible, but ideally it wouldn't be necessary since
we'd have both Pacemaker and an updated Heartbeat both using cluster-glue as a
common base.

> 
> - Is using this better than using heartbeat for pacemaker? Or are they the
> same? 

Its a subset, essentially: clplumbing + lrmd + stonith

> If the same, perhaps we hold off on adding this to fedora for now, let
> pacemaker use heartbeat and then down the road when enough folks have moved to
> pacemaker submit this and obsolete heartbeat? 

I don't think that's possible, since heartbeat 2.1 conflicts massively with
Pacemaker.
I suspect there'd be all sorts of compiler problems, not least of all due to
multiple copies of "crm" header files.

> - Perhaps pacemaker could fold in the needed parts of this and just use them
> internally and avoid conflict with heartbeat that way?

That was considered undesirable.

If we were intending to fork the code, then putting it in Pacemaker would make
more sense.
But that would be a last possible resort.


Does that help?

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 509533] Review Request: sap - A small CLI audio player

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #28 from Jussi Lehtola   2009-07-26 16:58:56 
EDT ---
(In reply to comment #27)
> (In reply to comment #25)
> > No F-10 branch?
> The latest vala in F10 is too old for this package, so unfortunately not.  

Right, I had forgotten about that.

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 512217] Review Request: geany-plugins - A bundle of plugins for Geany

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Dominic Hopf  changed:

   What|Removed |Added

   Flag||fedora-cvs?




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 512217] Review Request: geany-plugins - A bundle of plugins for Geany

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #41 from Dominic Hopf   2009-07-26 16:50:51 EDT 
---
New Package CVS Request
===
Package Name: geany-plugins
Short Description: A bundle of plugins for Geany
Owners: dmaphy jgu pingou
Branches: F-11
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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 513878] New: MD5 sum mismach error when installing SRPM packages gotten from mirrors.

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: MD5 sum mismach error when installing SRPM packages gotten from 
mirrors.

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

   Summary: MD5 sum mismach error when installing SRPM packages
gotten from mirrors.
   Product: Fedora
   Version: 11
  Platform: x86_64
OS/Version: Linux
Status: NEW
  Severity: high
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: daviverci...@fedoraproject.org
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


User-Agent:   Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.1.1)
Gecko/20090717 Fedora/3.5.1-1.fc11 Firefox/3.5.1

When I try to install a source RPM package gotten from a official Fedora 11
mirror using the command "rpm -ivh" I got a MD5 sum mismatch error.

"[r...@mars fedora]# rpm -ivh syslinux-3.75-1.fc11.src.rpm
warning: syslinux-3.75-1.fc11.src.rpm: Header V3 RSA/SHA256 signature: NOKEY,
key ID d22e77f2
  1:syslinux   warning: user mockbuild does not exist - using root
warning: group mockbuild does not exist - using root
### [100%]
error: unpacking of archive failed on file /usr/src/redhat/SOURCES/syslinux-
3.75.tar.bz2;4a675da8: cpio: MD5 sum mismatch"

Reproducible: Always

Steps to Reproduce:
1. Get a src=rpm from any Fedora 11 mirror.
2. Try to install using "rpm -ivh"
Actual Results:  
cpio: MD5 sum mismatch"

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 502227] Review Request: virtuoso-opensource - A high-performance object-relational SQL database

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Rex Dieter  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE




--- Comment #28 from Rex Dieter   2009-07-26 16:34:35 EDT 
---
imported, builds 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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 510668] Review Request: canorus - Music Score Editor

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #14 from Orcan 'oget' Ogetbil   2009-07-26 
16:28:40 EDT ---
Thanks for the extensive review! I fixed most of the issues and I contacted
upstream and ask them about the license issues. I will update the SRPM
according to their reply:
   https://lists.berlios.de/pipermail/canorus-devel/2009-July/000989.html

To make my understanding clear, even if the midi and musicxml files are free,
can I not put them in the package?

The upstream is aware of the crashes (I talked to them on IRC). They say that
the software is still at alpha state and most of these issues will be fixed in
time. Whereas there are still nice things you can do with canorus, it should
not be used professionally yet.

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 477199] Review Request: PolicyKit-kde - PolicyKit integration for the KDE desktop

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Rex Dieter  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 492600] Review Request: oxygen-icon-theme - Oxygen Icon Theme

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Rex Dieter  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 509533] Review Request: sap - A small CLI audio player

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #27 from Julian Aloofi   2009-07-26 
16:25:47 EDT ---
(In reply to comment #25)
> No F-10 branch?
The latest vala in F10 is too old for this package, so unfortunately 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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 508349] Review Request: kdepim-runtime - KDE PIM Runtime Environment

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Rex Dieter  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 513733] Review Request: entertainer - A simple media center

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #21 from Julian Aloofi   2009-07-26 
16:17:17 EDT ---
I'll set up another system in a virtual machine, using x64, and see whether I
can reproduce that error. I'd say we should hold this package as long as we
can't say it's exclusively happening to Rahul.

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 513874] New: Review Request: perl-Math-Calc-Units - Human-readable unit-aware calculator

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: perl-Math-Calc-Units - Human-readable unit-aware 
calculator

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

   Summary: Review Request: perl-Math-Calc-Units - Human-readable
unit-aware calculator
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: ru...@rubenkerkhof.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://ruben.fedorapeople.org/perl-Math-Calc-Units.spec
SRPM URL: perl-Math-Calc-Units-1.06-1.fc11.src.rpm
Description:
Math::Calc::Units is a simple calculator that keeps track of units. It
currently handles combinations of byte sizes and duration only, although
adding any other multiplicative types is easy. Any unknown type is treated
as a unique user type (with some effort to map English plurals to their
singular forms).

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 513874] Review Request: perl-Math-Calc-Units - Human-readable unit-aware calculator

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #1 from Ruben Kerkhof   2009-07-26 16:12:48 
EDT ---
Oops, srpm is here:
http://ruben.fedorapeople.org/perl-Math-Calc-Units-1.06-1.fc11.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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 513733] Review Request: entertainer - A simple media center

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #20 from Rahul Sundaram   2009-07-26 16:08:49 
EDT ---


Confirmed that the split works however the same problem with the application
not even starting. 100% reproducible. I turned off the firewall. Made sure I am
online and it makes no difference. 

Not sure you want to hold on before importing it but I request you to test more
and report the problem upstream before importing if you can. If I am the only
person experiencing it, it wouldn't matter much.

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 502227] Review Request: virtuoso-opensource - A high-performance object-relational SQL database

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Kevin Fenzi  changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #27 from Kevin Fenzi   2009-07-26 16:07:06 EDT ---
Ah, excellent. Sorry for any confusion here. 

cvs done.

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 502227] Review Request: virtuoso-opensource - A high-performance object-relational SQL database

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Kevin Kofler  changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #26 from Kevin Kofler   2009-07-26 16:00:55 
EDT ---
Lorenzo's FAS account is "arbiter".

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 510979] Review Request: phpSmug - PHP wrapper for the SmugMug API

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #11 from Kevin Fenzi   2009-07-26 15:38:19 EDT ---
That sounds fine to me... any objections?

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 435576] Review Request: ncdu - Text-based disk usage viewer

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Kevin Fenzi  changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #11 from Kevin Fenzi   2009-07-26 15:48:59 EDT ---
cvs done.

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 436637] Review Request: disktype - Detect the content format of a disk or disk image

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Kevin Fenzi  changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #21 from Kevin Fenzi   2009-07-26 15:49:24 EDT ---
cvs done.

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 220683] Review Request: rubygems - the Ruby standard for packaging ruby libraries

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Kevin Fenzi  changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #14 from Kevin Fenzi   2009-07-26 15:47:51 EDT ---
cvs done.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 508523] Review Request: miredo - Tunneling of IPv6 over UDP through NATs

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Kevin Fenzi  changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #15 from Kevin Fenzi   2009-07-26 15:48:32 EDT ---
cvs done.

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 511895] Review Request: clutter-imcontext - IMContext Framework Library for Clutter

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Kevin Fenzi  changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #14 from Kevin Fenzi   2009-07-26 15:45:54 EDT ---
cvs done.

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 513733] Review Request: entertainer - A simple media center

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Kevin Fenzi  changed:

   What|Removed |Added

 CC||ke...@tummy.com




--- Comment #19 from Kevin Fenzi   2009-07-26 15:44:06 EDT ---
Shall we hold off here on cvs until Rahul's concerns are all solved?

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 509533] Review Request: sap - A small CLI audio player

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Kevin Fenzi  changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #26 from Kevin Fenzi   2009-07-26 15:42:33 EDT ---
cvs done.

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 502227] Review Request: virtuoso-opensource - A high-performance object-relational SQL database

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Kevin Fenzi  changed:

   What|Removed |Added

 CC||ke...@tummy.com
   Flag|fedora-cvs? |




--- Comment #25 from Kevin Fenzi   2009-07-26 15:41:08 EDT ---
Humm... Lorenzo: I don't see you in the packager group. 
Is your account lvillani? Are you a packager? 

Only packagers can officially approve reviews... of course anyone is welcome to
comment.

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 513784] Review Request: lxc - Linux Resource Containers

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Kevin Fenzi  changed:

   What|Removed |Added

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




--- Comment #3 from Kevin Fenzi   2009-07-26 15:37:41 EDT ---
cvs done.

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 512263] Review Request: audtty - A ncurses based terminal client for the Audacious

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Kevin Fenzi  changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #7 from Kevin Fenzi   2009-07-26 15:39:09 EDT ---
cvs done.

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 471915] Review Request: jbossweb2 - JBoss Web Server based on Apache Tomcat

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Milos Jakubicek  changed:

   What|Removed |Added

   Flag||needinfo?(dwall...@redhat.c
   ||om)




--- Comment #34 from Milos Jakubicek   2009-07-26 15:28:59 
EDT ---
Ping...where does it stuck?

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 511895] Review Request: clutter-imcontext - IMContext Framework Library for Clutter

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Peter Robinson  changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #13 from Peter Robinson   2009-07-26 15:31:16 
EDT ---
New Package CVS Request
===
Package Name: clutter-imcontext
Short Description: IMContext Framework Library for Clutter
Owners: pbrobinson
Branches: F-11
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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 511212] Review Request: cluster-glue - reusable clustering components

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Kevin Fenzi  changed:

   What|Removed |Added

 CC||ke...@tummy.com




--- Comment #9 from Kevin Fenzi   2009-07-26 15:26:37 EDT ---
ok. Finally found some time to look at this a bit. ;) 

So, my understanding is that this is a newerversion/fork of heartbeat thats
stripped down and only has the parts pacemaker needs/wants in it. Is that
correct or am I misunderstanding? :) 

First, Fedora really would prefer to avoid Conflicts if at all possible. 
(See https://fedoraproject.org/wiki/Packaging/Conflicts). 
So, is there some way we can avoid that here? 

- Can this package just rename the dirs/files that conflict with 'cluster-glue'
or something and that way they could both be installed and pacemaker could have
a compile time option to use whichever one. (Or autodetect it based on the
files found, etc). 

- Is using this better than using heartbeat for pacemaker? Or are they the
same? If the same, perhaps we hold off on adding this to fedora for now, let
pacemaker use heartbeat and then down the road when enough folks have moved to
pacemaker submit this and obsolete heartbeat? 

- Perhaps pacemaker could fold in the needed parts of this and just use them
internally and avoid conflict with heartbeat that way?

Thoughts?

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 511895] Review Request: clutter-imcontext - IMContext Framework Library for Clutter

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Orcan 'oget' Ogetbil  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #12 from Orcan 'oget' Ogetbil   2009-07-26 
15:25:11 EDT ---
Thanks! Imagine all packages take 3 seconds longer to build. These would add up
to total build times that can't be ignored. I believe that it's good to keep
things at minimum (e.g. remove unnecessary BR's, commands etc).

I think everything is good now


This package (clutter-imcontext) is APPROVED by oget


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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 473835] Review Request: autoarchive - Simple backup tool

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Jussi Lehtola  changed:

   What|Removed |Added

 CC||jussi.leht...@iki.fi




--- Comment #5 from Jussi Lehtola   2009-07-26 15:11:46 
EDT ---
A couple of additional notes - I just sponsored Julian.


(In reply to comment #4)
> Notes:
> -Your package should BuildRequire: python-devel instead of python  

Actually, python-setuptools-devel pulls in python-devel, so that's not a
problem. But it's better to BR it exclusively.

**

(The ./ in front of setup.py is unnecessary when run via python.)

**

I suggest using autoarchive instead of %{name} in the %files section for
consistency.

**

Requires: lzma is not enough IMHO. The package seems to need also tar, gzip and
bzip2.

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 499476] Review Request: orbited - A browser(javascript)->tcp bridge

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Brennan Ashton  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||bash...@brennanashton.com
 AssignedTo|nob...@fedoraproject.org|bash...@brennanashton.com
   Flag||needinfo?(tcall...@redhat.c
   ||om)




--- Comment #3 from Brennan Ashton   2009-07-26 
14:59:46 EDT ---
Ping?
could you please add the daemon and config file.
http://www.orbited.org/wiki/Deployment

Once I see this activity to verify this is not stale I will start 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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 468226] Review Request: python-tw-jquery - ToscaWidgets wrapping for jQuery

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Brennan Ashton  changed:

   What|Removed |Added

 CC||bash...@brennanashton.com




--- Comment #7 from Brennan Ashton   2009-07-26 
14:53:10 EDT ---
ping?

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 479021] Review Request: python-repoze-what-plugins-sql - The repoze.what SQL plugin

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Brennan Ashton  changed:

   What|Removed |Added

 CC||bash...@brennanashton.com




--- Comment #7 from Brennan Ashton   2009-07-26 
14:49:32 EDT ---
can this please get pushed into testing

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 503592] Review Request: python-sprox - A package for creation of web widgets directly from database schema

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Brennan Ashton  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||bash...@brennanashton.com
 AssignedTo|nob...@fedoraproject.org|bash...@brennanashton.com
   Flag||fedora-review+




--- Comment #6 from Brennan Ashton   2009-07-26 
14:47:09 EDT ---
this also looks clean to me.  I see no need for those tests,as long as they are
run at build. Some times it is nice to have the tests for example code, but I
do not see a need for that in this case.

Why did these reviews get dropped this late in the review stage?

I see no reason to APPROVE this package.

Go ahead for CVS.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 513869] New: Review Request: perl-Beanstalk-Client - Client class to talk to a beanstalkd server

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: perl-Beanstalk-Client - Client class to talk to a 
beanstalkd server

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

   Summary: Review Request: perl-Beanstalk-Client - Client class
to talk to a beanstalkd server
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: ru...@rubenkerkhof.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://ruben.fedorapeople.org/perl-Beanstalk-Client.spec
SRPM URL:
http://ruben.fedorapeople.org/perl-Beanstalk-Client-1.05-1.fc11.src.rpm
Description:
Beanstalk::Client provides a Perl API of protocol version 1.0 to the
beanstalkd server, a fast, general-purpose, in-memory workqueue service by
Keith Rarick.

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 512217] Review Request: geany-plugins - A bundle of plugins for Geany

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Jochen Schmitt  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #40 from Jochen Schmitt   2009-07-26 
14:41:40 EDT ---
Ok, I have tanke a last look on the package and it's look nice, so the package
is APPROVED and I will sponsored you.

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 473835] Review Request: autoarchive - Simple backup tool

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Julian Aloofi  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||julian.fed...@googlemail.co
   ||m
 AssignedTo|nob...@fedoraproject.org|julian.fed...@googlemail.co
   ||m
   Flag||fedora-review?




--- Comment #4 from Julian Aloofi   2009-07-26 
14:38:35 EDT ---
rpmlint output of all files is clean:
2 packages and 1 specfiles checked; 0 errors, 0 warnings.

MUST: The package does not yet exist in Fedora. The Review Request is not a
duplicate. OK
MUST: The spec file for the package is legible and macros are used
consistently. 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}. 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. NEEDSWORK
-Please add COPYING to the doc section

MUST: The sources used to build the package must match the upstream source, as
provided in the spec URL. OK
MUST: The package MUST successfully compile and build into binary rpms. OK
MUST: The spec file MUST handle locales properly. N/A
MUST: Optflags are used and time stamps preserved. OK
MUST: Packages containing shared library files must call ldconfig. N/A
MUST: A package must own all directories that it creates or require the package
that owns the directory. OK
MUST: Files only listed once in %files listings. OK
MUST: Debuginfo package is complete. N/A
MUST: Permissions on files must be set properly. OK
MUST: Clean section exists. OK
MUST: Large documentation files must go in a -doc subpackage. N/A
MUST: All relevant items are included in %doc. Items in %doc do not affect
runtime of application. NEEDSWORK
-As I said before, you need to include COPYING

MUST: Header files must be in a -devel package. N/A
MUST: Static libraries must be in a -static package. N/A
MUST: Packages containing pkgconfig(.pc) files must 'Requires: pkgconfig'. N/A
MUST: If a package contains library files with a suffix then library files
ending in .so must go in a -devel package. N/A
MUST: In the vast majority of cases, devel packages must require the base
package using a fully versioned dependency. N/A
MUST: Packages does not contain any .la libtool archives. N/A
MUST: Desktop files are installed properly. N/A
MUST: No file conflicts with other packages and no general names. OK
MUST: Buildroot cleaned before install. OK
SHOULD: %{?dist} tag is used in release. OK
SHOULD: If the package does not include license text(s) as separate files from
upstream, the packager should query upstream to include it. OK
SHOULD: The package builds in mock. OK 


Notes:
-Your package should BuildRequire: python-devel instead of python

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 503590] Review Request: python-tgext-admin - Admin Controller add-on for basic TG identity model

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Brennan Ashton  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||bash...@brennanashton.com
 AssignedTo|nob...@fedoraproject.org|bash...@brennanashton.com
   Flag||fedora-review+




--- Comment #10 from Brennan Ashton   2009-07-26 
14:38:24 EDT ---
I am not sure why Parag dropped this, but it looks good to me. I confirm the
changes fix the issues noted, and APPROVE. Go ahead with CVS 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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 436637] Review Request: disktype - Detect the content format of a disk or disk image

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Richard Fearn  changed:

   What|Removed |Added

   Flag|fedora-cvs+ |fedora-cvs?




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 436637] Review Request: disktype - Detect the content format of a disk or disk image

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #20 from Richard Fearn   2009-07-26 
14:34:10 EDT ---
Package Change Request
==
Package Name: disktype
New Branches: EL-5
Owners: richardfearn

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 435576] Review Request: ncdu - Text-based disk usage viewer

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Richard Fearn  changed:

   What|Removed |Added

   Flag|fedora-cvs+ |fedora-cvs?




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 435576] Review Request: ncdu - Text-based disk usage viewer

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #10 from Richard Fearn   2009-07-26 
14:34:09 EDT ---
Package Change Request
==
Package Name: ncdu
New Branches: EL-5
Owners: richardfearn

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 491790] Review Request: mediawiki-SpamBlacklist - An extension to provide a Spam Blacklist management system

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #4 from John Guthrie   2009-07-26 
13:12:28 EDT ---
Alas, I have not received any kind of response from the author as yet.  I will
try to contact him again.  If I still don't get a response, then we'll probably
have to close the review request.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 511966] Review Request: zbar - bar code reader

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #14 from Mamoru Tasaka   2009-07-26 
12:57:43 EDT ---
For -3:

* SourceURL
  - Missed in the previous comment, however for sourceforge hosted
source, please follow:
https://fedoraproject.org/wiki/Packaging/SourceURL#Sourceforge.net

* Requires / BuildRequires
  - "BuildRequires: libX11-devel" is redundant because
gtk2-devel requires this (and this is also in BuildRequires).

  https://fedoraproject.org/wiki/Packaging/Guidelines#Requires
  - Now "Requires: ImageMagick-c++" on the main package is redundant
(and should be removed) as the dependencies for libraries automatically
detected by rpmbuild should pull in this dependency:

  - And again "Requires: gtk2" on -gtk subpackage, and "Requires: qt >= 4"
on -qt subpackage are redundant (and should be removed).

  - "Requires: %{name} = %{version}-%{release}" on -pygtk subpackage
is redundant because -pygtk subpackage requires -gtk subpackage and
-gtk subpackage requires main package.

* Provides
  - "Provides: libzbar-static" on -devel subpackage must be removed
because no static archives are now included (and same on other
devel related subpackages)

* Directory ownership issue
  - The %files entry
-
%files
%{_includedir}/zbar
-
contains the directory %_includedir/zbar itself and all 
files/directories/etc under this directory (please check
what $ rpm -ql zbar-devel returns), while
-
%files
%dir %{_includedir}/zbar
-
contains the directory %_includedir/zbar only.

* Macros in %changelog
  - In changelog, please use %% instead of % to keep macros from
being expanded ($ rpm -q --changelog zbar
shows what is actually occuring)

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 507106] Review Request: msp430-libc - C library for use with GCC on Texas Instruments MSP430 microcontrollers

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #4 from Robert Spanton   2009-07-26 12:52:18 
EDT ---
Patches are now upstream.  Reflected in this specfile:

Spec URL: http://users.ecs.soton.ac.uk/rds/rpm/mspgcc/msp430-libc.spec
SRPM URL:
http://users.ecs.soton.ac.uk/rds/rpm/mspgcc/msp430-libc-0-3.20090726cvs.fc11.src.rpm

Rob

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 507030] Review Request: pymunk - Python wrapper for the chipmunk 2D physics engine

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #10 from Robert Spanton   2009-07-26 12:53:12 
EDT ---
I hope to get time for this tomorrow.

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 510979] Review Request: phpSmug - PHP wrapper for the SmugMug API

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #10 from Paul W. Frields   2009-07-26 12:32:53 
EDT ---
http://pfrields.fedorapeople.org/packages/SPECS/phpSmug.spec
http://pfrields.fedorapeople.org/packages/SRPMS/php-phpSmug-2.0.2-3.fc11.src.rpm

Name has been changed as Jason noted.  I'll figure out how to get phpFlickr and
phpLightweightPicasaAPI fixed appropriately.  Not sure if this package needs
re-review, but otherwise it's ready for CVS and I can set up a new comment here
for that if that's OK.

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 507223] Review Request: dalston - Moblin System Information Icons

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #13 from Peter Robinson   2009-07-26 12:30:28 
EDT ---
New upstream release. 

SPEC: http://pbrobinson.fedorapeople.org/dalston.spec
SRPM: http://pbrobinson.fedorapeople.org/dalston-0.0.29-1.fc11.src.rpm
koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=1523479

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 507223] Review Request: dalston - Moblin System Information Icons

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #12 from Peter Robinson   2009-07-26 12:28:19 
EDT ---
(In reply to comment #11)
> New upstream release
> 
> SRPM: http://pbrobinson.fedorapeople.org/bisho-0.10.7-1.fc11.src.rpm
> koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=1502659  

Ignore this. Wrong bug.

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 511895] Review Request: clutter-imcontext - IMContext Framework Library for Clutter

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #11 from Peter Robinson   2009-07-26 12:23:42 
EDT ---
Added in post and tested "sed -i '/configure/d' autogen.sh"

koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=1523493
SRPM: http://pbrobinson.fedorapeople.org/clutter-imcontext-0.1.2-4.fc11.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.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 510979] Review Request: phpSmug - PHP wrapper for the SmugMug API

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #9 from Paul W. Frields   2009-07-26 12:17:28 
EDT ---
I'm going to fix this up as php-phpSmug which should satisfy the current
guidelines and at least make the package searchable as expected.

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 469470] Review Request: mausezahn - A fast versatile packet generator

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #33 from Vivek Shah   2009-07-26 12:03:01 EDT 
---
Hi Mamoru,
  With Fedora specific flags turned on, I can see there are buffer
overflow issues in the program. I have notified upstream about it. Will be
patching them up and then update the srpm in a couple of days. It needs to
fixed I believe before it goes in the repos more so since the program needs
root privileges to run.

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 506825] Review Request: bickley - A meta data management API and framework

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Bug 506825 depends on bug 507392, which changed state.

Bug 507392 Summary: update to clutter-gst 0.9.0
https://bugzilla.redhat.com/show_bug.cgi?id=507392

   What|Old Value   |New Value

 Status|NEW |CLOSED
 Resolution||RAWHIDE



-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 506825] Review Request: bickley - A meta data management API and framework

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #3 from Peter Robinson   2009-07-26 11:50:40 
EDT ---
Updated to the latest upstream release.

SPEC: http://pbrobinson.fedorapeople.org/bickley.spec
SRPM: http://pbrobinson.fedorapeople.org/bickley-0.4.3-1.fc11.src.rpm
koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=1523364

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 513848] New: Review Request: fbzx - A ZX Spectrum emulator for FrameBuffer

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: fbzx - A ZX Spectrum emulator for FrameBuffer

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

   Summary: Review Request: fbzx - A ZX Spectrum emulator for
FrameBuffer
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: musur...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://www.webalice.it/musuruan/RPMS/reviews/fbzx.spec
SRPM URL: http://www.webalice.it/musuruan/RPMS/reviews/fbzx-2.1b-1.fc10.src.rpm
Description: 
FBZX is a Sinclair Spectrum emulator, designed to work at full screen using 
the FrameBuffer or under X-Windows.

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 470696] Review Request: rubygem-passenger - Passenger Ruby on Rails deployment system

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #52 from Hongli Lai   2009-07-26 10:09:27 EDT 
---
I'll give a few quick hints then.
- You see I modified the boost::thread class. The modifications are necessary
in order to implement oxt::thread.
- Some exception classes like thread_resource_error are modified to inherit
from oxt::tracable_exception.

oxt::thread and oxt::tracable_exception and their purposes are documented in
their respective header files.

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 470696] Review Request: rubygem-passenger - Passenger Ruby on Rails deployment system

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #51 from Jeroen van Meeuwen   2009-07-26 
07:57:33 EDT ---
I can wait until October, but I do want to understand in full what changes
we're talking about so that we may actually get some work done in October ;-)

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 513733] Review Request: entertainer - A simple media center

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #18 from Julian Aloofi   2009-07-26 
07:49:02 EDT ---
Splitted up, rebuilt and checked in mock for F10 and F11:
SPEC: http://julian.fedorapeople.org/entertainer/entertainer.spec
SRPM:
http://julian.fedorapeople.org/entertainer/entertainer-0.4.2-4.fc11.src.rpm

Regarding the socket error: Are you online when running entertainer? There seem
to be some issues when not connected to the internet, see upstream bug report:
https://bugs.launchpad.net/entertainer/+bug/243309
Could also be a firewall problem.

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 220683] Review Request: rubygems - the Ruby standard for packaging ruby libraries

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Jeroen van Meeuwen  changed:

   What|Removed |Added

 CC||kana...@kanarip.com
   Flag|fedora-cvs+ |fedora-cvs?




--- Comment #13 from Jeroen van Meeuwen   2009-07-26 
06:58:09 EDT ---
Package Change Request
==
Package Name: rubygems
Owner: kanarip
[New Branches: ] EL-4

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 470696] Review Request: rubygem-passenger - Passenger Ruby on Rails deployment system

2009-07-26 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #50 from Hongli Lai   2009-07-26 06:54:31 EDT 
---
Sorry, I haven't had the time to post a proper reply.

Please feel free to go renegade if you can't wait until October.

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


  1   2   >