updatecenter2
  1. updatecenter2
  2. UPDATECENTER2-1786

Separate IPS package for pkg-java javadocs content

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.2
    • Fix Version/s: B41
    • Component/s: pkg-java
    • Labels:
      None
    • Environment:

      Operating System: All
      Platform: All

    • Issuezilla Id:
      1,786
    • Tags:

      Description

      This is spillover from GlassFish issue:

      https://glassfish.dev.java.net/issues/show_bug.cgi?id=9294

      In order to minimize the initial size of GlassFish distributions, it would be
      preferable to move pkg-java javadocs content into separate IPS package which
      would not be installed in the initial distribution image.

        Activity

        Hide
        Snjezana Sevo-Zenzerovic added a comment -

        Just another note: given that this saves about 80KB once it is zipped up, you
        should be able to tell me to go away

        Show
        Snjezana Sevo-Zenzerovic added a comment - Just another note: given that this saves about 80KB once it is zipped up, you should be able to tell me to go away
        Hide
        Tom Mueller added a comment -

        The uncompressed size of the javadocs is about 446KB. The pkg-client.jar file
        is about 270KB. However, as Snjezana pointed out, the javadocs compress much
        more than the JAR files.

        Show
        Tom Mueller added a comment - The uncompressed size of the javadocs is about 446KB. The pkg-client.jar file is about 270KB. However, as Snjezana pointed out, the javadocs compress much more than the JAR files.
        Hide
        Tom Mueller added a comment -

        Lowering priority as this is a nice to have.

        Show
        Tom Mueller added a comment - Lowering priority as this is a nice to have.
        Hide
        scm_issue_link added a comment -

        Code changed in updatecenter2
        User: : tmueller
        Path:
        trunk/project/build.xml
        https://updatecenter2.dev.java.net/source/browse/updatecenter2/trunk/project/build.xml?view=diff&rev=2519&p1=trunk/project/build.xml&p2=trunk/project/build.xml&r1=2518&r2=2519
        trunk/project/packaging/pkg-java-docs_proto.py
        https://updatecenter2.dev.java.net/source/browse/updatecenter2/trunk/project/packaging/pkg-java-docs_proto.py?view=auto&rev=2519
        trunk/project/packaging/pkg-java_proto.py
        https://updatecenter2.dev.java.net/source/browse/updatecenter2/trunk/project/packaging/pkg-java_proto.py?view=diff&rev=2519&p1=trunk/project/packaging/pkg-java_proto.py&p2=trunk/project/packaging/pkg-java_proto.py&r1=2518&r2=2519
        trunk/project/packaging/pkg-toolkit-incorporation_manifest
        https://updatecenter2.dev.java.net/source/browse/updatecenter2/trunk/project/packaging/pkg-toolkit-incorporation_manifest?view=diff&rev=2519&p1=trunk/project/packaging/pkg-toolkit-incorporation_manifest&p2=trunk/project/packaging/pkg-toolkit-incorporation_manifest&r1=2518&r2=2519
        trunk/project/packaging/pkg-toolkit_manifest
        https://updatecenter2.dev.java.net/source/browse/updatecenter2/trunk/project/packaging/pkg-toolkit_manifest?view=diff&rev=2519&p1=trunk/project/packaging/pkg-toolkit_manifest&p2=trunk/project/packaging/pkg-toolkit_manifest&r1=2518&r2=2519
        trunk/project/tests/sanity-unix
        https://updatecenter2.dev.java.net/source/browse/updatecenter2/trunk/project/tests/sanity-unix?view=diff&rev=2519&p1=trunk/project/tests/sanity-unix&p2=trunk/project/tests/sanity-unix&r1=2518&r2=2519
        trunk/project/tests/sanity-unix-nightly
        https://updatecenter2.dev.java.net/source/browse/updatecenter2/trunk/project/tests/sanity-unix-nightly?view=diff&rev=2519&p1=trunk/project/tests/sanity-unix-nightly&p2=trunk/project/tests/sanity-unix-nightly&r1=2518&r2=2519
        trunk/project/tests/sanity-windows
        https://updatecenter2.dev.java.net/source/browse/updatecenter2/trunk/project/tests/sanity-windows?view=diff&rev=2519&p1=trunk/project/tests/sanity-windows&p2=trunk/project/tests/sanity-windows&r1=2518&r2=2519
        trunk/project/tests/sanity-windows-nightly
        https://updatecenter2.dev.java.net/source/browse/updatecenter2/trunk/project/tests/sanity-windows-nightly?view=diff&rev=2519&p1=trunk/project/tests/sanity-windows-nightly&p2=trunk/project/tests/sanity-windows-nightly&r1=2518&r2=2519

        Log:
        UPDATECENTER2-1786 Separate IPS package for pkg-java javadocs content
        Created a new package, pkg-java-docs, that contains the javadocs content for the pkg(5) Java API. Added this package to the pkg-toolkit and pkg-toolkit-incorporation dependendies. This new package is installed into the toolkit images so that the content of those images stays the same. But adopting projects can now choose to not install the javadocs if they don't want them. Also updated the sanity tests with some minor changes.

        Show
        scm_issue_link added a comment - Code changed in updatecenter2 User: : tmueller Path: trunk/project/build.xml https://updatecenter2.dev.java.net/source/browse/updatecenter2/trunk/project/build.xml?view=diff&rev=2519&p1=trunk/project/build.xml&p2=trunk/project/build.xml&r1=2518&r2=2519 trunk/project/packaging/pkg-java-docs_proto.py https://updatecenter2.dev.java.net/source/browse/updatecenter2/trunk/project/packaging/pkg-java-docs_proto.py?view=auto&rev=2519 trunk/project/packaging/pkg-java_proto.py https://updatecenter2.dev.java.net/source/browse/updatecenter2/trunk/project/packaging/pkg-java_proto.py?view=diff&rev=2519&p1=trunk/project/packaging/pkg-java_proto.py&p2=trunk/project/packaging/pkg-java_proto.py&r1=2518&r2=2519 trunk/project/packaging/pkg-toolkit-incorporation_manifest https://updatecenter2.dev.java.net/source/browse/updatecenter2/trunk/project/packaging/pkg-toolkit-incorporation_manifest?view=diff&rev=2519&p1=trunk/project/packaging/pkg-toolkit-incorporation_manifest&p2=trunk/project/packaging/pkg-toolkit-incorporation_manifest&r1=2518&r2=2519 trunk/project/packaging/pkg-toolkit_manifest https://updatecenter2.dev.java.net/source/browse/updatecenter2/trunk/project/packaging/pkg-toolkit_manifest?view=diff&rev=2519&p1=trunk/project/packaging/pkg-toolkit_manifest&p2=trunk/project/packaging/pkg-toolkit_manifest&r1=2518&r2=2519 trunk/project/tests/sanity-unix https://updatecenter2.dev.java.net/source/browse/updatecenter2/trunk/project/tests/sanity-unix?view=diff&rev=2519&p1=trunk/project/tests/sanity-unix&p2=trunk/project/tests/sanity-unix&r1=2518&r2=2519 trunk/project/tests/sanity-unix-nightly https://updatecenter2.dev.java.net/source/browse/updatecenter2/trunk/project/tests/sanity-unix-nightly?view=diff&rev=2519&p1=trunk/project/tests/sanity-unix-nightly&p2=trunk/project/tests/sanity-unix-nightly&r1=2518&r2=2519 trunk/project/tests/sanity-windows https://updatecenter2.dev.java.net/source/browse/updatecenter2/trunk/project/tests/sanity-windows?view=diff&rev=2519&p1=trunk/project/tests/sanity-windows&p2=trunk/project/tests/sanity-windows&r1=2518&r2=2519 trunk/project/tests/sanity-windows-nightly https://updatecenter2.dev.java.net/source/browse/updatecenter2/trunk/project/tests/sanity-windows-nightly?view=diff&rev=2519&p1=trunk/project/tests/sanity-windows-nightly&p2=trunk/project/tests/sanity-windows-nightly&r1=2518&r2=2519 Log: UPDATECENTER2-1786 Separate IPS package for pkg-java javadocs content Created a new package, pkg-java-docs, that contains the javadocs content for the pkg(5) Java API. Added this package to the pkg-toolkit and pkg-toolkit-incorporation dependendies. This new package is installed into the toolkit images so that the content of those images stays the same. But adopting projects can now choose to not install the javadocs if they don't want them. Also updated the sanity tests with some minor changes.
        Hide
        Tom Mueller added a comment -

        Fix integrated in revision 2519.

        Show
        Tom Mueller added a comment - Fix integrated in revision 2519.
        Hide
        faouzia added a comment -

        Verified in B44

        Show
        faouzia added a comment - Verified in B44

          People

          • Assignee:
            Tom Mueller
            Reporter:
            Snjezana Sevo-Zenzerovic
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: