[PATCH] Add option to operate on symbolic links for sys::fs::remove() and sys::fs::status()
rafael.espindola at gmail.com
Thu Mar 20 10:25:54 PDT 2014
> Yes, this seems better.
Cool. It LGTM :-)
> But isn’t the sys::fs::status change to allow stat’ing symlinks good in general ? Otherwise how is sys::fs::is_symlink() supposed to work, it doesn’t work currently.
sys::fs is a bit odd. It is very llvm specific (like stating to make
sure we cannot remove /dev/null), but is somewhat over general for
that. I would like to make it include only the small bits that llvm
actually uses, even if one day sys::fs is implemented on top of a
generic, standard c++ api, that api would necessarily be much more
complicated than sys::fs.
So the answer is that we should remove is_symlink, since it is dead. I
will do that in a sec.
More information about the llvm-commits