Bug 460928

Summary:	Review Request: sugar-chat
Product:	[Fedora] Fedora	Reporter:	Jeremy Katz <katzj>
Component:	Package Review	Assignee:	Nobody's working on this, feel free to take it <nobody>
Status:	CLOSED RAWHIDE	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	rawhide	CC:	fedora-package-review, notting, pbrobinson, simon
Target Milestone:	---	Flags:	simon: fedora-review+
Target Release:	---
Hardware:	All
OS:	Linux
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2008-09-18 17:54:25 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Jeremy Katz 2008-09-02 17:38:43 UTC

Spec URL:
http://katzj.fedorapeople.org/review/sugar-chat/sugar-chat.spec
SRPM URL:
http://katzj.fedorapeople.org/review/sugar-chat/sugar-chat-46-1.fc10.src.rpm
Description: Chat client for Sugar Environment

Comment 1 Simon Schampijer 2008-09-09 08:17:03 UTC

rpmlint looks good. 

I am not so happy that we have to remove pseudo.po in the rpm. It gets into the tarball because it is in git - the problem is that ''we need to remove it from Pootle as well, and right now, there is no straightforward way to remove stuff from Pootle (thanks to the convoluted symlinking scheme that we use)''. So an option would be to ignore the file when we create the tarball and remove the file completely for 0.84.

I guess we need to add the BR gettext temporarily as well like in the terminal activity.

Comment 2 Simon Schampijer 2008-09-15 06:01:36 UTC

So we fixed sugar-toolkit that is why you can remove the pseudo.po part from the spec http://koji.fedoraproject.org/koji/buildinfo?buildID=63008

Morgan has released a new version of chat that contains the license field in activity.info http://dev.laptop.org/pub/sugar/sources/chat-activity/Chat-47.tar.bz2

All the rest looks good.

Comment 3 Jeremy Katz 2008-09-17 18:25:50 UTC

Cool, updated to 47.

New Package CVS Request
=======================
Package Name: sugar-chat
Short Description: Chat client for Sugar
Owners: katzj
Branches: F-9
InitialCC: mpg

Comment 4 Kevin Fenzi 2008-09-17 18:56:21 UTC

cvs done.

Comment 5 Jeremy Katz 2008-09-18 17:54:25 UTC

Building

Comment 6 Peter Robinson 2010-06-10 21:09:54 UTC

Package Change Request
======================
Package Name: sugar-chat
New Branches: EL-6
Owners: pbrobinson sdz

Comment 7 Kevin Fenzi 2010-06-11 04:36:15 UTC

Have you checked with tuxbrewr to see if he would like to maintain in EPEL?