Commit Graph

21 Commits

Author SHA1 Message Date
Patrice Neff
6bf22df41d Allow puppet validation to be disabled
This way only puppet-lint is used.

This is necessary on Windows where the Puppet executable is extremely slow.
2012-08-31 17:28:37 +02:00
Martin Grenfell
9eeaaf3abb puppet: cache versions
Cache the version numbers when we first extract them.

Remove the "Extract" from the version number function names since it is
redundant and fugly.
2012-08-30 18:12:07 +01:00
Martin Grenfell
2bf439a7fc puppet: extract version checking logic into a function 2012-08-30 18:03:20 +01:00
Richard Brown
1af59829df puppet: Fix the puppet-lint version comparison.
The original comparison would disable puppet-lint for version 1.2.3 because 3 <
10 even though 2 > 1
2012-08-29 15:05:10 +01:00
Richard Brown
4467ab2663 puppet: Fix puppet-lint check
Original verison would always call puppet-lint even if the executable wasn't
present. Fixes issue #331
2012-08-29 15:04:17 +01:00
Richard Brown
6314305019 puppet: fix version extractors to ignore stderr 2012-08-29 15:02:48 +01:00
Gabriel Filion
d798286bb7 Enable pass arguments to puppet-lint
Some users might want to disable some checks for puppet-lint if they do
not intend to follow all of the style recommendations. There is
currently no way of doing that.

By adding a global variable synstastic_puppet_lint_arguments, one can
now set arguments to pass to puppet-lint via .vimrc.

Signed-off-by: Gabriel Filion <lelutin@gmail.com>
2012-06-17 03:33:49 -04:00
Martin Grenfell
03a58be0fe puppet: fix a bug with the syntastic_puppet_lint_disabled option
The logic for this option was reversed in 2 places. Puppet-lint should
have been getting disabled for old versions and if it wasnt installed,
but the logic was wrong.
2012-05-01 14:44:15 +01:00
Jos Houtman
4c8ee38782 Abstracted the makeprg selection for puppet away into its own function 2012-03-20 08:53:43 +01:00
Jos Houtman
ca6a4b23c3 Added puppet-lint support with the use of subtypes 2012-03-20 05:56:51 +01:00
Mikael Fridh
f4a2433c16 puppet: enable --storeconfigs on 2.7
without --storeconfigs it's going to complain alot if you have any imported
resources in your manifests.
2012-02-14 10:15:17 +01:00
Martin Grenfell
cb6205254e puppet: update errorformat to ignore a general message
The checker was always outputting this if syntax errors were detected:
    err: Try 'puppet help parser validate' for usage,

So just ignore it. See #157
2012-02-13 09:55:43 +00:00
Martin Grenfell
6d34c70713 puppet: skip --ignoreimport for versions >= 2.7.10
This option was removed in 2.7.10 - as noted in #157.
2012-02-13 09:55:43 +00:00
Martin Grenfell
258812abc0 puppet: fix the puppet version number extraction
fix a bug where the version number had a newline after the last version
number component e.g. 2.7.9\n
2012-02-08 23:29:50 +00:00
Martin Grenfell
2f796a7ca1 puppet: fix indenting - make it consistent at 4 spaces 2012-02-08 13:41:37 +00:00
Martin Grenfell
b4e7f8598a puppet checker: only check the version once
Previously it was calling `system()` to check the puppet version on
every syntax check. Now we do this only once and cache the result.
2012-02-08 13:40:02 +00:00
S. Zachariah Sprackett
d0a7546f1e Make this work with releases > 2.7 2012-02-07 11:27:32 -05:00
S. Zachariah Sprackett
4b0888740e Need this on new puppet as well 2011-11-24 22:48:56 -05:00
S. Zachariah Sprackett
239aed5c09 Add --ignoreimport to puppet parse 2011-11-23 14:59:07 -05:00
Roman Gonzalez
50e99cc2c1 Improving the puppet syntax checker.
The current implementation only supports puppet < 2.6, when using
2.7.x it will complain all the time saying that the executable to check
syntax has changed. With this patch, it will work for version <= 2.7.x
2011-08-09 22:19:47 -07:00
Eivind Uggedal
e71df80fa1 Syntax checker for Puppet manifests. 2011-04-28 08:48:54 +02:00