[Bug 676544] Review Request: Pidgin-logviewer - User friendly and intuitive way of viewing chat logs in Pidgin

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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


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

Rahul Sundaram <metherid@xxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |metherid@xxxxxxxxx

--- Comment #1 from Rahul Sundaram <metherid@xxxxxxxxx> 2011-02-10 23:30:20 EST ---

I am not a sponsor and hence cannot approve but I will do a review.  You do not
need to define a buildroot or have a %clean section any longer in the spec
file.  These are defined by RPM automatically. 

Also you the license tag in the spec says GPLv2+ but the logplugin.c under src
directory which is the only source file has a copyright notice that says:

--
/* Improved Log Viewer for Pidgin.
 * Tirtha Chatterjee
 * This code is licensed under GPL v2
 */ 
---

You need to talk to upstream on whether this is just GPLv2 or whether it is
GPLv2 or later.  The plan for the author is to eventually merge this into
Pidgin and hence I think GPLv2 is preferable but do confirm first.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]