Re: xinput-calibrator startup question

[Max Krummenacher <max.oss.09@gmail.com>]

Hi Stefan

Am Donnerstag, den 21.04.2016, 13:11 -0700 schrieb Stefan Agner:
> On 2016-04-19 14:01, Jussi Kukkonen wrote:
> > On 19 April 2016 at 21:24, Koen Kooi <koen@dominion.thruhere.net>
> > wrote:
> > > 
> > > > Op 19 apr. 2016, om 17:12 heeft Jussi Kukkonen <
> > > > jussi.kukkonen@intel.com> het volgende geschreven:
> > > > 
> > > > Hi Koen,
> > > > 
> > > > I was looking at xinput-calibrator in preparation for bug 9365
> > > > "Remove
> > > > xtscal in preference of xinput-calibrator", and I'm wondering
> > > > if you
> > > > remember the logic behind this commit:
> > > > 
> > > > commit 6464bcd67d10ab9967ac83c27c413c1014be707e
> > > > Author: Koen Kooi <koen@dominion.thruhere.net>
> > > > Date:   Wed Apr 30 11:33:23 2014 +0200
> > > > 
> > > >    xinput-calibrator: fix XDG launch
> > > > 
> > > >    In the move from meta-oe to OE-core XDG based launched was
> > > > dropped
> > > >    without noting it in the commit message, so fix that
> > > > regression.
> > > > 
> > > >    Gnome-session will now launch the calibrator again.
> > > > 
> > > > The commit installs the app desktop file in the XDG autostart
> > > > directory. But we already have a XSession script for a similar
> > > > purpose
> > > > -- the result in Sato is that xinput-calibrator runs twice (and
> > > > I
> > > > would expect that to happen in gnome as well)?
> 
> It does not run twice in our case (LXDE). Can you point to the
> XSession
> script you are referring to?

This is not entirely true.

For the Colibri iMX.7 xinput-calibrator was launched twice initially.
Once by the XDG launch and once by the XSession script
/etc/X11/Xsession.d/30xinput_calibrate.sh.

The formfactor file https://github.com/Freescale/meta-fsl-arm/blob/mast
er/recipes-bsp/formfactor/formfactor/mx7/machconfig sets
HAVE_TOUCHSCREEN=1.
As a workaround we now override machconfig for our machine.

So for me the question remains if installing xinput-calibrator by
itself should launch the calibrator or if this should be conditional on
a config provided by the formfactor package.

Max

> 
> --
> Stefan