search

scijava / scijava-common

A plugin framework and application container with built-in extensibility mechanism :electric_plug:
BSD 2-Clause "Simplified" License
87 stars 52 forks source link

CI release builds fail due to multi-platform build targets #472

Closed ctrueden closed 10 months ago

ctrueden commented 11 months ago

When the CI for this project cuts a release, it uses the same build.yml workflow as non-release builds, thus triggering a matrix of the three target OSes. But then all three OSes try to deploy the built release to OSS Sonatype, which results in build failure due to immutable artifacts. And anyway there is no reason to attempt to deploy more than once—it should just be the Linux build that gets deployed/released. The other two platforms should maybe not build release builds at all, or at most should only sanity check the builds without deploying.

Further thinking is needed first about the best place to cause this behavior to change. Mayhap this should be fixed in the ci-build.sh script?

ctrueden commented 10 months ago

Addressed by 43164babd71c2429e4fa2e8f59f7eda767ebe120 along with scijava/scijava-scripts@7bd22bf4749d6f92eeefafab0ead47ea4f617785.