[Bug 193898] Review Request: Jython - Java source interpreter

[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: Jython -  Java source interpreter


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





------- Additional Comments From ifoox@xxxxxxxxxx  2006-09-08 20:51 EST -------
New files:
http://people.redhat.com/ifoox/extras/jython-2.2-0.a0.2jpp_2fc.src.rpm
http://people.redhat.com/ifoox/extras/jython.spec

(In reply to comment #2)
> Just some quick comments from looking at parts of the spec file..
> 
> - Summary is odd: "Java source interpreter"
Fixed
> - Source lines should be URLs to sources
The source appears to be a CVS snapshot, but I'll check to make sure whether
it's not downloaded.
> - Remove Epoch
Fixed.
> - Change Group to Development/Libraries
Fixed.
> - Make BuildRoot standard	
Fixed.
> - Remove these lines...
> Distribution:	JPackage
> Vendor:		JPackage Project
Fixed. 

Also fixed:
- Add dist tag.
- Fix compile line to use the system Python libraries instead of the python2.2
source.
- Remove Source1 (python2.2 library).
- Move buildroot removal from prep to install.
- Use libedit (EditLine) instead of GNU readline.



(In reply to comment #3)
> BTW, I don't think we should be linking libreadline-java with jython since I
> believe the licenses are incompatible.  See this thread for discussion and a
> potential solution:
> http://sourceforge.net/mailarchive/message.php?msg_id=707283

I've built the latest version of libreadline-java against libedit (EditLine)
instead of GNU readline, libedit is LGPL, and is what the poster in the thread
you mention proposes. So this should be fine.  See bug #193896.

There are still many issues to resolve with this package, I will try to resolve
most of them tomorrow. But I have a few questions which hopefully someone can
answer. 

1. rpmlint gives me errors like this when running it on the main rpm:
E: jython non-executable-script /usr/share/jython/Lib/Cookie.py 0644
It doesn't give it for all the .py files in that directory, even though they all
have the same permissions. Should these be 755, or should these files be moved
to /usr/lib instead?

2. What should be the group for jython-demo?
I put it in Development/Documentation along with the manual and javadoc, but I'm
not sure this is right.

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