Jenkins-kde-ci: kdelibs KDE-4.14 stable-qt4 » Linux,gcc - Build # 49 - Fixed!

2016-08-25 Thread no-reply

GENERAL INFO

BUILD SUCCESS
Build URL: 
https://build.kde.org/job/kdelibs%20KDE-4.14%20stable-qt4/PLATFORM=Linux,compiler=gcc/49/
Project: PLATFORM=Linux,compiler=gcc
Date of build: Thu, 25 Aug 2016 22:35:16 +
Build duration: 20 min

CHANGE SET
Revision dd1c2da9d26fd4cfc7fe0a25f413e536d56cf2db by Albert Astals Cid: 
(Backport karchive fix for out of directory files)
  change: edit kdecore/io/karchive.cpp
  change: add kdecore/tests/tar_relative_path_outside_archive.tar.bz2
  change: edit kdecore/tests/karchivetest.cpp
  change: edit kdecore/tests/karchivetest.h


JUNIT RESULTS

Name: (root) Failed: 0 test(s), Passed: 163 test(s), Skipped: 0 test(s), Total: 
163 test(s)

COBERTURA RESULTS

Cobertura Coverage Report
  

By packages
  

Jenkins-kde-ci: kdelibs KDE-4.14 stable-qt4 » Linux,gcc - Build # 49 - Fixed!

2016-08-25 Thread no-reply

GENERAL INFO

BUILD SUCCESS
Build URL: 
https://build.kde.org/job/kdelibs%20KDE-4.14%20stable-qt4/PLATFORM=Linux,compiler=gcc/49/
Project: PLATFORM=Linux,compiler=gcc
Date of build: Thu, 25 Aug 2016 22:35:16 +
Build duration: 20 min

CHANGE SET
Revision dd1c2da9d26fd4cfc7fe0a25f413e536d56cf2db by Albert Astals Cid: 
(Backport karchive fix for out of directory files)
  change: edit kdecore/io/karchive.cpp
  change: add kdecore/tests/tar_relative_path_outside_archive.tar.bz2
  change: edit kdecore/tests/karchivetest.cpp
  change: edit kdecore/tests/karchivetest.h


JUNIT RESULTS

Name: (root) Failed: 0 test(s), Passed: 163 test(s), Skipped: 0 test(s), Total: 
163 test(s)

COBERTURA RESULTS

Cobertura Coverage Report
  

By packages
  

Jenkins-kde-ci: kdelibs KDE-4.14 latest-qt4 » Linux,gcc - Build # 48 - Fixed!

2016-08-25 Thread no-reply

GENERAL INFO

BUILD SUCCESS
Build URL: 
https://build.kde.org/job/kdelibs%20KDE-4.14%20latest-qt4/PLATFORM=Linux,compiler=gcc/48/
Project: PLATFORM=Linux,compiler=gcc
Date of build: Thu, 25 Aug 2016 22:35:16 +
Build duration: 10 min

CHANGE SET
Revision dd1c2da9d26fd4cfc7fe0a25f413e536d56cf2db by Albert Astals Cid: 
(Backport karchive fix for out of directory files)
  change: add kdecore/tests/tar_relative_path_outside_archive.tar.bz2
  change: edit kdecore/tests/karchivetest.cpp
  change: edit kdecore/io/karchive.cpp
  change: edit kdecore/tests/karchivetest.h


JUNIT RESULTS

Name: (root) Failed: 0 test(s), Passed: 163 test(s), Skipped: 0 test(s), Total: 
163 test(s)

COBERTURA RESULTS

Cobertura Coverage Report
  

By packages
  

Jenkins-kde-ci: kdelibs KDE-4.14 latest-qt4 » Linux,gcc - Build # 48 - Fixed!

2016-08-25 Thread no-reply

GENERAL INFO

BUILD SUCCESS
Build URL: 
https://build.kde.org/job/kdelibs%20KDE-4.14%20latest-qt4/PLATFORM=Linux,compiler=gcc/48/
Project: PLATFORM=Linux,compiler=gcc
Date of build: Thu, 25 Aug 2016 22:35:16 +
Build duration: 10 min

CHANGE SET
Revision dd1c2da9d26fd4cfc7fe0a25f413e536d56cf2db by Albert Astals Cid: 
(Backport karchive fix for out of directory files)
  change: add kdecore/tests/tar_relative_path_outside_archive.tar.bz2
  change: edit kdecore/tests/karchivetest.cpp
  change: edit kdecore/io/karchive.cpp
  change: edit kdecore/tests/karchivetest.h


JUNIT RESULTS

Name: (root) Failed: 0 test(s), Passed: 163 test(s), Skipped: 0 test(s), Total: 
163 test(s)

COBERTURA RESULTS

Cobertura Coverage Report
  

By packages
  

Re: Review Request 128749: Backport karchive fix for out of directory files

2016-08-25 Thread Albert Astals Cid

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128749/
---

(Updated Aug. 25, 2016, 10:34 p.m.)


Status
--

This change has been marked as submitted.


Review request for kdelibs, Andreas Cord-Landwehr and David Faure.


Changes
---

Submitted with commit dd1c2da9d26fd4cfc7fe0a25f413e536d56cf2db by Albert Astals 
Cid to branch KDE/4.14.


Repository: kdelibs


Description
---

What the summary says


Diffs
-

  kdecore/io/karchive.cpp eb0bf2e 
  kdecore/tests/karchivetest.h e64e0ed 
  kdecore/tests/karchivetest.cpp 7786b98 

Diff: https://git.reviewboard.kde.org/r/128749/diff/


Testing
---


Thanks,

Albert Astals Cid



Re: Review Request 128749: Backport karchive fix for out of directory files

2016-08-25 Thread Andreas Cord-Landwehr

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128749/#review98647
---


Ship it!




Yeah, thanks applying it!
LGTM, only point is that I cannot see the test-tar archive in the diff, but I 
presume that reviewboard cannot display binary files

- Andreas Cord-Landwehr


On Aug. 24, 2016, 10:36 nachm., Albert Astals Cid wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128749/
> ---
> 
> (Updated Aug. 24, 2016, 10:36 nachm.)
> 
> 
> Review request for kdelibs, Andreas Cord-Landwehr and David Faure.
> 
> 
> Repository: kdelibs
> 
> 
> Description
> ---
> 
> What the summary says
> 
> 
> Diffs
> -
> 
>   kdecore/io/karchive.cpp eb0bf2e 
>   kdecore/tests/karchivetest.h e64e0ed 
>   kdecore/tests/karchivetest.cpp 7786b98 
> 
> Diff: https://git.reviewboard.kde.org/r/128749/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Albert Astals Cid
> 
>



Re: Review Request 128749: Backport karchive fix for out of directory files

2016-08-25 Thread David Faure

---
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128749/#review98624
---


Ship it!




Thanks Albert, you rock.

- David Faure


On Aug. 24, 2016, 10:36 p.m., Albert Astals Cid wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128749/
> ---
> 
> (Updated Aug. 24, 2016, 10:36 p.m.)
> 
> 
> Review request for kdelibs, Andreas Cord-Landwehr and David Faure.
> 
> 
> Repository: kdelibs
> 
> 
> Description
> ---
> 
> What the summary says
> 
> 
> Diffs
> -
> 
>   kdecore/io/karchive.cpp eb0bf2e 
>   kdecore/tests/karchivetest.h e64e0ed 
>   kdecore/tests/karchivetest.cpp 7786b98 
> 
> Diff: https://git.reviewboard.kde.org/r/128749/diff/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Albert Astals Cid
> 
>