Re: [PATCH 1/1] xinput-calibrator: move it from meta-oe to oe-core

[Laurentiu Palcu <laurentiu.palcu@intel.com>]

On Mon, Jun 17, 2013 at 04:15:58PM +0100, Burton, Ross wrote:
> On 17 June 2013 13:26, Laurentiu Palcu <laurentiu.palcu@intel.com> wrote:
> > People using xserver-xorg that need to calibrate their touchscreen
> > devices would also need meta-oe. Bringing the recipes to oe-core will
> > make it easier for them.
> >
> > Aditionaly, drop xterm RDEPENDS. Terminal is not needed to run the menu
> > item.
> 
> I don't really like how there's a xdg-autostart desktop file and a
> systemd service file for loading the calibration and performing
> calibration - a single xsession file as xtscal does will be sufficient
> and actually work in Sato as it can block the rest of the UI before
> continuing.

I'm not sure I understand what's wrong with using xdg-autostart.
Xsession parses each .desktop file in xdg/autostart and executes the
application listed in 'Exec'. Having a separate xsession file created
in /etc/X11/Xsession.d (like xtscal does) is, basically, the same thing.
Only the moment of execution differs. Or, maybe, I misunderstood your
point.

> 
> Also persistent calibration doesn't work with non-root X users, so
> maybe the tool should use ~/.pointercal.xinput if it can't write to
> /etc.

We could easily change xinput_calibrator_once.sh to save/read the calibration
data either to/from /etc or ~.

Laurentiu

> 
> Apart from that I've got a touchscreen that isn't X-axis-reversed now. :)
> 
> Ross