! Please note that this is a snapshot of our old Bugzilla server, which is read only since May 29, 2020. Please go to gitlab.xfce.org for our new server !
Add dialog to choose the hostname, port and password
Status:
RESOLVED: FIXED
Severity:
enhancement

Comments

Description Vincent Legout 2009-02-02 20:45:23 CET
Hi,

I have written a patch to allow the user to choose the hostname, port and password to connect to MPD.
Could you please consider including it in xfmpc ?

I am not so familiar with C and GTK, there might be some mistakes, please tell me if I can improve it.

Vincent
Comment 1 Vincent Legout 2009-02-02 20:46:57 CET
Created attachment 2147 
add dialog to choose the hostname, port and password
Comment 2 Mike Massonnet editbugs 2009-02-03 05:19:25 CET
Hey thanks, I will review it and it already looks clean.
Comment 3 Mike Massonnet editbugs 2009-02-03 07:19:36 CET
From a quick point of view:
- It doesn't work with the MPD_HOST/PORT environment variables.
- I haven't though yet of a general preferences dialog, I'll revert it to a "Server Settings" dialog (a better name than "Server" is welcome).
- I will move the button to a menu item in the arrow button.
Comment 4 Vincent Legout 2009-02-03 12:08:07 CET
About the MPD_HOST/PORT environment variables, I think there is a mistake in xfmpc because it should be password@host and not host@password as it is in xfmpc (http://linux.die.net/man/1/mpc). That may be why it doesn't work for you.

I have moved the button to the arrow button and renamed it to "Server Settings".
Comment 5 Vincent Legout 2009-02-03 12:09:10 CET
Created attachment 2148 
updated patch
Comment 6 Mike Massonnet editbugs 2009-02-03 20:59:33 CET
I commited a modified version of the patch (revision 6634) which takes care of other bits.  Feel free to comment on it, but all in all it does its job well.  Thanks for your contribution, and btw good catch with the password@host bug ;-)

And by the way, as I'm not an MPD user for quiet some months anymore, I will enjoy further contributions.
Comment 7 Vincent Legout 2009-02-04 12:16:35 CET
I saw your commit, it looks better than my work :-)

I would like to work on xfmpc. Did you intend to work on something specific or want something to be done.
You wrote on the IDEAS file "XfmpcGoodies, a plugin interface on the right side pane", I don't understand what you mean by that.

Bug #4890

Reported by:
Vincent Legout
Reported on: 2009-02-02
Last modified on: 2009-07-14

People

Assignee:
Mike Massonnet
CC List:
0 users

Version

Version:
unspecified

Attachments

add dialog to choose the hostname, port and password (9.63 KB, patch)
2009-02-02 20:46 CET , Vincent Legout
no flags
updated patch (9.24 KB, patch)
2009-02-03 12:09 CET , Vincent Legout
no flags

Additional information