ICEfaces
  1. ICEfaces
  2. ICE-5003

Add source-code and JavaDoc to the ICEfaces Maven distributions

    Details

    • Type: New Feature New Feature
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.8.2
    • Fix Version/s: 3.1
    • Component/s: Release
    • Labels:
      None
    • Environment:
      Maven2

      Description

      It would be beneficial to add the ICEfaces source code and JavaDoc to the public Maven2 repository distributions.

        Issue Links

          Activity

          Ken Fyten created issue -
          Hide
          Ken Fyten added a comment -

          Contribution from ehillman:

          For your consideration, I've attached a patch to the core and component
          build.xml scripts that create a jar file (each) that contains their source
          code. These get build to each dist directory.

          Then these files can be deployed along with their appropriate artifact using
          the mvn deploy:deploy-file command, being sure to use the classifier option.
          For example, I can run the following from a command line...

          1. Generate the sources jar
            ant -f $ICEFACES_SRC_DIR/core/build.xml sources-jar
            mvn deploy:deploy-file
            -Dfile=$ICEFACES_SRC_DIR/core/dist/icefaces-sources.jar
            -DgroupId=org.icefaces -DartifactId=icefaces -Dversion=$ICEFACES_VERSION \
            -Dclassifier=sources -Dpackaging=jar -Durl=$REPOSITORY_URL
            -DrepositoryId=$REPOSITORY_ID

          If ICEfaces deployed the source files for each artifact as well, I believe
          it will be valued by the maven community using your artifacts.

          Thanks,
          Ed

          Show
          Ken Fyten added a comment - Contribution from ehillman: For your consideration, I've attached a patch to the core and component build.xml scripts that create a jar file (each) that contains their source code. These get build to each dist directory. Then these files can be deployed along with their appropriate artifact using the mvn deploy:deploy-file command, being sure to use the classifier option. For example, I can run the following from a command line... Generate the sources jar ant -f $ICEFACES_SRC_DIR/core/build.xml sources-jar mvn deploy:deploy-file -Dfile=$ICEFACES_SRC_DIR/core/dist/icefaces-sources.jar -DgroupId=org.icefaces -DartifactId=icefaces -Dversion=$ICEFACES_VERSION \ -Dclassifier=sources -Dpackaging=jar -Durl=$REPOSITORY_URL -DrepositoryId=$REPOSITORY_ID If ICEfaces deployed the source files for each artifact as well, I believe it will be valued by the maven community using your artifacts. Thanks, Ed
          Arran Mccullough made changes -
          Field Original Value New Value
          Salesforce Case [5007000000A77e8]
          Arran Mccullough made changes -
          Attachment build-script-sources.patch [ 12060 ]
          Hide
          Arran Mccullough added a comment -

          Addition comment:

          Please note: I discovered myself that when deploying the files in this sequence

          mvn deploy:deploy-file <artifact file>
          mvn deploy:deploy-file <source JAR>

          I needed to use the -DgeneratePom=false option for the second command (the command that deploys the Source jar). Otherwise, the original POM file gets replaced, losing any dependency delcarations the original POM might have had. Which makes subsequent compiles fail.

          Show
          Arran Mccullough added a comment - Addition comment: Please note: I discovered myself that when deploying the files in this sequence mvn deploy:deploy-file <artifact file> mvn deploy:deploy-file <source JAR> I needed to use the -DgeneratePom=false option for the second command (the command that deploys the Source jar). Otherwise, the original POM file gets replaced, losing any dependency delcarations the original POM might have had. Which makes subsequent compiles fail.
          Ken Fyten made changes -
          Fix Version/s 1.8.3 [ 10211 ]
          Ken Fyten made changes -
          Fix Version/s EE-2.1.0 [ 10262 ]
          Fix Version/s 1.8.3 [ 10211 ]
          Tyler Johnson made changes -
          Salesforce Case [5007000000A77e8] [5007000000A77e8, 5007000000Gv9Wk]
          Ken Fyten made changes -
          Link This issue is duplicated by ICE-4361 [ ICE-4361 ]
          Ken Fyten made changes -
          Link This issue blocks ICE-6713 [ ICE-6713 ]
          Ken Fyten made changes -
          Fix Version/s 3.1 [ 10312 ]
          Fix Version/s EE-3.0.0.GA [ 10262 ]
          Ken Fyten made changes -
          Assignee Deryk Sinotte [ deryk.sinotte ]
          Ken Fyten made changes -
          Fix Version/s 3.1.0.BETA2 [ 10336 ]
          Assignee Priority P2
          Ken Fyten made changes -
          Fix Version/s 3.1.0.RC1 [ 10337 ]
          Fix Version/s 3.1.0.BETA2 [ 10336 ]
          Deryk Sinotte made changes -
          Link This issue blocks ICE-8230 [ ICE-8230 ]
          Ken Fyten made changes -
          Issue Type Improvement [ 4 ] New Feature [ 2 ]
          Ken Fyten made changes -
          Fix Version/s 3.1.0.RC1 [ 10337 ]
          Affects [Documentation (User Guide, Ref. Guide, etc.), Sample App./Tutorial, Compatibility/Configuration]
          Assignee Priority P2 P1
          Hide
          Deryk Sinotte added a comment -

          Resolving as fixed. The improvements done through ICE-8230 include adding source and javadoc jars for all the sub-projects except ICEpush.

          Show
          Deryk Sinotte added a comment - Resolving as fixed. The improvements done through ICE-8230 include adding source and javadoc jars for all the sub-projects except ICEpush.
          Deryk Sinotte made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Ken Fyten made changes -
          Status Resolved [ 5 ] Closed [ 6 ]

            People

            • Assignee:
              Deryk Sinotte
              Reporter:
              Ken Fyten
            • Votes:
              3 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: