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

Update Adobe CMaps URL and license #6155

Merged
merged 1 commit into from
Jul 1, 2015
Merged

Conversation

TimothyGu
Copy link
Contributor

They are now using Apache 2.0 as well.

I also checked the 2015 version of all the resources. They are unchanged to the version currently checked in, besides the license change.

They are now using Apache 2.0 as well.
@Snuffleupagus
Copy link
Collaborator

Can we update this without also updating about:license#adobecmap in Firefox (https://dxr.mozilla.org/mozilla-central/source/toolkit/content/license.html)?
If the CMaps are now licensed under Apache 2.0, do we even need an about:license entry any more?

@TimothyGu
Copy link
Contributor Author

I don't think we need it anymore since we already have an Apache 2.0 entry.

@TimothyGu
Copy link
Contributor Author

Forgot to add: should I make a patch to mozilla-central then? Or will someone else do it?

@Snuffleupagus
Copy link
Collaborator

Before moving forwards with this patch, and any subsequent ones, I think that we should wait for the developers to sign off on the changes.

@brendandahl
Copy link
Contributor

Looks good. I agree we should patch mc on the next uplift so I filed https://bugzilla.mozilla.org/show_bug.cgi?id=1179358

brendandahl added a commit that referenced this pull request Jul 1, 2015
Update Adobe CMaps URL and license
@brendandahl brendandahl merged commit 9bb2908 into mozilla:master Jul 1, 2015
@TimothyGu TimothyGu deleted the cmap branch January 23, 2016 01:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants