This repository has been archived by the owner. It is now read-only.

mobile-shell 1.2.4 fails to install on OS X 10.9 #23992

Closed
vandrijevik opened this Issue Nov 5, 2013 · 8 comments

Comments

Projects
None yet
5 participants
@vandrijevik
Contributor

vandrijevik commented Nov 5, 2013

https://gist.github.com/vandrijevik/5d362c874f6b0d4ca7e6

Note: brew install --env=std mobile-shell results in the same error.

@MikeMcQuaid

This comment has been minimized.

Member

MikeMcQuaid commented Nov 5, 2013

This is likely an upstream issue. Please report it there.

@vandrijevik

This comment has been minimized.

Contributor

vandrijevik commented Nov 5, 2013

Cheers! I have reported this upstream at mobile-shell/mosh#475 (in case anyone is interested in tracking or cross-referencing).

@keithw

This comment has been minimized.

keithw commented Nov 5, 2013

(Hi from Mosh.) We have seen this with other users -- we're told the problem is fixed if they remove the protobuf libraries and rebuild everything that Mosh depends on.

Our understanding is that this is a bug in Homebrew. Homebrew does not seem to correctly handle the case of rebuilding a package's dependencies after the user has gone over an architecture transition.

Here we're not sure if it's a 32-to-64-bit transition or just 10.8 to 10.9, but either way the dependency (protobuf) needs to be rebuilt to match the package that depends on it, and Homebrew is not aware enough to rebuild the dependency automatically. We'd need to replicate the scenario to be sure, though.

@adamv

This comment has been minimized.

Contributor

adamv commented Nov 5, 2013

This analysis makes sense; we specifically don't rebuild anything ever unless a user asks. Doing a major arch transition (32->64, previous compilers -> xcode 5) will definitely break stuff.

@vandrijevik does rebuilding protobuf fix this?

@adamv

This comment has been minimized.

Contributor

adamv commented Nov 5, 2013

@keithw sorry for causing upstream churn

@mistydemeo

This comment has been minimized.

Contributor

mistydemeo commented Nov 5, 2013

@keithw While we don't usually force users to rebuild on a major transition, we do track the biggest transition to 10.9 (the new default C++ stdlib) and refuse to allow incompatible linkage. It's possible the protobuf installation predates when we were tracking that though.

@keithw

This comment has been minimized.

keithw commented Nov 6, 2013

@adamv No worries, thanks for the quick response. We'd rather hear about problems with Mosh sooner rather than later and appreciate your looping us in.

@vandrijevik

This comment has been minimized.

Contributor

vandrijevik commented Nov 6, 2013

Thanks for the quick responses, everyone (and pardon my own tardiness). I can confirm that brew rm protobuf && brew install protobuf fixes the issue and mosh installs fine after that. Cheers!

@vandrijevik vandrijevik closed this Nov 6, 2013

@Homebrew Homebrew locked and limited conversation to collaborators Feb 17, 2016

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.