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

Fix bug (test -d fails), add example ~/.m2/settings.xml, fix opensuse issues #12

Closed noelmcloughlin closed 7 years ago

noelmcloughlin commented 7 years ago

trivial bug fix

noelmcloughlin commented 7 years ago

This PR delivers an example maven-settings.xml file which is deployed to ~/.m2/ directory. The values for the maven settings.xml file are populated by jinja context. The required variables scmhost, repohost, and orgdomain are added to settings.xml.

noelmcloughlin commented 7 years ago

Fixes https://github.com/saltstack-formulas/maven-formula/issues/11

noelmcloughlin commented 7 years ago

The PR includes fixes required for opensuse testing. Had problems with saltsack/salt#5567 and salt.grains.get(os_family) == 'Suse' seems to not work in jinja - trying (os) == 'SUSE'

noelmcloughlin commented 7 years ago

Closing this pull request for now. Will open better fresh PR with detailed explanation and squashed commits over next few days.