application indicator (needed for maverick integration)

Hi Jon, if you need any help porting this to application indicators please let me know!

Jon are you working on this bug? I see status set to 'in progress' but unassigned.
I've done some work on that and I could share if needed.

Michael Speth has sent me his first draft on this; that's why I marked it as in progress. I have asked him to share his work here so we can collaborate. Francesco, please post any work you have done on this.

Added a link to the branch.

The configure script does not check if the appindicator python module is available. Comments are welcome, thanks.

Looks great. Had I implemented it I would probably have made it check for appindicator in the python script at runtime and have the script fall back to statusicon if it isn't found. I think your solution, where either statusicon or appindicator is built in, is superior because only one of them is really needed. What I would like is a configure parameter, let's call it --enable-gui, that would take one of three argument: "no", "statusicon" or "appindicator". Hope you get the idea. I can do this when I merge your branch, though, if you don't want to mess too much with the autoconf script.

Yep, I converted the script to be more class centric and the syntax to be more python like. I'll take any suggestions on improving this. Also, if you like whats in the script, how should this be merged into Redshift? Should there be a new directory similar to gtk-redshift?

I forgot to include the new svg for when redshift is in the "off state" that the script requires

Il giorno ven, 04/06/2010 alle 19.47 +0000, Jon Lund Steffensen ha
scritto:
> Looks great. Had I implemented it I would probably have made it check
> for appindicator in the python script at runtime and have the script
> fall back to statusicon if it isn't found. I think your solution, where
> either statusicon or appindicator is built in, is superior because only
> one of them is really needed. What I would like is a configure
> parameter, let's call it --enable-gui, that would take one of three
> argument: "no", "statusicon" or "appindicator". Hope you get the idea. I
> can do this when I merge your branch, though, if you don't want to mess
> too much with the autoconf script.
Got the idea, and I agree. Please do it yourself, I'm not very handy
with autoconf.

I'm going to push a couple of commits and propose for merging my branch
so you can do a final review and merge.

cheers,
Francesco

@Michael: In the meantime Francesco Marella has provided a clean and working solution. Please have a look at it.