Enhancement: Allow ` shell commands in config file

[GoogleCodeExporter]

I like to encrypt my password in the config file, so it looks like this:

[gcalcli]
user: graham@example.com
pw: `gpg --decrypt ~/.googlepw.gpg`

I'm attaching a patch which looks for ` as the first character of a config 
value, and if found runs it through subprocess.check_output.

Original issue reported on code.google.com by gra...@gkgk.org on 14 Oct 2011 at 3:59

Attachments:

• gcalcli-shell.diff

[GoogleCodeExporter]

This should be 'Enhancement', not 'Defect'. I'm not sure how to change it.

Original comment by gra...@gkgk.org on 14 Oct 2011 at 4:02

[GoogleCodeExporter]

I think this should be added to the next release. This is a great idea. Maybe 
repost this as enhancement to make sure it's added? (I am new to this)
Thanks for the patch!!

Original comment by OrdiRu...@gmail.com on 6 Nov 2011 at 2:28

[GoogleCodeExporter]

I was just trying to write this patch myself when I stumbled upon this thread. 
Excellent work! Does the job well.

I also wanted to add that there's also the more intimidating `gpg --batch 
--decrypt ~/.googlepw.gpg` :)

This would make a great enhancement. I would say that you might also want to 
search for other shell variants such as $(*) to try to help users who want to 
use some specialized shell variants. But this may be tricky as some shells can 
have quirky syntax.

Original comment by derekAthomas@gmail.com on 13 Apr 2012 at 6:12

[GoogleCodeExporter]

Moved to https://github.com/insanum/gcalcli/pull/3

Original comment by gra...@gkgk.org on 12 May 2012 at 3:36

[GoogleCodeExporter]

gcalcli moved to github: https://github.com/insanum/gcalcli

This enhancement was made on github.

Thanks graham! :-)

Original comment by eda...@insanum.com on 15 May 2012 at 2:40

• Changed state: Fixed