[Bug 698362] Review Request: writetype - Light word processor

bugzilla at redhat.com bugzilla at redhat.com
Sat Jun 18 20:25:14 UTC 2011


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=698362

--- Comment #7 from PRABIN KUMAR DATTA <linux.n.pkd at gmail.com> 2011-06-18 16:25:13 EDT ---
Updated package:-
Spec URL: http://prabindatta.fedorapeople.org/SPECS/writetype.spec
SRPM URL:
http://prabindatta.fedorapeople.org/SRPMS/writetype-1.2.130-5.fc15.src.rpm

> 1. The license is GPLv3+ and not GPLv3. (notice the line "or (at your option)
> any later version.")
Updated.

> 2. Strange premissions on the desktop file. Why?
After extraction from tarball file permission for  writetype.desktop is "755".
Now, in Makefile file under install section, code to install desktop file is:
"cp writetype.desktop $(ICONDIR)/writetype.desktop"
That why, the file permission which should have to 644 is 755.

> 3. Doesn't build in mock, missing build requirement on desktop-file-utils
Updated. But, here I like to mention one thing that pyttsx is required for this
package which in under review request[1]. Thou, pyttsx is not mandatory since
enchant and festival are already there performing similar function.

-
[1]. https://bugzilla.redhat.com/show_bug.cgi?id=702998

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



More information about the package-review mailing list