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

Modify Sun copyright to follow Oracle style #3685

Closed
wants to merge 1 commit into from
Closed

Modify Sun copyright to follow Oracle style #3685

wants to merge 1 commit into from

Conversation

richsalz
Copy link
Contributor

Master only.

@richsalz richsalz added the branch: master Merge to master branch label Jun 15, 2017
@richsalz richsalz self-assigned this Jun 15, 2017
@richsalz richsalz added the hold: need omc decision The OMC needs to make a decision label Jun 15, 2017
@richsalz
Copy link
Contributor Author

This is on hold until Oracle confirms it to me.

@kroeckx
Copy link
Member

kroeckx commented Jun 15, 2017

I thought I added a comment about this earlier, but it seems to be lost. I think it should probably keep saying Sun Microsystems, even if Oracle now owns it.

@richsalz
Copy link
Contributor Author

richsalz commented Jun 15, 2017 via email

@richsalz richsalz added approval: done This pull request has the required number of approvals and removed hold: need omc decision The OMC needs to make a decision labels Jun 20, 2017
@richsalz
Copy link
Contributor Author

Oracle has approved this, and that approval is recorded in our license folder.
Oracle purchased Sun and all its assets, including all intellectual property and copyrights.
I'm merging this now.

@richsalz
Copy link
Contributor Author

Argh, typo in the addrev call, but this is merged.

@richsalz richsalz closed this Jun 20, 2017
@richsalz richsalz deleted the mv-sun-copyright branch June 20, 2017 15:20
@richsalz richsalz mentioned this pull request Jun 20, 2017
1 task
davidben added a commit to google/boringssl that referenced this pull request Jan 4, 2025
This imports the following changes from OpenSSL:

* openssl/openssl@aa8f3d7
* openssl/openssl@5aba2b6
* openssl/openssl@c80149d

See the following links for some related discussion in OpenSSL's
repository:

* openssl/openssl#3663
* openssl/openssl#3684
* openssl/openssl#3585 (comment)
* openssl/openssl#3685

The copyright_summary script may be used to compare this CL. Note there
is one change to ecdsa_test.cc to align with OpenSSL. See
go/openssl-copyright-consolidation-comparison (internal-only).

Bug: 364634028
Change-Id: I987c4e145d2ccd0c32bbf9e7bb2cc69e89019d35
Reviewed-on: https://boringssl-review.googlesource.com/c/boringssl/+/74712
Reviewed-by: Adam Langley <[email protected]>
Commit-Queue: David Benjamin <[email protected]>
pi-314159 pushed a commit to open-quantum-safe/boringssl that referenced this pull request Jan 20, 2025
This imports the following changes from OpenSSL:

* openssl/openssl@aa8f3d7
* openssl/openssl@5aba2b6
* openssl/openssl@c80149d

See the following links for some related discussion in OpenSSL's
repository:

* openssl/openssl#3663
* openssl/openssl#3684
* openssl/openssl#3585 (comment)
* openssl/openssl#3685

The copyright_summary script may be used to compare this CL. Note there
is one change to ecdsa_test.cc to align with OpenSSL. See
go/openssl-copyright-consolidation-comparison (internal-only).

Bug: 364634028
Change-Id: I987c4e145d2ccd0c32bbf9e7bb2cc69e89019d35
Reviewed-on: https://boringssl-review.googlesource.com/c/boringssl/+/74712
Reviewed-by: Adam Langley <[email protected]>
Commit-Queue: David Benjamin <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approval: done This pull request has the required number of approvals branch: master Merge to master branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants