Change in libosmo-netif[master]: Stream examples: fix typos in error message

2019-02-06 Thread Harald Welte
Harald Welte has submitted this change and it was merged. ( 
https://gerrit.osmocom.org/12825 )

Change subject: Stream examples: fix typos in error message
..

Stream examples: fix typos in error message

Likely a copy-paste error from corresponding client example code.

Change-Id: I47cc8c7340eb188d85a49d4f41c295bed0e1bee4
---
M examples/stream-server.c
1 file changed, 2 insertions(+), 2 deletions(-)

Approvals:
  Harald Welte: Looks good to me, approved
  osmith: Looks good to me, but someone else must approve
  Jenkins Builder: Verified



diff --git a/examples/stream-server.c b/examples/stream-server.c
index c1086ec..f3ee315 100644
--- a/examples/stream-server.c
+++ b/examples/stream-server.c
@@ -148,7 +148,7 @@

srv = osmo_stream_srv_link_create(tall_test);
if (srv == NULL) {
-   fprintf(stderr, "cannot create client\n");
+   fprintf(stderr, "cannot create server link\n");
exit(EXIT_FAILURE);
}
osmo_stream_srv_link_set_addr(srv, "127.0.0.1");
@@ -156,7 +156,7 @@
osmo_stream_srv_link_set_accept_cb(srv, accept_cb);

if (osmo_stream_srv_link_open(srv) < 0) {
-   fprintf(stderr, "cannot open client\n");
+   fprintf(stderr, "cannot open server link\n");
exit(EXIT_FAILURE);
}


--
To view, visit https://gerrit.osmocom.org/12825
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: libosmo-netif
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I47cc8c7340eb188d85a49d4f41c295bed0e1bee4
Gerrit-Change-Number: 12825
Gerrit-PatchSet: 3
Gerrit-Owner: Max 
Gerrit-Reviewer: Harald Welte 
Gerrit-Reviewer: Jenkins Builder (102)
Gerrit-Reviewer: Max 
Gerrit-Reviewer: osmith 
Gerrit-CC: daniel 


Change in libosmo-netif[master]: Stream examples: fix typos in error message

2019-02-06 Thread Harald Welte
Harald Welte has posted comments on this change. ( 
https://gerrit.osmocom.org/12825 )

Change subject: Stream examples: fix typos in error message
..


Patch Set 2: Code-Review+2


--
To view, visit https://gerrit.osmocom.org/12825
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: libosmo-netif
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I47cc8c7340eb188d85a49d4f41c295bed0e1bee4
Gerrit-Change-Number: 12825
Gerrit-PatchSet: 2
Gerrit-Owner: Max 
Gerrit-Reviewer: Harald Welte 
Gerrit-Reviewer: Jenkins Builder (102)
Gerrit-Reviewer: Max 
Gerrit-Reviewer: osmith 
Gerrit-CC: daniel 
Gerrit-Comment-Date: Wed, 06 Feb 2019 08:09:26 +
Gerrit-HasComments: No
Gerrit-HasLabels: Yes


Change in libosmo-netif[master]: Stream examples: fix typos in error message

2019-02-05 Thread osmith
osmith has posted comments on this change. ( https://gerrit.osmocom.org/12825 )

Change subject: Stream examples: fix typos in error message
..


Patch Set 2: Code-Review+1


--
To view, visit https://gerrit.osmocom.org/12825
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: libosmo-netif
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I47cc8c7340eb188d85a49d4f41c295bed0e1bee4
Gerrit-Change-Number: 12825
Gerrit-PatchSet: 2
Gerrit-Owner: Max 
Gerrit-Reviewer: Jenkins Builder (102)
Gerrit-Reviewer: Max 
Gerrit-Reviewer: osmith 
Gerrit-CC: Harald Welte 
Gerrit-CC: daniel 
Gerrit-Comment-Date: Tue, 05 Feb 2019 15:29:21 +
Gerrit-HasComments: No
Gerrit-HasLabels: Yes


Change in libosmo-netif[master]: Stream examples: fix typos in error message

2019-02-05 Thread Max
Max has posted comments on this change. ( https://gerrit.osmocom.org/12825 )

Change subject: Stream examples: fix typos in error message
..


Patch Set 2:

(1 comment)

https://gerrit.osmocom.org/#/c/12825/1/examples/stream-server.c
File examples/stream-server.c:

https://gerrit.osmocom.org/#/c/12825/1/examples/stream-server.c@145
PS1, Line 145:  fprintf(stderr, "cannot create server link\n");
> here as well
Done



--
To view, visit https://gerrit.osmocom.org/12825
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: libosmo-netif
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I47cc8c7340eb188d85a49d4f41c295bed0e1bee4
Gerrit-Change-Number: 12825
Gerrit-PatchSet: 2
Gerrit-Owner: Max 
Gerrit-Reviewer: Jenkins Builder (102)
Gerrit-Reviewer: Max 
Gerrit-CC: Harald Welte 
Gerrit-CC: daniel 
Gerrit-CC: osmith 
Gerrit-Comment-Date: Tue, 05 Feb 2019 15:25:00 +
Gerrit-HasComments: Yes
Gerrit-HasLabels: No


Change in libosmo-netif[master]: Stream examples: fix typos in error message

2019-02-05 Thread Max
Hello Jenkins Builder,

I'd like you to reexamine a change. Please visit

https://gerrit.osmocom.org/12825

to look at the new patch set (#2).

Change subject: Stream examples: fix typos in error message
..

Stream examples: fix typos in error message

Likely a copy-paste error from corresponding client example code.

Change-Id: I47cc8c7340eb188d85a49d4f41c295bed0e1bee4
---
M examples/stream-server.c
1 file changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/libosmo-netif refs/changes/25/12825/2
--
To view, visit https://gerrit.osmocom.org/12825
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: libosmo-netif
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I47cc8c7340eb188d85a49d4f41c295bed0e1bee4
Gerrit-Change-Number: 12825
Gerrit-PatchSet: 2
Gerrit-Owner: Max 
Gerrit-Reviewer: Jenkins Builder (102)
Gerrit-CC: daniel