[maemo-developers] [ANNOUNCE] N9/N950 TV out control application

[From: Ville Syrjälä]

On Mon, Jan 09, 2012 at 10:46:14AM +0100, Pali Rohár wrote:
> On Monday 09 January 2012 11:20:19 you wrote:
> > On Mon, Jan 09, 2012 at 09:48:27AM +0100, Pali Rohár wrote:
> > > On Monday 09 January 2012 10:39:43 Ville =?ISO-8859-1?Q?Syrj=E4l=E4?= 
> wrote:
> > > > I finally managed to make some progress on "porting"
> > > > maemo-tvout-control to harmattan.
> > > > 
> > > > Code is here:
> > > > https://gitorious.org/maemo-tvout-control/n9-tvout-control
> > > > 
> > > > What you'll find there is a a very simple QWidget based GUI,
> > > > and backend code utilizing XRandR.
> > > > 
> > > > The backend supports a few extra knobs, when compared with
> > > > fremantle. However, I was too lazy to write the extra GUI
> > > > code. So the current GUI offers the same controls that
> > > > maemo-tvout-control has.
> > > > 
> > > > The ultimate plan is to write a nice QML GUI for this stuff,
> > > > although I don't know when I'll have time to learn QML.
> > > > 
> > > > Another idea I have is to move the backend code to a separate
> > > > library. The same should be done for the fremantle Xv based
> > > > backend code. This would provide a unified TV out control API
> > > > for both platforms.
> > > 
> > > Hello,
> > > 
> > > last year I merged maemo-tvout-control with control panel settings plugin
> > > for Nokia N900. Code is now in Community SSU on gitorious
> > > 
> > > https://gitorious.org/community-ssu/maemo-tvout-control
> > > 
> > > and is based on your orignal code
> > > https://gitorious.org/maemo-tvout-control/maemo-tvout-control
> > 
> > Ah, I did see the mail, but was swamped with something at the time and then
> > I just totally forgot to respond. Sorry.
> > 
> > > I created new tvout status menu plugin, it is in that CSSU repository too.
> > 
> > Great. Multiple different GUIs was the reason I tried to keep the
> > backend code separate.
> > 
> > > It is possible to merge my changes on CSSU to your project? I think it is
> > > not good idea to have 2 separated projects which has same code.
> > 
> > Yeah we could do some merging. But perhaps now would a good time to
> > move the backend code into a separate library. I'm not sure if the
> > harmattan and fremantle code can coexist nicely inside a single library,
> > or if that would cause problems with the packaging and whatnot. But at
> > least we could provide a library with the same name, and the same API,
> > for both platforms.
> 
> That version which is in CSSU working as:
> 
> * status menu plugin is loaded at startup and only this plugin have access to 
> tvout-ctl.c code. It monitor gconf tvout keys and when some is changed, it 
> call tvout-ctl function to change xv properties. (also monitor for xv changes 
> and change gconf keys)
> 
> * control panel plugin and desktop widget (and other plugins that can be 
> implemented) only changing gconf tvout keys, nothing more.
> 
> 
> In tvout-ctl is minimum changes. There was only removed hardcoded function 
> update_ui and tvout_ui_set_* Instead that, tvout_ctl_init function get pointer 
> to function which is called.
> 
> So tvout-ctl.c should be same in fremantle and harmattan with my changes. I do 
> not know harmattan, but I think that haramttan does not use any Gtk, gconf 
> libraries, so desktop widget, control and status menu plugins are only for 
> fremantle.

Yep. At least for now there are no desktop/menu plugins.

> I think that the best will be to have one repository for tvout-ctl, 
> one for fremantle (here will be fremantle plugins) in CSSU project and one for 
> harmattan. What do you think?

Sounds good. We don't really need a harmattan specific GUI repo,
at least yet. The standalone Qt GUI should work in both fremantle
and harmattan if the tvout-ctl API is the same. Some clever way to
handle packaging may be needed if there are differences. Perhaps
keep the packaging stuff in separate branches and merge master to
them when the actual code changes.

-- 
Ville Syrjälä
syrjala at sci.fi
http://www.sci.fi/~syrjala/