1255588 - Disable Firefox on MacOS <10.9

Status: RESOLVED FIXED

(Core :: Widget: Cocoa, defect)

Description

[Benjamin Smedberg]

Announced on firefox-dev, we're planning on dropping support for MacOS versions <10.9. We need to make that explicit in the product so that users get an appropriate error message.

Comment 1

[Stefan [:stefanh]]

We could just use LSMinimumSystemVersion in browser/app/macbuild/Contents/Info.plist.in. Will we support running a 32-bit firefox on 10.9?

Comment 2

[Benjamin Smedberg]

Yes, until Firefox 52.

Comment 3

[Grant Galitz]

(In reply to Benjamin Smedberg  [:bsmedberg] from comment #2)
> Yes, until Firefox 52.

Which is weird, because Mac OS X 10.8 requires a 64 bit CPU. IIRC it's why Apple upped the system requirements then, they explicitly left the 32 bit yonah and lower chips behind in 10.7 and earlier.

Comment 4

[Stefan [:stefanh]]

I pushed a patch to try for a while ago and the 10.7 builders failed: https://treeherder.mozilla.org/#/jobs?repo=try&revision=d8c12ff4e462&selectedJob=20427468

I guess they launch the application. The 10.7 builders needs to be turned off for 49 or some tests needs to be disabled.

Comment 5

[Stefan [:stefanh]]

Attachment: Make 10.9.0 the minimum version

So, this will make Firefox not start if the OS version is < 10.9.0 (tested on 10.7.5)

The logs from the try run also shows that this works:
https://treeherder.mozilla.org/logviewer.html#?job_id=20427356&repo=try#L48239
https://treeherder.mozilla.org/logviewer.html#?job_id=20427356&repo=try#L48268

Comment 6

[Stephen A Pohl [:spohl]]

(In reply to Stefan [:stefanh] from comment #5)
> Created attachment 8751452 [details] [diff] [review]
> Make 10.9.0 the minimum version
> 
> So, this will make Firefox not start if the OS version is < 10.9.0 (tested
> on 10.7.5)

Can you post a screenshot of what you see when you try to start Firefox on OSX < 10.9.0?

Comment 7

[Stefan [:stefanh]]

Attachment: Make 10.9.0 the minimum version v1.0

Whoops, wrong patch.

Comment 8

[Stefan [:stefanh]]

(In reply to Stephen A Pohl [:spohl] from comment #6)
> Can you post a screenshot of what you see when you try to start Firefox on
> OSX < 10.9.0?

Sure.

Comment 9

[Stefan [:stefanh]]

Attachment: 10.7.5-launch.png

Right, I have Swedish version...

Translation:
"This version of the program Nightly can not be used with this version of Mac OS X

You have Mac OS X 10.7.5. The program requires Mac OS X 10.9 or later."

Comment 10

[Stephen A Pohl [:spohl]]

Comment on attachment 8751453 [details] [diff] [review]
Make 10.9.0 the minimum version v1.0

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

As discussed via IRC, please don't land this until tests are disabled on OSX < 10.9.0 or you get the green light to land it despite the tests failing. Thanks!

Comment 11

[Stefan [:stefanh]]

(In reply to Stefan [:stefanh] from comment #5)
> The logs from the try run also shows that this works:
> https://treeherder.mozilla.org/logviewer.
> html#?job_id=20427356&repo=try#L48239
> https://treeherder.mozilla.org/logviewer.
> html#?job_id=20427356&repo=try#L48268

Hmm, these links doen't seem to work. But this is the message:
13:31:14     INFO - PROCESS | 5887 | application requires at least Mac OS X version 10.9.0, but is being run on 10.7.2, and so is exiting

Comment 12

[Benjamin Smedberg]

I just pushed an expanded version of Stefan's patch which disables selftest.py, to see if that will succeed. Ted says we may also run xpcshell as part of package.py (I can't test directly because I no longer have a mac).

https://treeherder.mozilla.org/#/jobs?repo=try&revision=37e0a11521ea

Comment 13

[Benjamin Smedberg]

Attachment: try: -b do -p macosx64,macosx64-st-an,macosx64-mulet,macosx64_gecko -u all -t all

Comment 14

[Joel Maher ( :jmaher ) (UTC -8)]

Comment on attachment 8752341 [details] [diff] [review]
try: -b do -p macosx64,macosx64-st-an,macosx64-mulet,macosx64_gecko -u all -t all

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

::: testing/xpcshell/moz.build
@@ +10,5 @@
> +    # Disabled on Mac due to our builders still being on MacOS 10.7,
> +    # see bug 1255588
> +    PYTHON_UNIT_TESTS += [
> +        'selftest.py',
> +    ]

this will remove selftest.py from the 'make check' step of the build job.  from what I can tell this is what we want to do here.

Comment 15

[Benjamin Smedberg]

I did a try run and everything looks good, but I don't want to land this before the weekend because I won't be around to respond. So I plan on pushing this on Monday unless Stefan you want to do it earlier.

Comment 16

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/7df6fed09fb8
https://hg.mozilla.org/integration/mozilla-inbound/rev/1665c708ebab

Comment 17

[Stefan [:stefanh]]

(In reply to Benjamin Smedberg  [:bsmedberg] from comment #15)
> I did a try run and everything looks good, but I don't want to land this
> before the weekend because I won't be around to respond. So I plan on
> pushing this on Monday unless Stefan you want to do it earlier.

Thanks. I couldn't resist landing it. Inbound looks OK: https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&revision=1665c708ebabf7b6e31c741cd4590f14e3e5beba

Comment 18

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/7df6fed09fb8
https://hg.mozilla.org/mozilla-central/rev/1665c708ebab