Added suport to drive calling janitor_directory through attribute settin...#2
Added suport to drive calling janitor_directory through attribute settin...#2arieplugge wants to merge 3 commits into
Conversation
…tings. See the README + Fixed a few things - when it encounters a directory the file delete fails obviously, only include real files in the list. - declare windosw a supported platform, I have tested it. Forward slahs as separator though: F:/backup/log as a directory - one instance of an empty array catch was missing - recursive did not work properly. - bumped the version to 1.0.4
|
👍 |
|
+1 this fixed my problem with the LWRP not being loaded and getting an error However, the regex examples in the README still don't work. the first example is missing the closing backslash I think? so ruby syntax checker complains. The other 2 result in I also think it should be noted that at least one of the attributes (exclude_all, include_only, age, size) is required for the LWRP to work or you run into the same error as above. I had tried to just have Either way, thanks for the fix! |
|
@arieplugge I have submitted a PR on your fork to fix the issues I experienced |
|
This update will need some rework to integrate it properly now that the other changes have been merged. Also, I don't like the idea of shipping a default recipe in this library only cookbook. The intended use here is to include this cookbook in your own via a depends, and use the resources from there. |
...gs.
See the README
include real files in the list.
Forward slahs as separator though: F:/backup/log as a directory