1405389 - mach bootstrap uses wrong macports port name

Status: RESOLVED FIXED

(Firefox Build System :: General, enhancement)

Description

[André Reinald]

mach bootstrap wants macports to install py27-readline.
That port has been renamed to py27-gnureadline.

Comment 1

[André Reinald]

Attachment: bug-1405389-1.patch

Comment 2

[Nathan Froyd [:froydnj]]

Thanks for the patch!  I'm going to assume that as the author of the patch, you have verified this fix works on a macports system, since I don't have such a system close at hand. :)

How recently did the port get renamed?  Do we have to try and handle both names, for people using macports who haven't updated to the latest-and-greatest?

Did you mean to ask for review on the patch, or were you putting it here for reference?

Comment 3

[André Reinald]

Comment on attachment 8914826 [details] [diff] [review]
bug-1405389-1.patch

Yes, the fix works on a current installation of MacPorts.

Thanks for pointing that out: after some digging it looks the package name change occurred about 3-4 years ago. MacPorts requires a full update of ports at every major system upgrade, and a few of those happened in this period. Plus compiling mozcentral requires ports which are much more recent than 3-4 years (clang/llvm 4.0). So I think it's safe to assume targeted developers have a sufficiently current MacPorts.

I'd be glad to have a review and land the patch, I just didn't know who to ask.

Comment 4

[Sylvestre Ledru [:Sylvestre]]

If the change occurred several years ago and nobody reported that before, it is probably safe to say that the Macports bootstrap support is not used by Firefox contributors or dev. Maybe we should remove that.

Comment 5

[André Reinald]

Or maybe potential contributors gave up?
It looks like this script is the only place where we "support" macports anyway, and I only hit this issue when running it.
As a macports user, I'd be happy with human readable instructions about build dependencies: maybe one day I'll create a portfile to build Firefox from macports :-)

Comment 6

[Nathan Froyd [:froydnj]]

Comment on attachment 8914826 [details] [diff] [review]
bug-1405389-1.patch

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

Thank you!

Comment 8

[André Reinald]

Attachment: Change port name from py27-readline to py27-gnureadline to enable mach bootstrap on current macports

MozReview-Commit-ID: E9AlS8RCSp9

Comment 9

[André Reinald]

Changed previous patch comment to respect guidelines.
Carrying over r+

Comment 10

[Sylvestre Ledru [:Sylvestre]]

Comment on attachment 8914826 [details] [diff] [review]
bug-1405389-1.patch

>diff --git a/python/mozboot/mozboot/osx.py b/python/mozboot/mozboot/osx.py
>--- a/python/mozboot/mozboot/osx.py
>+++ b/python/mozboot/mozboot/osx.py
>@@ -370,17 +370,17 @@ class OSXBootstrapper(BaseBootstrapper):
>         missing = [package for package in packages if package not in installed]
>         if missing:
>             print(PACKAGE_MANAGER_PACKAGES % ('MacPorts',))
>             self.run_as_root([self.port, '-v', 'install'] + missing)
> 
>     def ensure_macports_system_packages(self):
>         packages = [
>             'python27',
>-            'py27-readline',
>+            'py27-gnureadline',
>             'mercurial',
>             'autoconf213',
>             'gnutar',
>             'watchman',
>         ]
> 
>         self._ensure_macports_packages(packages)
>         self.run_as_root([self.port, 'select', '--set', 'python', 'python27'])

Comment 11

[Sylvestre Ledru [:Sylvestre]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/85d26f1a75cf677beeacb9967a9bf490306e0af6
Bug 1405389 - Change port name from py27-readline to py27-gnureadline to enable mach bootstrap on current macports. r=nfroyd

Comment 12

[Sebastian Hengst [:aryx] (needinfo me if it's about an intermittent or backout)]

https://hg.mozilla.org/mozilla-central/rev/85d26f1a75cf

Comment 13

[Gregory Szorc [:gps]]

(In reply to Sylvestre Ledru [:sylvestre] from comment #4)
> If the change occurred several years ago and nobody reported that before, it
> is probably safe to say that the Macports bootstrap support is not used by
> Firefox contributors or dev. Maybe we should remove that.

We've definitely had people use the MacPorts bootstrapper in the past year. I suspect MacPorts maintained some backwards compatibility for the old name and only recently dropped it? Who knows.