Merge lp:~michihenning/storage-framework/remove-non-blocking into lp:storage-framework/devel

Proposed by Michi Henning
Status: Merged
Approved by: James Henstridge
Approved revision: 41
Merged at revision: 45
Proposed branch: lp:~michihenning/storage-framework/remove-non-blocking
Merge into: lp:storage-framework/devel
Diff against target: 25 lines (+2/-2)
2 files modified
src/qt/client/internal/local_client/DownloaderImpl.cpp (+1/-1)
src/qt/client/internal/local_client/UploaderImpl.cpp (+1/-1)
To merge this branch: bzr merge lp:~michihenning/storage-framework/remove-non-blocking
Reviewer Review Type Date Requested Status
unity-api-1-bot continuous-integration Approve
James Henstridge Approve
Review via email: mp+302493@code.launchpad.net

Commit message

Removed O_NONBLOCK from local client socket pair.

Description of the change

Removed O_NONBLOCK from local client socket pair.

To post a comment you must log in.
Revision history for this message
unity-api-1-bot (unity-api-1-bot) wrote :

FAILED: Continuous integration, rev:41
https://jenkins.canonical.com/unity-api-1/job/lp-storage-framework-ci/58/
Executed test runs:
    FAILURE: https://jenkins.canonical.com/unity-api-1/job/build/333/console
    SUCCESS: https://jenkins.canonical.com/unity-api-1/job/build-0-fetch/339
    SUCCESS: https://jenkins.canonical.com/unity-api-1/job/build-1-sourcepkg/release=vivid+overlay/264
    SUCCESS: https://jenkins.canonical.com/unity-api-1/job/build-1-sourcepkg/release=xenial+overlay/264
    SUCCESS: https://jenkins.canonical.com/unity-api-1/job/build-1-sourcepkg/release=yakkety/264
    FAILURE: https://jenkins.canonical.com/unity-api-1/job/build-2-binpkg/arch=amd64,release=vivid+overlay/194/console
    FAILURE: https://jenkins.canonical.com/unity-api-1/job/build-2-binpkg/arch=amd64,release=xenial+overlay/194/console
    FAILURE: https://jenkins.canonical.com/unity-api-1/job/build-2-binpkg/arch=amd64,release=yakkety/194/console
    FAILURE: https://jenkins.canonical.com/unity-api-1/job/build-2-binpkg/arch=armhf,release=vivid+overlay/194/console
    FAILURE: https://jenkins.canonical.com/unity-api-1/job/build-2-binpkg/arch=armhf,release=xenial+overlay/194/console
    FAILURE: https://jenkins.canonical.com/unity-api-1/job/build-2-binpkg/arch=armhf,release=yakkety/194/console
    FAILURE: https://jenkins.canonical.com/unity-api-1/job/build-2-binpkg/arch=i386,release=vivid+overlay/194/console
    FAILURE: https://jenkins.canonical.com/unity-api-1/job/build-2-binpkg/arch=i386,release=xenial+overlay/194/console
    FAILURE: https://jenkins.canonical.com/unity-api-1/job/build-2-binpkg/arch=i386,release=yakkety/194/console

Click here to trigger a rebuild:
https://jenkins.canonical.com/unity-api-1/job/lp-storage-framework-ci/58/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
James Henstridge (jamesh) wrote :

Looks good!

review: Approve
Revision history for this message
unity-api-1-bot (unity-api-1-bot) :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/qt/client/internal/local_client/DownloaderImpl.cpp'
2--- src/qt/client/internal/local_client/DownloaderImpl.cpp 2016-07-22 00:17:24 +0000
3+++ src/qt/client/internal/local_client/DownloaderImpl.cpp 2016-08-09 23:48:44 +0000
4@@ -249,7 +249,7 @@
5 {
6 // Set up socket pair.
7 int fds[2];
8- int rc = socketpair(AF_UNIX, SOCK_STREAM | SOCK_NONBLOCK, 0, fds);
9+ int rc = socketpair(AF_UNIX, SOCK_STREAM, 0, fds);
10 if (rc == -1)
11 {
12 // LCOV_EXCL_START
13
14=== modified file 'src/qt/client/internal/local_client/UploaderImpl.cpp'
15--- src/qt/client/internal/local_client/UploaderImpl.cpp 2016-07-26 01:51:26 +0000
16+++ src/qt/client/internal/local_client/UploaderImpl.cpp 2016-08-09 23:48:44 +0000
17@@ -351,7 +351,7 @@
18 {
19 // Set up socket pair.
20 int fds[2];
21- int rc = socketpair(AF_UNIX, SOCK_STREAM | SOCK_NONBLOCK, 0, fds);
22+ int rc = socketpair(AF_UNIX, SOCK_STREAM, 0, fds);
23 if (rc == -1)
24 {
25 // LCOV_EXCL_START

Subscribers

People subscribed via source and target branches

to all changes: