ros-infrastructure / bloom

A release automation tool which makes releasing catkin (http://ros.org/wiki/catkin) packages easier.
Other
58 stars 91 forks source link

Testing release repository after completion #102

Open wjwwood opened 11 years ago

wjwwood commented 11 years ago

Bloom tutorials should detail how to test a release repository before push to the farm.

Better yet, bloom should offer to check the state of the release repository before pushing and updating the rosdistro file, but this might be in the farther future.

ahornung commented 11 years ago

Some ideas: Catch obvious errors so that the build farm has less to do in these "easy" cases.

wjwwood commented 11 years ago

The package.xml is parsed by catkin_pkg several times in bloom, so that is checked.

Building and running unit tests should happen before release, aka pre-release. I do not think this is bloom's responsibility, but rather the role of a continuous integration system.

ahornung commented 11 years ago

The building does not need to be done by bloom, a simple line in the tutorials would suffice to tell users how to compile from the source package before pushing. Otherwise the turnaround time for a single error could be 24h on the build farm. If I forgot to add 3 files to my repo then this will take three days to fix and get a proper release out, whereas it could be caught locally almost immediately (while taking away unnecessary load of the build farm).

aleeper commented 11 years ago

Has this been addressed? The release process with pre-release test builds is still not clear. http://www.ros.org/wiki/release tells people to go look at bloom for releasing catkin stacks, but the bloom page doesn't mention anything about build tests.

In any case, I agree with Armin. It's taken me 3 days to attempt a new package release.