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

chore(license): updates to LICENSE, README.md, CONTRIBUTING.md #3875

Merged
merged 3 commits into from
Aug 21, 2023

Conversation

srl295
Copy link
Member

@srl295 srl295 commented Aug 15, 2023

- Drop SPDX for now - change license to license-file in Cargo.toml
https://doc.rust-lang.org/cargo/reference/manifest.html#the-license-and-license-file-fields
(the SPDX identifier will come back soon!)
- Central tracking: unicode-org/.github#6
- followon to #2303
sffc
sffc previously approved these changes Aug 16, 2023
README.md Show resolved Hide resolved
@gregtatum gregtatum removed their request for review August 16, 2023 13:41
Manishearth
Manishearth previously approved these changes Aug 17, 2023
Copy link
Member

@Manishearth Manishearth left a comment

Choose a reason for hiding this comment

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

By when will we have an SPDX? We're planning on having a release in a month or so

@srl295
Copy link
Member Author

srl295 commented Aug 17, 2023

By when will we have an SPDX? We're planning on having a release in a month or so

I would say do not wait for it. Even if we had it in 2 weeks, i'm probably not going to do another run through everything in the next month.

@annebright ?

@srl295 srl295 dismissed stale reviews from Manishearth and sffc via 00e2c97 August 17, 2023 20:06
Co-authored-by: Shane F. Carr <[email protected]>
@srl295 srl295 requested review from Manishearth and sffc August 17, 2023 20:07
@Manishearth
Copy link
Member

I would say do not wait for it. Even if we had it in 2 weeks, i'm probably not going to do another run through everything in the next month.

Ah, sure; we can do that ourselves if needed.

Partly my question is from the POV of internal compliance: the next time we update ICU4X within Google we'll need to get this license approved and it's easier if there's an SPDX. But if that's not going to be ready in time, would I be okay in getting preliminary internal approval for https://www.unicode.org/license.txt and say the SPDX is coming? Is there a preliminary SPDX I can get them to use internally?

@annebright
Copy link

I don't know how long it takes for SPDX to assign an identifier to a new license. Our outside counsel on this is on vacation this week, but will be submitting the new license to SPDX (and OSI) next week. I have to think it will take at least a couple weeks after that. Once outside counsel is back from vacation, I'll ask him again, but I don't have an answer right now.

@srl295
Copy link
Member Author

srl295 commented Aug 17, 2023

I would say do not wait for it. Even if we had it in 2 weeks, i'm probably not going to do another run through everything in the next month.

Ah, sure; we can do that ourselves if needed.

Partly my question is from the POV of internal compliance: the next time we update ICU4X within Google we'll need to get this license approved and it's easier if there's an SPDX. But if that's not going to be ready in time, would I be okay in getting preliminary internal approval for https://www.unicode.org/license.txt and say the SPDX is coming? Is there a preliminary SPDX I can get them to use internally?

you could have them compare it to Unicode-DFS-2016 perhaps?

@annebright
Copy link

annebright commented Aug 17, 2023 via email

@Manishearth
Copy link
Member

Manishearth commented Aug 17, 2023

Who does the approving within Google - is it Dashiell?

I think in part, I plan to email the open source licensing group in a while. Most of the tooling around this uses SPDX but I suppose we could figure out something temporarily to use internally.

you could have them compare it to Unicode-DFS-2016 perhaps?

The tricky thing is not the review, the tricky thing is once the review has completed the documentation and tooling needs to be updated to say that something is approved and it's nice when that thing has a name. But like I said we can make something up.

@sffc
Copy link
Member

sffc commented Aug 17, 2023

Basically landing a LICENSE file without an SPDX triggers a compliance workflow (Dashiell might be CC'd but it's mainly driven by people on the tech side), and since we land ICU4X as a bunch of individual crates we'll probably need to spin this up for each one of them. It's all doable, just more paperwork and red tape than if we landed this with an SPDX.

@Manishearth
Copy link
Member

Basically landing a LICENSE file without an SPDX triggers a compliance workflow

I don't think this is true fwiw, as long as the internal tooling is aware of the license.

@Manishearth
Copy link
Member

Manishearth commented Aug 17, 2023

However, it is true that updating the license (regardless of SPDX) is going to make the import process much more painful. But that's inevitable; I think there are workarounds I can come up with.

@annebright
Copy link

annebright commented Aug 17, 2023 via email

@Manishearth
Copy link
Member

No, it's fine, we can merge this, just wanted an idea.

@srl295
Copy link
Member Author

srl295 commented Aug 18, 2023

minor update, license title name change

Copy link

@annebright annebright left a comment

Choose a reason for hiding this comment

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

Looks like all of the License updates are using the old license name - should be changed to UNICODE LICENSE V#. And the notice at the bottom of the ReadMe needs its title "Copyright & Licenses"

@srl295
Copy link
Member Author

srl295 commented Aug 18, 2023

Looks like all of the License updates are using the old license name - should be changed to UNICODE LICENSE V#. And the notice at the bottom of the ReadMe needs its title "Copyright & Licenses"

That's not what I'm seeing… You might be seeing the diffs (comparisons), but if you click the "…" menu and choose "View File" you will see what the updated file looks like.

Otherwise lines starting with a - are removed, lines starting with a + are added.

image

@annebright
Copy link

I understand how to View File and I did. I was just in the wrong "chore". So, approved now.

@srl295 srl295 merged commit b9419ce into main Aug 21, 2023
@srl295 srl295 deleted the srl295/licensev3 branch August 21, 2023 16:47
srl295 added a commit that referenced this pull request Aug 21, 2023
- feature added in #3875, other license files in #2722
- when #2722 was merged, #3875 broke
sffc pushed a commit that referenced this pull request Aug 21, 2023
- feature added in #3875, other license files in #2722
- when #2722 was merged, #3875 broke
@sffc sffc removed their request for review August 29, 2023 17:08
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

Successfully merging this pull request may close these issues.

4 participants