[Bug 567348] Review Request: dreampie - A graphical cross-platform interactive Python shell

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

--- Comment #2 from Ionuț Arțăriși <mapleoin@xxxxxxxxxxx> 2010-03-03 12:36:13 EST ---
Thank you, Fabian

(In reply to comment #1)
> Just csome quick comments on your spec file:
> 
> - Is 'CFLAGS="$RPM_OPT_FLAGS"' really needed? It's a noarch package.
> - Please use '%global' instead of '%define'
>  
> https://fedoraproject.org/wiki/Packaging:Guidelines#.25global_preferred_over_.25define

Done.

> - The README is missing in %doc

I've not included the README because it contains installation instructions for
Windows and Debian that I think are unneeded by fedora users.

"Irrelevant documentation include build instructions, the omnipresent INSTALL
file containing generic build instructions, for example, and documentation for
non-Linux systems, e.g. README.MSDOS."
https://fedoraproject.org/wiki/Packaging/Guidelines#Documentation

> - The license is GPLv3+, source header says '3 of the License, or (at your
> option) any later version.'

I've updated it and added LGPLv2+ too after finding the
build/lib/dreampielib/gui/SimpleGladeApp.py module.

> - The latest release is 1.0.2
> 
> Please take a look at he following page, too :
> https://fedoraproject.org/wiki/Packaging:Python    

I have. Could you please be a bit more specific as to what exactly you think is
wrong?

http://mapleoin.fedorapeople.org/pkgs/dreampie/dreampie.spec
http://mapleoin.fedorapeople.org/pkgs/dreampie/dreampie-1.0.2-1.fc12.src.rpm

%changelog
* Wed Mar  3 2010 Ionuț C. Arțăriși <mapleoin@xxxxxxxxxxxxxxxxx> - 1.0.2-1
- fixed spelling error in description
- don't make subp_main.py executable
- don't pass cflags to build command
- use global instead of define macros
- changed license field to GPLv3+ and LGPLv2+
- updated to 1.0.2

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