Skip to content
This repository has been archived by the owner on Jun 27, 2019. It is now read-only.

Oic - Allows more than one server's instace #1391

Closed
wants to merge 2 commits into from

Conversation

ceolin
Copy link
Contributor

@ceolin ceolin commented Feb 3, 2016

No description provided.

Set REUSEADDR option to coap sockets for well known ports.

Signed-off-by: Flavio Ceolin <[email protected]>
@thiagomacieira
Copy link
Contributor

I'd like to ask you to wait. Your first commit will conflict with my splitting of the Linux socket code into Unix-generic and Linux-specific. Your change is easier to re-do after the conflict, mine is a little more difficult.

@barbieri
Copy link

barbieri commented Feb 3, 2016

okay @thiagomacieira but do you have an early version so we can check how you're doing it?

I think it could be done by moving the common functions to a -posix.c file and use those symbols when setting the "impl".

@thiagomacieira
Copy link
Contributor

See #1395

@ceolin
Copy link
Contributor Author

ceolin commented Feb 16, 2016

@thiagomacieira i intend to merge this commit today later because there is an old issue opened that is fixed with it

@edersondisouza
Copy link
Contributor

Do I need something special to test this patch? On my machine, if I start two light-server and light-scan app, it doesn't matter to which server I try to 'pair', client will only communicate with the one that started after. So, at least for me, this series doesn't work =/

@otaviobp
Copy link

In order to have 2 servers running in same machine, you need to set SOL_MACHINE_ID variable in one of them, so we will have 2 servers with different device ids.
But, anyway, you are right. I am now facing the same problem, only the last server is receiving packets. @ceolin, can you check that?

@edersondisouza
Copy link
Contributor

@otaviobp Yes, one of them has a different SOL_MACHINE_ID.

@thiagomacieira
Copy link
Contributor

I've been busy and haven't had the time to act on people's feedback on #1395 yet. I definitely won't have time for it today before EOB in your timezone, but could try to do it if you give me 24 hours.

@ceolin
Copy link
Contributor Author

ceolin commented Feb 16, 2016

@thiagomacieira sure, i can wait. I was thinking you would not have time to review you pr.

@thiagomacieira
Copy link
Contributor

I don't. I will try to do something today, during meetings.

@ceolin
Copy link
Contributor Author

ceolin commented Feb 16, 2016

@thiagomacieira ok, thanks a lot !

@ceolin
Copy link
Contributor Author

ceolin commented Feb 23, 2016

closed, new version #1467

@ceolin ceolin closed this Feb 23, 2016
@ceolin ceolin deleted the oic branch February 25, 2016 19:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants