Crash Mobile build 758 on Android 5.1.1

[Lubomir I. Ivanov]

On 31 January 2016 at 12:29, Rick Walsh <rickmwalsh at gmail.com> wrote:
>
> That is interesting. Either there's something your phone doesn't like about
> my builds (work fine on my phone), or I was wrong about 675 being commit
> 1030cb2. I assumed 758 was built from the curreny git master and I counted
> back. But maybe 758 was a few commits earlier, in which case I think 675
> would be a few commits before 1030cb2. Dirk can you confirm?
>

hello Rick,

*) QML UI: Don't explicitly set width of GridLayout items
^ this one is a suspect in terms of what the stacktrace shows -
QGridLayoutEngine::sizeHint()

*) it could be a regression in the newer Qt that Dirk is using and it
could work with the older Qt you are using.
it's best to test with the newest available Qt.

*) to get the XXX digit of a commit:
> git describe HEAD --tags --abbrev=12
v4.5.2-758-g0eae59bcd29b
> git describe 1030cb2 --tags --abbrev=12
v4.5.2-676-g1030cb265a46

*) i've tried to figure out if the plasma-mobile components are a
factor, but it seems that GridLayout is only used in the *Gallery
components, yet those are not used directly in the subsurface
codebase. i could be wrong...

plasma-mobile being outside of the subsurface tree is the correct
decision, but as a third party library it can create a scenario where
for instance:
- subsurface-758 works
- plasma-mobile updates
- subsurface-759 is a small translation change, but no longer works

so this has to be monitored as well.

lubomir
--