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

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

Closed
ceolin wants to merge 2 commits intosolettaproject:masterfrom
ceolin:oic
Closed

Oic - Allows more than one server's instace#1391
ceolin wants to merge 2 commits intosolettaproject:masterfrom
ceolin:oic

Conversation

@ceolin
Copy link
Copy Markdown
Contributor

@ceolin ceolin commented Feb 3, 2016

No description provided.

Signed-off-by: Flavio Ceolin <flavio.ceolin@intel.com>
Set REUSEADDR option to coap sockets for well known ports.

Signed-off-by: Flavio Ceolin <flavio.ceolin@intel.com>
@thiagomacieira
Copy link
Copy Markdown
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
Copy Markdown

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
Copy Markdown
Contributor

See #1395

@ceolin
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown

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
Copy Markdown
Contributor

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

@thiagomacieira
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Contributor

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

@ceolin
Copy link
Copy Markdown
Contributor Author

ceolin commented Feb 16, 2016

@thiagomacieira ok, thanks a lot !

@ceolin
Copy link
Copy Markdown
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