[Bug 483730] Review Request: kde-plasma-translatoid - A Google Translation Plasmoid

[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=483730





--- Comment #1 from Jaroslav Reznik <jreznik@xxxxxxxxxx>  2009-02-03 08:29:48 EDT ---
rpmlint output:

kde-plasma-translatoid.src: E: invalid-spec-name
- missing 's' in spec filename

kde-plasma-translatoid.src: E: no-cleaning-of-buildroot %install
- same as for #483728

kde-plasma-translatoid.src: W: summary-not-capitalized translator using google
translator
- capitalize summary, description too

kde-plasma-translatoid.x86_64: W: spurious-executable-perm
/usr/share/doc/kde-plasma-translatoid-0.4.1/README


kde-plasma-translatoid.x86_64: W: incoherent-version-in-changelog 0.4.1
['0.4.1-1.fc10', '0.4.1-1']
- same as for #483728


- license GPLv2+
- I'm not sure about Google trademarks, let me check
- URL should be
http://www.kde-look.org/content/show.php/translatoid?content=97511
- you should use 97511-translatoid%{version} in Source0 
- remove flags due to Fedora policy
- use build root consistently - rm -rf $RPM_BUILD_ROOT -> rm -rf ${buildroot}

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

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

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