[Differential] [Accepted] D4118: if is not an archive, always copy

Dan Leinir Turthra Jensen noreply at phabricator.kde.org
Wed Jan 18 11:31:32 UTC 2017


leinir accepted this revision.
leinir added a comment.
This revision is now accepted and ready to land.


  Right. I think we can sensibly pick this. I would like a comment added, though, that future "don't simply install things" logic should go into the section above this one, rather than into the if statement there... but, other than that, yes, the logic itself is good, let's go for it :)

REPOSITORY
  R304 KNewStuff

BRANCH
  phab/isarchive

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

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: mart, #plasma, leinir
Cc: plasma-devel, #frameworks, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20170118/39e4bf3f/attachment.html>


More information about the Kde-frameworks-devel mailing list