D28105: Send keyboard map from a read only source

davidedmundson added inline comments.

INLINE COMMENTS

> zzag wrote in xkb.cpp:313
> Noob question: why do we need another QFile object?
> 
> I'm just trying to understand why we can't do something along these lines
> 
>   tmp->setPermissions(QFileDevice::ReadOwner);
>   m_seat->setKeymap(tmp->handle(), size);

I tired that first.

Both with Qt API as well as fchmod directly on this fd and on a dup'd version.

It didn't work.

There's a really good test in the bug report

REPOSITORY
  R108 KWin

REVISION DETAIL
  https://phabricator.kde.org/D28105

To: davidedmundson, #kwin
Cc: apol, zzag, kwin, Orage, cacarry, LeGast00n, The-Feren-OS-Dev, cblack, jraleigh, zachus, fbampaloukas, GB_2, mkulinski, ragreen, jackyalcine, iodelay, crozbo, bwowk, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, hardening, romangg, jensreuterberg, abetts, sebas, ahiemstra, mart