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.

Sign up for GitHub

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

Jump to bottom

Fix copyright disclaimer. #20517

Closed
wants to merge 1 commit into from
Closed

Fix copyright disclaimer. #20517

wants to merge 1 commit into from

Conversation

paulidale
Copy link
Contributor

The mention of the GPL shouldn't have been there.

@paulidale
Fix copyright disclaimer.
f50c995 
The mention of the GPL shouldn't have been there.
@paulidale paulidale added branch: master Merge to master branch approval: review pending This pull request needs review by a committer branch: 1.1.1 Merge to OpenSSL_1_1_1-stable branch approval: otc review pending This pull request needs review by an OTC member branch: 3.0 Merge to openssl-3.0 branch branch: 3.1 Merge to openssl-3.1 tests: exempted The PR is exempt from requirements for testing labels Mar 15, 2023
@paulidale paulidale self-assigned this Mar 15, 2023
@t-j-h t-j-h removed the approval: otc review pending This pull request needs review by an OTC member label Mar 15, 2023
@t8m t8m added approval: done This pull request has the required number of approvals triaged: documentation The issue/pr deals with documentation (errors) and removed approval: review pending This pull request needs review by a committer labels Mar 15, 2023
@levitte
Copy link
Member

levitte commented Mar 16, 2023

Side note: that note was added in 2013 (e0202d9), before we had our CLAs. Not sure we can say whether it should or shouldn't have been added at that time...

@t-j-h
Copy link
Member

t-j-h commented Mar 16, 2023

Side note: that note was added in 2013 (e0202d9), before we had our CLAs. Not sure we can say whether it should or shouldn't have been added at that time...

It should not have been added then either as the statement is actually both confusing (to some) and incorrect.

@tom-cosgrove-arm
Copy link
Contributor

I've "reached out" to Ard to get him to confirm. Otherwise we can raise with Linaro if you like

@t8m
Copy link
Member

t8m commented Mar 16, 2023

I've "reached out" to Ard to get him to confirm. Otherwise we can raise with Linaro if you like

We've already got confirmation from him.

@t8m t8m added approval: ready to merge The 24 hour grace period has passed, ready to merge and removed approval: done This pull request has the required number of approvals labels Mar 16, 2023
@t8m
Copy link
Member

t8m commented Mar 16, 2023

Merged to all active branches. Thank you.

@t8m t8m closed this Mar 16, 2023
openssl-machine pushed a commit that referenced this pull request Mar 16, 2023
@paulidale @t8m
Fix copyright disclaimer.
c879f8a 
The mention of the GPL shouldn't have been there.

Reviewed-by: Tim Hudson <tjh@openssl.org>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from #20517)
openssl-machine pushed a commit that referenced this pull request Mar 16, 2023
@paulidale @t8m
Fix copyright disclaimer.
332093f 
The mention of the GPL shouldn't have been there.

Reviewed-by: Tim Hudson <tjh@openssl.org>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from #20517)

(cherry picked from commit c879f8a)
openssl-machine pushed a commit that referenced this pull request Mar 16, 2023
@paulidale @t8m
Fix copyright disclaimer.
19d22e7 
The mention of the GPL shouldn't have been there.

Reviewed-by: Tim Hudson <tjh@openssl.org>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from #20517)

(cherry picked from commit c879f8a)
openssl-machine pushed a commit that referenced this pull request Mar 16, 2023
@paulidale @t8m
Fix copyright disclaimer.
131c167 
The mention of the GPL shouldn't have been there.

Reviewed-by: Tim Hudson <tjh@openssl.org>
Reviewed-by: Tomas Mraz <tomas@openssl.org>
(Merged from #20517)

(cherry picked from commit c879f8a)
@paulidale paulidale deleted the fix-copyright branch March 16, 2023 23:32
@gernot-h gernot-h mentioned this pull request Sep 18, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@t8m t8m t8m approved these changes

@t-j-h t-j-h t-j-h approved these changes

Assignees

@paulidale paulidale

Labels
approval: ready to merge The 24 hour grace period has passed, ready to merge branch: master Merge to master branch branch: 1.1.1 Merge to OpenSSL_1_1_1-stable branch branch: 3.0 Merge to openssl-3.0 branch branch: 3.1 Merge to openssl-3.1 tests: exempted The PR is exempt from requirements for testing triaged: documentation The issue/pr deals with documentation (errors)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@paulidale @levitte @t-j-h @tom-cosgrove-arm @t8m