Comment 8 for bug 588086
Revision history for this message
| Francesco Marella (francesco-marella) wrote Re: [Bug 588086] Re: application indicator (needed for maverick integration) | #8 |
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
35a32c5
(Get the code!)
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