rlishtaba / rubyripper

Automatically exported from code.google.com/p/rubyripper
0 stars 0 forks source link

rubyripper should create missing directories in Log #489

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
As of right now, the master branch of Rubyripper crashes when it is unable to 
create a "ripping.log" file (e.g. if the directory it is creating the log file 
in does not yet exist).  This can be fixed by iterating over the path and 
creating any directories which don't already exist.

I've attached a small patch which fixes this issue.

Original issue reported on code.google.com by comradec...@gmail.com on 1 Nov 2011 at 5:24

Attachments:

GoogleCodeExporter commented 8 years ago
This is a recurring theme on several places in the code. Please move this logic 
to the FileAndDir class and update the Log class to use that instead.

Original comment by boukewou...@gmail.com on 1 Nov 2011 at 7:07

GoogleCodeExporter commented 8 years ago
This patch makes use of FileAndDir to accomplish the same.

Original comment by comradec...@gmail.com on 2 Nov 2011 at 1:35

Attachments:

GoogleCodeExporter commented 8 years ago
Your patch is merged with the master branch :)

Original comment by boukewou...@gmail.com on 5 Nov 2011 at 8:28