Skip to content

Add an Apple privacy info file for OpenSSL#24251

Closed
takeh1k0 wants to merge 1 commit intoopenssl:masterfrom
takeh1k0:apple_privacyinfo
Closed

Add an Apple privacy info file for OpenSSL#24251
takeh1k0 wants to merge 1 commit intoopenssl:masterfrom
takeh1k0:apple_privacyinfo

Conversation

@takeh1k0
Copy link
Copy Markdown
Contributor

for #24032 (comment)

I have created PrivacyInfo.xcprivacy file using Xcode 15.1 GUI.

image

CLA: trivial

@openssl-machine openssl-machine added the hold: cla required The contributor needs to submit a license agreement label Apr 24, 2024
@t8m
Copy link
Copy Markdown
Member

t8m commented Apr 24, 2024

Although this is probably not copyrightable, it is outside of what we usually accept with CLA: trivial. Would you be willing to submit a regular CLA? https://www.openssl.org/policies/cla.html

@t8m t8m added branch: master Applies to master branch approval: review pending This pull request needs review by a committer approval: otc review pending triaged: documentation The issue/pr deals with documentation (errors) branch: 3.0 Applies to openssl-3.0 branch branch: 3.1 Applies to openssl-3.1 (EOL) tests: exempted The PR is exempt from requirements for testing branch: 3.2 Applies to openssl-3.2 (EOL) branch: 3.3 Applies to openssl-3.3 labels Apr 24, 2024
Copy link
Copy Markdown
Member

@mattcaswell mattcaswell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved subject to CLA resolution

@takeh1k0
Copy link
Copy Markdown
Contributor Author

Although this is probably not copyrightable, it is outside of what we usually accept with CLA: trivial. Would you be willing to submit a regular CLA? https://www.openssl.org/policies/cla.html

Thanks for the advice. I will submit a regular CLA.

@hlandau
Copy link
Copy Markdown
Member

hlandau commented Apr 24, 2024

Does this really need to go in the root directory?

@takeh1k0
Copy link
Copy Markdown
Contributor Author

@t8m I have submitted my CLA via email.

@t8m
Copy link
Copy Markdown
Member

t8m commented Apr 24, 2024

Does this really need to go in the root directory?

I think we could add Apple subdir and move it there. We already have ms and VMS subdirs.

@t8m
Copy link
Copy Markdown
Member

t8m commented Apr 24, 2024

Does this really need to go in the root directory?

I think we could add Apple subdir and move it there. We already have ms and VMS subdirs.

Or even better create Apple subdir under os-dep and move it there. At some point we should move ms and VMS under os-dep too.

@takeh1k0
Copy link
Copy Markdown
Contributor Author

My CLA has been received. I was told to close and reopen any open pull requests to remove the "needs CLA" tag, so I will close this pull request and will reopen.

@mattcaswell
Copy link
Copy Markdown
Member

My CLA has been received. I was told to close and reopen any open pull requests to remove the "needs CLA" tag, so I will close this pull request and will reopen.

It was sufficient to close and reopen this PR...but never mind.

@takeh1k0
Copy link
Copy Markdown
Contributor Author

@mattcaswell I misunderstood the meaning of reopen... Thanks for pointing that out!

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

Labels

approval: review pending This pull request needs review by a committer branch: master Applies to master branch branch: 3.0 Applies to openssl-3.0 branch branch: 3.1 Applies to openssl-3.1 (EOL) branch: 3.2 Applies to openssl-3.2 (EOL) branch: 3.3 Applies to openssl-3.3 hold: cla required The contributor needs to submit a license agreement tests: exempted The PR is exempt from requirements for testing triaged: documentation The issue/pr deals with documentation (errors)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants