[Bug 253691] Review Request: java-1.7.0-icedtea - IcedTea runtime and development environments

[bugzilla at redhat.com]

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: java-1.7.0-icedtea - IcedTea runtime and development environments


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





------- Additional Comments From fitzsim at redhat.com  2007-08-27 15:04 EST -------
(In reply to comment #24)

> Release critical items:
> 
> 1. move plugin on x86_64 as per Kostas' patch

Will do.

> 2. remove useless-explicit-provides jdbc-stdext

Although I said I should remove this in the comments above, after re-reading the
comment I left myself in the spec file, I think both these explicit provides are
necessary:

# Both these versioned provides need to be here since either may be
# required explicitly.  Requiring the first one means requiring a
# version of the JDBC API; requiring the second one means requiring
# the JDBC API provided by this version of the JDK.  This convention
# comes from JPackage.

> 3. are you going to disable soname generation for the dlopen'd .sos?

I'd rather that this wait until post-F8 since fixing it will be fiddly and it
won't negatively affect the initial release.

> 4. should there be documentation for -plugin and -src?

Yeah, probably a good idea, just to explain what they are.  I'll write two READMEs.

Some new minor licensing issues came to our attention today.  They're very
similar to the ones we discussed above.  Lillian Angel and I are currently
working around the newly-discovered issues and we'll keep the workaround list
up-to-date in the IcedTea README.  I consider these issues release-critical, but
we should have things ready-to-go by later today.

> 
> Non-critical but please fix before F8:
> 
> 1. fix missing shebang and permissions on
> sample/scripting/scriptpad/src/scripts/memory.sh
> 2. other non-critical items brought up by rpmlint (see comments above and
> attached rpmlint output)

Yes, thanks for the summary.


-- 
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, or are watching someone who is.