Tony Mechelynck [:tonymec] Reporter
	Description • 12 years ago

At the current comm-central changeset 17c2cee4815c, the file suite/common/app-license.html includes an MPL2 comment at its top, but the body of the file links to http://www.mozilla.org/legal/eula/ which (in turn) consists only of a set of links to the text of the MPL 1.1.

I found this app-license.html serendipitously by searching MXR, I don't know if it is "user-visible". In particular it is not used by the SeaMonkey about:license, which AFAICT is identical with the one used by Firefox (a Toolkit file, maybe?)

The whole file is very short, as follows:

<!-- This Source Code Form is subject to the terms of the Mozilla Public
   - License, v. 2.0. If a copy of the MPL was not distributed with this
   - file, You can obtain one at http://mozilla.org/MPL/2.0/. -->

    <p>Official <b>binaries</b> of this product released by the 
    <a href="http://www.mozilla.org/">Mozilla Foundation</a>
    are made available under 
    <a href="http://www.mozilla.org/legal/eula/">the corresponding 
    EULA</a>.</p>

	Tony Mechelynck [:tonymec] Reporter
	Comment 1 • 12 years ago

Oops: the initial paragraph in the Firefox about:license seems to correspond to this file, but it is missing (not replaced by this text) in the SeaMonkey about:license.

	Tony Mechelynck [:tonymec] Reporter
	Comment 2 • 12 years ago

A little digging shows that the top comment was added by bug 757018 at comm-central changeset 84ac3c711098 ; otherwise the file hadn't changed since the initial import from CVS into comm-central. My guess (just a guess) is that this file has become dead code since the switchover of SeaMonkey to MPL2.

	Tony Mechelynck [:tonymec] Reporter
	Comment 3 • 12 years ago

A fact being better than a guess, I did the following search: http://mxr.mozilla.org/comm-central/search?string=app-license.html

…but I'm not sure how to interpret the results.

	Tony Mechelynck [:tonymec] Reporter
	Comment 4 • 12 years ago

I find the following block suggestive, at the end of suite/common/Makefile.in :

# DEFINES for preprocessing
# Use suite/common/app-license.html as input when generating 
# chrome://content/communicator/license.html to override 
# chrome://global/content/license.html (about:license)
DEFINES += -DAPP_EULA_BLOCK=$(abs_srcdir)/app-license.html \
           -DMOZ_APP_VERSION=$(MOZ_APP_VERSION) \
           -DMOZ_MACBUNDLE_NAME=$(MOZ_MACBUNDLE_NAME) \
           $(NULL)



The string APP_EULA_BLOCK is found nowhere else in the whole source.

	Gervase Markham [:gerv]
	Comment 5 • 12 years ago

The MPL header at the top of the file is a red herring. The actual problem here is that SeaMonkey is not under a "Mozilla Foundation EULA"; binaries of Mozilla software released by Mozilla are themselves released under the MPL (now MPL 2), as of a couple of years ago.

So the contents of this file need updating to say that. The equivalent fragment in Firefox, as seen in about:license, says:

"Binaries of this product have been made available to you by the Mozilla Project under the Mozilla Public License 2.0 (MPL). Know your rights."

(with various links). SeaMonkey should say that also.

Gerv

	Tony Mechelynck [:tonymec] Reporter
	Comment 6 • 12 years ago

In reply to comment #5: Maybe replace APP_EULA_BLOCK by APP_LICENSE_BLOCK in the SeaMonkey Makefile as it has been done in Firefox & Thunderbird, and at the same time replace this file by a copy of the Firefox app-license.txt as it is today?

Gerv, Neil, Callek, what do you think?

	Tony Mechelynck [:tonymec] Reporter
	Comment 7 • 12 years ago

P.S. …perhaps a "suitably adjusted" copy of the Firefox app-license.txt, including e.g. a link to

<a href="http://www.seamonkey-project.org/">the SeaMonkey Project</a>

?

	Tony Mechelynck [:tonymec] Reporter
	Comment 8 • 12 years ago

P.P.S. …and (before my 1st cup of tea after my afternoon siesta) I meant the firefox app-license.html of course.

	Tony Mechelynck [:tonymec] Reporter
	Comment 9 • 12 years ago

See also bug 762264.

	Philip Chee Assignee
	Comment 10 • 10 years ago

Ports the following bugs:

Bug 464994 - about:license and about:rights potentially contradict each other
Bug 730814 - update about:license to MPL 2. Also add a link to the FAQ,
correct a few inaccuracies about what license applies to what, and make the
document validate as HTML5

	neil@parkwaycc.co.uk
	Comment 11 • 10 years ago

Comment on attachment 8380358 [details] [diff] [review]
Patch v1.0 Proposed fix.

r=me based on code inspection only.

	Philip Chee Assignee
	Comment 12 • 10 years ago

Comment on attachment 8380358 [details] [diff] [review]
Patch v1.0 Proposed fix.

Cancelling review requests. We are missing a bit of code that references this file (like Firefox and Thunderbird do).

	Philip Chee Assignee
	Comment 13 • 10 years ago

> -CHROME_DEPS += $(abs_srcdir)/app-license.html
Bug 934361 - Remove CHROME_DEPS

> -DEFINES += -DAPP_EULA_BLOCK=$(abs_srcdir)/app-license.html \
> +DEFINES += -DAPP_LICENSE_BLOCK=$(abs_srcdir)/app-license.html \
Bug 464994 fixed this for Firefox and Thunderbird. I guess they forgot SeaMonkey - again.

> +    <p><b>Binaries</b> of this product have been made available to you by the
> +    <a href="http://www.seamonkey-project.org/">The SeaMonkey Project</a>
> +    under the Mozilla Public License 2.0 (MPL).
> +    <a href="about:rights">Know your rights</a>.</p>
I changed:
<a href="http://www.mozilla.org/">Mozilla Project</a>
to:
<a href="http://www.seamonkey-project.org/">The SeaMonkey Project</a>

> Bug 730814 ... and make the document validate as HTML5
Despite the commit message, license.html never was and certainly is not HTML5 compliant.

	Philip Chee Assignee
	Comment 14 • 10 years ago

> Bug 464994 fixed this for Firefox and Thunderbird. I guess they forgot SeaMonkey
> - again.
To test this patch go to about:license
Check that the app-license.html snippet is included properly in about:license

	Philip Chee Assignee
	Comment 15 • 10 years ago

Callek suggested that I do a dep build after touching app-license.html so I changed "Mozilla Public License 2.0" to "SeaMonkey Public License 2.0" and rebuilt. The change worked.

	Justin Wood (:Callek)
	Comment 16 • 10 years ago

Comment on attachment 8380649 [details] [diff] [review]
Patch v1.1 better fix

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

logistical r+

r? to gerv, karsten or mcsmurf for legal implications

	Frank Wein [:mcsmurf]
	Comment 17 • 10 years ago

Comment on attachment 8380649 [details] [diff] [review]
Patch v1.1 better fix

about:license says with this patch:
"made available to you by the The SeaMonkey Project "

double "the"? I think one is enough. Just linkify the "SeaMonkey Project" and leave out the "The".

	Philip Chee Assignee
	Comment 18 • 10 years ago

> double "the"? I think one is enough. Just linkify the "SeaMonkey Project" and 
> leave out the "The".
Fixed.

	Philip Chee Assignee
	Comment 19 • 10 years ago

Comment on attachment 8392984 [details] [diff] [review]
Patch v1.2 with no the The.

(In reply to Justin Wood (:Callek) from comment #16)
> r? to gerv, karsten or mcsmurf for legal implications
Re-adding Karsten and Gerv. Not sure what Justin means about legal implications. app-license.html had been broken for ages. This patch just un-breaks it and brings us up to par with Thunderbird and Firefox.

	Gervase Markham [:gerv]
	Comment 20 • 10 years ago

Comment on attachment 8392984 [details] [diff] [review]
Patch v1.2 with no the The.

Assuming this has the obvious effect, LGTM. r=gerv.

Gerv

	Philip Chee Assignee
	Comment 21 • 10 years ago

Comment on attachment 8392984 [details] [diff] [review]
Patch v1.2 with no the The.

Poke mcsmurf and Mnyromyr

	Frank Wein [:mcsmurf]
	Comment 22 • 10 years ago

Comment on attachment 8392984 [details] [diff] [review]
Patch v1.2 with no the The.

Looks and works fine

	Philip Chee Assignee
	Comment 23 • 10 years ago

Pushed to comm-central:
http://hg.mozilla.org/comm-central/rev/d853681a4c05