michaelzoech / pyimc

a Python script to control Pidgin and Skype via command-line
Other
2 stars 0 forks source link

remove "redundant" config from script? #6

Closed michaelzoech closed 14 years ago

michaelzoech commented 14 years ago

maybe we should put the complete call to dmenu with params into exactly one property for the user to configure in the config file.

pros:

anpieber commented 14 years ago

mhm, this sounds quite nice; additionally it will reduce the required code in the script immense :)

to write scripts about the positioning maybe it will help to be able to put all configurations per commend line to the script? WDYT?

I set this issue as a task for the next release!

michaelzoech commented 14 years ago

what do you mean exactly with "put all configurations per command line to the script" ? do you mean that it should be possible to configure the script from config file and command line too?

anpieber commented 14 years ago

yep; i think this will be nice. I think about something like -config.NAME VALUE

configuration given via cmd is only value for one run;but this allows you to configure the tools via scripts. I think of this very handy in case of using some windowmanagers such as ratpoison where you can bound entire, very complex scripts to short keys

michaelzoech commented 14 years ago

+1

-config.NAME VALUE is ok, but what about the more *nix way of --NAME=VALUE ?

anpieber commented 14 years ago

+1; but would also add this to pyimc-0.2.0 and release the pyimc-0.1.0 now :)

michaelzoech commented 14 years ago

this issue should remove the required dmenu-vertical dependency, update PKGBUILD accordingly

anpieber commented 14 years ago

created own issue for dmenu-vertical reference

michaelzoech commented 14 years ago

done