summaryrefslogtreecommitdiff
path: root/lib
Commit message (Collapse)AuthorAgeFilesLines
* Issue 4334: Because set(property, nil) gets turned into get(property), add a ↵cd/fix-spurious-warningsChris Doherty2016-03-112-5/+6
| | | | 'nil_set' parameter to get() to skip a spurious warning when that happens.
* Merge pull request #4690 from chef/lcg/fix-escape-globLamont Granquist2016-03-1121-30/+30
|\ | | | | s/escape_glob/escape_glob_dir/
| * s/escape_glob/escape_glob_dir/lcg/fix-escape-globLamont Granquist2016-03-1021-30/+30
| | | | | | | | | | Removes all remaining references to the deprecated escape_glob helper routine.
* | Vbox's shared filesystem driver uses non-file device names too. Noah Kantrowitz2016-03-101-1/+1
|/
* Fix property coercion in both places, because they were different and Noah Kantrowitz2016-03-092-2/+2
| | | one in Chef::Mixin::Properties would result in duplicated values.
* Fix typo in whyrun messageMichael Pereira2016-03-081-1/+1
|
* Merge pull request #4661 from davide125/systemd-userPhil Dibowitz2016-03-084-11/+67
|\ | | | | Extend service resource to support systemd user services
| * error out if provider doesn't support user servicesDavide Cavalca2016-03-043-0/+18
| |
| * double quotes to make rubocop happyDavide Cavalca2016-03-031-6/+6
| |
| * Extend service resource to support systemd user servicesDavide Cavalca2016-03-032-11/+49
| |
* | Merge pull request #4673 from dduponchel/GH-2561Thom May2016-03-081-3/+9
|\ \ | | | | | | preferred_manifest_record: fix pretty print.
| * | preferred_manifest_record: fix pretty print.David Duponchel2016-03-071-3/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The `preferred_manifest_record` function pretty prints the list of existing files when the asked file can't be found. To do that, the root path of the cookbook is removed from each existing file but the update is done on the `existing_files` variable, not on a copy. When called several times, this variable will eventually be equals to `[nil]`, leading to a NoMethodError. Without the fix, the new unit test fails with > expected Chef::Exceptions::FileNotFound, got #<NoMethodError: undefined method `[]' for nil:NilClass> with backtrace: > # ./lib/chef/cookbook_version.rb:321:in `block in preferred_manifest_record' > # ./lib/chef/cookbook_version.rb:319:in `map!' > # ./lib/chef/cookbook_version.rb:319:in `preferred_manifest_record' > # ./spec/unit/cookbook_version_file_specificity_spec.rb:318:in `block (3 levels) in <top (required)>' > # ./spec/unit/cookbook_version_file_specificity_spec.rb:317:in `block (2 levels) in <top (required)>' Fixes #2561.
* | | Bump version to 12.8.112.8.1jk/bump-versionJohn Keiser2016-03-071-1/+1
|/ /
* | Merge pull request #4617 from chef/lcg/chefstyle-perfLamont Granquist2016-03-0454-83/+133
|\ \ | | | | | | Autofixing new Perf cops in 0.37.2
| * | add some @yield docsLamont Granquist2016-03-0416-5/+53
| | | | | | | | | | | | added some FIXMEs because trying to sort out types for random undocumented code was making me completely aggro
| * | remove unused block argumentslcg/chefstyle-perfLamont Granquist2016-02-2318-22/+22
| | |
| * | Autofixing new Perf cops in 0.37.2Lamont Granquist2016-02-2349-59/+61
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6 Performance/Casecmp 18 Performance/Detect 1 Performance/RangeInclude 27 Performance/RedundantBlockCall 6 Performance/RedundantMatch 5 Performance/RedundantMerge 18 Performance/StringReplacement
* | | Adding specs for rehash updatetyler-ball2016-03-042-3/+4
| | |
* | | Changing the knife behavior when using the cached plugins to only display 1 ↵tyler-ball2016-03-041-3/+21
| | | | | | | | | | | | error message when plugin files are missing from the cache
* | | First part of fixing the rehash command, it now always loads gems from disc ↵tyler-ball2016-03-043-1/+15
| | | | | | | | | | | | instead of using the hash file
* | | Merge pull request #4664 from chef/salam/databag-fixThom May2016-03-041-1/+0
|\ \ \ | | | | | | | | Data bag item hash can have name key
| * | | Data bag item hash can have name keysalam/databag-fixSalim Alam2016-03-031-1/+0
| | | |
* | | | Merge pull request #4478 from chef/lcg/rfc-060-gem-metadataLamont Granquist2016-03-038-5/+198
|\ \ \ \ | |/ / / |/| | | RFC-060 gem metadata MVP
| * | | need to require bundlerlcg/rfc-060-gem-metadataLamont Granquist2016-02-231-0/+1
| | | | | | | | | | | | | | | | for Bundler::UI
| * | | remove now-unnecessary requireLamont Granquist2016-02-091-1/+0
| | | |
| * | | rubocopismsLamont Granquist2016-02-091-2/+2
| | | |
| * | | add comments and cleanupLamont Granquist2016-02-093-7/+14
| | | |
| * | | remove unnecessary paranoiaLamont Granquist2016-02-091-5/+1
| | | |
| * | | make rubygems.org configurableLamont Granquist2016-02-092-2/+2
| | | | | | | | | | | | | | | | for bonus points gem_package will pick this up as well.
| * | | get the formatting for cookbook gems correctLamont Granquist2016-02-096-18/+119
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | this leverages the PR: https://github.com/bundler/bundler/pull/4245 to redirect output from bundler into chef's logger/formatter to get the UI right. if you don't have that in the external bundler you'll just get output on STDOUT. fix is to upgrade bundler.
| * | | fixed my .vimrc auto-copyright headerLamont Granquist2016-02-091-1/+1
| | | |
| * | | RFC-060 gem metadata MVPLamont Granquist2016-02-095-1/+90
| | | |
* | | | Merge pull request #4655 from coderanger/solobagNoah Kantrowitz2016-03-031-0/+1
|\ \ \ \ | |_|_|/ |/| | | Clearer exception for loading non-existent data bag items in solo mode.
| * | | Clearer exception for loading non-existent data bag items in solo mode.Noah Kantrowitz2016-03-021-0/+1
| | | |
* | | | Merge pull request #4307 from davide125/systemd-maskPhil Dibowitz2016-03-033-2/+67
|\ \ \ \ | |/ / / |/| | | Extend service resource to support masking
| * | | add tests for is_masked?Davide Cavalca2016-02-221-1/+1
| | | |
| * | | is_masked? fixDavide Cavalca2016-02-221-1/+1
| | | |
| * | | rubocop doesn't like single quotesDavide Cavalca2016-02-221-1/+1
| | | |
| * | | rework to use service resource directly instead of subclassingDavide Cavalca2016-02-224-84/+52
| | | |
| * | | rework to use a subclassed SystemdService resourceDavide Cavalca2016-02-224-50/+84
| | | |
| * | | raise UnsupportedAction for mask / unmask in the base classDavide Cavalca2016-02-221-0/+8
| | | |
| * | | Extend service resource to support maskingDavide Cavalca2016-02-223-2/+57
| | | |
* | | | Revert "Merge pull request #4465 from ↵tm/revert_4465Thom May2016-03-021-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | artursitarski/change-shell-for-locked-user" This reverts commit 94494e761763b877c3ef3b52eee9ac9df2186340, reversing changes made to a95a87d50c9e5af2fbd583afcf972880656d1a4b.
* | | | Merge pull request #4646 from coderanger/negative-lengthNoah Kantrowitz2016-03-011-0/+5
|\ \ \ \ | | | | | | | | | | Handle negative content length headers too.
| * | | | Split the negative check to its own log message.Noah Kantrowitz2016-02-291-1/+6
| | | | |
| * | | | Handle negative content length headers too.Noah Kantrowitz2016-02-291-1/+1
| | | | | | | | | | | | | | | This is equivalent to not passing one, we can't check for bad downloads.
* | | | | Merge pull request #4645 from chef/msiMatt Wrock2016-02-291-1/+1
|\ \ \ \ \ | | | | | | | | | | | | fixes #4639 and returns true from define_resource_requirements for MSIs
| * | | | | fixes #4639 and returns true from define_resource_requirements for MSIsmsiMatt Wrock2016-02-291-1/+1
| |/ / / /
* | | | | if no module name is found for a valid dsc resource default to ↵dscMatt Wrock2016-02-291-5/+2
|/ / / / | | | | | | | | | | | | PSDesiredStateConfiguration
* | | | use Pathname#cleanpath instead of Chef::Util::PathHelper.cleanpath to ↵Matt Wrock2016-02-291-2/+2
| | | | | | | | | | | | | | | | compare globed files with forward slashes