Thanks Erik, I will take care of those.
Kumar
On 11/20/2013 1:52 AM, Erik Joelsson wrote:
Looks good, but one style nit.
195 $(JDK_OUTPUTDIR)/demo/nbproject/%:
$(JDK_TOPDIR)/src/closed/share/demo/nbproject/%
196 $(call install-file)
197 $(CHMOD) -f ug+w $@
198 ifeq ($(OPENJDK_TARGET_OS), solaris)
199 $(RM) -r $(JDK_OUTPUTDIR)/demo/nbproject/jfc/SwingApplet
200 endif
Make conditionals in recipes should be indented the same as the
recipe, but with spaces so not to confuse make:
195 $(JDK_OUTPUTDIR)/demo/nbproject/%:
$(JDK_TOPDIR)/src/closed/share/demo/nbproject/%
196 $(call install-file)
197 $(CHMOD) -f ug+w $@
198 ifeq ($(OPENJDK_TARGET_OS), solaris)
199 $(RM) -r $(JDK_OUTPUTDIR)/demo/nbproject/jfc/SwingApplet
200 endif
/Erik
On 2013-11-20 00:36, Kumar Srinivasan wrote:
Hello,
Please review these changes to remove applet related demos from the
images and thereby
from the Solaris distributions. Note: at this time appletviewer is
still available in the distros,
as there are tests dependent on it.
http://cr.openjdk.java.net/~ksrini/8028645/webrev.0/
Thanks
Kumar