Crash Mobile build 758 on Android 5.1.1
Lubomir I. Ivanov
neolit123 at gmail.com
Sun Jan 31 05:19:14 PST 2016
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?
*) QML UI: Don't explicitly set width of GridLayout items
^ this one is a suspect in terms of what the stacktrace shows -
*) 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
> git describe 1030cb2 --tags --abbrev=12
*) 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
- 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.
More information about the subsurface