[mythtv-users] [mythtv] 20180705 Android Observations

[David Hampton]

On July 13, 2018 3:50:11 PM EDT, David Engel <david at istwok.net> wrote:
>How did this get on mythtv-users?  I added mythtv-dev back to the cc
>list.
>
>On Fri, Jul 13, 2018 at 06:18:44PM +0100, jksj wrote:
>> On 13/07/18 15:55, Lennart Sorensen wrote:
>> > On Thu, Jul 12, 2018 at 04:35:21PM -0500, David Engel wrote:
>> > > On Thu, Jul 12, 2018 at 04:13:07PM -0400, Lennart Sorensen wrote:
>> > > > On Thu, Jul 12, 2018 at 02:56:16PM -0500, David Engel wrote:
>> > > > > It was using timestretch at 1.5x.  It didn't happen last
>night.  I did
>> > > > > see multiple, but not too many, short pauses, as you
>described.  They
>> > > > > were annoying, but certainly not as annoying as exiting
>playback
>> > > > > altogether.  I later tried some talking heads shows at 1.8x
>> > > > > timestretch, but I couldn't take the jitter.
>> > > > > 
>> > > > > Yes, it shouldn't hurt.  Hopefully, they will take interest. 
>Our
>> > > > > timestretch and fast forward/rewind use is definitely more of
>a
>> > > > > torture test than with any other player of which I'm aware.
>> > > > All this talk of mythtv on android got me curious, so I just
>tried doing
>> > > > a build to see how it would be on my Sony TV, but since (I
>believe)
>> > > > it runs 32 bit, I had to build for that, and those 32 bit large
>file
>> > > > support bugs hit and that seems just not fixable.  Trying to go
>to API
>> > > > 24 that supposedly fixes that problem causes other things to
>not work.
>> > > > 
>> > > > Oh well, I guess the sony TV is probably too ram limited anyhow
>to run
>> > > > the frontend.
>> > > > 
>> > > > I think the readme file might as well state building for 32 bit
>is broken
>> > > > for now.  Would have saved some time.
>> > > What doesn't work for you?  I have a 32-bit, Anroid box on which
>I do
>> > > some very, basic testing.  I haven't noticed anything not
>working.
>> > Get an error about undefined ::fgetpos in freesurround.cpp which
>from what
>> > I found is due to using android API less than 24 and
>_FILE_OFFSET_BITS=64
>> > 
>> > Following the readme uses API 21.
>> > 
>> > This is where the build ends:
>> > 
>> > In file included from freesurround.cpp:20:0:
>> >
>/home/lsorense/android/android-ndk/sources/cxx-stl/gnu-libstdc++/4.9/include/cstdio:107:11:
>error: '::fgetpos' has not been declared
>> >     using ::fgetpos;
>> >             ^
>> > [...]
>> > 
>> My successfull 32 bit build was against
>> 
>> API 24,
>> 
>> SDK Tools 26.1.1
>> 
>> android-ndk-r15c
>> 
>> SDK Platform Tools 28.00 - I only used this because I did not know
>how to
>> load the earlier version
>> 
>> Two changes were required
>> 
>> workdir/mythtv/mythtv/configure
>> 
>> #export ANDROID_BUILD_TOOLS_REVISION=27.0.3
>> export ANDROID_BUILD_TOOLS_REVISION=28.0.0
>> 
>> workdir/packaging/android/config
>> 
>> #check_cppflags -D_FILE_OFFSET_BITS=64
>> 
>> #check_cxx -D_FILE_OFFSET_BITS=64 <<EOF && add_cxxppflags
>> -D_FILE_OFFSET_BITS=64
>
>Can you please open a ticket for this and attach your patch?  That
>will help make sure I or one of the other devs don't forget about it.

I have an uncommitted patch that looks to see if there is only one version of the build tools installed, and if so, uses it whatever it is. It falls back to the hard coded value only when multiple versions of the build tools are installed. I'l try and commit it next week. 

David