Version: (using KDE KDE 3.4.0) Installed from: Compiled From Sources If Konqueror is not installed when a user tries to use musicbrainz the error message is Could not launch the browser: Could not find service 'kfmclient A more user friendly message would be "Konqueror does not seem to be installed, it is required for musicbrainz." https://bugzilla.ubuntu.com/show_bug.cgi?id=10448
There's no reason Konqueror should be required for musicbrainz. Its possible the user was clicking the wrong button. I'm on a new computer so didn't have musicbrainz yet, compiling amarok with it now, I'll chmod a-x kfmclient and investigate.
The problem is not konqueror, it's kfmclient. kfmclient is called by the invokeBrowser function, that will run whatever browser you set as default in kde: http://developer.kde.org/documentation/library/3.3-api/kdecore/html/kapplication_8cpp-source.html#l02272
OTOH, wouldn't it be just as easy to let the user pick which browser they want to use from within amarok? It would allow those without KDE to actually launch a browser.
I like Philip Brink's idea, but: I'm wondering if the KDE function invokeBrowser is meant to be hard coded to use kfmclient, or should it check the [General] BrowserApplication setting in the ~/.kde/share/config/kdeglobals file? If not, can Amarok do this (check the BrowserApplication setting), rather than simply using invokeBrowser? This should also allow Amarok to work without KDE, and without adding an extra option to Amarok.
SVN commit 490088 by markey: Added option for selecting the external web browser used by amarok. Now, kfmclient and kdebase is no longer required for browser invocation, instead we use amaroK::invokeBrowser(). Currently, konqueror is the default browser. Ideally I would like to use the global kde setting bs default, but I can't figure out how to access it. BUG: 106015 M +2 -0 ChangeLog M +87 -33 src/Options1.ui M +6 -3 src/Options1.ui.h M +1 -0 src/amarokcore/amarok.kcfg M +12 -1 src/configdialog.cpp M +8 -8 src/contextbrowser.cpp M +4 -1 src/firstrunwizard.ui.h M +1 -3 src/tagdialog.cpp