Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Latest autoupdate breaks gitx #207

Closed
balboah opened this issue Jul 3, 2013 · 19 comments
Closed

Latest autoupdate breaks gitx #207

balboah opened this issue Jul 3, 2013 · 19 comments
Labels

Comments

@balboah
Copy link

balboah commented Jul 3, 2013

On OS X 10.8.4.
Version: 0.14.89 dev

dyld: Library not loaded: @rpath/ObjectiveGit.framework/ObjectiveGit
  Referenced from: /usr/local/bin/gitx
  Reason: image not found
Trace/BPT trap: 5
@moeffju
Copy link

moeffju commented Jul 3, 2013

Console also says:

LSOpenURLsWithRole() failed for the application /Users/moeffju/Library/Application Support/GitX/GitX 0.14.89/GitX.app with error -10810 for the file /Users/moeffju/src/repo.

(Mac OS 10.9 DP 2, btw)

@sdegutis
Copy link

sdegutis commented Jul 3, 2013

Same for me, can't find ObjectiveGit. Betting it was accidentally removed from the copy-frameworks build phase.

@woodwardjd
Copy link

commenting to a) follow just this bug and b) to say "thanks for jumping on this so quickly" :)

@sdegutis
Copy link

sdegutis commented Jul 3, 2013

Welp, checked the build phase and the framework should still be copied. Betting @rowanj is doing something a little more custom, so probably only he can solve this one.

@tiennou
Copy link

tiennou commented Jul 3, 2013

I'd say gitx is a binary, not a package/.app, and thus @rpath (which tells dyld to go look in Contents/Resources/) doesn't makes sense. It's a side-effect of the move to Objective-Git as the backend of GitX.

What's the output of otool -L /usr/local/bin/gitx ? Mine says :

% otool -L /usr/local/bin/gitx
/usr/local/bin/gitx:
    @executable_path/../Frameworks/ObjectiveGit.framework/Versions/A/ObjectiveGit (compatibility version 1.0.0, current version 1.0.0)
    /System/Library/Frameworks/AppKit.framework/Versions/C/AppKit (compatibility version 45.0.0, current version 1187.37.0)
    /System/Library/Frameworks/CoreFoundation.framework/Versions/A/CoreFoundation (compatibility version 150.0.0, current version 744.18.0)
    /System/Library/Frameworks/ScriptingBridge.framework/Versions/A/ScriptingBridge (compatibility version 1.0.0, current version 1.0.0)
    /System/Library/Frameworks/Foundation.framework/Versions/C/Foundation (compatibility version 300.0.0, current version 945.16.0)
    /usr/lib/libobjc.A.dylib (compatibility version 1.0.0, current version 228.0.0)
    /usr/lib/libSystem.B.dylib (compatibility version 1.0.0, current version 169.3.0)

and the culprit ought to be the first one.

You could try to fix it by doing something like install_name_tool -change @rpath/ObjectiveGit.framework/ObjectiveGit @executable_path/../Frameworks/ObjectiveGit.framework/Versions/A/ObjectiveGit but I'm not sure it will work... It doesn't work with longer paths.

@moeffju
Copy link

moeffju commented Jul 3, 2013

I use git x, an alias which calls GitX.app via open -a.

@tiennou
Copy link

tiennou commented Jul 3, 2013

This one works : install_name_tool -add_rpath @loader_path/../Frameworks /usr/local/bin/gitx.

@moeffju You're running 10.9, and I'm not ;-). I'm trying to find what error -10810 is, but gitx works with the command above.

Edit: found a better way ;-).

@tiennou
Copy link

tiennou commented Jul 3, 2013

@moeffju Here's something : http://www.finetunedmac.com/forums/ubbthreads.php?ubb=showflat&Number=25369. Does that ring a bell ?

@schwa
Copy link

schwa commented Jul 3, 2013

Broken for me too. @moeffju's "git x" is a great idea. (And yes running 10.9)

@tiennou's install_name_tool fix is a good workaround. Thanks!

@digitaljhelms
Copy link

+1

This is a breaking change to functionality that has been existent in GitX since 0.3 back in 2008 (and likely back to 0.1 and alpha versions previous to 0.1) from the original GitX project by @pieter; per the README:

GitX itself is fairly simple. Most of its power is in the 'gitx' binary, which you should install through the menu. the 'gitx' binary supports most of git rev-list's arguments. For example, you can run gitx --all to display all branches in the repository, or gitx -- Documentation to only show commits relating to the 'Documentation' subdirectory. With gitx -Shaha, gitx will only show commits that contain the word 'haha'. Similarly, with gitx v0.2.1.., you will get a list of all commits since version 0.2.1.

I frequently open GitX from within a repository directory at the CLI by simply running gitx, and I oft use gitx -S<string> to expedite searches from the CLI; these are core features of the application, and always have been.

Of course we can monkey-patch around the issue, but ultimately this is something that should be mended without requiring users to run install_name_tool or create aliases in bash configuration file(s).

I'll be staying at 0.14.86 for now...

@sdegutis
Copy link

sdegutis commented Jul 3, 2013

Wait.. are we all talking about the same thing? I'm not talking about the "gitx" binary at all. When I open GitX.app, it crashes instantly, after updating to this morning's update.

@digitaljhelms
Copy link

@Evanescence you may be experiencing #208, this issue is specific to running gitx from the CLI.

@sdegutis
Copy link

sdegutis commented Jul 3, 2013

Ah, yes. My mistake. Apologies all around.

@simple10
Copy link

simple10 commented Jul 3, 2013

@tiennou's solution worked for me

install_name_tool -add_rpath @loader_path/../Frameworks /usr/local/bin/gitx

I was getting this error:

$ gitx
dyld: Library not loaded: @rpath/ObjectiveGit.framework/ObjectiveGit
  Referenced from: /usr/local/bin/gitx
  Reason: image not found
Trace/BPT trap: 5

@rowanj
Copy link
Owner

rowanj commented Jul 4, 2013

I was up 'till 2 this morning fixing this, not sure where it snuck in; but the 'Library not loaded' crash should be fixed in 0.14.92

http://builds.phere.net/GitX/development/GitX-dev-92.dmg

If that has other issues, you can find 0.14.86 (the previous working build) on the new GitHub releases page:
https://github.com/rowanj/gitx/releases

@rowanj
Copy link
Owner

rowanj commented Jul 4, 2013

  • Builds between 0.14.86 and 0.14.92 don't launch (Library not found) - please grab the latest or previous build manually; since it won't run as far as the auto-update.
  • In 0.14.92, gitx command line tool still has this problem (which I'll track as command line broke after last release of gitx #210)

rowanj added a commit that referenced this issue Jul 4, 2013
@rowanj
Copy link
Owner

rowanj commented Jul 8, 2013

Should be fixed in 0.14.95 (current auto-update version, also available from the website and GitHub page).

You may need to re-install the command line tools.

GitX 0.14.95 (download .dmg)

@digitaljhelms
Copy link

👍 fix confirmed on my end, ta muchly!

@rowanj
Copy link
Owner

rowanj commented Jul 8, 2013

Thanks for confirming that, @digitaljhelms - closing this issue.

@rowanj rowanj closed this as completed Jul 8, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

9 participants