{"push_id":552733936,"size":19,"distinct_size":19,"ref":"refs/heads/bug/master/PUP-1208-fix-checksum","head":"667a49334afdc697b37703a0633f6ec5ee5e680c","before":"d5d1aad5a843617152f56a3ec37f729a0aaff695","commits":[{"sha":"573bc48740d2f61e8d1c163ea1f6985b6091b125","author":{"email":"95e828102f92afbaa4d3ed92646a950670566628@cloudsmith.com","name":"Henrik Lindberg"},"message":"(maint) Fix missing args in EvaluatingParser.evaluate_file\n\nThe method evaluate_file was not used earlier and a problem with\nnot accepting and passing on scope went undetected.\n\nThis corrects the signature of the evalue_file method.","distinct":true,"url":"https://api.github.com/repos/MikaelSmith/puppet/commits/573bc48740d2f61e8d1c163ea1f6985b6091b125"},{"sha":"a0513ee9275a92917e23f3bf3414ee2835155b80","author":{"email":"95e828102f92afbaa4d3ed92646a950670566628@cloudsmith.com","name":"Henrik Lindberg"},"message":"(PUP-2351) Add a puppet epp command for validation and rendering\n\nThis adds a new command called puppet-epp that can validate\none or several epp templates, render them to text with\nvarious options for specifying facts and variables in scope,\ngiving variables to the template etc.\n\nThe command also has a dump action that is useful for debugging\nepp parsing. This command is similar to the puppet parser dump\naction.\n\nFor more details run 'puppet man epp'","distinct":true,"url":"https://api.github.com/repos/MikaelSmith/puppet/commits/a0513ee9275a92917e23f3bf3414ee2835155b80"},{"sha":"7a0d2f7dd6829c255bfb3a220a64706a219a8d7e","author":{"email":"95e828102f92afbaa4d3ed92646a950670566628@cloudsmith.com","name":"Henrik Lindberg"},"message":"(maint) Remove unreachable code","distinct":true,"url":"https://api.github.com/repos/MikaelSmith/puppet/commits/7a0d2f7dd6829c255bfb3a220a64706a219a8d7e"},{"sha":"8432e9fa25e82b926c96916386426d58c818d7c0","author":{"email":"95e828102f92afbaa4d3ed92646a950670566628@cloudsmith.com","name":"Henrik Lindberg"},"message":"(PUP-2531) Add epp application to make epp face available from cmdline","distinct":true,"url":"https://api.github.com/repos/MikaelSmith/puppet/commits/8432e9fa25e82b926c96916386426d58c818d7c0"},{"sha":"a235e991896b878d7152fec574897c3ea21c9036","author":{"email":"95e828102f92afbaa4d3ed92646a950670566628@cloudsmith.com","name":"Henrik Lindberg"},"message":"(PUP-2531) Fix comments from review\n\nThis fixes a number of small problems/typos as pointed out in review.","distinct":true,"url":"https://api.github.com/repos/MikaelSmith/puppet/commits/a235e991896b878d7152fec574897c3ea21c9036"},{"sha":"9e43b0847a8e44c50d0c7ada75ef79c0ed9f96a8","author":{"email":"95e828102f92afbaa4d3ed92646a950670566628@cloudsmith.com","name":"Henrik Lindberg"},"message":"(PUP-2531) Add support to validate a local file reference\n\nSince validate is mostly useful when developing a module and there\nis no modulepath to use for looking up a template using a\nmodulename/templatename reference it becomes cumbersome to test\na template in a local file. While it was possible to use an absolute\nreference, this becomes replacing one difficulty with another.\n\nThis commit adds logic that first checks if the given reference can\nbe loaded from the module path, and if that fails, tries to load it as\na file reference. This means that this now works:\n\npuppet epp validate foo.epp\npuppet epp validate ./somehwere/templates/foo.epp","distinct":true,"url":"https://api.github.com/repos/MikaelSmith/puppet/commits/9e43b0847a8e44c50d0c7ada75ef79c0ed9f96a8"},{"sha":"ea69436d9ba3f2da1f49ee0cb196e546788c6074","author":{"email":"95e828102f92afbaa4d3ed92646a950670566628@cloudsmith.com","name":"Henrik Lindberg"},"message":"(PUP-2531) Make missing files to epp dump exit with 1","distinct":true,"url":"https://api.github.com/repos/MikaelSmith/puppet/commits/ea69436d9ba3f2da1f49ee0cb196e546788c6074"},{"sha":"5b3d7c3854bd61a95f3d03079086f7ffd5a27ab0","author":{"email":"95e828102f92afbaa4d3ed92646a950670566628@cloudsmith.com","name":"Henrik Lindberg"},"message":"(PUP-2531) Add support to render an epp file with relative path\n\nThe render command only supported rendering arbitrary files if\nthey were given with a full path.\n\nThis commit changes this to use a relative path that is not a reference\nto a template in a module on the modulepath to be taken as a\nrelative file path.","distinct":true,"url":"https://api.github.com/repos/MikaelSmith/puppet/commits/5b3d7c3854bd61a95f3d03079086f7ffd5a27ab0"},{"sha":"914756cd3203b2d2d6ed56a3a5b523716f168483","author":{"email":"95e828102f92afbaa4d3ed92646a950670566628@cloudsmith.com","name":"Henrik Lindberg"},"message":"(PUP-2531) Fix formatting of multiple errors on epp render\n\nThe output was inconsistent with newlines inserted if there\nwhere more than one missing file.","distinct":true,"url":"https://api.github.com/repos/MikaelSmith/puppet/commits/914756cd3203b2d2d6ed56a3a5b523716f168483"},{"sha":"c1d0e85f0881f8919750de1ad7de5848fd011b9e","author":{"email":"95e828102f92afbaa4d3ed92646a950670566628@cloudsmith.com","name":"Henrik Lindberg"},"message":"(PUP-2531) Refactor error handling of epp face\n\nThe error handling and the final exit status was not handled\nconsistently. The Puppet Face API requires that actions produce\nthe string that is to be output to stdout, or that they raise an\nexception and that the exception raising causes an exit status != 0\nwhen run as an command line application.\n\nThe implementation called exit directly for some actions but not all.\n\nThis commit changes the behavior to print to a String buffer and then\nreturning that or raising an exception of the action should exit.\n\nThis has a small negative side effect that partial results can not be\nviewed when processing multiple files; output is either produced or the\naction fails. This is how the face api is designed; producing partial\nresult and also exiting is not possible without violating the API.","distinct":true,"url":"https://api.github.com/repos/MikaelSmith/puppet/commits/c1d0e85f0881f8919750de1ad7de5848fd011b9e"},{"sha":"f19fa99d9fff4434ddfb47163a64163ff17eb20b","author":{"email":"95e828102f92afbaa4d3ed92646a950670566628@cloudsmith.com","name":"Henrik Lindberg"},"message":"(PUP-2531) Refactor search for template in epp face\n\nThis refactors the search for a template file by making two\nidentical sequences call a common method to either return the\nabsolute template file, or an existing relative file.\n\nOne case of such mapping remains but it needs different handling\nsince it calls into code that only treats absolute paths\nas template file references in the file system, and all relative\npaths are module/template based.","distinct":true,"url":"https://api.github.com/repos/MikaelSmith/puppet/commits/f19fa99d9fff4434ddfb47163a64163ff17eb20b"},{"sha":"3ab1562022af42a0a3b54c61675c3777b90576ba","author":{"email":"95e828102f92afbaa4d3ed92646a950670566628@cloudsmith.com","name":"Henrik Lindberg"},"message":"(maint) Remove travis sudo: false setting\n\nThis removes the sudo: false setting as this is suspected to cause\nproblems when running tests on travis.","distinct":true,"url":"https://api.github.com/repos/MikaelSmith/puppet/commits/3ab1562022af42a0a3b54c61675c3777b90576ba"},{"sha":"ad54e56afab718a2f918960b80b2886fa9c823d1","author":{"email":"c028c213ed5efcf30c3f4fc7361dbde0c893c5b7@puppetlabs.com","name":"Josh Cooper"},"message":"(PUP-2699) Remove as_platform shared context\n\nCommit 772f7566 added the ability to emulate posix or windows by\nstubbing the corresponding platform facts, and file & path separators.\nWhile a good idea in practice, it causes problems. For example, when\nexecuting `as_platform => :posix` tests on a Windows host, the path\nseparator is changed to ':'. However, if any code calls into facter, it\nwill use the default fact search path of\n`C:\\ProgramData\\PuppetLabs\\facter\\facts.d` which is interpreted to mean\ntwo directories, 'C' and '\\ProgramData\\...' emitting warnings of the form:\n\n    Excluding C from search path. Fact file paths must be an absolute directory\n\nAnother source of problems is when trying to use a file-based log\ndestination in a test. For example, if you create a log destination\nusing a posix path, but try to execute an `as_platform => :windows` test,\nthe file separator will be changed from '/' to '\\\\', which causes the\nlog destination to report that it is not a valid absolute path.\n\nThis commit eliminates the as_platform shared context and modifies the\nrespective tests to only execute on the corresponding platforms. It\nmeans some unit tests will only execute on certain platforms, but that\nis a small subset, and leads to more predictable results.","distinct":true,"url":"https://api.github.com/repos/MikaelSmith/puppet/commits/ad54e56afab718a2f918960b80b2886fa9c823d1"},{"sha":"264abcc5373e2b9f629198e7c605e1bdb114388d","author":{"email":"7cdd076e25adef0e25d3fedc5490d7fb59f5b1c0@puppetlabs.com","name":"Kylo Ginsberg"},"message":"Merge pull request #3498 from joshcooper/ticket/master/PUP-2699-remove_as_platform\n\n(PUP-2699) Remove as_platform shared context","distinct":true,"url":"https://api.github.com/repos/MikaelSmith/puppet/commits/264abcc5373e2b9f629198e7c605e1bdb114388d"},{"sha":"5224eadbb6e9503aa5f7aeedc6ca8aabd2fe5738","author":{"email":"75ef9faee755c70589550b513ad881e5a603182c@puppetlabs.com","name":"Nick Lewis"},"message":"Merge pull request #3357 from hlindberg/PUP-2531_add-epp-validate-and-render-face\n\n(PUP-2531) add epp validate and render face","distinct":true,"url":"https://api.github.com/repos/MikaelSmith/puppet/commits/5224eadbb6e9503aa5f7aeedc6ca8aabd2fe5738"},{"sha":"9f0062ac94cb330e540724cfc257c4bf33bc45e4","author":{"email":"dc76e9f0c0006e8f919e0c515c66dbba3982f785@27cbe62520b4.(none)","name":"root"},"message":"(PUP-3887) Fix checksum option in file_server\n\nThe file_server uses the requested checksum_type to calculate a\nfile checksum, rather than always defaulting to md5.","distinct":true,"url":"https://api.github.com/repos/MikaelSmith/puppet/commits/9f0062ac94cb330e540724cfc257c4bf33bc45e4"},{"sha":"51395c71ab6b67144599f560ddf059a629ecdcdb","author":{"email":"dc76e9f0c0006e8f919e0c515c66dbba3982f785@0c320f3f5201.(none)","name":"root"},"message":"(PUP-3889) Fix checksum_type for file\n\nWhen checking whether the local file must be modified, ensure the metadata\nincludes the requested checksum, rather than defaulting to md5.","distinct":true,"url":"https://api.github.com/repos/MikaelSmith/puppet/commits/51395c71ab6b67144599f560ddf059a629ecdcdb"},{"sha":"ec2ae565c124718326c6d30226646e7397bfe1f1","author":{"email":"5fa21ba918749479f1a2164cc11f556766c4eb5f@puppetlabs.com","name":"Michael Smith"},"message":"(PUP-3889) Fix streaming checksums\n\nStreaming checksums are compared to file checksums, so they must behave\nthe same way on a stream that is being written to a file.","distinct":true,"url":"https://api.github.com/repos/MikaelSmith/puppet/commits/ec2ae565c124718326c6d30226646e7397bfe1f1"},{"sha":"667a49334afdc697b37703a0633f6ec5ee5e680c","author":{"email":"5fa21ba918749479f1a2164cc11f556766c4eb5f@puppetlabs.com","name":"Michael Smith"},"message":"(PUP-3890) Fix mtime/ctime checksum type\n\nMakes file contents that use mtime or ctime for checksum_type\ncompare to ensure the target mtime/ctime is newer than the source.","distinct":true,"url":"https://api.github.com/repos/MikaelSmith/puppet/commits/667a49334afdc697b37703a0633f6ec5ee5e680c"}]}