Re: Review Request 125537: Use largest timestamp in subdirectory as resource directory timestamp.

2015-10-06 Thread Xuetian Weng
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125537/ --- (Updated Oct. 6, 2015, 9:42 p.m.) Status -- This change has been mar

Re: Review Request 125537: Use largest timestamp in subdirectory as resource directory timestamp.

2015-10-06 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125537/#review86429 --- Ship it! Perfect thanks! I'll create a new tag to include th

Re: Review Request 125537: Use largest timestamp in subdirectory as resource directory timestamp.

2015-10-06 Thread Xuetian Weng
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125537/ --- (Updated Oct. 6, 2015, 11:06 a.m.) Review request for KDE Frameworks, Alb

Re: Review Request 125537: Use largest timestamp in subdirectory as resource directory timestamp.

2015-10-06 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125537/#review86416 --- Nice and original solution, I like it. Just some last fixes I

Re: Review Request 125537: Use largest timestamp in subdirectory as resource directory timestamp.

2015-10-06 Thread Xuetian Weng
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125537/ --- (Updated Oct. 6, 2015, 9:53 a.m.) Review request for KDE Frameworks, Albe

Re: Review Request 125537: Use largest timestamp in subdirectory as resource directory timestamp.

2015-10-06 Thread David Faure
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125537/#review86409 --- Good catch, thanks - and nice to see a unittest as well. src

Re: Review Request 125537: Use largest timestamp in subdirectory as resource directory timestamp.

2015-10-05 Thread Luca Beltrame
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125537/#review86407 --- +1. I hit this issue today and this patch fixes it. - Luca Be

Review Request 125537: Use largest timestamp in subdirectory as resource directory timestamp.

2015-10-05 Thread Xuetian Weng
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/125537/ --- Review request for KDE Frameworks, Albert Astals Cid and David Faure. Rep