[Bug 192912] Review Request: paps

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: paps


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-12 02:33 EST ---
(In reply to comment #9)
> Main thing that needs to be done before pango-view can be used like paps is to
> make it break text into paragraphs before laying out (for performance 
> reasons.)
>  I'm actually not sure that paps does that.  But anyway, I probably will get 
> to
> doing that sooner or later, but can't make any promise at this point.

I'm not sure that I understood what you mean though, paps splits up each lines
into the paragraphs.

> Ok, what about having a simple shell script called u2ps shipped and advertised
> in Core with a documented command line interface, and make it call paps as the
> implementation for now, but leave it open to switch to pango-view later on... 
> The interface should be quite simple, a cat-like tool with the following 
> options:

It's not a bad idea though, I imagined gnome-u2ps which is on GNOME CVS, but
anyway. I don't know how it is recognized in the world, I at least got confused.
I like a concept to provide a standard interface, otherwise.

> That should be enough for now, and (except for footer?) paps supports the rest
> already, with different namings possibly.

Well, there are some required features from CUPS side too. using this as an
replacement of a commandline printing tool such as a2ps is also one of the way
though, the main thing are to work together on CUPS and to replaces the various
printing filters which is just kept to get CJK printing working as I described
earlier. no particular advantages there.
That is in my todo anyway.

> Another thing that should work in u2ps but is not currently working in paps is
> setting default paper size based on LC_PAPER.  We can make the wrapper figure
> out the paper and pass it to paps for now, and use it to set page size later
> with pango-view.

Ok, it's probably a feature that needs to be implemented.


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

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


[Bug 193929] Review Request: knetstats

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: knetstats


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|[EMAIL PROTECTED] |[EMAIL PROTECTED]
OtherBugsDependingO|163776  |163778
  nThis||




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

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


[Bug 193929] Review Request: knetstats

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: knetstats


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-12 02:22 EST ---
(In reply to comment #12)
> Re: comment #10
> > -update-desktop-database is deprecated
> 
> It certainly is not deprecated (what makes you say that?).  
> 

Sorry, deprected in deed is the wrong way to formulate this, what I meant is
usually calling update-desktop-database is no longer needed unless:

> the app's .desktop file contains MimeType= entries, ScriptletSnippets are
> clear that use of update-desktop-database in %post/%postun is required. 

---

(In reply to comment #13)
> (In reply to comment #12)
> > Re: comment #10
> > > -update-desktop-database is deprecated
> > 
> > It certainly is not deprecated (what makes you say that?).  
> 
> Hans, you want to comment on this ?
> 

See above, the knetstats does not contain any MimeType's, so the call to
update-desktop-database should be removed (and the Requires).

> Anyone want to sponsor me for knetstats ? Hans ?
> 

Quoting myself from comment #6:

I see (from google) that you're already active in other area's of the Fedora
project, good! As such I'm willing to sponsor you, but first you must show some
more / better understanding of the FE packaging guidelines.

There are 2 ways to show this better understanding:
1) You review a couple of packages from others see FE-NEW for a list of
   Review Requests that need a Reviewer, don't worry about not being competent
   enough todo a review, just add me to the CC-list and I'll watch over your 
   back.
2) Create some more packages and put me in the CC for the Review Request

So yes, I'm willing to sponsor you (eventually) provided that you show a
somewhat better understanding of the packaging guidelines then you've done
sofar. (Sofar is ok, you're still learning I understand!).

So lets first get this package approved (but not yet imported as your not
sponsored), and then I think it would be a good idea for you to package
something else (maybe you already have) and get that reviewed too, usually after
2 packages most people are beginning to understand the guidelines good enough to
get sponsored (good enough for my taste atleast).

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

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


[Bug 194132] Review Request: yum-metadata-parser

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: yum-metadata-parser


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-12 02:04 EST ---
1. I don't want to make the yum package arch-specific
2. it's not a necessary component of yum - it's optional but a significant 
speed up
3. it's a separate package in upstream and I'd like to diminish the confusion
b/t upstream pkgs and fedora.
4. in the same way that we split out urlgrabber from yum base - I'd like this to
be done the same.
5. it makes for easier upgrades b/c if we fix a bug in the metadata-parser we
don't have to ship a whole new yum pkg - some people get nervous about updates
to their updating system for obvious reasons.

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

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


[Bug 192049] Review Request: gnash - GNU Flash player

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: gnash - GNU Flash player


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Attachment #130941|0   |1
is obsolete||




--- Additional Comments From [EMAIL PROTECTED]  2006-06-12 01:53 EST ---
Created an attachment (id=130942)
 --> (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=130942&action=view)
other file

sorry for the typo here is the other file.

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

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


[Bug 192049] Review Request: gnash - GNU Flash player

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: gnash - GNU Flash player


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-12 01:47 EST ---
Created an attachment (id=130941)
 --> (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=130941&action=view)
buildlog

here it is

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

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


[Bug 194132] Review Request: yum-metadata-parser

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: yum-metadata-parser


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-12 01:47 EST ---
Don't really see why it can't just be part of yum (who needs noarch?) but, if it
can't, go ahead.

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

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


[Bug 192912] Review Request: paps

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: paps


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-12 01:46 EST ---
> >My other concern with paps as it stands now
> > is its command line interface that we have to keep if we push it into Core. 
> > Instead of --fontscale and --family for example, it should have a single 
> > --font
> > that takes a Pango font description.
> 
> It sounds good. let me push it to upstream then.

Ok good.  While communicating with upstream, suggest that he ports paps to
pangocairo over the weekend, and we may actually have it next week :-).


> > Such a tool can be included in Pango upstream in fact.
> 
> Not as an example? it would be nice if it will continues to be maintained.

Yeah, I'm already trying to push the pango-view tool as a stable maintained tool
(instead of an example), and it's been packaged in pango-devel for a while now.
 I also have wanted to add PS/PDF output support to it for a while.

Main thing that needs to be done before pango-view can be used like paps is to
make it break text into paragraphs before laying out (for performance reasons.)
 I'm actually not sure that paps does that.  But anyway, I probably will get to
doing that sooner or later, but can't make any promise at this point.


> > If there's no strong reason for having paps or a similar tool in Core for 
> > FC6, I
> > suggest postponing this and working on the replacement tool.
> 
> We are focusing to improve the CIJK handling of the text printing and paps 
> was a
> better candidate at that time - this was being developed since PS/PDF backend
> for cairo was experimental or before that, which wasn't relied on - We have
> decided to work on paps because it may be close to become successful at RHEL5
> timeframe so that the improvement of CIJK text printing is our goal for RHEL5.
> plus, we have no enough time to make an another replacement from scratch so 
> that
> FC6t1 will be coming soon.

Ok, what about having a simple shell script called u2ps shipped and advertised
in Core with a documented command line interface, and make it call paps as the
implementation for now, but leave it open to switch to pango-view later on... 
The interface should be quite simple, a cat-like tool with the following 
options:

  --landscape
  --portrait (Default)
  --font
  (--font-size, --font-family, ...?)
  --margin
  --margin-left
  --margin-right
  --margin-top
  --margin-bottom
  --header=[TEXT]
  --footer=[TEXT]

That should be enough for now, and (except for footer?) paps supports the rest
already, with different namings possibly.

Another thing that should work in u2ps but is not currently working in paps is
setting default paper size based on LC_PAPER.  We can make the wrapper figure
out the paper and pass it to paps for now, and use it to set page size later
with pango-view.

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

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


[Bug 193929] Review Request: knetstats

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: knetstats


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-12 01:42 EST ---
(In reply to comment #12)
> Re: comment #10
> > -update-desktop-database is deprecated
> 
> It certainly is not deprecated (what makes you say that?).  

Hans, you want to comment on this ?

Anyone want to sponsor me for knetstats ? Hans ?



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

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


[Bug 192049] Review Request: gnash - GNU Flash player

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: gnash - GNU Flash player


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-12 01:41 EST ---
(In reply to comment #14)
> > This is with the above srpm?
> yes

I don't know how to reproduce that error,
so could you attach the full buildlog please?
(Feel free to gzip it to save space, thanks.:)

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

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


[Bug 192049] Review Request: gnash - GNU Flash player

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: gnash - GNU Flash player


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-12 01:30 EST ---
(In reply to comment #13)
> (In reply to comment #12)
> > build fails here on x86_64 due to multilib issue (it tryes to link the 64bit
> > files with 32bit libs)
> 
> This is with the above srpm?
yes
> And you have mesa-libGL-devel.x86_64 installed?
yes:
rpm -q mesa-libGL-devel.x86_64
mesa-libGL-devel-6.4.2-6



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

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


[Bug 192912] Review Request: paps

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: paps


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-12 01:03 EST ---
(In reply to comment #7)
> Just a note that most (if not all) of the code in libpaps.c is essentially
> deprecated now that cairo has PS/PDF backends being enabled for FC6.  It 
> should
> be a matter of a weekend's work to get someone write a paps-like a2ps
> replacement using pangocairo.
> 
> This can be fixed later of course, and the upstream author already knows about
> this and may in fact do it himself.

Yes, upstream is aware of that.

>  My other concern with paps as it stands 
> now
> is its command line interface that we have to keep if we push it into Core. 
> Instead of --fontscale and --family for example, it should have a single 
> --font
> that takes a Pango font description.

It sounds good. let me push it to upstream then.

> Such a tool can be included in Pango upstream in fact.

Not as an example? it would be nice if it will continues to be maintained.

> If there's no strong reason for having paps or a similar tool in Core for 
> FC6, I
> suggest postponing this and working on the replacement tool.

We are focusing to improve the CIJK handling of the text printing and paps was a
better candidate at that time - this was being developed since PS/PDF backend
for cairo was experimental or before that, which wasn't relied on - We have
decided to work on paps because it may be close to become successful at RHEL5
timeframe so that the improvement of CIJK text printing is our goal for RHEL5.
plus, we have no enough time to make an another replacement from scratch so that
FC6t1 will be coming soon.


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

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


[Bug 194305] Review Request: gtypist - GNU typing tutor

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: gtypist - GNU typing tutor


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-12 00:58 EST ---
Spec URL: http://gtypist.zing.fastmail.fm/gtypist.spec
SRPM URL: http://gtypist.zing.fastmail.fm/gtypist-2.7-3.src.rpm

* Mon Jun 12 2006 Zing <[EMAIL PROTECTED]> - 2.7-3
- do not use makeinstall macro
- rm info dir file from buildroot?

Could you explain the /usr/share/info/dir error?  I don't see that in my i386
builds; Is it a x86-64 build error?  Is the above the correct fix?

Thanks.

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

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


[Bug 192049] Review Request: gnash - GNU Flash player

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: gnash - GNU Flash player


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-12 00:28 EST ---
(In reply to comment #12)
> build fails here on x86_64 due to multilib issue (it tryes to link the 64bit
> files with 32bit libs)

This is with the above srpm?
And you have mesa-libGL-devel.x86_64 installed?

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

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


[Bug 192577] Review Request: perl-OpenFrame

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: perl-OpenFrame


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-12 00:16 EST ---
I take it there's been no progress, which is too bad.  The author still seems to
be active and the Openframe author posted a blog just a couple of weeks ago. 
Unfortunately I just can't find anything that would work as a blanket license
for this package other than a statement that everything in the upstream SVN
repository is released under an OSI-approved license. 

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

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


[Bug 192912] Review Request: paps

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: paps


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 23:49 EST ---
Just a note that most (if not all) of the code in libpaps.c is essentially
deprecated now that cairo has PS/PDF backends being enabled for FC6.  It should
be a matter of a weekend's work to get someone write a paps-like a2ps
replacement using pangocairo.

This can be fixed later of course, and the upstream author already knows about
this and may in fact do it himself.  My other concern with paps as it stands now
is its command line interface that we have to keep if we push it into Core. 
Instead of --fontscale and --family for example, it should have a single --font
that takes a Pango font description.

Such a tool can be included in Pango upstream in fact.

If there's no strong reason for having paps or a similar tool in Core for FC6, I
suggest postponing this and working on the replacement tool.

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

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


[Bug 192912] Review Request: paps

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: paps


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 23:13 EST ---
Ok, updated in Extras CVS, except dist tag.


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

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


[Bug 194787] Review Request: manaworld - 2D mmorpg game

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: manaworld - 2D mmorpg game


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|163778  |163779
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 22:53 EST ---
APPROVED

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

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


[Bug 194519] Review Request: q - Equational programming language

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: q - Equational programming language


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 22:49 EST ---
Hmmm.  Things are looking good, but what's /usr/lib/q/libtool?

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

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


[Bug 194730] Review Request: guichan - A Portable C++ GUI library for games using Allegro, SDL and OpenGL

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: guichan - A Portable C++ GUI library for games using 
Allegro, SDL and OpenGL


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 22:43 EST ---
Package imported and built into devel. Closing.
Thanks!

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

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


[Bug 194787] Review Request: manaworld - 2D mmorpg game

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: manaworld - 2D mmorpg game


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


Bug 194787 depends on bug 194730, which changed state.

Bug 194730 Summary: Review Request: guichan - A Portable C++ GUI library for 
games using Allegro, SDL and OpenGL
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=194730

   What|Old Value   |New Value

 Resolution||NEXTRELEASE
 Status|ASSIGNED|CLOSED



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

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


[Bug 194787] Review Request: manaworld - 2D mmorpg game

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: manaworld - 2D mmorpg game


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|[EMAIL PROTECTED] |[EMAIL PROTECTED]
OtherBugsDependingO|163776  |163778
  nThis||




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

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


[Bug 192912] Review Request: paps

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: paps


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 21:38 EST ---

> > Question
> > - Any way to make this so that we don't have to use autotools to build it?  
> > This
> > is fragile and ugly.
> 
> Hm, I could contain the chunks of Makefile.in in a patch though, it may causes
> an issue that is hard to maintain it.

Its really your choice.  Which ever method you feel more comfortable supporting.



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

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


[Bug 194730] Review Request: guichan - A Portable C++ GUI library for games using Allegro, SDL and OpenGL

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: guichan - A Portable C++ GUI library for games using 
Allegro, SDL and OpenGL


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|163778  |163779
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 21:35 EST ---
All MUSTFIX items fixed.  I learned another trick on f-e-l to fix the autoxxx
issue.  Instead of touching all of the Makefiles, you can restore the original
timestamps on configure/configure.in with:

touch -r configure configure.stamp
touch -r configure.in configure.in.stamp
%patch -p1
touch -r configure.stamp configure
touch -r configure.in.stamp configure.in

No need to change this before checking in, but's it's good to know for future
reference.

APPROVED

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

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


[Bug 192912] Review Request: paps

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: paps


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEEDINFO_REPORTER   |ASSIGNED




--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 21:27 EST ---
(In reply to comment #3)
> NEEDSWORK
> - Brew doesn't support %{?dist} tag anymore, so this will not evaluate when 
> built.

Yes, I was aware of this and was about to modify it before next build.

> - BuildRequires: automake
> - %makeinstall no longer acceptable according to guidelines.  Try "make 
> install
> DESTDIR=$RPM_BUILD_ROOT" instead.

Ok, I'll update.

> Question
> - Any way to make this so that we don't have to use autotools to build it?  
> This
> is fragile and ugly.

Hm, I could contain the chunks of Makefile.in in a patch though, it may causes
an issue that is hard to maintain it.

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

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


[Bug 194787] Review Request: manaworld - 2D mmorpg game

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: manaworld - 2D mmorpg game


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 21:08 EST ---
My Review:

MUST (OK)

 * rpmlint does not return any warning/errors (good)
 * Package is named according to Package Naming Guidelines
 * Spec filename matches package base name
 * Package license is open-source compliant
 * License field matches package's license
 * License file (COPYING) is included in %doc
 * Spec file is written in American English
 * Spec file is legible
 * Source file from package matches the upstream source with md5sum:
   b142f603c75819a04ac50d876776e92b  tmw-0.0.19.tar.gz
 * Package compiled and built fine in i386
 * BuildRequires is used well
 * Package does not contain locale files
 * Package does not contain shared library files
 * Package is not relocatable
 * Package owns all directories it creates
 * No duplicate files in %files
 * macro use consistent
 * No -doc subpackage needed
 * %doc contains only files that do not affect application runtime
 * No -devel subpackage needed
 * No pkgconfig files (.pc) needed
 * Package does not contain libtool archives (.la)
 * Desktop file installed with desktop-file-install
 * Package does not own other packages' directories

SHOULD (OK)

 * Scriptlets for icon cache are used in proper way
 * Package installs and runs just fine!



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

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


[Bug 191017] Review Request: eclipse-subclipse

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: eclipse-subclipse


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 19:10 EST ---
updated

http://www.marcanoonline.com/downloads/fedora/package_submissions/subclipse/eclipse-subclipse.spec
http://www.marcanoonline.com/downloads/fedora/package_submissions/subclipse/eclipse-subclipse-1.0.1-6.src.rpm

I applied the debuginfo workaround explained on bug #191014

Checked rpmlint warnings:
 non-standard-group Text Editors/Integrated Development Environments (IDE)
   I am using the same group that is using eclipse
 eclipse-subclipse invalid-license CPL, Apache Software License
   subclipse and svnClientAdapter has those differente licenses

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

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


[Bug 191015] Review Request: javasvn

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: javasvn


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 19:06 EST ---
updated

http://www.marcanoonline.com/downloads/fedora/package_submissions/subclipse/javasvn.spec
http://www.marcanoonline.com/downloads/fedora/package_submissions/subclipse/javasvn-1.0.4-3.src.rpm

I applied the debuginfo workaround explained on bug #191014

Checked rpmlint warnings:,
 invalid-license TMate License - http://tmate.org/svn/license.html
   What to to about it, it is a BSD license with an added clause about the
availiablity of the source code

 wrong-file-end-of-line-encoding for HTML files is not fixed because it is not
needed

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

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


[Bug 191014] Review Request: ganymed

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: ganymed


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 19:03 EST ---
updated

http://www.marcanoonline.com/downloads/fedora/package_submissions/subclipse/ganymed.spec
http://www.marcanoonline.com/downloads/fedora/package_submissions/subclipse/ganymed-209-3.src.rpm

I applied the debuginfo workaround, (there are still warnings for the java inner
classes, maybe find-debuginfo.sh must ignore class with $ in their names)

Checked rpmlint warnings, the only no fixed warnings are
"wrong-file-end-of-line-encoding" for HTML files.

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

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


[Bug 193929] Review Request: knetstats

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: knetstats


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 18:23 EST ---
Re: comment #10
> -update-desktop-database is deprecated

It certainly is not deprecated (what makes you say that?).  

If the app's .desktop file contains MimeType= entries, ScriptletSnippets are
clear that use of update-desktop-database in %post/%postun is required, (though
IMO, there shouldn't be need for additional Requires for this, just like the
icon cache case).

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

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


[Bug 194832] Review Request: pyxmms

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: pyxmms


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||194830
  nThis||




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

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


[Bug 194830] Review Request: CastPodder

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: CastPodder


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn||194832




--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 17:04 EST ---
Spec URL: http://www.knox.net.nz/~nodoid/CastPodder.spec
SRPM URL: http://www.knox.net.nz/~nodoid/CastPodder-5.0-2.src.rpm

Fixed the requires - d'oh!

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

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


[Bug 194832] New: Review Request: pyxmms

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.




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

   Summary: Review Request: pyxmms
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: normal
  Priority: normal
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL: http://www.knox.net.nz/~nodoid/pyxmms.spec
SRPM URL: http://www.knox.net.nz/~nodoid/pyxmms-2.06-1.src.rpm
Description: 

pyxmms is a set of python bindings for xmms

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

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


[Bug 193929] Review Request: knetstats

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: knetstats


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 16:55 EST ---
Hello Hans,
thanks for your remarks,

> Some (semi) initial remarks:
> -You BuildRequire qt-devel and you also BuildRequire kdelibs-devel, however
>  kdelibs-devel Requires qt-devel itself, so the BR qt-devel is redundant 
> remove 

Removed.

>  please.
> -update-desktop-database is deprecated, remove please as well as the belongen
>  Requires(%post[un])
> -as already mentioned in comment #8 you must properly update the icon cache in
>  %post[un] see the wiki scriptlets page
> -don't call /sbin/ldconfig in %post[un] this package does not seem to contain
>  any libs

Updated to

%post
touch --no-create %{_datadir}/icons/hicolor || :

%postun
touch --no-create %{_datadir}/icons/hicolor || :

> -instead of:
>  ###
>  rm -rf $RPM_BUILD_ROOT%{_datadir}/applnk/Internet/%{name}.desktop
> 
>  desktop-file-install --vendor fedora \
>  --add-category Network \
>  --dir $RPM_BUILD_ROOT%{_datadir}/applnk/Internet/ \
>  %{name}.desktop
>  ###
>  use:
>  ###
>  desktop-file-install --vendor fedora \
>  --add-category X-Fedora \
>  --add-category Network \
>  --delete-original \
>  --dir $RPM_BUILD_ROOT%{_datadir}/applications/ \
>  $RPM_BUILD_ROOT%{_datadir}/applnk/Internet/%{name}.desktop
>  ###
>  Notice that the changed --dir, desktop files should be installed in
>  %{_datadir}/applications/ nowadays. Also notice the --delete-original
>  replacing the seperate rm command and last notice the additional
>  "--add-category X-Fedora" param which all fedora packages should use.
>  Also don't forget to update %files for the changed dir.
> 

done .

> 
> 
> > Nevertheless, I was unable to fix the
> > 
> > mkdir -p /usr/share/doc/HTML/en/knetstats && cd 
> > /usr/share/doc/HTML/en/knetstats
> > && rm -f common && ln -s ../common commonln: creating symbolic link 
> > `common' to
> > `../common': Permission denied
> > 
> > change the build so that it doesnot try to install files outside of the
> buildroot ?
> > How can I 
> 
> Okay, this is because of a dirty hack in upstreams sources, there are 2 
> possible
> fixes:
> 1) ignore the error (scons already does this) and add the following at the 
> end 
>of %install:
>ln -s ../common $RPM_BUILD_ROOT%{_datadir}/doc/HTML/en/knetstats/common
> 2) patch admin/kde.py to properly honor DESTDIR.
> 

I was able to overcome that Permission Denied issue with 
%install
rm -fr $RPM_BUILD_ROOT
scons prefix=$RPM_BUILD_ROOT%{_prefix} install

but with rpmlint -i knetstats-1.5-5.i386.rpm, Ive fallen on
W: knetstats dangling-relative-symlink /usr/share/doc/HTML/en/knetstats/common
../common
The relative symbolic link points nowhere.

With f13's advice I've patched admin/kde.py accordingly.

Again with f13's advise, I've included scons as BR.

Updated:
Spec URL: http://beta.glwb.info/knetstats/knetstats.spec
SRPM URL: http://beta.glwb.info/knetstats/knetstats-1.5-5.src.rpm


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

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


[Bug 194470] Review Request: php-magickwand

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: php-magickwand


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 16:40 EST ---
Neither trick works :|
I'll look into this for you, but it might take a day or 2 before I find the time
for this.


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

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


[Bug 194830] New: Review Request: CastPodder

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.




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

   Summary: Review Request: CastPodder
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: normal
  Priority: normal
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL: http://www.knox.net.nz/~nodoid/CastPodder.spec
SRPM URL: http://www.knox.net.nz/~nodoid/CastPodder-5.0-1.src.rpm
Description: 

Cast podder is a media agregator (it's primary use is as a pod caster). It is 
written in Python and uses wxPython.

I can see problems with the spec file, but would appreciate some advice on 
where some of the parts should go as currently things go to /opt

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

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


[Bug 193342] Review Request: cegui - Free library providing windowing and widgets for graphics APIs / engines

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: cegui - Free library providing windowing and widgets 
for graphics APIs / engines


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|163778  |163779
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 16:24 EST ---
(In reply to comment #25)
> No problem Hans, it's been a good learning process and I've a learnt a lot :-)
> Here's the latest
> 
> http://dribble.org.uk/cegui.spec
> http://dribble.org.uk/cegui-0.4.1-8.src.rpm
> 

Looks good, approved!

> PS.
> I have submitted the appropriate patches for this and the others upstream.

Good!


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

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


[Bug 194470] Review Request: php-magickwand

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: php-magickwand


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 16:12 EST ---
I can't reproduce the binary-or-shlib-defines-rpath on my ix86 system, looks 
like it is x86_64 related. Is adding --disable-rpath to %configure enough or 
will I have to something like this - used in php-extras, IIRC:

# Cause libtool to avoid passing -rpath when linking
# (this hack is well-known as "libtool rpath workaround")
sed -i 's|^hardcode_libdir_flag_spec|hardcode_libdir_flag_spec=" -
D__LIBTOOL_IS_A_FOOL__ "|' libtool

And AFAIK, only php-extras is using this hack of the php packages in Extras...

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

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


[Bug 193929] Review Request: knetstats

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: knetstats


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 16:08 EST ---
(In reply to comment #9)
> Updated:
> Spec URL: http://beta.glwb.info/knetstats/knetstats.spec
> SRPM URL: http://beta.glwb.info/knetstats/knetstats-1.5-4.src.rpm
> 
> I have made the necessary modifications and read the guidelines as well :)
> Now I need to put that into action :)
> 

Okay,

Some (semi) initial remarks:
-You BuildRequire qt-devel and you also BuildRequire kdelibs-devel, however
 kdelibs-devel Requires qt-devel itself, so the BR qt-devel is redundant remove 
 please.
-update-desktop-database is deprecated, remove please as well as the belongen
 Requires(%post[un])
-as already mentioned in comment #8 you must properly update the icon cache in
 %post[un] see the wiki scriptlets page
-don't call /sbin/ldconfig in %post[un] this package does not seem to contain
 any libs
-instead of:
 ###
 rm -rf $RPM_BUILD_ROOT%{_datadir}/applnk/Internet/%{name}.desktop

 desktop-file-install --vendor fedora \
 --add-category Network \
 --dir $RPM_BUILD_ROOT%{_datadir}/applnk/Internet/ \
 %{name}.desktop
 ###
 use:
 ###
 desktop-file-install --vendor fedora \
 --add-category X-Fedora \
 --add-category Network \
 --delete-original \
 --dir $RPM_BUILD_ROOT%{_datadir}/applications/ \
 $RPM_BUILD_ROOT%{_datadir}/applnk/Internet/%{name}.desktop
 ###
 Notice that the changed --dir, desktop files should be installed in
 %{_datadir}/applications/ nowadays. Also notice the --delete-original
 replacing the seperate rm command and last notice the additional
 "--add-category X-Fedora" param which all fedora packages should use.
 Also don't forget to update %files for the changed dir.




> Nevertheless, I was unable to fix the
> 
> mkdir -p /usr/share/doc/HTML/en/knetstats && cd 
> /usr/share/doc/HTML/en/knetstats
> && rm -f common && ln -s ../common commonln: creating symbolic link `common' 
> to
> `../common': Permission denied
> 
> change the build so that it doesnot try to install files outside of the
buildroot ?
> How can I 

Okay, this is because of a dirty hack in upstreams sources, there are 2 possible
fixes:
1) ignore the error (scons already does this) and add the following at the end 
   of %install:
   ln -s ../common $RPM_BUILD_ROOT%{_datadir}/doc/HTML/en/knetstats/common
2) patch admin/kde.py to properly honor DESTDIR.


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

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


[Bug 193342] Review Request: cegui - Free library providing windowing and widgets for graphics APIs / engines

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: cegui - Free library providing windowing and widgets 
for graphics APIs / engines


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 16:08 EST ---
No problem Hans, it's been a good learning process and I've a learnt a lot :-)
Here's the latest

http://dribble.org.uk/cegui.spec
http://dribble.org.uk/cegui-0.4.1-8.src.rpm

PS.
I have submitted the appropriate patches for this and the others upstream.

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

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


[Bug 174866] Review Request: polypaudio: Improved Linux sound server

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: polypaudio: Improved Linux sound server


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|163776  |
  nThis||




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

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


[Bug 193342] Review Request: cegui - Free library providing windowing and widgets for graphics APIs / engines

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: cegui - Free library providing windowing and widgets 
for graphics APIs / engines


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 15:22 EST ---
Ok, solved, my bad:
The output of "libtool --help --mode=link" contains:
  -rpath LIBDIR the created library will eventually be installed in LIBDIR

So this is a different rpath option as -Wl,rpath passed to gcc / g++ or -rpath
passed to ld. I guess it might cause a -Wl,rpath in the link command when LIBDIR
is a non standard dir and hence it has the name, but appeareantly libtool is
smart enough not to pass -rpath when LIBDIR (as passed to libtool) is a standard
libdir (which it is in our case: /usr/lib[64]).

I've verified this with my local installed build of -5 (+tinyxml + 64 bit 
fixes):
[EMAIL PROTECTED] ~]$ objdump -x /usr/lib64/libCEGUILuaScriptModule.so | grep 
-i rpath
[EMAIL PROTECTED] ~]$

So the -rpath in this case can stay, soryyy. Still this dance wasn't in
(entirely) vain as the other -rpath's where really a problem.


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

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


[Bug 174866] Review Request: polypaudio: Improved Linux sound server

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: polypaudio: Improved Linux sound server


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||NOTABUG




--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 15:19 EST ---
Pierre: Go ahead and open a new review request for this package. 

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

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


[Bug 191014] Review Request: ganymed

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: ganymed


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 14:31 EST ---
(In reply to comment #9)
> # Link source files to fix -debuginfo generation.
> rm -f ch
> ln -s src/ch

Yeah that should work. find-debuginfo.sh seems to be broken for java packages
right now. It's on my list of things to investigate for FC6 but if someone wants
to help out here that would be great.


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

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


[Bug 176374] Review Request: nagios-plugins

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: nagios-plugins


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 14:24 EST ---
One step ahead of me.  It is now built cleanly.  I'm going to wait and make sure
everything looks good in devel before I build it for FC-X.

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

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


[Bug 193342] Review Request: cegui - Free library providing windowing and widgets for graphics APIs / engines

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: cegui - Free library providing windowing and widgets 
for graphics APIs / engines


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 14:18 EST ---
The Makefile.in now has:

libCEGUILuaScriptModule_la_LIBADD = -llua -ltolua++

rather than adding them directly on the link line. Makefile.am is now using:

libCEGUILuaScriptModule_la_LIBADD = -llua -ltolua++

instead of the LIBS variable. I have a problem, if remove "-rpath $(libdir)"
from the Makefiles, it fails on installation with:

/usr/bin/install -c .libs/libCEGUILuaScriptModule.lai
/usr/local/lib/libCEGUILuaScriptModule.la
/usr/bin/install: cannot stat `.libs/libCEGUILuaScriptModule.lai': No such file
or directory

If "-rpath $(libdir)" is present, it installs OK. I'm not sure what to do about
this. 

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

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


[Bug 191589] Review Request: qt4-qsa: Qt Script for Applications

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: qt4-qsa: Qt Script for Applications


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

Summary|Review Request: qsa: Qt |Review Request: qt4-qsa: Qt
   |Script for Applications |Script for Applications




--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 14:10 EST ---
The maning is only a little problem:)

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

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


[Bug 191589] Review Request: qsa: Qt Script for Applications

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: qsa: Qt Script for Applications


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 13:56 EST ---
I forgot about the need for 2 qsa's.  The packaging guidelines are relatively
clear on this, that addons to a package foo should be named (something like)
foo-addon.

In our case(s), I'd suggest naming *this* one qt4-qsa, and the other (qt3) one
qt-qsa so it is immediately clear looking at the name which qt is assiciated
with which qsa pkg.

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

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


[Bug 194011] Review Request: curry - MÃ ¼nster Curry compiler

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: curry -  Münster Curry compiler


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 13:09 EST ---
Built on FC4, FC5 and FC6. Added entry in owners.list file.
Thanks again for the review!

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

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


[Bug 194051] Review Request: abcMIDI - ABC to/from MIDI conversion utilities

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: abcMIDI - ABC to/from MIDI conversion utilities


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 12:44 EST ---
Built on FC4, FC5 and FC6. Added entry to owners.list file.
Thanks for the review!

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

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


[Bug 193960] Review Request: perl-Net-LibIDN

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: perl-Net-LibIDN


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 12:44 EST ---
Yes, that sounds well. BTW, I've got updated all four packages on June, 9th to 
have a better rpmlint output. And as I'm new to the Fedora Extras stuff, just 
contact me, if there's something...

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

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


[Bug 191589] Review Request: qsa: Qt Script for Applications

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: qsa: Qt Script for Applications


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 12:04 EST ---
No because it is good enugh when your package wil be called qsa3. The additional
qt is not needed.

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

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


[Bug 191589] Review Request: qsa: Qt Script for Applications

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: qsa: Qt Script for Applications


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 11:37 EST ---
Thaks, I'll try to roll up a 1.1.4 package in the next few days. As for the
naming convention, it seems stat if qt 4.1.x will make its way into fc5, it will
be called qt4, while qt 3.3.x will be called just qt. So why not to rename your
package to qsa4 (or qsa-qt4), while mine will be called just qsa (or qsa-qt3)?
Wouldn't it be more consistent? I don't mean to be intrusive.

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

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


[Bug 194551] Review Request: ifd-egate

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: ifd-egate


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 11:34 EST ---
Created an attachment (id=130931)
 --> (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=130931&action=view)
Suggested corresponding specfile patch


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

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


[Bug 194551] Review Request: ifd-egate

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: ifd-egate


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 11:30 EST ---
Created an attachment (id=130930)
 --> (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=130930&action=view)
Suggested Makefile patch

I don't have hardware to test this with, but here's some observations:

- Cosmetic: placement of %post is somewhat unusual, it's usually after %clean
- pcscd not restarted on final removal
- $RPM_OPT_FLAGS not honored
- The makefile hunk of the timing patch (which seems misplaced in this patch)
  defines USB_DRIVER_DIR which is unused.
- USB_CFLAGS, USB_LDFLAGS, PCSC_LDFLAGS undefined
- Unowned install directories
- License is a bit unclear, both LGPL and BSD are included
- "This should be installed with the base OS" sounds odd in the %description

Attached is a suggested replacement for the Makefile hunk in the timing patch.

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

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


[Bug 193820] Review Request: libcm

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: libcm


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|NEEDINFO_REPORTER




--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 11:29 EST ---
NEEDSWORK
- Why no URL?  No way to verify upstream sources
- No changelog

Rpmlint Errors:

E: libcm no-description-tag
E: libcm no-changelogname-tag
W: libcm no-url-tag
W: libcm devel-file-in-non-devel-package /usr/lib64/pkgconfig/cm.pc
E: libcm-debuginfo no-changelogname-tag
W: libcm-debuginfo no-url-tag
E: libcm-devel no-changelogname-tag
W: libcm-devel no-url-tag


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

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


[Bug 176374] Review Request: nagios-plugins

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: nagios-plugins


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 11:24 EST ---
Mike,

Adding the lines below to the %install section should fix it

%ifarch ppc ppc64
%{__rm} -f %{buildroot}%{_libdir}/nagios/plugins/check_sensors
%endif

jpo

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

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


[Bug 191589] Review Request: qsa: Qt Script for Applications

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: qsa: Qt Script for Applications


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 11:19 EST ---
You can use it, but it will need mutch changes. Because for Qt3 you must use an
other version on QSA. And the you shut call your package qsa3 so that there will
be no conflicts.

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

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


[Bug 192912] Review Request: paps

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: paps


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEEDINFO|NEEDINFO_REPORTER




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

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


[Bug 194551] Review Request: ifd-egate

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: ifd-egate


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |NEEDINFO
   Flag||needinfo?([EMAIL PROTECTED]
   ||i)2006




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

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


[Bug 192912] Review Request: paps

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: paps


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|needinfo?2006   |




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

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


[Bug 194320] Review Request: im-chooser

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: im-chooser


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|188265  |
  nThis||




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

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


[Bug 192460] Review Request: pygobject2-2.10.1

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: pygobject2-2.10.1


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|188265  |
  nThis||




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

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


[Bug 191589] Review Request: qsa: Qt Script for Applications

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: qsa: Qt Script for Applications


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 11:06 EST ---
Would you mind if I used your spec to prepare a qsa package for qt3/fc5?

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

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


[Bug 192912] Review Request: paps

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: paps


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |NEEDINFO
 AssignedTo|[EMAIL PROTECTED]  |[EMAIL PROTECTED]
 CC||[EMAIL PROTECTED]
OtherBugsDependingO|188265  |188267
  nThis||
   Flag||needinfo?2006




--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 11:04 EST ---
NEEDSWORK
- Brew doesn't support %{?dist} tag anymore, so this will not evaluate when 
built.
- BuildRequires: automake
- %makeinstall no longer acceptable according to guidelines.  Try "make install
DESTDIR=$RPM_BUILD_ROOT" instead.


Question
- Any way to make this so that we don't have to use autotools to build it?  This
is fragile and ugly.

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

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


[Bug 194607] Review Request: openssl-ibmca

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: openssl-ibmca


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |NEEDINFO
 CC||[EMAIL PROTECTED]
   Flag||needinfo?([EMAIL PROTECTED]
   ||m)2006




--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 10:52 EST ---
Indeed.  I don't think we're interested in carrying s390(x) specific packages in
Core, given that Core doesn't ship on those platforms.

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

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


[Bug 194606] Review Request: libica

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: libica


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |NEEDINFO
 CC||[EMAIL PROTECTED]
   Flag||needinfo?([EMAIL PROTECTED]
   ||m)2006




--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 10:51 EST ---
I'd like to know too.  Devel freeze for FC6Test1 is Wed, would need to complete
review by then for inclusion in Test1

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

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


[Bug 194551] Review Request: ifd-egate

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: ifd-egate


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]  |[EMAIL PROTECTED]
 CC||[EMAIL PROTECTED]
OtherBugsDependingO|188265  |188267
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 10:49 EST ---
Ville, would you be willing to continue the review?

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

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


[Bug 193820] Review Request: libcm

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: libcm


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|[EMAIL PROTECTED]  |[EMAIL PROTECTED]
 CC||[EMAIL PROTECTED]
OtherBugsDependingO|188265  |188267
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 10:47 EST ---
I take it this is necessary to build metacity (and thus needs to be in Core?)

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

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


[Bug 193187] Review Request: pcsc-lite & ccid

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: pcsc-lite & ccid


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]  |[EMAIL PROTECTED]
 CC||[EMAIL PROTECTED]
OtherBugsDependingO|188265  |188268
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 10:46 EST ---
Accepting the package.  Bill already approved for Core.  What changes would we
need to make in Anaconda or Comps to support these packages?

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

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


[Bug 194594] Review Request: wireshark

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: wireshark


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |NEEDINFO
   Flag||needinfo?([EMAIL PROTECTED])
   ||2006




--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 10:41 EST ---
Just a reminder, Devel freeze for FC6Test1 is this Wed.  This package will have
to be ready by then for inclusion into Test1.

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

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


[Bug 191200] Review Request: lvm2-cluster

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: lvm2-cluster


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 10:40 EST ---
Development freeze for FC6Test1 is Wed.  This package will need to be ready by
then for inclusion in Test1.

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

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


[Bug 194202] Review Request: scim-bridge (gtkimm module in C)

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: scim-bridge (gtkimm module in C)


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|MODIFIED|CLOSED
 Resolution||RAWHIDE




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

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


[Bug 193191] Review Request: rename of notify-daemon to notification-daemon

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: rename of notify-daemon to notification-daemon


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 10:36 EST ---
John, did this get built into Rawhide?  If so, please close the bug.  Thanks.

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

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


[Bug 193157] Review Request: system-config-selinux

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: system-config-selinux


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|188268  |163779
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 10:35 EST ---
Since Bill thinks this is for Extras, I'll re-block on FE-ACCEPT since it passed
review.  Dan, do you currently have sponsership within Extras yet?

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

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


[Bug 194132] Review Request: yum-metadata-parser

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: yum-metadata-parser


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 10:33 EST ---
Bill, can I get the go ahead here to add this in?  I think we'll all be happy to
have this.

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

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


[Bug 194519] Review Request: q - Equational programming language

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: q - Equational programming language


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 10:14 EST ---
I have called the apache module "mod_q" following the usual naming rules.
I refrained from separating the octave package, since using the octave
module simply fails with an error that octave could not be found.

http://math.ifi.unizh.ch/fedora/5/i386/SRPMS.gemi/q-7.1-0.2.rc2.src.rpm

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

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


[Bug 193479] Review Request: xwrits

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: xwrits


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 09:41 EST ---
Sorry for the delay. The current spec/src.rpm were the new ones, but you're
correct that I should have bumped the release number.

I've done that and uploaded new versions:

http://people.redhat.com/jlayton/xwrits/xwrits.spec
http://people.redhat.com/jlayton/xwrits/xwrits-2.22-2.src.rpm


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

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


[Bug 190345] Review Request: vdr-femon - DVB frontend status monitor plugin for VDR

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: vdr-femon - DVB frontend status monitor plugin for VDR


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 09:37 EST ---
http://cachalot.mine.nu/5/SRPMS/vdr-femon-1.0.1-1.src.rpm

* Mon Jun 12 2006 Ville Skyttä  - 1.0.1-1
- 1.0.1, build for VDR 1.4.1.


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

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


[Bug 190343] Review Request: VDR - Video Disk Recorder

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: VDR - Video Disk Recorder


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 09:36 EST ---
http://cachalot.mine.nu/5/SRPMS/vdr-1.4.1-1.src.rpm

* Mon Jun 12 2006 Ville Skyttä  - 1.4.1-1
- 1.4.1, liemikuutio 1.6.


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

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


[Bug 191589] Review Request: qsa: Qt Script for Applications

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: qsa: Qt Script for Applications


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 09:20 EST ---
So here comes the next now with contains in the debug file:)
http://prdownloads.sourceforge.net/qsmartcard/qsa.spec?download
http://prdownloads.sourceforge.net/qsmartcard/qsa-1.2.1-10FC5.src.rpm?download

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

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


[Bug 193929] Review Request: knetstats

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: knetstats


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 08:19 EST ---
Updated:
Spec URL: http://beta.glwb.info/knetstats/knetstats.spec
SRPM URL: http://beta.glwb.info/knetstats/knetstats-1.5-4.src.rpm

I have made the necessary modifications and read the guidelines as well :)
Now I need to put that into action :)

Nevertheless, I was unable to fix the

mkdir -p /usr/share/doc/HTML/en/knetstats && cd /usr/share/doc/HTML/en/knetstats
&& rm -f common && ln -s ../common commonln: creating symbolic link `common' to
`../common': Permission denied

change the build so that it doesnot try to install files outside of the 
buildroot ?
How can I 

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

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


[Bug 193960] Review Request: perl-Net-LibIDN

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: perl-Net-LibIDN


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 07:45 EST ---
Ah I see, so it seems (from the large amount of Review Requests) that you're
seriously interested in becoming an FE contributer.

I'm able to sponsor people and as said I believe that you're seriously
interested (good!) So I would like to sponsor you once I get to know you a
little better. For this I would like to suggest that we work together to get 3
of your packages approved and once 3 have passed review I'll sponsor you.

Does this sound like a plan?

Notice that I've already started by reviewing php-magickwand.


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

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


[Bug 181445] Review Request: php-shout

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: php-shout


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 07:36 EST ---
Brandon,

Matthias currently is a bit swamped, so let me know if your still interested and
then I'll take a look.


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

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


[Bug 185325] Review Request: sparse

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: sparse


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 07:34 EST ---
Are you still interested in this? Ifso it would be nice if you could provide a 
complete SRPM as requested in Comment #9 .

Shouldn't you respond within one week from now, I'll presume you have
lost interest into getting this package into FE and close this PR.


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

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


[Bug 177580] Review Request: lat (LDAP Administration Tool)

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: lat  (LDAP Administration Tool)


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]  |[EMAIL PROTECTED]
OtherBugsDependingO|163776  |163778
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 07:23 EST ---
MUST:
=
* rpmlint output is:
E: lat hardcoded-library-path in /usr/lib
E: lat no-binary
E: lat only-non-binary-in-usr-lib
All because this is a mono app, all ok.
* Package and spec file named appropriately
* Packaged according to packaging guidelines
0 License (GPL) ok, but license file noy included
* spec file is legible and in Am. English.
* Source matches upstream
* Compiles and builds on devel-x86_64
* BR: ok
* Locales properly handled
* No shared libraries
* Not relocatable
0 Package does not own all dirs (see Must fix below)
* No duplicate files & Permissions ok
* %clean & macro usage OK
* Contains code only
0 no %doc
* no -devel package needed, no libs / .la files.
* .desktop file properly installed


MUST fix:
=
* %post and %postun call update-desktop-database this is deprecated and must
  be removed.
* no %doc, not even for COPYING!
* unowned dirs: /usr/share/omf /usr/share/gnome /use/gnome/help, I know this
  is somwhat controversial, but lat doesnot depend on any packages which provide
  these dirs, so it should own them. Feel free to discuss this on f-e-l if you 
  disagree.
* /usr/pixmaps/lat.png must be moved to /usr/share/icons/hicolor/22x22/apps/
  in %install and then the icon-cache should be properly updated in %post and
  %postun (see wiki scriptlets page).

Should fix:
===
* in %install you write:
# remove libtool archives
/usr/bin/find %{buildroot}%{_libdir} -name '*.la' -exec %{__rm} -f {} \;

This is a mono app, does it install .la files?


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

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


[Bug 194519] Review Request: q - Equational programming language

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: q - Equational programming language


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





--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 06:27 EST ---
(In reply to comment #9)
> I read up a bit and it does look like this is hopeless on any 64-bit arch.  
> So I
> suggest just doing an ExcludeArch and opening the usual tracking bug.  Maybe
> some 64-bit experts would be able to lend a hand.
Ok.

> W: q symlink-should-be-relative /usr/bin/gqbuilder
> /usr/share/q/gqbuilder/gqbuilder.q
In fact this script depends on gnocl (GTK/Gnome bindings for Tcl), which
I intend to submit some time. Probably best to remove this for now.

> E: q info-dir-file /usr/share/info/dir
> Don't package this file.
How is it that this file is sometimes created, sometimes not?

> Having a build, I can look at the dependency list.  It looks like this will 
> pull
> in all of TCL and Tk, plus Imagemagick and unixODBC.  That's pretty heavy, but
> not insane as if it pulled in octave or a web server.  By the way, it doesn't
> look like you build the Apache module.  I doubt it's worth it to do so,
> honestly, but you probably want to take that out of the description.
The octave module is built, however it isn't linked agains liboctave, so
there is no dependency. However using the module requires octave to be present.
Should we leave it as is, or create a separate package with the module,
that depends on octave?
Also, I try to build the apache module as a separate package, with the
name q-httpd.

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

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


[Bug 192049] Review Request: gnash - GNU Flash player

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: gnash - GNU Flash player


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 03:46 EST ---
build fails here on x86_64 due to multilib issue (it tryes to link the 64bit
files with 32bit libs)
---
g++ -shared -nostdlib /usr/lib/gcc/x86_64-redhat-linux/4.1.1/../../../../lib64/c
rti.o /usr/lib/gcc/x86_64-redhat-linux/4.1.1/crtbeginS.o  .libs/plugin.o .libs/n
pn_gate.o .libs/npp_gate.o .libs/np_entry.o  -Wl,--rpath -Wl,/home/dragoran/rpm/
BUILD/gnash-0.7.1/backend/.libs -Wl,--rpath -Wl,/home/dragoran/rpm/BUILD/gnash-0
.7.1/server/.libs -Wl,--rpath -Wl,/home/dragoran/rpm/BUILD/gnash-0.7.1/libgeomet
ry/.libs -Wl,--rpath -Wl,/home/dragoran/rpm/BUILD/gnash-0.7.1/libbase/.libs -L/u
sr/lib -L/usr/lib64 -L/usr/lib64/mozilla/plugins -lSDL /usr/lib64/libgtkglext-x1
1-1.0.so /usr/lib64/libgdkglext-x11-1.0.so -lGLU /usr/lib/libGL.so -lpangox-1.0
-lgtk-x11-2.0 -lgdk-x11-2.0 -lgdk_pixbuf-2.0 -lpangocairo-1.0 -lgobject-2.0 -lgm
odule-2.0 -ldl -lpango-1.0 -lcairo -latk-1.0 -lglib-2.0 -lxml2 -lz -ljpeg -lpng
-logg ../backend/.libs/libgnashbackend.so ../server/.libs/libgnashasobjs.so ../s
erver/.libs/libgnashserver.so ../libgeometry/.libs/libgnashgeo.so ../libbase/.li
bs/libgnashbase.so -lSDL_mixer -lrt -lX11 -lXi -lXmu -L/usr/lib/gcc/x86_64-redha
t-linux/4.1.1 -L/usr/lib/gcc/x86_64-redhat-linux/4.1.1/../../../../lib64 -L/lib/
../lib64 -L/usr/lib/../lib64 -lstdc++ -lm -lc -lgcc_s /usr/lib/gcc/x86_64-redhat
-linux/4.1.1/crtendS.o /usr/lib/gcc/x86_64-redhat-linux/4.1.1/../../../../lib64/
crtn.o  -m64 -mtune=generic -Wl,--export-dynamic -Wl,--export-dynamic -Wl,-sonam
e -Wl,libgnashplugin.so -o .libs/libgnashplugin.so
/usr/lib/libGL.so: could not read symbols: File in wrong format
collect2: ld returned 1 exit status
make[3]: *** [libgnashplugin.la] Error 1
make[3]: Leaving directory `/home/dragoran/rpm/BUILD/gnash-0.7.1/plugin'
make[2]: *** [all-recursive] Error 1
make[2]: Leaving directory `/home/dragoran/rpm/BUILD/gnash-0.7.1/plugin'
make[1]: *** [all-recursive] Error 1
make[1]: Leaving directory `/home/dragoran/rpm/BUILD/gnash-0.7.1'
make: *** [all] Error 2


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

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


[Bug 194810] Review Request: Openbox

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: Openbox


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||194811
  nThis||




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

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


[Bug 194811] Review Request: obconf

2006-06-11 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: obconf


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn||194810




--- Additional Comments From [EMAIL PROTECTED]  2006-06-11 02:57 EST ---
Adding a bug dependency for Openbox since it makes use of openbox-devel.

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

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