D8274: Don't create new symlinks when copying symlinks - copy the file's contents - like /usr/bin/cp

[Henrik Fehlauer]

rkflx added subscribers: dfaure, rkflx.
rkflx added a comment.


  Unfortunately, this two year old patch does not apply for me anymore to the current git master (while it did apply to v5.15.0, it would not compile then).
  
  In addition, I'm not convinced this should be changed at the Frameworks level. Developers might rely on the current behaviour, so the patch should probably rather clarify the API docs <https://api.kde.org/frameworks/kio/html/namespaceKIO.html#a8e3118adc0bb43d03ad15d67bc3d335c> from `cp` to `cp -a`. According to @dfaure, this is on purpose <https://bugs.kde.org/show_bug.cgi?id=124898#c2>. If your IRC logs say otherwise, feel free to post them here for reference.
  
  To give a practical example: Let's assume in Dolphin a user copies a folder to another location on the same partition, which he later plans to restore after trying out some things in the original folder. He would rightly expect to get back the original state, including all symlinks. With your patch (please correct me if I misunderstood) this would break.
  
  Therefore, I would encourage you (in addition to the API docs modification) to provide patches, or at least open bugzilla issues where still missing, against applications where a deep copy would be the appropriate behaviour indeed (e.g. Gwenview).

REPOSITORY
  R241 KIO

REVISION DETAIL
  https://phabricator.kde.org/D8274

To: shlomif
Cc: rkflx, dfaure, bcooksley, #frameworks
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20171013/80f90bca/attachment.html>