820117 - XQuartz verification no longer required in python/mozboot/mozboot/osx.py

Status: RESOLVED FIXED

(Firefox Build System :: General, defect)

Description

[Jorge Luis Mendez [:jlmendezbonini]]

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.8; rv:17.0) Gecko/20100101 Firefox/17.0
Build ID: 20121128204232

Steps to reproduce:

System:
OSX 10.8.2 and using homebrew

I ran the one-line Firefox build prerequisites bootstrap command:

wget https://hg.mozilla.org/mozilla-central/raw-file/default/python/mozboot/bin/bootstrap.py && python bootstrap.py



Actual results:

Received this message:

"Homebrew needs XQuartz installed in order to build some dependencies. Please
download and install XQuartz from the following URL:

    http://xquartz.macosforge.org/downloads/SL/XQuartz-2.7.3.dmg\

When that has finished, please relaunch bootstrap."




Expected results:

Homebrew no longer requires XQuartz to build Python.

See https://github.com/mxcl/homebrew/issues/14989 for the discussion and https://github.com/mxcl/homebrew/commit/8f0c5f795816690e903b1e74b343f2572dda45ff for the commit that removes the dependency.

Comment 1

[Jorge Luis Mendez [:jlmendezbonini]]

Attachment: Removed XQuartz verifications from python/mozboot/mozboot/osx.py

My first bug report so I'm not sure whether the submitted patch file was generated properly. Thanks.

Comment 2

[Gregory Szorc [:gps]]

Comment on attachment 690537 [details] [diff] [review]
Removed XQuartz verifications from python/mozboot/mozboot/osx.py

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

Looks good!

The patch is /almost/ proper. We'd like patches to have the header lines (author, commit message, etc) to make it easier to check things in. See https://developer.mozilla.org/en-US/docs/Mercurial_FAQ#How_can_I_generate_a_patch_for_somebody_else_to_check-in_for_me.3F (preferred) or |git format-patch|.

But, this being your first patch, I can normalize things for you.

Your name and email will be attached to the patch. Is it OK if I use Jorge Luis Mendez <[e-mail]>?

Comment 3

[Gregory Szorc [:gps]]

Please answer authorship question from previous comment or upload a properly-formatted patch. Then, I'll check this in for you.

Comment 4

[Jorge Luis Mendez [:jlmendezbonini]]

Attachment: Remove XQuartz verifications in python/mozboot/mozboot/osx.py

Generated with: git format-patch -U8 -W.
Hopefully this is good. If not, please go ahead and fix it for me if it's not too much trouble. I don't want to make you waste too much time with such a simple patch.

Thanks.

Comment 5

[Gregory Szorc [:gps]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/7ce1a8ae8ed6

This should be merged into the main tree within a few hours.

Congratulations on landing your first patch!

Comment 6

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/7ce1a8ae8ed6