Follow-up from "WIP: Resolve "Add support for symbolic links""
The following discussion from !5 (closed) should be addressed:
-
@rnou started a discussion: (+1 comment) Does the commit include other changes not related to HAS_SYMLINKS? If not the define should appear on more places (i.e., target_path()). Make sure that you are not missing anything.
target_path is not under HAS_SYMLINKS ifdef because I should also refactor the constructors of the Metadata class to do that. For the moment things have been tested only with HAS_SYMLINKS enabled, we should test them also when HAS_SYMLINKS is disabled