Uploaded image for project: 'JDK'
  1. JDK
  2. JDK-8006854

com.sun.security.auth.PrincipalComparator should not be included in compact1 or compact2

XMLWordPrintable


      The profiles builds currently include com.sun.security.auth.PrincipalComparator in compact1 and compact2 whereas it should only be in compact3. This was necessary while awaiting a fix to a dependency issue in the security libraries. This has is now fixed via JDK-7019834.

      While we are there then we should also exclude com.sun.security.auth.DialogCallbackHandler from compact3. This is a Swing based callback handler and should not be in any profile.

      Here are the diffs to fix these issues:

      diff --git a/makefiles/profile-rtjar-includes.txt b/makefiles/profile-rtjar-includes.txt
      --- a/makefiles/profile-rtjar-includes.txt
      +++ b/makefiles/profile-rtjar-includes.txt
      @@ -133,8 +133,7 @@
           sun/util/logging/resources \
           sun/util/resources
       
      -PROFILE_1_RTJAR_INCLUDE_TYPES := \
      - com/sun/security/auth/PrincipalComparator.class
      +PROFILE_1_RTJAR_INCLUDE_TYPES :=
       
       PROFILE_1_RTJAR_EXCLUDE_TYPES :=
       
      @@ -455,7 +454,12 @@
           javax/management/remote/rmi/_RMIConnectionImpl_Tie.class \
           javax/management/remote/rmi/_RMIConnection_Stub.class \
           javax/management/remote/rmi/_RMIServerImpl_Tie.class \
      - javax/management/remote/rmi/_RMIServer_Stub.class
      + javax/management/remote/rmi/_RMIServer_Stub.class \
      + com/sun/security/auth/callback/DialogCallbackHandler.class \
      + com/sun/security/auth/callback/DialogCallbackHandler\$$$$1.class \
      + com/sun/security/auth/callback/DialogCallbackHandler\$$$$2.class \
      + com/sun/security/auth/callback/DialogCallbackHandler\$$$$Action.class \
      + com/sun/security/auth/callback/DialogCallbackHandler\$$$$ConfirmationInfo.class
       
       PROFILE_3_INCLUDE_METAINF_SERVICES := \
           META-INF/services/javax.script.ScriptEngineFactory

            dholmes David Holmes
            alanb Alan Bateman
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: