-
Notifications
You must be signed in to change notification settings - Fork 172
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
How to handle gRPC repo w.r.t. attribution/licensing #1041
Comments
Hi @hibachrach! Thank you for these questions! I'm really not sure what the answers are and will bring them up in our next SIG meeting. If you'd like to join the meeting to be part of the discussion, here's a link to our Google Group, and the calendar invite should be sent to you after you join. We meet Tuesdays at 9:00AM PT. |
Hi @hibachrach, First off, thank you for building this gem! gRPC instrumentation has been wanted for a long time (see #10), and I'm excited to see it come together. We talked about your questions during the SIG yesterday and would like to welcome the gem into the Is there any level of engagement you'd be willing to commit to? Are there specific requirements you feel uncomfortable about fulfilling? (We're fine either way, and looking for feedback since these guidelines are relatively new) Despite our preference to bring If you'd prefer to keep the gem separate, we understand that too. To answer your questions, the safest legal path would be to retain the Apache 2.0 license, keep the license headers, and add your additional copyrights to the headers on the files you modified. There is also an Apache 2 NOTICE file you could use as well. Here's an example from apache httpd, and here's a file with retained copyright. Google Open Source has some additional recommendations on this topic: https://opensource.google/documentation/reference/copyright#files_derived_from_third_party_code |
Happy to help in any small way I can 😄
Cool! Expect a PR coming soon 😄 I'll just update all license stuff to match what y'all have.
I'm happy to try to help to the extent that I'm able with bugs but I only incidentally work with this code as it pertains to my company's system. I also know I get burnt out doing dev work in my free time when it's my primary job at work so I don't want to say one thing and do another. I can therefore commit primarily to trying, lol. |
Great! Thank you so much!
That makes a lot of sense. Committing to trying works for me! 😄 |
👋 This issue has been marked as stale because it has been open with no activity. You can: comment on the issue or remove the stale label to hold stale off for a while, add the |
Hi @hibachrach! Checking back in on this! Anything we can do to help? |
Hey sorry, have just had a busy personal life recently. Going to take a look at this this weekend. |
Hello,
I recently created
opentelemetry-instrumentation-grpc
(Ruby gem forthcoming) which is mostly just a fork of much of theopentelemetry-instrumentation-gruf
code.I had a few questions:
I would have opened a PR to just add the code to this repo but I wasn't sure if I could commit to maintaining it (which the contributing guidelines request).
Thank you!
The text was updated successfully, but these errors were encountered: