diff options
author | Charles Jones <cj32@txstate.edu> | 2013-02-18 18:37:48 -0600 |
---|---|---|
committer | Charles Jones <cj32@txstate.edu> | 2013-02-25 18:54:44 -0600 |
commit | b8837066dcaa389c00aaf22565bb6988ded3932e (patch) | |
tree | 5908955d9a31e49bd7d290aca1da990b44fee54e /actionpack | |
parent | 202041e762a98cb433c3a24a0b03308d4e05a99d (diff) | |
download | rails-b8837066dcaa389c00aaf22565bb6988ded3932e.tar.gz rails-b8837066dcaa389c00aaf22565bb6988ded3932e.tar.bz2 rails-b8837066dcaa389c00aaf22565bb6988ded3932e.zip |
Fix deletion of empty directories:
1. When comparing the directory to delete against the top level
cache_path, use File.realpath to make sure we aren't comparing two
unequal strings that point to the same path. This occurs, for
example, when cache_path has a trailing slash, which it does in the
default Rails configuration. Since the input to
delete_empty_directories never has a trailing slash, the comparison
will never be true and the top level cache directory (and above) may
be deleted. However…
2. File.delete raises EPERM when trying to delete a directory, so no
directories have ever been deleted. Changing the code to Dir.delete
fixes that.
Diffstat (limited to 'actionpack')
0 files changed, 0 insertions, 0 deletions