RFR: JDK-8028645: [infra] purge applet demos from the Solaris	distros
    Erik Joelsson 
    erik.joelsson at oracle.com
       
    Wed Nov 20 09:52:25 UTC 2013
    
    
  
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
>
    
    
More information about the build-dev
mailing list