summaryrefslogtreecommitdiff
path: root/lib/chef/provider/package
Commit message (Collapse)AuthorAgeFilesLines
* This commit is why AndOr needs to be bannedLamont Granquist2016-03-141-2/+2
| | | | | | | | I can barely understand what these intend. I'm pretty certain that the logic in the encrypted data bag item code was wrong (fixed now to match the comment) and I'm believe I converted the yum code to preserve the logic. But holy fuck its hard to read the yum code that I'm replacing.
* chefstyle Style/AndOr fixesLamont Granquist2016-03-145-43/+43
| | | | | this is part of our informal style guide, lets make it formal since clearly its not getting followed very well.
* s/escape_glob/escape_glob_dir/lcg/fix-escape-globLamont Granquist2016-03-102-3/+3
| | | | | Removes all remaining references to the deprecated escape_glob helper routine.
* Merge pull request #4617 from chef/lcg/chefstyle-perfLamont Granquist2016-03-042-2/+2
|\ | | | | Autofixing new Perf cops in 0.37.2
| * Autofixing new Perf cops in 0.37.2Lamont Granquist2016-02-232-2/+2
| | | | | | | | | | | | | | | | | | | | 6 Performance/Casecmp 18 Performance/Detect 1 Performance/RangeInclude 27 Performance/RedundantBlockCall 6 Performance/RedundantMatch 5 Performance/RedundantMerge 18 Performance/StringReplacement
* | Merge pull request #4478 from chef/lcg/rfc-060-gem-metadataLamont Granquist2016-03-031-1/+1
|\ \ | | | | | | RFC-060 gem metadata MVP
| * | make rubygems.org configurableLamont Granquist2016-02-091-1/+1
| | | | | | | | | | | | for bonus points gem_package will pick this up as well.
* | | fixes #4639 and returns true from define_resource_requirements for MSIsmsiMatt Wrock2016-02-291-1/+1
| | |
* | | Meant to use the value from the if..then, then put a line above the if.cd/fix-choco-errorChris Doherty2016-02-251-6/+4
| | |
* | | Add a spec to better mimic the acceptance test.Chris Doherty2016-02-251-6/+6
| | |
* | | Change the Chocolatey check to look for choco.exe in #choco.exe.unknown2016-02-241-11/+10
| |/ |/|
* | fixed exe package downloadsMatt Wrock2016-02-221-30/+39
| |
* | flip multiline function arguments aroundlcg/rubocop-0.37.2Lamont Granquist2016-02-191-2/+2
| | | | | | | | no enforced trailing comma on arguments...
* | deal with 0.37.2 renamed copsLamont Granquist2016-02-184-7/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 252 Style/TrailingCommaInLiteral 84 Style/TrailingCommaInArguments 15 Style/SpaceAroundKeyword -- 351 Total We already dealt with SpaceAroundKeyword under its old name SpaceBeforeModifierKeyword, it looks like it got stricter about spaces after keywords. TrailingComma also got split, and it looks like the TrailingCommaInArguments behavior is new?
* | zypper multipackage performance fixlcg/fix-zypperLamont Granquist2016-02-171-1/+1
| | | | | | | | | | helps to remember the values of expensive things that we go off and compute.
* | do not include source parameter when removing a chocolatey package and ↵no_webMatt Wrock2016-02-111-3/+4
| | | | | | | | ensure source is used on all functional tests
* | autofixing auto-inserted delimiterslcg/chefstyle-batchLamont Granquist2016-02-092-2/+2
| |
* | auto fixing some rubocopsLamont Granquist2016-02-095-5/+5
|/ | | | | | | | | Style/NegatedWhile Style/ParenthesesAroundCondition Style/WhileUntilDo Style/WordArray Performance/ReverseEach Style/ColonMethodCall
* autofixing whitespace copsLamont Granquist2016-02-0519-94/+92
| | | | | | | | | | | | | | | | | | | | | | 4174 Style/SpaceInsideHashLiteralBraces 1860 Style/SpaceAroundOperators 1336 Style/SpaceInsideBlockBraces 1292 Style/AlignHash 997 Style/SpaceAfterComma 860 Style/SpaceAroundEqualsInParameterDefault 310 Style/EmptyLines 294 Style/IndentationConsistency 267 Style/TrailingWhitespace 238 Style/ExtraSpacing 212 Style/SpaceBeforeBlockBraces 166 Style/MultilineOperationIndentation 144 Style/TrailingBlankLines 120 Style/EmptyLineBetweenDefs 101 Style/IndentationWidth 82 Style/SpaceAroundBlockParameters 40 Style/EmptyLinesAroundMethodBody 29 Style/EmptyLinesAroundAccessModifier 1 Style/RescueEnsureAlignment
* Chocolatey provider: Raise a MissingLibrary exception if we can't find ↵cd/choco-error-msgChris Doherty2016-02-031-8/+28
| | | | Chocolatey.
* Update for chef.io domain in copyright lines too.Noah Kantrowitz2016-02-021-1/+1
|
* Update all auth email address from opscode.com and getchef.com to chef.io.Noah Kantrowitz2016-02-027-8/+8
| | | Generated via git ls-files | xargs perl -pi -e "s/(Author.*?<[^@]+@)(?:opscode\\.com|getchef\\.com)(>)/\\1chef.io\\2/gi"
* Replace all Opscode copyrights with Chef Software.Noah Kantrowitz2016-02-028-8/+8
| | | Created via git ls-files | xargs perl -pi -e "s/(Copyright.*?), Opscode(,)? Inc(\.)?/\\1, Chef Software Inc./gi"
* Copyright year update for 2016 and massive cleanup.Noah Kantrowitz2016-02-0226-31/+31
| | | Generated via git ls-files | xargs perl -pi -e "s/[Cc]opyright (?:\([Cc]\) )?((?\!$(date +%Y))\\d{4})(-\\d{4})?([, ][ \d]+)*(,|(?= ))/Copyright \\1-$(date +%Y),/g"
* only require what we needlcg/zypper-multipackageLamont Granquist2016-01-251-3/+1
|
* zypper multipackage patchLamont Granquist2016-01-251-39/+52
|
* Autofixing Style/PercentLiteralDelimeterslcg/percentliteraldelimetersLamont Granquist2016-01-184-4/+4
| | | | | | See chef/chefstyle#11 for analysis and discussion. We select '{}' since audit of our source code shows that is the most common, and that used to be the dominant learning paradigm (e.g. in ruby 1.9 pickaxe book.
* fix chefstyle failuresLamont Granquist2016-01-151-1/+1
| | | | | | | | autocorrects some merges that will be broken when the next chefstyle is released. also runs chefstyle after the travis tests, because i'd rather know how the code is broken rather than how its not pretty.
* fix candidate version filteringMatt Wrock2016-01-141-11/+22
|
* Use double quotes by defaultThom May2016-01-1425-161/+161
| | | | | | | This is an entirely mechanically generated (chefstyle -a) change, to go along with chef/chefstyle#5 . We should pick something and use it consistently, and my opinion is that double quotes are the appropriate thing.
* autocorrecting Style/TrailingCommalcg/trailing_commaLamont Granquist2016-01-133-4/+4
| | | | chefstyle -a fixed 1044 occurrances
* Correct uses of fail to raisetm/fix_failThom May2016-01-131-1/+1
| | | | Style/SignalException - 3
* Merge pull request #4384 from chef/lcg/really-fix-copsLamont Granquist2016-01-121-24/+24
|\ | | | | really fix cops
| * really fix copslcg/really-fix-copsLamont Granquist2016-01-121-24/+24
| |
* | Merge pull request #4378 from chef/choco_editsMatt Wrock2016-01-121-2/+18
|\ \ | | | | | | assert candidates exist for alternate sources and when pinning versions
| * | assert candidates exist for alternate sources and when pinning versionsMatt Wrock2016-01-121-2/+18
| | |
* | | Merge pull request #4375 from chef/no_version_detectMatt Wrock2016-01-121-13/+1
|\ \ \ | |_|/ |/| | fixes #4354 and fixes #4372 stop trying to auto detect versions for exe installers
| * | fexes #4354 and fiexs #4372 stop trying to auto detect versions for exe ↵no_version_detectMatt Wrock2016-01-111-13/+1
| |/ | | | | | | installers
* | Merge pull request #4381 from chef/lcg/useless-to-sLamont Granquist2016-01-121-2/+2
|\ \ | | | | | | chefstyle: fix Lint/StringConversionInInterpolation
| * | chefstyle: fix Lint/StringConversionInInterpolationLamont Granquist2016-01-121-2/+2
| |/ | | | | | | useless use of `"#{foo.to_s}"`
* | fix some lint copsLamont Granquist2016-01-121-1/+1
|/ | | | | | | | | fixes the following offenses: 46 Lint/EndAlignment 21 Lint/BlockAlignment 3 Lint/SpaceBeforeFirstArg 1 Lint/DefEndAlignment
* chocolatey multipackage providerlcg/chocolatey-packageLamont Granquist2016-01-081-0/+230
|
* fixing post merge test breaksMatt Wrock2015-12-151-1/+1
|
* non msi packages must explicitly provide a source attribute on installMatt Wrock2015-12-153-16/+25
|
* Use properties in package resourcesjk/package-propertiesJohn Keiser2015-12-1510-22/+3
|
* Add missing exception variableJay Mundrawala2015-12-091-1/+1
|
* Merge pull request #4193 from chef/mwrock/packageSalim Alam2015-12-084-27/+350
|\ | | | | adds support for installer types inno, nsis, wise and installshield to windows_package resource
| * adds support to installer types inno, nsis, wise and installshield top the ↵Matt Wrock2015-12-074-27/+350
| | | | | | | | windows_package resource
* | restore handling of dpkg -s returns codes (w/tests)lcg/dpkg-s-return-codesLamont Granquist2015-12-071-1/+1
|/ | | | | | | | | | | | | | | | Previously dpkg -s would allow a 0 or 1 exit status and it wasn't entirely clear why this was so either from the code or the tests. This restores throwing an exception if we are outside of the [0,1] range, and then adds tests for both behaviors: - on old ubuntu/debian we get an exit(0) and output on stdout if the file is not installed - on newer ubuntu/debian we get an exit(1) and output on stderr if the file is not installed Added tests for both those cases, and the case where dpkg -s does some other kind of exitcode barf and we should raise.
* fix couple of issues in dpkg cleanupLamont Granquist2015-12-021-44/+5
| | | | | | | | | | | | this is just code rearrangement, no changes to behavior at all. pushes the source_array and resolved_source_array up into the superclass so that other providers can use it (as it was intended, but that slipped out of the last PR). also renamed it to something a little longer and less likely to clash with variables (which it was already doing). the block comment also refers to an earlier iteration that used coercions, which had to be backed out because of chefspec.