[Bug 516698] Review Request: fpaste - A simple tool for pasting info onto fpaste.org

2009-08-13 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=516698





--- Comment #8 from Ankur Sinha sanjay.an...@gmail.com  2009-08-13 01:59:33 
EDT ---
hi,

new spec: 
http://ankursinha.fedorapeople.org/fpaste/fpaste.spec

new srpm:
http://ankursinha.fedorapeople.org/fpaste/fpaste-0.3.2-2.fc11.src.rpm


rest of the mock build stuff at :

http://ankursinha.fedorapeople.org/fpaste/

regards,

Ankur

-- 
Configure 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 516698] Review Request: fpaste - A simple tool for pasting info onto fpaste.org

2009-08-13 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=516698


Rahul Sundaram sunda...@redhat.com changed:

   What|Removed |Added

 AssignedTo|nob...@fedoraproject.org|sunda...@redhat.com
   Flag||fedora-review+




--- Comment #9 from Rahul Sundaram sunda...@redhat.com  2009-08-13 02:04:33 
EDT ---
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 516698] Review Request: fpaste - A simple tool for pasting info onto fpaste.org

2009-08-13 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=516698


Ankur Sinha sanjay.an...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs+




--- Comment #10 from Ankur Sinha sanjay.an...@gmail.com  2009-08-13 02:27:19 
EDT ---
New Package CVS Request
===
Package Name: fpaste
Short Description: A simple tool for pasting info onto fpaste.org
Owners: ankursinha
Branches: F-10 F-11

-- 
Configure 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 470293] Review Request: sblim-cmpi-network - SBLIM Network Instrumentation

2009-08-13 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=470293


srinivas srinivas_ramana...@dell.com changed:

   What|Removed |Added

 CC||srinivas_ramana...@dell.com




--- Comment #10 from srinivas srinivas_ramana...@dell.com  2009-08-13 
03:06:24 EDT ---
Implemented the changes suggested.
Final rpmlint output:
# rpmlint sblim-cmpi-network.spec
0 packages and 1 specfiles checked; 0 errors, 0 warnings.
# rpmlint ../SRPMS/sblim-cmpi-network-1.3.8-1.fc11.src.rpm
1 packages and 0 specfiles checked; 0 errors, 0 warnings.
# rpmlint ../RPMS/x86_64/sblim-cmpi-network-*
sblim-cmpi-network-devel.x86_64: W: no-documentation
sblim-cmpi-network-test.x86_64: W: no-documentation
4 packages and 0 specfiles checked; 0 errors, 2 warnings.

Fixed these warnings. 

Spec URL:

http://linux.dell.com/files/fedora/sblim-cmpi-network/sblim-cmpi-network.spec

SRPM URL:

http://linux.dell.com/files/fedora/sblim-cmpi-network/sblim-cmpi-network-1.3.8-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 455507] Review Request: jsmin - JavaScript minifier

2009-08-13 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=455507


Christopher Curran ccur...@redhat.com changed:

   What|Removed |Added

 CC||ccur...@redhat.com




--- Comment #7 from Christopher Curran ccur...@redhat.com  2009-08-13 
03:10:04 EDT ---
IANAL but given the various interpretations of good and evil I think this is
not an issue. All we need is to find an appropriate thesis which shows good and
evil do not exist or that good and evil are overlapping variables. If good can
be evil in context and the equivalent event evil not good then there is no
problem as the license negates itself. I say that it is moral to accept his
software as sufficiently free and save discussions of morality for the pub,
where real philosophy occurs.

Chris

-- 
Configure 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 514342] Review Request: rubygem-fattr - Fatter attribute for Ruby

2009-08-13 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=514342


Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #4 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-08-13 
03:58:46 EDT ---
Approving, provided:

- Please add the following text file as %doc:
  http://codeforpeople.com/lib/license.txt
  (In the spec file add this with _full URL_ like
--
URL:http://rubyforge.org/projects/codeforpeople/
Source0:http://gems.rubyforge.org/gems/%{gemname}-%{version}.gem
Source1:http://codeforpeople.com/lib/license.txt
...
...
%install
rm -rf %{buildroot}
mkdir -p %{buildroot}%{gemdir}
...
...
install -p -m 644 %{SOURCE1} %{buildroot}%{geminstdir}/
...
...
%files
...
%doc %{geminstdir}/license

--

- If we say the license is the same as Ruby's, on Fedora this
  means it is licensed under GPLv2 or Ruby (on Fedora Ruby license
  means the lines below or the conditions below: of
  http://www.ruby-lang.org/en/LICENSE.txt), so please
  change the license tag to GPLv2 or Ruby (not GPLv2+ or)

-
This package (rubygem-fattr) is APPROVED by mtasaka
-

-- 
Configure 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 516578] Review Request: hyphen-fo - Faroese hyphenation rules

2009-08-13 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=516578


Caolan McNamara caol...@redhat.com 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 504424] Review Request: rubygem-json - A JSON implementation in Ruby

2009-08-13 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=504424


Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #12 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-08-13 
04:35:45 EDT ---
Okay.


  This package (rubygem-json) is APPROVED by mtasaka


-- 
Configure 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 516336] Review Request: mmapper - Graphical MUME mapper

2009-08-13 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=516336





--- Comment #8 from Fedora Update System upda...@fedoraproject.org  
2009-08-13 04:41:20 EDT ---
mmapper-2.0.4-2.final2.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/mmapper-2.0.4-2.final2.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 516336] Review Request: mmapper - Graphical MUME mapper

2009-08-13 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=516336





--- Comment #7 from Fedora Update System upda...@fedoraproject.org  
2009-08-13 04:41:15 EDT ---
mmapper-2.0.4-2.final2.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/mmapper-2.0.4-2.final2.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 226069] Merge Review: libXevie

2009-08-13 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=226069


Parag AN(पराग) panem...@gmail.com changed:

   What|Removed |Added

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




-- 
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 226068] Merge Review: libXdmcp

2009-08-13 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=226068


Parag AN(पराग) panem...@gmail.com changed:

   What|Removed |Added

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




-- 
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 226066] Merge Review: libXcursor

2009-08-13 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=226066


Parag AN(पराग) panem...@gmail.com changed:

   What|Removed |Added

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




-- 
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 226071] Merge Review: libXfixes

2009-08-13 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=226071


Parag AN(पराग) panem...@gmail.com changed:

   What|Removed |Added

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




-- 
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 226067] Merge Review: libXdamage

2009-08-13 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=226067


Parag AN(पराग) panem...@gmail.com changed:

   What|Removed |Added

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




-- 
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 226086] Merge Review: libXres

2009-08-13 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=226086


Parag AN(पराग) panem...@gmail.com changed:

   What|Removed |Added

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




-- 
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 512505] Review Request: php-ezc-AuthenticationDatabaseTiein - eZ Components AuthenticationDatabaseTiein

2009-08-13 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=512505


Guillaume Kulakowski llaum...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #3 from Guillaume Kulakowski llaum...@gmail.com  2009-08-13 
05:11:39 EDT ---
New Package CVS Request
===
Package Name: php-ezc-AuthenticationDatabaseTiein
Short Description: Provides a Database filter for the Authentication component
Owners: llaumgui
Branches: F-10 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 512506] Review Request: php-ezc-Feed - eZ Components Feed

2009-08-13 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=512506


Guillaume Kulakowski llaum...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #2 from Guillaume Kulakowski llaum...@gmail.com  2009-08-13 
05:12:50 EDT ---
New Package CVS Request
===
Package Name: php-ezc-Feed
Short Description: This component handles parsing and creating RSS1, RSS2 and
ATOM feeds
Owners: llaumgui
Branches: F-10 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 226065] Merge Review: libXcomposite

2009-08-13 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=226065


Parag AN(पराग) panem...@gmail.com changed:

   What|Removed |Added

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




-- 
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 510943] Review Request: ipplan - Web-based IP address manager and tracker

2009-08-13 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=510943





--- Comment #9 from Andrew Colin Kissa and...@topdog.za.net  2009-08-13 
06:46:30 EDT ---

Gary,

I have submitted a request to maintain php-adodb for EPEL, I will work on
bumping ipplan to 4.92 soon.

If you would be so kind as to look at
https://bugzilla.redhat.com/show_bug.cgi?id=513954 if you can, which is holding
up php-layers-menu which in turn will hold up ipplan.

Thanks in advance

-- 
Configure 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-08-13 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 #10 from Peter Robinson pbrobin...@gmail.com  2009-08-13 06:59:25 
EDT ---
(In reply to comment #7)
 There is an issue with bickley-0.4.pc on 64 bit:
 
 prefix=/usr
 exec_prefix=${prefix}
 libdir=${exec_prefix}/lib 
 includedir=${prefix}/include
 
 libdir should be libdir64  

The patch I provided has now been committed to upstream git. Will be in the
next release.

-- 
Configure 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 515143] Review Request: mtkbabel - Program to Operate the i-Blue 747 GPS Data Logger

2009-08-13 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=515143





--- Comment #12 from Fedora Update System upda...@fedoraproject.org  
2009-08-13 07:02:12 EDT ---
mtkbabel-0.8-1.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/mtkbabel-0.8-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 515143] Review Request: mtkbabel - Program to Operate the i-Blue 747 GPS Data Logger

2009-08-13 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=515143





--- Comment #11 from Fedora Update System upda...@fedoraproject.org  
2009-08-13 07:02:06 EDT ---
mtkbabel-0.8-1.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/mtkbabel-0.8-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 515081] Review Request: R-affy - Methods for Affymetrix Oligonucleotide Arrays

2009-08-13 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=515081


Bug 515081 depends on bug 515080, which changed state.

Bug 515080 Summary: Review Request: R-preprocessCore - A collection of 
pre-processing functions
https://bugzilla.redhat.com/show_bug.cgi?id=515080

   What|Old Value   |New Value

 Status|ASSIGNED|ON_QA
 Resolution||ERRATA
 Status|ON_QA   |CLOSED



--- Comment #7 from Pierre-YvesChibon pin...@pingoured.fr  2009-08-13 
07:11:42 EDT ---
License tag has been corrected in the cvs:
http://fgc.lsi.umich.edu/cgi-bin/blosxom3.cgi/affy 

R-preprocessCore is in stable now :-)

-- 
Configure 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 516466] Review Request: sys_basher - multi-threaded hardware tester

2009-08-13 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=516466





--- Comment #43 from Joshua Rosen bjro...@polybus.com  2009-08-13 07:18:48 
EDT ---
Hoefully I've gotten the spec right this time. Let me know what you want me to
look at.

-- 
Configure 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 516466] Review Request: sys_basher - multi-threaded hardware tester

2009-08-13 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=516466





--- Comment #45 from Jussi Lehtola jussi.leht...@iki.fi  2009-08-13 07:46:24 
EDT ---
Ugh, and the BuildRoot tag is wrong. Use
%(mktemp -ud %{_tmppath}/%{name}-%{version}-%{release}-XX)

-- 
Configure 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 516466] Review Request: sys_basher - multi-threaded hardware tester

2009-08-13 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=516466





--- Comment #44 from Jussi Lehtola jussi.leht...@iki.fi  2009-08-13 07:45:42 
EDT ---
OK: rpmlint output is clean

NEEDSWORK: source does not match upstream:
$ md5sum sys_basher-1.1.20.tar.gz ../SOURCES/sys_basher-1.1.20.tar.gz 
729bdb57a9477b9d3c0fdef6bfc31ba9  sys_basher-1.1.20.tar.gz
18b02243dde1bef9560016a947647728  ../SOURCES/sys_basher-1.1.20.tar.gz

**

Regenerate the SRPM using the correct tarball, or put the correct tarball on
the web site.

Do the informal reviews ASAP, give links to them here.

-- 
Configure 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 504424] Review Request: rubygem-json - A JSON implementation in Ruby

2009-08-13 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=504424


Xavier Lamien lxt...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #13 from Xavier Lamien lxt...@gmail.com  2009-08-13 07:45:45 EDT 
---
Thanks Mamoru.


New Package CVS Request
===
Package Name:   rubygem-json
Short Description:  A JSON implementation in Ruby
Owners: laxathom
Branches:   F-10 F-11 EL-5
InitialCC:  (nobody)

-- 
Configure 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 517191] Review Request: php-symfony-symfony - Open-Source PHP Web Framework

2009-08-13 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=517191


Andrew Colin Kissa and...@topdog.za.net changed:

   What|Removed |Added

 CC||and...@topdog.za.net




--- Comment #2 from Andrew Colin Kissa and...@topdog.za.net  2009-08-13 
07:52:35 EDT ---

I think you can fix the .htaccess errors by using an apache config file
%{_sysconfdir}/httpd/conf.d/symfony for all the configurations provided by the
.htaccess. This is recommended because some sites will not allow overriding
configurations using .htaccess 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 517300] New: Rename Request: ccss - A simple api for CSS Stylesheets

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

Summary: Rename Request: ccss - A simple api for CSS Stylesheets

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

   Summary: Rename Request: ccss - A simple api for CSS
Stylesheets
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: pbrobin...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


SPEC: http://pbrobinson.fedorapeople.org/ccss.spec
SRPM: http://pbrobinson.fedorapeople.org/ccss-0.5.0-1.fc11.src.rpm

Description:
Libccss offers a simple API to

* Parse CSS stylesheets.
* Query for style configurations on a user-provided document
  representation.
* Draw query results onto cairo surfaces.

Original package review (from 2 months ago) for libccss is:
https://bugzilla.redhat.com/show_bug.cgi?id=506712

Announcement of version 0.5.0 including the dropping of lib from the naming:
http://lwn.net/Articles/346658/

The above package is very similar to the previous package with some updates to
current packaging recommendations and splitting out the developers docs to a
-docs noarch sub package and the appropriate provides/obsoletes.

-- 
Configure 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 517191] Review Request: php-symfony-symfony - Open-Source PHP Web Framework

2009-08-13 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=517191





--- Comment #3 from Christof Damian chris...@damian.net  2009-08-13 08:38:32 
EDT ---
I don't think they need to be in the Apache configuration, because they are
either related to tests:

./test/functional/fixtures/project/web/.htaccess
./lib/plugins/sfDoctrinePlugin/test/functional/fixtures/web/.htaccess
./lib/plugins/sfPropelPlugin/test/functional/fixtures/web/.htaccess
./lib/plugins/sfCompat10Plugin/test/functional/fixtures/web/.htaccess

or are used on project creation as a template.

./lib/task/generator/skeleton/project/web/.htaccess

The symfony library directory itself is usually not accessible from the web
anyway.

-- 
Configure 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 516547] Review Request: Django-south - Intelligent schema migrations for Django apps

2009-08-13 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=516547





--- Comment #5 from Diego Búrigo Zacarão dieg...@gmail.com  2009-08-13 
08:42:28 EDT ---
Done!

SPEC: http://diegobz.fedorapeople.org/packages/Django-south/Django-south.spec
SRPM:
http://diegobz.fedorapeople.org/packages/Django-south/Django-south-0.6-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 515308] Review Request: usbmuxd - Daemon for communicating with Apple's iPod Touch and iPhone

2009-08-13 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=515308


Peter Robinson pbrobin...@gmail.com changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution||RAWHIDE




--- Comment #12 from Peter Robinson pbrobin...@gmail.com  2009-08-13 08:42:19 
EDT ---
And now in 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 517191] Review Request: php-symfony-symfony - Open-Source PHP Web Framework

2009-08-13 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=517191





--- Comment #4 from Andrew Colin Kissa and...@topdog.za.net  2009-08-13 
08:45:35 EDT ---

Oh okay thats fine then.

-- 
Configure 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 517300] Rename Request: ccss - A simple api for CSS Stylesheets

2009-08-13 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=517300


Yanko Kaneti yan...@declera.com changed:

   What|Removed |Added

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




--- Comment #1 from Yanko Kaneti yan...@declera.com  2009-08-13 09:14:38 EDT 
---
Diff looks good. Builds in mock. Provides/Obsoletes work as expected. 
APPROVED again

-- 
Configure 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 517300] Rename Request: ccss - A simple api for CSS Stylesheets

2009-08-13 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=517300


Peter Robinson pbrobin...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #2 from Peter Robinson pbrobin...@gmail.com  2009-08-13 09:28:40 
EDT ---
Thanks!

New Package CVS Request
===
Package Name: ccss
Short Description: A simple api for CSS Stylesheets
Owners: pbrobinson
Branches: F-11 F-10
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 510734] Review Request: x11vnc - VNC server for the current X11 session

2009-08-13 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=510734





--- Comment #50 from Pavel Alexeev (aka Pahan-Hubbitus) pa...@hubbitus.info  
2009-08-13 09:42:07 EDT ---
Well, I got answer from Karl J. Runge.

Since x11vnc 0.9.4 the full java source is available:
 http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=500970
So, I think we can redistribute its in src.rpm and delete binaries at built
time.

Also I ask him to devide x11vnc to separate packages with viewer applets - he
answer what not interested in doing that upstream.

I think we may leave all as is now.

Tom, thank you for the check and lifting FE-legal.

-- 
Configure 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 455507] Review Request: jsmin - JavaScript minifier

2009-08-13 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=455507





--- Comment #8 from Tom spot Callaway tcall...@redhat.com  2009-08-13 
10:10:06 EDT ---
Red Hat Legal (who are lawyers) disagree.

Its a bad license, with unenforceable clauses.

-- 
Configure 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 517338] New: Review Request: irssi-xmpp is an irssi plugin to connect to the Jabber network

2009-08-13 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: irssi-xmpp is an irssi plugin to connect to the Jabber 
network

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

   Summary: Review Request: irssi-xmpp is an irssi plugin to
connect to the Jabber network
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: npajk...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


SPEC: http://www.package-review.xf.cz/irssi-xmpp.spec
SRPM: http://package-review.xf.cz/irssi-xmpp-0.50-1.fc12.src.rpm
PATCH: http://www.package-review.xf.cz/irssi-xmpp-0.50-config.patch

Description:
irssi-xmpp is an irssi plugin to connect to the Jabber network

*  Sending and receiving messages in irssi's query windows
* A roster with contact  resource tracking (contact list)
* Contact management (add, remove, manage subscriptions)
* Tab completion of commands, JIDs and resources
* Many extensions supported (XEP) including Multi-User Chat (MUC)
* Support for multiple accounts
* Unicode support (UTF-8)
* SSL support
* ...

-- 
Configure 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 510734] Review Request: x11vnc - VNC server for the current X11 session

2009-08-13 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=510734





--- Comment #51 from Christian Krause c...@plauener.de  2009-08-13 11:08:19 
EDT ---
Pavel, thanks for asking upstream regarding this issue.

Since it looks like that most of the issues are addressed, I'll do the complete
review on the weekend. Please apologize the delay.

-- 
Configure 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 485638] Review Request: dmenu - Dynamic X menu

2009-08-13 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=485638


Rahul Sundaram sunda...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-review?




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

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


[Bug 485638] Review Request: dmenu - Dynamic X menu

2009-08-13 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=485638


Rahul Sundaram sunda...@redhat.com changed:

   What|Removed |Added

 CC||sunda...@redhat.com
 AssignedTo|nob...@fedoraproject.org|sunda...@redhat.com




--- Comment #13 from Rahul Sundaram sunda...@redhat.com  2009-08-13 11:27:12 
EDT ---

Remove the buildroot definition and you dont need to clean it in %install
section anymore.

https://fedoraproject.org/wiki/Packaging/Guidelines#BuildRoot_tag

Add a comment explaining what the patch does and it's upstream status

https://fedoraproject.org/wiki/Packaging/Guidelines#All_patches_should_have_an_upstream_bug_link_or_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 516466] Review Request: sys_basher - multi-threaded hardware tester

2009-08-13 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=516466





--- Comment #46 from Joshua Rosen bjro...@polybus.com  2009-08-13 11:25:57 
EDT ---
http://www.polybus.com/sys_basher/sys_basher.spec
http://www.polybus.com/sys_basher/sys_basher-1.1.21-1.fc11.src.rpm
http://www.polybus.com/sys_basher/sys_basher-1.1.21-1.fc11.x86_64.rpm

I've created a script that runs rpmlint, md5sums and spectool so I shouldn't
have anymore problems with 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 485638] Review Request: dmenu - Dynamic X menu

2009-08-13 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=485638





--- Comment #14 from Till Maas opensou...@till.name  2009-08-13 11:40:32 EDT 
---
(In reply to comment #13)
 Remove the buildroot definition and you dont need to clean it in %install
 section anymore.
 
 https://fedoraproject.org/wiki/Packaging/Guidelines#BuildRoot_tag

To be more precise: It's is only not needed, if the package is not going into
EPEL 5 or 4.

-- 
Configure 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 517366] New: Review Request: emacs-haskell-mode - Haskell editing mode for Emacs

2009-08-13 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: emacs-haskell-mode -  Haskell editing mode for Emacs

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

   Summary: Review Request: emacs-haskell-mode -  Haskell editing
mode for Emacs
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: saga...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://sagarun.fedorapeople.org/SPECS/emacs-haskell-mode.spec
SRPM URL:
http://sagarun.fedorapeople.org/SRPMS/emacs-haskell-mode-2.4-1.fc11.src.rpm
Description: 
This package adds a Haskell major mode to Emacs.  The Haskell mode
supports font locking, declaration scanning, documentation,
indentation, and interaction with Hugs and GHCi.

Links to koji builds:
Fedora 10:  http://koji.fedoraproject.org/koji/taskinfo?taskID=1603637
Fedora 11:  http://koji.fedoraproject.org/koji/taskinfo?taskID=1603639

Note:This is my first package.

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

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


[Bug 517183] Review Request: mipv6-daemon - IPv6 Mobility Daemon

2009-08-13 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=517183


Jarod Wilson jwil...@redhat.com changed:

   What|Removed |Added

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




--- Comment #1 from Jarod Wilson jwil...@redhat.com  2009-08-13 12:14:59 EDT 
---
I've started reviewing the package, have a few spec changes to recommend, will
get some more details in here after lunch...

-- 
Configure 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 517366] Review Request: emacs-haskell-mode - Haskell editing mode for Emacs

2009-08-13 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=517366


Arun SAG saga...@gmail.com changed:

   What|Removed |Added

 CC||saga...@gmail.com
Version|rawhide |11
 Blocks||177841(FE-NEEDSPONSOR)




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

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


[Bug 507695] Review Request: python-sqlalchemy0.5 - Modular and flexible ORM library for python

2009-08-13 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=507695


Luke Macken lmac...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #16 from Luke Macken lmac...@redhat.com  2009-08-13 12:42:04 EDT 
---
New Package CVS Request
===
Package Name: python-sqlalchemy0.5
Short Description: Modular and flexible ORM library for python
Owners: lmacken, toshio
Branches: F-10 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 516466] Review Request: sys_basher - multi-threaded hardware tester

2009-08-13 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=516466





--- Comment #47 from Jussi Lehtola jussi.leht...@iki.fi  2009-08-13 12:47:52 
EDT ---
OK, I think this package is good to go now.

But I won't approve it formally yet, you have to do the informal reviews first
so I can sponsor 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 517366] Review Request: emacs-haskell-mode - Haskell editing mode for Emacs

2009-08-13 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=517366





--- Comment #1 from Shakthi Kannan shakthim...@gmail.com  2009-08-13 13:13:32 
EDT ---
#01 It should be:

Patch0:
Patch1:

with capital 'P'.

#02 Don't rename the source file. Although the latest sources say
haskell-mode.tar.gz, please use the exact version:

  haskell-mode-2.4.tar.gz

You have mentioned the same in the .spec file as Version: 2.4

#03 You should add the -b .patch extension when applying %patch -p1

#04 In the Makefile patch, substitute install command with a variable, like:

INSTALL = /usr/bin/install

#05 Put the 'clean' target in the Makefile as the last target, at the bottom.

#06 Please remove control characters in haskell-site-file.el (^L), and check
for the same in other files.

#07 Don't include ~ files in the source package!

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

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


[Bug 516466] Review Request: sys_basher - multi-threaded hardware tester

2009-08-13 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=516466





--- Comment #48 from Joshua Rosen bjro...@polybus.com  2009-08-13 13:33:52 
EDT ---
OK let me know what you want me to review.

Also I have a CentOS question. In installed the rpmdevtools on CentOS5.3 and
built the package. The distro name was not included in the rpm name,

sys_basher-1.1.21-1.i386.rpm

Is there something different that I need to do on CentOS to get it to build the
packages with the distro name included?

-- 
Configure 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 517191] Review Request: php-symfony-symfony - Open-Source PHP Web Framework

2009-08-13 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=517191





--- Comment #5 from Jason Tibbitts ti...@math.uh.edu  2009-08-13 13:47:20 EDT 
---
Is this essentially the same thing as bug 351441?  Do the same issues which
caused the closure of that review still apply?

-- 
Configure 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 486721] Review Request: beagle-xesam - Xesam adaptor for Beagle

2009-08-13 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=486721


Jason Tibbitts ti...@math.uh.edu changed:

   What|Removed |Added

 Blocks||201449(FE-DEADREVIEW)




-- 
Configure 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 516779] Review Request: libbitmask - multi-word bitmask library

2009-08-13 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=516779


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

   What|Removed |Added

 Status|NEW |ASSIGNED




-- 
Configure 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 516874] Review Request: eqntott - Generates truth tables from Boolean equations

2009-08-13 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=516874


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

   What|Removed |Added

 Status|NEW |ASSIGNED




-- 
Configure 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 516698] Review Request: fpaste - A simple tool for pasting info onto fpaste.org

2009-08-13 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=516698


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

   What|Removed |Added

 Status|NEW |ASSIGNED




-- 
Configure 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 516547] Review Request: Django-south - Intelligent schema migrations for Django apps

2009-08-13 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=516547


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

   What|Removed |Added

 Status|NEW |ASSIGNED




-- 
Configure 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 516782] Review Request: libcpuset - Processor and memory placement library

2009-08-13 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=516782


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

   What|Removed |Added

 Status|NEW |ASSIGNED




-- 
Configure 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 516466] Review Request: sys_basher - multi-threaded hardware tester

2009-08-13 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=516466





--- Comment #49 from Jussi Lehtola jussi.leht...@iki.fi  2009-08-13 13:50:51 
EDT ---
(In reply to comment #48)
 OK let me know what you want me to review.

https://fedoraproject.org/wiki/PackageReviewProcess

Have your pick from the review list at
https://fedoraproject.org/wiki/PackageMaintainers/ReviewRequests - unassigned
review requests

but please review only bugs not tagged with FE-NEEDSPONSOR.


 Also I have a CentOS question. In installed the rpmdevtools on CentOS5.3 and
 built the package. The distro name was not included in the rpm name,
 
 sys_basher-1.1.21-1.i386.rpm
 
 Is there something different that I need to do on CentOS to get it to build 
 the
 packages with the distro name included?  

Actually I'm not sure which package ships the %{dist} macro, I tried to search
for it today. But if you build the package in mock for EPEL it will get the
correct value. (And the Fedora build system uses mock.)

-- 
Configure 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 478659] Review Request: xfce4-screenshooter - Screenshot utility for the Xfce desktop

2009-08-13 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=478659


Jason Landstrom jjlandst...@gmail.com changed:

   What|Removed |Added

 CC||jjlandst...@gmail.com




--- Comment #7 from Jason Landstrom jjlandst...@gmail.com  2009-08-13 
13:51:13 EDT ---
I'm a newbie running Fedora 11 trying to run xfce4-screenshooter.  Everything
works great except when I select a region with copy to clipboard I am unable to
paste it into OpenOffice Impress like I can with gnome screenshot.

-- 
Configure 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 516364] Review Request: xrdp - Open source remote desktop protocol (RDP) server

2009-08-13 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=516364





--- Comment #10 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-08-13 
13:58:48 EDT ---
For -0.1:

* _initddir
  - Well, actually the macro written at the top should be
_initddir, not initddir, sorry...


%{!?_initddir: %global _initddir %{_sysconfdir}/rc.d/init.d}


* Macros
  - If you want to use %__install and %__rm, please use
%__sed, %__make for consistency (and you also use
rm)

* Directory ownership issue
  - Still the directory %_datadir/xrdp itself is not owned
by any packages.

* About %_sysconfdir/xrdp/rsakeys.ini:
  - You have to use %verify(not ), like:
-
%install
rm -rf %{buildroot}
make install DESTDIR=%{buildroot}

# rsakeys.ini
touch %{buildroot}%{_sysconfdir}/xrdp/rsakeys.ini
chmod 06?? %{buildroot}%{_sysconfdir}/xrdp/rsakeys.ini


%files

%verify(not size md5 mtime) %{_sysconfdir}/xrdp/rsakeys.ini

-
  * By the way, when I try xrdp-keygen, rsakeys.ini
is created with 0600 permission.
- If this permission (0600) is correct, %attr(0600,root,root)
  is also needed.
- If the permission should be 0644 or so, %post scriptlet
  needs fixing.
  ( I think anyway changing permission on %post explicitly
is preferred )

  * Currently rsakeys.ini is recreated every time xrdp
is upgraded. Is this correct?
If not, something like
-
[ -s %{_sysconfdir}/xrdp/rsakeys.ini ] || \
 xrdp-keygen xrdp %{_sysconfdir}/xrdp/rsakeys.ini /dev/null
chmod 06?? %{_sysconfdir}/xrdp/rsakeys.ini
-
is needed so that rsakeys.ini won't be overwritten.
( I used -s here, not -f, because at first install,
  0 size rsakeys.ini is installed )

  * Note that I think /dev/null is okay, however /dev/null 21
should be avoided. If this command outputs some errors, we
should check them and examine what was wrong.

-- 
Configure 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 478659] Review Request: xfce4-screenshooter - Screenshot utility for the Xfce desktop

2009-08-13 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=478659





--- Comment #8 from Kevin Fenzi ke...@tummy.com  2009-08-13 14:15:27 EDT ---
This closed review is not the place for bugs. ;) 

Please file a new bug on the xfce4-screenshooter package?

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

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


[Bug 516466] Review Request: sys_basher - multi-threaded hardware tester

2009-08-13 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=516466





--- Comment #50 from Joshua Rosen bjro...@polybus.com  2009-08-13 14:37:59 
EDT ---
I have a review question. I've downloaded the files for 481030, pmd-emacs and
fetched the source file using spectool -g. Are all the sources including
patches required to be in that file or is reviewer supposed to install the
SRPM? Does the source archive format matter? This package was supplies as a
.zip instead of a .tar.gz or a .tar.bz2. Zip is not a standard way to archive
files on a *nix system although it's fully supported.

Also should I be posting my review comments on that bug report given that I'm
doing this as a learning experience?

-- 
Configure 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 516466] Review Request: sys_basher - multi-threaded hardware tester

2009-08-13 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=516466





--- Comment #51 from Jason Tibbitts ti...@math.uh.edu  2009-08-13 14:44:34 
EDT ---
Patches will be in the src.rpm; if all you have is the spec, you will generally
not have what you need to build the package although in many situations it's
enough.

We generally have to consume what upstream provides when it comes to archive
format.  When given a choice, we prefer whatever is smaller.  If upstream
provides a .zip, we use a .zip.  (And this is by far not uncommon, and
certainly nothing against the package.)

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

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


[Bug 516466] Review Request: sys_basher - multi-threaded hardware tester

2009-08-13 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=516466





--- Comment #52 from Joshua Rosen bjro...@polybus.com  2009-08-13 14:59:23 
EDT ---
One more question. Is there a way to unpack an SRPM into a local directory
rather than installing it or should I just do this inside of a VM?

-- 
Configure 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 516466] Review Request: sys_basher - multi-threaded hardware tester

2009-08-13 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=516466





--- Comment #53 from Jussi Lehtola jussi.leht...@iki.fi  2009-08-13 15:13:09 
EDT ---
(In reply to comment #52)
 One more question. Is there a way to unpack an SRPM into a local directory
 rather than installing it or should I just do this inside of a VM?  

I do all my reviews in mock. It uses a chroot to build the packages, so your
build directory is clean.

Another option is to have a second user on your system with which you build the
packages (better this way, so you don't end up blowing all your files away when
building a crappily written spec file). Just run
 $ rpmdev-setuptree
and build the SRPM with
 $ rpmbuild --rebuild package.src.rpm

Or you can build using your normal user name, this way your SPEC, SOURCES and
RPMS directories just get messier with every package you 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 478659] Review Request: xfce4-screenshooter - Screenshot utility for the Xfce desktop

2009-08-13 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=478659





--- Comment #9 from Christoph Wickert christoph.wick...@googlemail.com  
2009-08-13 15:16:17 EDT ---
Filed as bug 517408.

-- 
Configure 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 517409] New: Review Request: django-authority - A Django app for generic per-object permissions and custom permission checks

2009-08-13 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: django-authority - A Django app for generic per-object 
permissions and custom permission checks

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

   Summary: Review Request: django-authority - A Django app for
generic per-object permissions and custom permission
checks
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: dieg...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL:
http://diegobz.fedorapeople.org/packages/django-authority/django-authority.spec
SRPM URL:
http://diegobz.fedorapeople.org/packages/django-authority/django-authority-0.3-1.fc11.src.rpm

Description:
This is a Django app for generic per-object permissions, custom permission
checks and permission requests. It also includes view decorators and template 
tags for ease of use.

This package is needed to suppress a dependency for the new Transifex 0.7-RC1.

-- 
Configure 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 517412] New: Review Request: django-profile - Django pluggable user profile zone

2009-08-13 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: django-profile - Django pluggable user profile zone

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

   Summary: Review Request: django-profile - Django pluggable user
profile zone
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: dieg...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL:
http://diegobz.fedorapeople.org/packages/django-profile/django-profile.spec
SRPM URL:
http://diegobz.fedorapeople.org/packages/django-profile/django-profile-0.6-0.1.20090813svnr420.fc11.src.rpm

Description: 

This is a user private zone/profile management application, allowing
the user to take control of his account and insert information about
him in his profile.

Inside this package you will find a demo application which will show
you what can be accomplished with the rest of the utilities included
in the package.

This package is needed to suppress a dependency for the new Transifex 0.7-RC1.

-- 
Configure 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 517410] New: Review Request: django-piston - A mini-framework for Django for creating RESTful APIs

2009-08-13 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: django-piston - A mini-framework for Django for 
creating RESTful APIs

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

   Summary: Review Request: django-piston - A mini-framework for
Django for creating RESTful APIs
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: dieg...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL:
http://diegobz.fedorapeople.org/packages/django-piston/django-piston.spec
SRPM URL:
http://diegobz.fedorapeople.org/packages/django-piston/django-piston-0.2.2-1.fc11.src.rpm

Description: 

Piston is a relatively small Django application that lets you
create application programming interfaces (API) for your sites.

This package is needed to suppress a dependency for the new Transifex 0.7-RC1.

-- 
Configure 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 517191] Review Request: php-symfony-symfony - Open-Source PHP Web Framework

2009-08-13 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=517191





--- Comment #6 from Christof Damian chris...@damian.net  2009-08-13 15:23:51 
EDT ---
(In reply to comment #5)
 Is this essentially the same thing as bug 351441?  Do the same issues which
 caused the closure of that review still apply?  

I didn't see that bug. But I guess they still apply. I had a quick grep through
the sources and found these in the vendor directories:

./lib/plugins/sfCompat10Plugin/lib/vendor: phpmailer
./lib/plugins/sfDoctrinePlugin/lib/vendor: doctrine
./lib/plugins/sfPropelPlugin/lib/vendor: phing,  propel,  propel-generator
./lib/vendor: lime

- phpmailer, phing, propel and propel-generator are already available in
Fedora. 
- doctrine isn't.
- lime is just used by symfony anyway, as far as I know. It is just one file.

Personally I think it is not a big problem, because symfony requires certain
versions of the libraries. For example if I create a doctrine package of the
current 1.1 it probably won't work with symfony 1.2. 

I would suggest bringing symfony into Fedora as is, with the plan to remove
these duplicated libraries in one of the next releases where possible. I think
this was how it was done in other packages which bundled libraries.

Then I probably would:
- remove the sfCompat10Plugin with the ancient phpmailer completely. 
- create the doctrine package, which I planned anyway.
- check if symmfony works with the fedora propel  phing packages. If it
doesn't I would remove that plugin too.
- create a patch to make symfony look at the default library locations instead
of vendor directories.

Thank god I don't have to think about EPEL, because RHEL is still stuck at PHP
5.1 .

-- 
Configure 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 517416] New: Review Request: awl - Andrew's Web Libraries - PHP Utility Libraries

2009-08-13 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: awl - Andrew's Web Libraries - PHP Utility Libraries

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

   Summary: Review Request: awl - Andrew's Web Libraries - PHP
Utility Libraries
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: serein...@online.fr
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://tmp.sereinity.homelinux.org/rpm/awl.spec
SRPM URL: http://tmp.sereinity.homelinux.org/rpm/awl-0.37-1.fc11.src.rpm
Description: 
This package contains Andrew's Web Libraries.  This is a set
of hopefully lightweight libraries for handling a variety of
useful things for web programming, including:
 - Session management
 - User management
 - DB Records
 - Simple reporting
 - DB Schema Updating
 - iCalendar parsing

-- 
Configure 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 481030] Review Request: pmd-emacs - an interface to PMD for (X)Emacs

2009-08-13 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=481030


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

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||jussi.leht...@iki.fi
 AssignedTo|nob...@fedoraproject.org|jussi.leht...@iki.fi
   Flag||fedora-review?




--- Comment #1 from Jussi Lehtola jussi.leht...@iki.fi  2009-08-13 15:58:44 
EDT ---
Taking over review. Joshua Rosen will perform an informal review first.

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

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


[Bug 516547] Review Request: Django-south - Intelligent schema migrations for Django apps

2009-08-13 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=516547


Toshio Ernie Kuratomi a.bad...@gmail.com changed:

   What|Removed |Added

 Blocks|177841(FE-NEEDSPONSOR)  |
   Flag|fedora-review?  |fedora-review+




--- Comment #6 from Toshio Ernie Kuratomi a.bad...@gmail.com  2009-08-13 
16:32:32 EDT ---
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 457160] Review Request: Zorba - General purpose XQuery processor implemented in C++

2009-08-13 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=457160


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

   What|Removed |Added

 CC||martin.giesek...@uos.de




--- Comment #34 from Martin Gieseking martin.giesek...@uos.de  2009-08-13 
16:33:36 EDT ---
SPEC: http://mgieseki.fedorapeople.org/zorba/zorba.spec
SRPM: http://mgieseki.fedorapeople.org/zorba/zorba-0.9.5-7.fc11.src.rpm

I'd also like to see zorba packaged for Fedora. Unfortunately, the review
process seems to have stalled. To keep it alive, I digged into the source and
fixed the build problems. There are still some spec file issues to be resolved
but that should be doable. 
Paul, if you can afford the time, please continue working on this package. :)


$ rpmlint ../RPMS/i586/zorba-*
zorba.i586: W: shared-lib-calls-exit /usr/lib/libzorba_simplestore.so.0.9.5
e...@glibc_2.0
zorba.i586: E: zero-length
/usr/share/doc/zorba-0.9.5/cxx/html/options_8h__incl.map
zorba.i586: E: zero-length
/usr/share/doc/zorba-0.9.5/cxx/html/path_8h__incl.map
zorba.i586: E: zero-length
/usr/share/doc/zorba-0.9.5/cxx/html/zorbastring_8h__incl.map
zorba.i586: E: zero-length
/usr/share/doc/zorba-0.9.5/c/html/options_8h__incl.map
zorba.i586: E: zero-length
/usr/share/doc/zorba-0.9.5/cxx/html/properties__base_8h__incl.map
zorba.i586: E: zero-length
/usr/share/doc/zorba-0.9.5/cxx/html/simplestorec_8h__incl.map
zorba.i586: E: zero-length
/usr/share/doc/zorba-0.9.5/cxx/html/simplestore_8h__incl.map
zorba.i586: E: zero-length
/usr/share/doc/zorba-0.9.5/cxx/html/identtypes_8h__incl.map
zorba.i586: W: devel-file-in-non-devel-package /usr/lib/libzorba_simplestore.so
zorba.i586: E: zero-length
/usr/share/doc/zorba-0.9.5/cxx/html/smart__ptr_8h__incl.map
zorba.i586: E: zero-length
/usr/share/doc/zorba-0.9.5/cxx/html/debugger__exception_8h__incl.map
zorba.i586: E: zero-length
/usr/share/doc/zorba-0.9.5/cxx/html/store__consts_8h__incl.map
zorba.i586: E: zero-length
/usr/share/doc/zorba-0.9.5/cxx/html/version_8h__incl.map
zorba-python.i586: W: summary-not-capitalized zorba Python module for zorba
zorba-python.i586: W: unstripped-binary-or-object
/usr/lib/python2.6/site-packages/_zorba_api.so
zorba-python.i586: W: no-documentation
zorba-ruby.i586: W: summary-not-capitalized zorba Ruby module
zorba-ruby.i586: W: unstripped-binary-or-object
/usr/lib/ruby/site_ruby/1.8/i386-linux/zorba_api.so
zorba-ruby.i586: W: no-documentation
5 packages and 0 specfiles checked; 12 errors, 8 warnings.

-- 
Configure 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 517183] Review Request: mipv6-daemon - IPv6 Mobility Daemon

2009-08-13 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=517183





--- Comment #2 from Jarod Wilson jwil...@redhat.com  2009-08-13 16:37:44 EDT 
---
Okay, so a few issues to work out...

1) without solid justification, the initscript should not be enabled by
default, it should be up to the user to enable it.

2) in one place, the initscript says 'config: /etc/mip6d.conf', then later you
test for /etc/sysconfig/mip6, then you source /etc/sysconfig/mip6d. And the
package has no config file whatsoever mentioned anywhere. :)

To resolve the enabled-by-default and three different names for the config, I'd
suggest:

--- mip6d.init.orig 2009-08-13 16:09:39.219807916 -0400
+++ mip6d.init 2009-08-13 16:22:46.888931134 -0400
@@ -2,11 +2,11 @@
 #
 # mip6d  Start script for the Mobile IPv6 daemon
 #
-# chkconfig: 2345 55 25
+# chkconfig: - 55 25
 # description: The mobile IPv6 daemon allows nodes to remain \
 #  reachable while moving around in the IPv6 Internet.
 # processname: mip6d
-# config: /etc/mip6d.conf
+# config: /etc/sysconfig/mip6d
 #
 ### BEGIN INIT INFO
 # Provides: mipv6-daemon
@@ -14,7 +14,7 @@
 # Required-Stop: $local_fs $remote_fs $network
 # Should-Start: $syslog
 # Should-Stop: $network $syslog
-# Default-Start: 2 3 4 5
+# Default-Start:
 # Default-Stop: 0 1 6
 # Short-Description: Start and stop Mobile IPV6 daemon
 # Description: The mobile IPv6 daemon allows nodes to remain
@@ -24,7 +24,7 @@
 # Source function library.
 . /etc/rc.d/init.d/functions

-if [ -f /etc/sysconfig/mip6 ]; then
+if [ -f /etc/sysconfig/mip6d ]; then
  . /etc/sysconfig/mip6d
 fi



On to the spec file itself...

1) as previously mentioned, the config file isn't accounted for anywhere in the
spec, and it needs to be, complete with %config(noreplace) labelling.

2) missing a chkconfig --add %post call

3) BuildRequires: /etc/init.d looks bogus, just flex and bison should be
sufficient.

4) 'Requires: initscripts, chkconfig' should be sufficient for Requires.

5) Generally, you don't want to inclue INSTALL files in %doc, because they're
irrelevant for a packaged install. Also, traditionally, %doc is the first line
after %defattr in the %files section.

6) use %{_initrddir} instead of /etc/rc.d/init.d/

7) %attr shouldn't be necessary on the initscript, you already install -m755'd
it.

I'd suggest the following spec file changes:

--- mipv6-daemon-orig.spec 2009-08-13 11:36:05.068932552 -0400
+++ mipv6-daemon.spec 2009-08-13 16:26:20.352931018 -0400
@@ -11,9 +11,8 @@ Source1: mip6d.init
 Patch0:  mipv6-daemon-header-fix.patch
 BuildRoot: %(mktemp -ud %{_tmppath}/%{name}-%{version}-%{release}-XX)

-BuildRequires: flex bison /etc/init.d
-Requires: initscripts
-Requires(post): chkconfig = 0.9, /sbin/service
+BuildRequires: flex bison
+Requires: initscripts, chkconfig

 %description
 The mobile IPv6 daemon allows nodes to remain
@@ -31,8 +30,10 @@ make %{?_smp_mflags}
 rm -rf $RPM_BUILD_ROOT
 make install DESTDIR=$RPM_BUILD_ROOT

-install -d $RPM_BUILD_ROOT/etc/rc.d/init.d
-install -m755 %{SOURCE1} $RPM_BUILD_ROOT/etc/rc.d/init.d/mip6d
+install -d $RPM_BUILD_ROOT%{_initrddir}
+install -m755 %{SOURCE1} $RPM_BUILD_ROOT%{_initrddir}/mip6d
+install -d $RPM_BUILD_ROOT%{_sysconfdir}/sysconfig
+touch $RPM_BUILD_ROOT%{_sysconfdir}/sysconfig/mip6d

 %clean
 rm -rf $RPM_BUILD_ROOT
@@ -44,6 +45,9 @@ then
  /sbin/chkconfig --del mip6d
 fi

+%post
+/sbin/chkconfig --add mip6d
+
 %postun
 if [ $1 -ge 1 ]; then
  /sbin/service mip6d condrestart  /dev/null 21 ||:
@@ -51,12 +55,13 @@ fi

 %files
 %defattr(-,root,root,-)
+%doc AUTHORS BUGS COPYING NEWS README README.IPsec THANKS extras
+%ghost %config(noreplace) %{_sysconfdir}/sysconfig/mip6d
+%{_initrddir}/mip6d
 %{_sbindir}/*
 %{_mandir}/man1/*
 %{_mandir}/man5/*
 %{_mandir}/man7/*
-%attr(0755,root,root) /etc/rc.d/init.d/mip6d
-%doc AUTHORS BUGS COPYING INSTALL INSTALL.kernel NEWS README README.IPsec
THANKS extras

 %changelog
 * Tue Aug 12 2009 Thomas Graf tgraf at, redhat.com 0.4-1


With these changes applied, the only noise I get from rpmlint is a complaint
about the initscript name not matching the package name, which we can ignore.

-- 
Configure 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 517183] Review Request: mipv6-daemon - IPv6 Mobility Daemon

2009-08-13 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=517183





--- Comment #3 from Jarod Wilson jwil...@redhat.com  2009-08-13 16:39:21 EDT 
---
Oh, I just did a touch and %ghost for the config file, but if you have a
default config file (one of the example files maybe?) you want to drop in
place, just with all the options commented out, that might not be a bad idea
either, so that people know what the config file is supposed to look like
without going hunting.

-- 
Configure 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 514342] Review Request: rubygem-fattr - Fatter attribute for Ruby

2009-08-13 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=514342


Jeroen van Meeuwen kana...@kanarip.com changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #5 from Jeroen van Meeuwen kana...@kanarip.com  2009-08-13 
16:48:29 EDT ---
Thanks;

New SPEC: http://www.kanarip.com/custom/SPECS/rubygem-fattr.spec
New SRPM:
http://www.kanarip.com/custom/f11/SRPMS/rubygem-fattr-1.0.3-3.fc11.src.rpm

New Package CVS Request
===
Package Name: rubygem-fattr
Short Description: Fatter attribute for Ruby
Owners: kanarip
Branches: EL-4 EL-5 F-10 F-11 devel
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 516547] Review Request: Django-south - Intelligent schema migrations for Django apps

2009-08-13 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=516547


Diego Búrigo Zacarão dieg...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #7 from Diego Búrigo Zacarão dieg...@gmail.com  2009-08-13 
17:03:32 EDT ---
New Package CVS Request
===
Package Name: Django-south
Short Description: Intelligent schema migrations for Django apps
Owners: diegobz
Branches: F-10 F-11 EL-5
InitialCC: diegobz

-- 
Configure 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 516364] Review Request: xrdp - Open source remote desktop protocol (RDP) server

2009-08-13 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=516364





--- Comment #11 from Itamar Reis Peixoto ita...@ispbrasil.com.br  2009-08-13 
17:42:10 EDT ---
done

http://ispbrasil.com.br/xrdp/xrdp-0.5.0-0.2.20090811cvs.fc12.src.rpm
http://ispbrasil.com.br/xrdp/xrdp.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 515032] Review Request: nss-util - Utilities for Network Security Services (NSS) and Softoken module

2009-08-13 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=515032


Bob Relyea rrel...@redhat.com changed:

   What|Removed |Added

 CC||rrel...@redhat.com
 AssignedTo|nob...@fedoraproject.org|rrel...@redhat.com
   Flag||fedora-review?




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

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


[Bug 470293] Review Request: sblim-cmpi-network - SBLIM Network Instrumentation

2009-08-13 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=470293





--- Comment #11 from Praveen K Paladugu praveen_palad...@dell.com  2009-08-13 
17:57:32 EDT ---
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 470293] Review Request: sblim-cmpi-network - SBLIM Network Instrumentation

2009-08-13 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=470293


Praveen K Paladugu praveen_palad...@dell.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




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

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


[Bug 515032] Review Request: nss-util - Utilities for Network Security Services (NSS) and Softoken module

2009-08-13 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=515032


Bob Relyea rrel...@redhat.com changed:

   What|Removed |Added

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




--- Comment #14 from Bob Relyea rrel...@redhat.com  2009-08-13 18:03:19 EDT 
---
1. URL - points to a generic NSS page. Is there a worry that mozilla may move
the actual FTP site around. See
http://fedoraproject.org/wiki/Packaging/SourceURL for examples. I believe this
tar was pulled for an NSS cvs tag and then the split.sh was ran on it. The wiki
has examples for this case (you've partially included the right thing with the
comment and conclusing of the nss.split.sh script.

2. You are creating a new empty directory %{libdir}/nss3 is there a reason (if
so comment it, if not remove it).

3. the %define nspr-version should technically be %global

4. uncomment the Requires: nss-util in the nss-util-devel package section.

5. This last on is the biggest one: The issue showed up in multiple locations,
including rpmlint.

In the %build you are creating nss-util.pc and nss-util-config and placing them
into the build root. This is a non-standard use of the build root, and requires
the first command in %install which clears the buildroot to be removed. A
better way would be to create nss-util.pc and nss-util-config in %build and
place the results in mozilla/dist (where NSS build results are placed), then
install them into their final location in the %include.

-- 
Configure 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 515032] Review Request: nss-util - Utilities for Network Security Services (NSS) and Softoken module

2009-08-13 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=515032


Bob Relyea rrel...@redhat.com changed:

   What|Removed |Added

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




-- 
Configure 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 515032] Review Request: nss-util - Utilities for Network Security Services (NSS) and Softoken module

2009-08-13 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=515032


Bob Relyea rrel...@redhat.com changed:

   What|Removed |Added

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




--- Comment #15 from Bob Relyea rrel...@redhat.com  2009-08-13 18:05:47 EDT 
---
rpmlint output:

bobs-laptop(79) rpmlint nss-util-3.12.3.99.3-8.fc11.test.1.src.rpm 
nss-util.src: W: strange-permission nss-split-util.sh 0775
nss-util.src:97: W: rpm-buildroot-usage %build %{__mkdir_p}
$RPM_BUILD_ROOT/%{_libdir}/pkgconfig
nss-util.src:104: W: rpm-buildroot-usage %build
$RPM_BUILD_ROOT/%{_libdir}/pkgconfig/nss-util.pc
nss-util.src:114: W: rpm-buildroot-usage %build %{__mkdir_p}
$RPM_BUILD_ROOT/%{_bindir}
nss-util.src:122: W: rpm-buildroot-usage %build 
$RPM_BUILD_ROOT/%{_bindir}/nss-util-config
nss-util.src:124: W: rpm-buildroot-usage %build chmod 755
$RPM_BUILD_ROOT/%{_bindir}/nss-util-config
1 packages and 0 specfiles checked; 0 errors, 6 warnings.

 and just for the .spec, which has better line numbers 

rpmlint nss-util.spec
nss-util.spec:100: W: rpm-buildroot-usage %build %{__mkdir_p}
$RPM_BUILD_ROOT/%{_libdir}/pkgconfig
nss-util.spec:107: W: rpm-buildroot-usage %build
$RPM_BUILD_ROOT/%{_libdir}/pkgconfig/nss-util.pc
nss-util.spec:117: W: rpm-buildroot-usage %build %{__mkdir_p}
$RPM_BUILD_ROOT/%{_bindir}
nss-util.spec:125: W: rpm-buildroot-usage %build 
$RPM_BUILD_ROOT/%{_bindir}/nss-util-config
nss-util.spec:127: W: rpm-buildroot-usage %build chmod 755
$RPM_BUILD_ROOT/%{_bindir}/nss-util-config
0 packages and 1 specfiles checked; 0 errors, 5 warnings.

-- 
Configure 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 517409] Review Request: django-authority - A Django app for generic per-object permissions and custom permission checks

2009-08-13 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=517409


Allisson Azevedo allis...@gmail.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||allis...@gmail.com
 AssignedTo|nob...@fedoraproject.org|allis...@gmail.com




--- Comment #1 from Allisson Azevedo allis...@gmail.com  2009-08-13 18:30:26 
EDT ---
I'll make review soon as posible :)

-- 
Configure 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 517412] Review Request: django-profile - Django pluggable user profile zone

2009-08-13 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=517412


Allisson Azevedo allis...@gmail.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||allis...@gmail.com
 AssignedTo|nob...@fedoraproject.org|allis...@gmail.com




--- Comment #1 from Allisson Azevedo allis...@gmail.com  2009-08-13 18:29:53 
EDT ---
I'll make review soon as posible :)

-- 
Configure 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 517410] Review Request: django-piston - A mini-framework for Django for creating RESTful APIs

2009-08-13 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=517410


Allisson Azevedo allis...@gmail.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||allis...@gmail.com
 AssignedTo|nob...@fedoraproject.org|allis...@gmail.com




--- Comment #1 from Allisson Azevedo allis...@gmail.com  2009-08-13 18:29:10 
EDT ---
I'll make review soon as posible :)

-- 
Configure 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 515032] Review Request: nss-util - Utilities for Network Security Services (NSS) and Softoken module

2009-08-13 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=515032


Elio Maldonado Batiz emald...@redhat.com changed:

   What|Removed |Added

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




--- Comment #16 from Elio Maldonado Batiz emald...@redhat.com  2009-08-13 
20:10:44 EDT ---
Bob, Please check the updated .spec and src.rpm files at
http://fedorapeople.org/~emaldonado/nss-util/devel/nss-util.spec
and at
http://fedorapeople.org/~emaldonado/nss-util/devel/nss-util-3.12.3.99.3-8.fc12.test.1.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 515032] Review Request: nss-util - Utilities for Network Security Services (NSS) and Softoken module

2009-08-13 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=515032


Bob Relyea rrel...@redhat.com changed:

   What|Removed |Added

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




--- Comment #17 from Bob Relyea rrel...@redhat.com  2009-08-13 20:36:31 EDT 
---
neither the original nss, not this package ever puts any shared libraries in
/usr/lib/nss3, so you should remove the creation of that directory and it's
ownership in the .spec file

-- 
Configure 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 515032] Review Request: nss-util - Utilities for Network Security Services (NSS) and Softoken module

2009-08-13 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=515032


Elio Maldonado Batiz emald...@redhat.com changed:

   What|Removed |Added

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




--- Comment #18 from Elio Maldonado Batiz emald...@redhat.com  2009-08-13 
20:55:13 EDT ---
(In reply to comment #17) Line and comments above removed. Updated the .spec
and srpm.

-- 
Configure 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 515032] Review Request: nss-util - Utilities for Network Security Services (NSS) and Softoken module

2009-08-13 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=515032


Bob Relyea rrel...@redhat.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #19 from Bob Relyea rrel...@redhat.com  2009-08-13 21:04:15 EDT 
---
OK, much better. ACCEPT.

bob

-- 
Configure 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 517338] Review Request: irssi-xmpp is an irssi plugin to connect to the Jabber network

2009-08-13 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=517338


Caius 'kaio' Chance ccha...@redhat.com changed:

   What|Removed |Added

 CC||ccha...@redhat.com
   Flag||fedora-review?




--- Comment #1 from Caius 'kaio' Chance ccha...@redhat.com  2009-08-13 
21:51:22 EDT ---
$ rpmlint irssi-xmpp.spec 
0 packages and 1 specfiles checked; 0 errors, 0 warnings.

FYI (not mandatory), you could use marcos like %__rm and %__make to replace rm
and make.

-- 
Configure 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 517338] Review Request: irssi-xmpp is an irssi plugin to connect to the Jabber network

2009-08-13 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=517338





--- Comment #2 from Caius 'kaio' Chance ccha...@redhat.com  2009-08-13 
21:55:20 EDT ---
mock failed:

DEBUG util.py:256:  ##
DEBUG util.py:256:  error: unpacking of archive failed on file
/builddir/build/SOURCES/irssi-xmpp-0.50.tar.gz;4a84c470: cpio: read failed -
Invalid argument
DEBUG util.py:256:  error:
/builddir/build/originals/irssi-xmpp-0.50-1.fc12.src.rpm cannot be installed
DEBUG util.py:319:  Child returncode was: 1

-- 
Configure 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 517021] Review Request: perl-DBIx-Simple - Easy-to-use OO interface to DBI

2009-08-13 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=517021


Caio 'kaio' Chance k...@kaio.me changed:

   What|Removed |Added

 CC||k...@kaio.me




--- Comment #1 from Caio 'kaio' Chance k...@kaio.me  2009-08-13 22:01:44 EDT 
---
$ rpmlint perl-DBIx-Simple.spec 
0 packages and 1 specfiles checked; 0 errors, 0 warnings.

-- 
Configure 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 517021] Review Request: perl-DBIx-Simple - Easy-to-use OO interface to DBI

2009-08-13 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=517021


Caio 'kaio' Chance k...@kaio.me changed:

   What|Removed |Added

   Flag||fedora-review?




--- Comment #2 from Caio 'kaio' Chance k...@kaio.me  2009-08-13 22:05:05 EDT 
---
mock done successfully:

INFO: Done(perl-DBIx-Simple-1.32-1.fc11.src.rpm) Config(fedora-rawhide-i386) 1
minutes 28 seconds

-- 
Configure 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 516971] Review Request: tokyotyrant - A network interface to Tokyo Cabinet

2009-08-13 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=516971


Caio 'kaio' Chance k...@kaio.me changed:

   What|Removed |Added

 CC||k...@kaio.me




--- Comment #1 from Caio 'kaio' Chance k...@kaio.me  2009-08-13 22:09:53 EDT 
---
tokyocabinet is still 1.4.23 at the moment.

-- 
Configure 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 516717] Review Request: viewnior - Elegant Image Viewer

2009-08-13 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=516717


Caio 'kaio' Chance k...@kaio.me changed:

   What|Removed |Added

 CC||k...@kaio.me




--- Comment #1 from Caio 'kaio' Chance k...@kaio.me  2009-08-13 22:17:16 EDT 
---
rpmlint is done:

$ rpmlint viewnior.spec 
0 packages and 1 specfiles checked; 0 errors, 0 warnings.

-- 
Configure 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 516717] Review Request: viewnior - Elegant Image Viewer

2009-08-13 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=516717


Caio 'kaio' Chance k...@kaio.me changed:

   What|Removed |Added

   Flag||fedora-review?




--- Comment #2 from Caio 'kaio' Chance k...@kaio.me  2009-08-13 22:19:32 EDT 
---
mock is done:

INFO: Done(viewnior-0.6-1.fc12.src.rpm) Config(fedora-rawhide-i386) 4 minutes
35 seconds

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