[ 2016-10-16, Eric reopens and provides improved wording ]
The current PR makes using copy(...) to copy/create a directory symlink an error. For example, the following is now an error:
copy("/", "root", copy_options::create_symlinks);
However the current PR doesn't handle the case where both copy_options::create_symlinks and copy_options::recursive are specified. This case is still incorrectly handled by bullet (3.6) [fs.op.copy].
I suggest we move the PR before this bullet so that it catches the recursive copy case, since currently the conditions are ordered:
3.6 Otherwise if is_directory(f) && (bool(options & copy_options::recursive) || ...)
3.X Otherwise if is_directory(f) && bool(options & copy_options::create_symlinks)
So 3.6 catches create_symlinks | recursive but I believe we want 3.X to handle it instead.