Currently the behavior with symlinks is undefined + untested.

Add test coverage, decide the new behavior, do a breaking release.

As part of this change, fix behavior when there are cyclic symlinks #1728