glassfish
  1. glassfish
  2. GLASSFISH-14019

make concept of install directory consistent

    Details

    • Type: Bug Bug
    • Status: Open
    • Priority: Minor Minor
    • Resolution: Unresolved
    • Affects Version/s: 3.1
    • Fix Version/s: future release
    • Component/s: embedded
    • Labels:
      None
    • Environment:

      Operating System: All
      Platform: All

    • Issuezilla Id:
      14,019

      Description

      Recently, the admin commands for nodes (create-node-ssh, install-node) were
      modified to provide a consistent interface for the --installdir option. They both
      now except an argument that refers to the directory that contains glassfish, mq,
      jdbc, and bin. However, when a user calls EmbeddedFileSystem.Builder.installRoot
      api or pass "org.glassfish.ejb.embedded.glassfish.installation.root" property to
      createEJBContainer call they are expected to point to glassfish3/glassfish dir.
      Maybe these APIs should also use the notion of the product dir rather than
      pointing to the glassfish dir.

        Activity

        Hide
        Bhavanishankar added a comment -

        Although the asadmin commands use --installdir to point to GlassFish product dir, the GlassFish implementation (including embedded GF) refers com.sun.aas.installRoot to point to the install dir where domains/, lib/, config/ are located.

        So, I am not sure of changing this now. Will look into this in the future release.

        Show
        Bhavanishankar added a comment - Although the asadmin commands use --installdir to point to GlassFish product dir, the GlassFish implementation (including embedded GF) refers com.sun.aas.installRoot to point to the install dir where domains/, lib/, config/ are located. So, I am not sure of changing this now. Will look into this in the future release.

          People

          • Assignee:
            Bhavanishankar
            Reporter:
            Tom Mueller
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated: