Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

NXBT-2399: split and version nuxeo packaging Jenkinsfile #3

Merged
merged 12 commits into from Dec 20, 2018

Conversation

alexistimic
Copy link
Contributor

No description provided.

@nuxeojenkins
Copy link

View issue in JIRA

Jenkinsfile Outdated
[$class: 'RebuildSettings', autoRebuild: false, rebuildDisabled: false],
[$class: 'ParametersDefinitionProperty', parameterDefinitions: [
[$class: 'StringParameterDefinition', defaultValue: '6.0-SNAPSHOT', description: 'Product version to build', name: 'NUXEO_VERSION'],
[$class: 'StringParameterDefinition', defaultValue: '', description: 'Optional - Use the specified URL (eg a link to staging) as the source for the distribution instead of maven', name: 'DISTRIBUTION_URL'],
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reword DISTRIBUTION_URL description to:

Optional - Alternate URL to download the distribution from instead of the default Maven artifact download.
For instance: http://community.nuxeo.com/static/snapshots/nuxeo-server-tomcat-10.3-SNAPSHOT.zip

Jenkinsfile Outdated
[$class: 'BooleanParameterDefinition', defaultValue: true, description: 'Publish .exe package', name: 'PUBLISH_EXE'],
[$class: 'StringParameterDefinition', defaultValue: '/var/www/community.nuxeo.com/static/staging/', description: 'Staging publishing destination path (for scp)', name: 'STAGING_PATH'],
[$class: 'StringParameterDefinition', defaultValue: 'nuxeo@lethe.nuxeo.com', description: 'Publishing destination host (for scp)', name: 'DEPLOY_HOST']]],
pipelineTriggers([])])
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what is the empty pipelineTriggers for?

Jenkinsfile Outdated
echo "*** "$(date +"%H:%M:%S")" Cloning/updating nuxeo-packaging-windows"
if [ ! -d nuxeo-packaging-windows ]; then
git clone git@github.com:nuxeo/nuxeo-packaging-windows.git
fi
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

checkout scm ?

@jcarsique
Copy link
Contributor

Add a header to the file.
👍 beside the few remarks and improvements.


<groupId>org.nuxeo.packaging</groupId>
<artifactId>windows-installer-3parties</artifactId>
<version>3.1</version>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Usage:

  • under Continuous Integration: SNAPSHOT version
  • on release: fixed version
    => 3.2-SNAPSHOT

</unsubscribe>
<archive>http://lists.nuxeo.com/pipermail/nuxeo-dev/</archive>
</mailingList>
</mailingLists>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.


<issueManagement>
<system>jira</system>
<url>http://jira.nuxeo.com/browse/NXP</url>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.


<ciManagement>
<system>Jenkins</system>
<url>http://qa.nuxeo.org/jenkins/</url>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

link to the job is even better

<excludes>
<exclude>org.nuxeo:nuxeo-ftest:1.8.6-SNAPSHOT:zip</exclude>
<!-- <exclude>org.jboss.seam:*</exclude> -->
</excludes>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not needed exclusion

</profile>

<profile>
<id>qapriv</id>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove profile even if harmless

</developers>

<scm>
<connection>scm:git:git://github.com/nuxeo/nuxeo.git</connection>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

<repository>
<id>public-releases</id>
<url>
http://mavenin.nuxeo.com/nexus/content/repositories/public-releases
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

<snapshotRepository>
<id>public-snapshots</id>
<url>
http://mavenin.nuxeo.com/nexus/content/repositories/public-snapshots
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

<distributionManagement>
<site>
<id>maven-website</id>
<url>scpexe://gironde.nuxeo.com/home/mavenweb/site/</url>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lethe...

@alexistimic alexistimic force-pushed the feature-NXBT-2399-split&fix branch 3 times, most recently from acfa7e1 to 1ee1dc6 Compare October 4, 2018 10:04
@jcarsique
Copy link
Contributor

👍

@alexistimic alexistimic merged commit bc06f38 into master Dec 20, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants