PHP bundler logs far too many deletions on install

shawnrice / alfred-bundler

a utility / workflow to handle dependencies

MIT License

21 stars 3 forks source link

PHP bundler logs far too many deletions on install #51

Closed deanishe closed 10 years ago

deanishe commented 10 years ago

Whenever I install the bundler, the majority of the log entries are PHP code reporting some cache directory/file it's deleted.

Most of the logged paths are contained within other deleted paths. Would it be possible to just log the deletion of the top-level directory in each case? If /some/dir has been deleted, it goes without saying that /some/dir/more_stuff/… and everything in it has been deleted.

shawnrice commented 10 years ago

We can do that. I put the report function in a recursive rmdir method, so that's why it just log spews so much.

I'll instead attach it to the instance calls. When relevant.

deanishe commented 10 years ago

This has been fixed, right? Want to close it?

shawnrice commented 10 years ago

It was pseudo-fixed. Now it's 'fixed' in that it was just pulled out. I'll reimplement less obtrusive logging soon.