Closed Bug 316621 Opened 19 years ago Closed 3 years ago

rt.jar and classes.zip may not be needed

Categories

(JSS Graveyard :: Library, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: wtc, Assigned: wtc)

Details

In mozilla/security/coreconf/jdk.mk, we define JAVA_CLASSES and JRE_CLASSES whose values contain rt.jar (was classes.zip) in the JDK or JRE. I found that JRE_CLASSES is defined but not used. And in fact, JRE does not have rt.jar. This means the JRE_CLASSES variable is useless. Also, the fact that we could build JSS with the nonexistent classes.zip in the value of JAVA_CLASSES before means JAVA_CLASSES may not be needed. JAVA_CLASSES is only used to define JDK_CLASSPATH, which is passed as argument to the -classpath option for the various Java command: 285 # define a default JDK classpath 286 ifeq ($(JDK_CLASSPATH),) 287 JDK_CLASSPATH = '$(JAVA_DESTPATH)$(PATH_SEPARATOR)$(JAVA_SOURCEPATH)$(PATH_SEPARATOR)$(JAVA_CLASSES)' 288 endif 289 290 # by default, override CLASSPATH environment variable using the JDK classpath option with $(JDK_CLASSPATH) 291 ifeq ($(JDK_CLASSPATH_OPT),) 292 JDK_CLASSPATH_OPT = -classpath $(JDK_CLASSPATH) 293 endif Perhaps it is not necessary to specify that -classpath option?
QA Contact: libraries
Status: NEW → ASSIGNED
QA Contact: libraries → jss-qa
JSS development has moved from the Mozilla community to the Dogtag PKI community. Please re-file this bug at https://github.com/dogtagpki/jss if it is still relevant. Thank you!
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.