[Bug 191218] Review Request: PyScript - Postscript graphics with Python

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

Summary: Review Request: PyScript - Postscript graphics with Python


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





------- Additional Comments From tjikkun@xxxxxxxxx  2006-05-12 12:46 EST -------
Ok, I had a little bit more time now, sorry for not giving all directions at
once but i am also still learning.
(In reply to comment #4)
A couple of things:
1) %prep section:
make it:
%setup -q
I don't know if this is a must, but I think it becomes easier if you do this.
First of all you have one line in %prep instead of 3, second you can remove a
lot of %{name}-%{version} so the specfile becomes easier to read

2) I read http://fedoraproject.org/wiki/Packaging/Python , you should do that
too. I hinted to replace paths with libdir but in this page you can read about
%{python_sitelib}, use this instead.

3) use wildcards in the %files section, instead of listing all files seperatly
(see the url in 2 for an example)

4) python setup.py install --prefix=$RPM_BUILD_ROOT/usr still has /usr hardcoded

5) time is up again for me, just a general hint that you can find a lot of info
in the fedora extras wiki.

6) don't take everything i say for granted, i could be wrong sometimes so if you
think i might be just say so.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.

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