bug #9351
closedInclude new locations for cdmlib-print and cdmlib-remote in jenkins release script
100%
Description
cdmlib-print and cdmlib-remote are now only available in taxeditor.printpublisher.
Therefore the according links to the libraries are now located in the printpublisher Manifest.MF and build.properties, not in taxeditor.cdmlib anymore.
The release scripts need to be adapted accordingly.
This is critical because the provision job for taxeditor does not work otherwise.
Related issues
Updated by Andreas Müller over 3 years ago
- Related to feature request #9204: Update cdmlib and taxeditor dependencies added
Updated by Andreas Müller over 3 years ago
- Due date set to 08/26/2020
- Start date changed from 12/08/2020 to 08/26/2020
- Follows task #9208: Move cdmlib-print and cdmib-remote to taxeditor.printpublisher added
Updated by Andreas Müller over 3 years ago
- Copied from task #9359: Update cdmlib, taxeditor, cdmserver etc. dependencies added
Updated by Andreas Kohlbecker about 3 years ago
- Target version changed from Release 5.19 to Release 5.21
Updated by Andreas Kohlbecker about 3 years ago
- Status changed from New to Feedback
- Assignee changed from Andreas Kohlbecker to Andreas Müller
This ticket is unclear to me what is meant with "jenkins release script"?
I checked server-scripts/jenkins-ci/taxeditor/
and could not find anything related.
So this must refer to the jenkins jobs directly but none of these configfiles contains any line with "SNAPSHOT" or "print":
./taxeditor-SA-JRE-INTEGRATION/config.xml ./taxeditor-SITE-DEPLOY/config.xml ./taxeditor-INTEGRATION/config.xml ./taxeditor-SA-INTEGRATION/config.xml ./taxeditor-PROVISION/config.xml ./taxeditor-SNAPSHOT/config.xml ./taxeditor-JRE-INTEGRATION/config.xml ./taxeditor-RLS-FINISH/config.xml ./taxeditor-HFX-START/config.xml ./taxeditor-JRE-SNAPSHOT/config.xml ./taxeditor-RLS-START/config.xml ./taxeditor-parent-SITE-DEPLOY/config.xml ./taxeditor-HFX-FINISH/config.xml
Are you 100% sure that there is anything that need to be changed in jenkins?
Updated by Andreas Müller about 3 years ago
- Status changed from Feedback to In Progress
- Assignee changed from Andreas Müller to Andreas Kohlbecker
Please refer to my commits above. This changing of the version in eu.etaxonomy.taxeditor.printpublisher/META-INF/MANIFEST.MF and eu.etaxonomy.taxeditor.printpublisher/build.properties similar to what we do in poms.
Updated by Andreas Müller about 3 years ago
Änderung ist: cdmlib-remote und cdmlib-print waren bislang dependencies von taxeditor.cdmlib. Jetzt sind es nur noch dependencies von taxeditor.printpublisher. Daher müssen jetzt die ManifestMF und die build.properties von taxeditor.printpublisher geupdated werden, genauso wie auch die ManifestMF und build.properties in taxeditor.cdmlib geupdated werden. Letzteres funktioniert auch weiterhin, aber eben nicht für remote und print da diese jetzt eben nur in der ManifestMF und build.properties von printpublisher stehen.
Updated by Andreas Kohlbecker about 3 years ago
the following lines in server-scripts/jenkins-ci/taxeditor/repo-update-git.sh
need to be adapted, so that the "ManifestMF und die build.properties von taxeditor.printpublisher geupdated werden":
#update the cdmlib.version (and taxeditor.version) values in the parent pom.xml to $cdmlib_version if [ -n "${cdmlib_version}" ]; then $_my_path/taxedUpdateConfig.pl $cdmlib_version #update cdmlib jars cd $WORKSPACE/eu.etaxonomy.taxeditor.cdmlib #update MANIFEST.MF, classpath and build files to keep eclipse environment consistent sed -i "s/\(lib\/cdmlib\-.*-\)[[:digit:]]\+\.[[:digit:]]\+\.[[:digit:]]\+\(-SNAPSHOT\)\?\(\(-sources\)\?\.jar\)/\1$cdmlib_version\3/g" ./META-INF/MANIFEST.MF # no longer needed since #4227 is completed: # sed -i "s/\(lib\/cdmlib\-.*-\)[[:digit:]]\+\.[[:digit:]]\+\.[[:digit:]]\+\(-SNAPSHOT\)\?\(\.jar\)/\1$cdmlib_version\3/g;s/\(lib\/cdmlib\-.*-\)[[:digit:]]\+\.[[:digit:]]\+\.[[:digit:]]\+\(-SNAPSHOT\)\?\(-sources\.jar\)/\1$cdmlib_version\3/g" ./.classpath sed -i "s/\(lib\/cdmlib\-.*-\)[[:digit:]]\+\.[[:digit:]]\+\.[[:digit:]]\+\(-SNAPSHOT\)\?\(\(-sources\)\?\.jar\)/\1$cdmlib_version\3/g" ./build.properties fi
- create a function for that!
- make a dry run to test the script
Updated by Andreas Kohlbecker about 3 years ago
- Target version changed from Release 5.21 to Release 5.20
Updated by Andreas Kohlbecker about 3 years ago
- Status changed from In Progress to Closed
- % Done changed from 0 to 100
fixed in server-scripts:
svn ci -m "ref #9351 support for setting version in multiple projects & fixing some problems" ─╯ Sending repo-update-git.sh Transmitting file data .done Committing transaction... Committed revision 29014.