[
https://issues.apache.org/jira/browse/BEAM-810?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15604018#comment-15604018
]
Kenneth Knowles edited comment on BEAM-810 at 10/25/16 3:14 AM:
----------------------------------------------------------------
PR [#1181|https://github.com/apache/incubator-beam/pull/1181] should fix one
issue created by the use of {{mvn verify}}. Perhaps a similar approach will
work for the archetypes. It is unfortunate to do so much useless work.
was (Author: kenn):
PR [#1181|https://github.com/apache/incubator-beam/pull/1181] should fix one
issue created by the use of `mvn verify`. Perhaps a similar approach will work
for the archetypes. It is unfortunate to do so much useless work.
> Beam's uses of archetypes are problematic
> -----------------------------------------
>
> Key: BEAM-810
> URL: https://issues.apache.org/jira/browse/BEAM-810
> Project: Beam
> Issue Type: Bug
> Components: build-system
> Affects Versions: Not applicable
> Reporter: Daniel Halperin
>
> * Using a version range in the archetype pom.xml is inaccurate, because we
> don't have a stable API. We can't actually expect an older version to work as
> we make backwards-incompatible changes.
> * In PR #936, a decision was made to pin the version of the archetype. This
> requires manual changes to update many places (archetype-resources), which
> breaks the release process in a few different ways. See [BEAM-806].
> * Travis uses `mvn verify`, not `mvn install` -- so archetypes are not
> properly tested and instead we're actually picking up the SNAPSHOT archetypes
> deployed the prior night (IIRC).
> * There was some other issue with parallelizing maven where archetypes are
> not properly marked as depending on "you can't test the archetype until
> you've installed the examples packages".
> * We're not actually testing that the archetype-generated projects actually
> compile. [BEAM-641].
> This is all pretty ugly. Suggestions wanted.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
