search

saltstack-formulas / maven-formula

A minimalistic way to install just the maven distribution without any dependencies
http://docs.saltstack.com/en/latest/topics/development/conventions/formulas.html
Other
4 stars 10 forks source link

Incremental improvements: onchanges, add archetype-catalog.xml pillar support #16

Closed noelmcloughlin closed 7 years ago

noelmcloughlin commented 7 years ago

Small incremental improvements:

(1) Currently maven or maven.env state failure triggers dependent states failure unnecessarily. (2) Both file.absent states in maven state can be consolidated. (3) Jinja check (-{%- if maven.source_hash %}) is unnecessary in maven state (4) If maven archetype-catalog url is found in pillar use it.

Fixed these

noelmcloughlin commented 7 years ago

@sroegner or @whiteinge Could you merge this if looks okay to you? Please close #14 as already fixed. I have a couple of incremental updates to maven formula coming so breaking into small pieces