Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ease of porting from Sming to SmingRTOS #84

Open
patrickjahns opened this issue Mar 9, 2016 · 2 comments
Open

Ease of porting from Sming to SmingRTOS #84

patrickjahns opened this issue Mar 9, 2016 · 2 comments

Comments

@patrickjahns
Copy link
Member

I just ported the small DNSServer addition and find it quite cumbersome do provide functionality for both SMING and SmingRTOS.
Some things to drip over is for example the naming convention i.e. lower case ../wiring/Wstring.h (RTOS) and upper case ../Wiring/Wstring.h (NONOS). Can we streamline the naming conventions for both projects?

Best case scenario would be, if the structure of both projects is the same - then it would be most easy to provide patches for both (for example security/problem related patches).

I know SMING will be deprecated and SmingRTOS is supposed to continue from there on. The structure of SmingRTOS has changed quite some and actually it would be great if SmingRTOS could still be a "drop-in-replacement" with as few changes as needed.

Any thoughts on this?

@hreintke
Copy link
Collaborator

hreintke commented Mar 9, 2016

@patrickjahns :
The change from capital Wiring to lowercase wiring was a "wish" from some to get sming development less prone to errors in capital sensitive systems.
I did change the directory structure because the "Sming_NONOS" was (in my opinion) unclear and not consistent.

Decision to take this in the SmingRTOS conversion was that this conversion did not have backward compatibility anyhow.

I did not see a possibilty to keep drop-in replacement and restructuring at the same time.

@patrickjahns
Copy link
Member Author

@hreintke
I understand your design decision.

Would it be possible to create a short Wikipage outlining the changed structure so people developing features for Sming RTOS and NONOS have it easier to keep thing compatible ?

I don`t know if it seems reasonable, but backporting the new structure to nonos would also be great - since the issue with case sensitive systems might still appear in NONOS

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants