Uploaded image for project: 'Tools (JBoss Tools)'
  1. Tools (JBoss Tools)
  2. JBIDE-20177

migrate skipRevisionCheckWhenPublishing and checkLatestPublishedSHA into jbosstools-releng-publish deploy mojo

    XMLWordPrintable

Details

    • Bug
    • Resolution: Won't Do
    • Major
    • 4.5.3.Final
    • 4.3.0.Beta1
    • build
    • None

    Description

      If we can migrate this logic into the mojo...

      if [[ ${skipRevisionCheckWhenPublishing} == "true" ]] || [[ $(. ${WORKSPACE}/sources/util/checkLatestPublishedSHA.sh -s ${WORKSPACE}/sources/aggregate/${projectName}-site/target/fullSite/all/repo -t http://download.jboss.org/jbosstools/mars/snapshots/builds/jbosstools-build-sites.aggregate.${projectName}-site_${jbosstools_site_stream}/latest/all/repo/) == "true" ]]; then
      

      ... we'll be able to remove this step in all jobs:

          <hudson.tasks.Maven>
            <targets>-U -e org.apache.maven.plugins:maven-dependency-plugin:2.9:unpack</targets>
            <mavenName>maven-3.2.5</mavenName>
            <properties>artifact=org.jboss.tools.releng:jbosstools-releng-publish:4.3.0.Beta2-SNAPSHOT:zip
      outputDirectory=${WORKSPACE}/sources
      trimVersion=true
      mdep.stripClassifier=true
      mdep.stripVersion=true</properties>
            <usePrivateRepository>false</usePrivateRepository>
            <settings class="jenkins.mvn.DefaultSettingsProvider"/>
            <globalSettings class="jenkins.mvn.DefaultGlobalSettingsProvider"/>
          </hudson.tasks.Maven>
      

      ... and instead, just pass in those vars to maven:

      -DskipRevisionCheckWhenPublishing=${skipRevisionCheckWhenPublishing}
      -DcheckLatestPublishedSHA_SOURCE_PATH=${WORKSPACE}/sources/aggregate/${projectName}-site/target/fullSite/all/repo
      -DcheckLatestPublishedSHA_TARGET_PATH=http://download.jboss.org/jbosstools/mars/snapshots/builds/jbosstools-build-sites.aggregate.${projectName}-site_${jbosstools_site_stream}/latest/all/repo/
      

      Attachments

        Issue Links

          Activity

            People

              nickboldt Nick Boldt
              nickboldt Nick Boldt
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: