Issue #9670 has been updated by James Turnbull. Category set to parser Status changed from Unreviewed to Needs More Information Assignee set to Anchor Systems
What version is this? Can you submit a patch (if so can you please sign a Contributor License Agreement in the top right menu) so we can review it. Thanks. ---------------------------------------- Bug #9670: puppet parser validate --ignoreimport ignores a bit too much https://projects.puppetlabs.com/issues/9670 Author: Anchor Systems Status: Needs More Information Priority: Normal Assignee: Anchor Systems Category: parser Target version: Affected Puppet version: Keywords: Branch: Attempting to validate changed manifests in a git pre-commit hook, I found that a regular `puppet parser validate` was complaining about not being able to find imported files. So, I whacked `--ignoreimport` in there and all seemed fine... until I realised it wasn't finding legitimate syntax errors. Turns out that `--ignoreimport` ignores a bit too much, and doesn't actually perform a parse. Quick fix was to remove the first line of `perform_initial_import` (in `node/environment.rb`) so that it didn't return immediately. What other effects that might have, I couldn't rightly say, but at least I know when I've broken my manifests now. -- You have received this notification because you have either subscribed to it, or are involved in it. To change your notification preferences, please click here: http://projects.puppetlabs.com/my/account -- You received this message because you are subscribed to the Google Groups "Puppet Bugs" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/puppet-bugs?hl=en.
