1190440 - Add React license to about:license for latest versions of React

Status: RESOLVED FIXED

(Hello (Loop) :: Client, defect, P2)

Description

[Mark Banner (:standard8)]

Per bug 1185704, we need to add the license to about:license to cover the latest versions of React that we're using.

Moving here so that we can use tracking flags, as I want to uplift to Aurora so that we can get it out as soon as is reasonable.

Comment 1

[Mark Banner (:standard8)]

Attachment: Add React License for latest versions of React.

As per bug 1185704, adding the react license to about:license.

Copied & pasted from the LICENSE file here: https://github.com/facebook/react

Comment 2

[Gervase Markham [:gerv]]

Comment on attachment 8642462 [details] [diff] [review]
Add React License for latest versions of React.

Review of attachment 8642462 [details] [diff] [review]:
-----------------------------------------------------------------

r=gerv.

Gerv

Comment 3

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/1aac59a31bc3

Comment 4

[Mark Banner (:standard8)]

Comment on attachment 8642462 [details] [diff] [review]
Add React License for latest versions of React.

Approval Request Comment
[Feature/regressing bug #]: bug 1096399
[User impact if declined]: None - licensing fix
[Describe test coverage new/current, TreeHerder]: N/A
[Risks and why]: Low. Updates about:license to include a license for a change that was made in Fx37 when we upgraded the library but missed the license change. Suggesting uplift so that we correct it as soon as reasonably possible.
[String/UUID change made/needed]: None

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/1aac59a31bc3

Comment 6

[Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)]

Mark, does any changes to licensing info have a l10n impact? Or are these always English only? Thanks.

Comment 7

[Mark Banner (:standard8)]

(In reply to Ritu Kothari (:ritu) from comment #6)
> Mark, does any changes to licensing info have a l10n impact? Or are these
> always English only? Thanks.

Its English-only, there's no localisation going on here.

Comment 8

[Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)]

Comment on attachment 8642462 [details] [diff] [review]
Add React License for latest versions of React.

[Triage Comment]
Licensing related low-risk change, should be safe to uplift to Beta41

Comment 9

[Mark Banner (:standard8)]

https://hg.mozilla.org/releases/mozilla-beta/rev/2a2048c6c5ea