After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 774478 - [patch attached] Banshee 2.6.1 fails to launch on macOS 10.12.1
[patch attached] Banshee 2.6.1 fails to launch on macOS 10.12.1
Status: RESOLVED WONTFIX
Product: banshee
Classification: Other
Component: general
2.6.1
Other Mac OS
: Normal critical
: ---
Assigned To: Banshee Maintainers
Banshee Maintainers
gnome[unmaintained]
Depends on:
Blocks:
 
 
Reported: 2016-11-15 14:46 UTC by Nate Graham
Modified: 2020-03-17 10:01 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Tested if speculative fix for the crash-on-launch issue on macOS (1.96 KB, patch)
2016-11-16 04:51 UTC, Nate Graham
none Details | Review
Fixes the path-not-found issue on macOS 10.12.1 (731 bytes, patch)
2016-11-16 15:25 UTC, Nate Graham
none Details | Review
Fixes an issue with incorrect mono library paths and environment variables on macOS 10.12.1 (1.12 KB, application/mbox)
2016-11-16 15:27 UTC, Nate Graham
  Details
ixes an issue with incorrect mono library paths and environment variables on macOS 10.12.1 (1.12 KB, patch)
2016-11-16 15:29 UTC, Nate Graham
none Details | Review
Fixes an issue with absolute path substitution for mono libraries and tango modules on macOS 10.12.1 (1.31 KB, patch)
2016-11-16 15:30 UTC, Nate Graham
none Details | Review
macOS: fix "no such file" issue (740 bytes, patch)
2017-06-22 21:32 UTC, Nate Graham
none Details | Review
macOS: fix failure to launch issue (1.23 KB, patch)
2017-06-22 21:33 UTC, Nate Graham
none Details | Review
macOS: fix incorrect mono path issue (1.08 KB, patch)
2017-06-23 03:36 UTC, Nate Graham
none Details | Review
macOS: fix no such file issue (993 bytes, patch)
2017-06-23 04:18 UTC, Nate Graham
none Details | Review
macOS: fix failure to locate bundle resources (1.49 KB, patch)
2017-06-23 04:20 UTC, Nate Graham
none Details | Review
macOS: fix incorrect mono path issue (1.26 KB, patch)
2017-06-23 04:20 UTC, Nate Graham
none Details | Review
macOS: fix failure to locate bundle resources (1.39 KB, patch)
2017-06-29 04:40 UTC, Nate Graham
none Details | Review

Description Nate Graham 2016-11-15 14:46:03 UTC
Banshee fails to launch on macOS Sierra 10.12.1; it immediately dies. Opening it from terminal reveals the error:

Warboss:~ nate$ /Applications/Banshee.app/Contents/MacOS/Banshee 
Application bundle has moved. Adjusting bundle...
find: /Users/nateApplications/Banshee.app/Contents/Resources/etc: No such file or directory

The path "/Users/nateApplications" is obviously not valid. The string "/Users/<user>" is being pre-pended to the bundle path, no matter where I put it. For example, if I move the bundle to the desktop:

Warboss:~ nate$ /Users/nate/Desktop/Banshee.app/Contents/MacOS/Banshee 
Application bundle has moved. Adjusting bundle...
find: /Users/nateUsers/nate/Desktop/Banshee.app/Contents/Resources/etc: No such file or directory


Whatever is prepending the path with "/Users/<user>" needs to stop doing this. This looks the same as https://bugzilla.gnome.org/show_bug.cgi?id=725454, but whatever DMG was regenerated there is invalid.

Please just stop pre-pending any path at all in the macOS version. It's unnecessary.
Comment 1 Nate Graham 2016-11-15 14:57:43 UTC
The problem is somewhere in banshee.darwin.in. Investigating.
Comment 2 Nate Graham 2016-11-15 16:11:29 UTC
The opening code that determines APP_ROOT (line 2) was wrong and did not need "$PWD/" prepended. That fixed the initial crash, but exposes another one with the GTK libraries having invalid paths. The part that adjusts the paths in the .in files doesn't seem to be doing its job and substituting ${APP_RESOURCES} into ${RES_DIR}.
Comment 3 Nate Graham 2016-11-15 17:09:05 UTC
Fixing that with `sed -i -E "s|\${APP_RESOURCES}|${RES_DIR}|g" ${file}` doesn't seem to eliminate the crash:

[Info  09:51:04.313] Running Banshee 2.6.1: [git-checkout (darwin11.4.0, x86_64) @ 2013-04-19 10:31:24 CEST]
Exception has been thrown by the target of an invocation.
System.Reflection.TargetInvocationException: Exception has been thrown by the target of an invocation. ---> System.DllNotFoundException: glibsharpglue-2

I'm missing something here--as can be expected with my first foray into Banshee development, I suppose.

Possibly relevant: http://stackoverflow.com/questions/13885454/mono-on-osx-couldnt-find-gtksharpglue-2-dll#15655635
Comment 4 Nate Graham 2016-11-15 20:21:09 UTC
This diff fixes the problem for me and now I can use Banshee normally:
http://pastebin.com/1Kj0Wsbe

I'm not sure if this is the 100% correct fix or how to submit a patch, but hopefully it should illustrate to someone more familiar with the codebase the sort of thing that will be needed to resolve the issue.
Comment 5 Andrés G. Aragoneses (IRC: knocte) 2016-11-16 01:32:49 UTC
Thanks for the patch!

Could you attach it in a more manageable format please? You would do this by cloning the repository with git, making the commit with your changes, and then issuing `git format-patch HEAD~1`, which generates the file that you can attach here.
Comment 6 Nate Graham 2016-11-16 04:51:02 UTC
Created attachment 339989 [details] [review]
Tested if speculative fix for the crash-on-launch issue on macOS

This patch fixes the crash-on-launch issue encountered on macOS. Please feel free to rewrite it if necessary, as I am not yet familiar with the style guidelines for this project.
Comment 7 Andrés G. Aragoneses (IRC: knocte) 2016-11-16 05:18:16 UTC
Awesome, that's more readable.

Only thing is that you're fixing 3 problems here in 1 single commit.

Ideally, please attach 3 patches. Each of them will fix a single issue. And in each commit message you would need to state the explanation of the change (basically enough to copy+paste what you wrote in the comments of this bug).

Also please state the exact version of MacOS that you're using, in each of the commit messages. After that I can commit the files for you in the repo (preserving authorship). Thanks!
Comment 8 Andrés G. Aragoneses (IRC: knocte) 2016-11-16 05:19:23 UTC
>  commit the files ...

I meant, commit the patches.
Comment 9 Nate Graham 2016-11-16 15:16:49 UTC
All right, let me try that again.
Comment 10 Nate Graham 2016-11-16 15:24:45 UTC
Review of attachment 339989 [details] [review]:

Ignore this.
Comment 11 Nate Graham 2016-11-16 15:24:51 UTC
Review of attachment 339989 [details] [review]:

Ignore this.
Comment 12 Nate Graham 2016-11-16 15:25:49 UTC
Created attachment 340017 [details] [review]
Fixes the path-not-found issue on macOS 10.12.1

This fixes the first of three issues that were preventing Banshee 2.6.1 from launching on macOS 10.12.1
Comment 13 Nate Graham 2016-11-16 15:27:03 UTC
Created attachment 340018 [details]
Fixes an issue with incorrect mono library paths and environment variables on macOS 10.12.1

This fixes the second of three issues that were preventing Banshee 2.6.1 from launching on macOS 10.12.1
Comment 14 Nate Graham 2016-11-16 15:29:14 UTC
Created attachment 340019 [details] [review]
ixes an issue with incorrect mono library paths and environment variables on macOS 10.12.1

This fixes the second of three issues that were preventing Banshee 2.6.1 from launching on macOS 10.12.1.
Comment 15 Nate Graham 2016-11-16 15:30:33 UTC
Created attachment 340020 [details] [review]
Fixes an issue with absolute path substitution for mono libraries and tango modules on macOS 10.12.1

This fixes the third of three issues that were preventing Banshee 2.6.1 from launching on macOS 10.12.1. Applying all three patches should restore Banshee's ability to launch on the Mac.
Comment 16 Nate Graham 2016-11-16 15:30:52 UTC
Done.
Comment 17 Nate Graham 2017-06-10 23:13:35 UTC
Ping?
Comment 18 Andrés G. Aragoneses (IRC: knocte) 2017-06-22 03:56:51 UTC
Oops, I'm very sorry, I missed your comment#16 on my mailbox, luckily you pinged me (next time ping directly, no need to go through Bugzilla :) ).

Patches look good to me technically, but they don't adjust to the commit-message guidelines.

For example, first one:

```
Fix a path-not-found issue on macOS 10.12.1
```

When it should be (first line as short as possible but including bug number, 3rd line and rest of lines explaining the problem as much detail as possible, not just the solution). For example:

```
macOS: fix "No such file or directory" at launch

Banshee failed to launch on macOS Sierra 10.12.1; it immediately died. Opening it from terminal reveals the error:

nate$ /Applications/Banshee.app/Contents/MacOS/Banshee 
Application bundle has moved. Adjusting bundle...
find: /Users/nateApplications/Banshee.app/Contents/Resources/etc: No such file or directory

The path "/Users/nateApplications" is obviously not valid. The string "/Users/<user>" was being pre-pended to the bundle path, no matter where I put it. For example, if I moved the bundle to the desktop:

nate$ /Users/nate/Desktop/Banshee.app/Contents/MacOS/Banshee 
Application bundle has moved. Adjusting bundle...
find: /Users/nateUsers/nate/Desktop/Banshee.app/Contents/Resources/etc: No such file or directory

The problem is in banshee.darwin.in. The opening code that determines APP_ROOT (line 2) was wrong and did not need "$PWD/" prepended.
```

And so on with the other patches.
Comment 19 Andrés G. Aragoneses (IRC: knocte) 2017-06-22 03:58:25 UTC
I missed the bug number, 1st line of commit message should be:

macOS: fix "No such file or directory" at launch (bgo#774478)
Comment 20 Nate Graham 2017-06-22 21:32:53 UTC
Created attachment 354283 [details] [review]
macOS: fix "no such file" issue
Comment 21 Nate Graham 2017-06-22 21:33:25 UTC
Created attachment 354284 [details] [review]
macOS: fix failure to launch issue
Comment 22 Nate Graham 2017-06-22 21:33:59 UTC
Done. Let me know if any further work is needed.

Also, is Banshee still actively developed?
Comment 23 Andrés G. Aragoneses (IRC: knocte) 2017-06-23 02:31:06 UTC
> Done. Let me know if any further work is needed.

I only see 1 line in your commit messages. In comment#18 I said that you should add all the details.

> Also, is Banshee still actively developed?

Unfortunately, not in the last months by me. But I plan to resurrect it at some point. (Hint: help is welcome ;) )
Comment 24 Nate Graham 2017-06-23 03:27:13 UTC
Are the rest of those details really that important? I didn't think so, but if you feel differently, I can add some more.

As for Banshee's future, if you don't already know this, you should be aware that there's been some activity on banshee-list@gnome.org (https://mail.gnome.org/archives/banshee-list/) by people who are trying to restart development. Might make sense to coordinate and centralize efforts to prevent duplication.
Comment 25 Nate Graham 2017-06-23 03:36:06 UTC
Created attachment 354291 [details] [review]
macOS: fix incorrect mono path issue

Oops, forgot the third reformatted patch.
Comment 26 Andrés G. Aragoneses (IRC: knocte) 2017-06-23 03:55:28 UTC
> Are the rest of those details really that important? I didn't think so, but if > you feel differently, I can add some more.

If you don't add those details, you're just adding the *what* and not the *why*. For example: fix incorrect mono path issue. Why is it incorrect?

> As for Banshee's future

Thanks for letting me know, I'll take that in account.
Comment 27 Nate Graham 2017-06-23 04:17:41 UTC
Okay, I can elaborate a bit.
Comment 28 Nate Graham 2017-06-23 04:18:27 UTC
Created attachment 354292 [details] [review]
macOS: fix no such file issue
Comment 29 Nate Graham 2017-06-23 04:20:11 UTC
Created attachment 354293 [details] [review]
macOS: fix failure to locate bundle resources
Comment 30 Nate Graham 2017-06-23 04:20:46 UTC
Created attachment 354294 [details] [review]
macOS: fix incorrect mono path issue
Comment 31 Nate Graham 2017-06-28 23:01:04 UTC
Is this thing on?
Comment 32 Andrés G. Aragoneses (IRC: knocte) 2017-06-29 03:20:45 UTC
Sorry to be picky but the first two patches contain the same description. Each patch should have a description unique to the problem that is fixing.
Comment 33 Nate Graham 2017-06-29 04:40:18 UTC
Created attachment 354670 [details] [review]
macOS: fix failure to locate bundle resources

They're all fixing the same problem: Banshee not launching on macOS. To be honest I'm not sure why we even need three patches in the first place since the actual problem won't be solved unless all three are applied in tandem.

Regardless, I've updated one of the patches to have a unique description.
Comment 34 André Klapper 2020-03-17 10:01:53 UTC
Banshee is not under active development anymore and had its last code changes more than three years ago. Its codebase has been archived.

Closing this report as WONTFIX as part of Bugzilla Housekeeping to reflect
reality. Please feel free to reopen this ticket (or rather transfer the project
to GNOME Gitlab, as GNOME Bugzilla is being shut down) if anyone takes the
responsibility for active development again.
See https://gitlab.gnome.org/Infrastructure/Infrastructure/issues/264 for more info.