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

Include in mongoose-os-libs #1

Open
pimvanpelt opened this issue Apr 30, 2020 · 6 comments
Open

Include in mongoose-os-libs #1

pimvanpelt opened this issue Apr 30, 2020 · 6 comments

Comments

@pimvanpelt
Copy link

Hoi,

I stumbled across your library here as I was starting a project with Modbus/RS485 transceiver. I was wondering if you'd like to host this library in the Mongoose OS github on github.com/mongoose-os-libs/modbus/ ?

If yes, I can help you import it, as maintainer of the -libs and -apps org. If no, no worries, then feel free to close this issue so I'm aware.
There'd be a little bit of work to import it (some of the copyrights point to Cesanta at the moment, and I think that should be you, assuming you're the original author of the code here), some .c / .h files are not tagged with an owner or might be third party (like the CRC16 code), etc. Nothing major, a few PRs would be sufficient to get this tidied up and published. Just let me know if you'd like to move it over.

Do you have a demonstration application as well (could go alongside github.com/mongoose-os-apps/modbus-demo/ for example). Let me know and I'd be happy to help you out here.

@suyashmathema
Copy link
Owner

suyashmathema commented May 6, 2020 via email

@pimvanpelt
Copy link
Author

Hi Suyash - I was working on my modbus project and was reminded of this offer here.
I have created https://github.com/mongoose-os-libs/modbus for your and invited you as maintainer of the repo. You're more than welcome to upload your current repo into that central spot, and it'll be tagged and published alongside all the other mongoose libraries.

Due to the rename (just modbus rather than mongoose-os-modbus, the tweak to mos.yml that renames it will no longer be necessary.

@pimvanpelt
Copy link
Author

By the way, the copyright points: I see you've added an APACHE 2.0 license, great! When tagging the files you wrote, you copied over the license blurbs from Cesanta, but I'm pretty sure they are not the copyright holders or authors :)

You could consider replacing * Copyright (c) 2014-2018 Cesanta Software Limited with your own name, or that of the people who inspired your project (like the Arduino project you linked in README.md)

@suyashmathema
Copy link
Owner

Hey
I will make the necessary changes and upload the code to the repository you mentioned.

@suyashmathema
Copy link
Owner

I updated the copyright names and updated the central mongoose os repository. Do we need to add tags or does it auto tag from the new version?

@pimvanpelt
Copy link
Author

pimvanpelt commented Jan 23, 2021 via email

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

No branches or pull requests

2 participants