gpsd-dev
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[gpsd-dev] Fw: gpsd client libraries for Android


From: Gary E. Miller
Subject: [gpsd-dev] Fw: gpsd client libraries for Android
Date: Tue, 2 Apr 2019 11:17:55 -0700

Yo All!

See below for work on an Android version of gpsd.  Comments anyone?

RGDS
GARY
---------------------------------------------------------------------------
Gary E. Miller Rellim 109 NW Wilmington Ave., Suite E, Bend, OR 97703
        address@hidden  Tel:+1 541 382 8588

            Veritas liberabit vos. -- Quid est veritas?
    "If you can’t measure it, you can’t improve it." - Lord Kelvin


Begin forwarded message:

Date: Tue, 2 Apr 2019 10:24:50 -0400
From: Larry Boyd <address@hidden>
To: "Gary E. Miller" <address@hidden>
Subject: Re: gpsd client libraries for Android


Here's a better one. It accomplishes the same thing, but uses the new
build system.

On Tue, Apr 2, 2019 at 9:42 AM Larry Boyd <address@hidden> wrote:

> I'm looking for comments on the attached patch as an approach to this.
>
> Instead of using the config file, I have the variables set as compiler
> flags. The majority of the changes in this patch are to disinclude
> gpsd_config.h when it is being built on Android. This is done by
> wrapping the include lines with #ifndef __ANDROID__ #endif so that it
> doesn't impact any other platform.
>
> I've also disabled AIS to remove that dependency.
>
> The "big hack" in this commit, is to replace gps_maskdump with a macro
> that returns an empty string. It allows it to compile, but doesn't
> return anything useful at this point. Most of the debugging won't
> work on Android as it is anyway, since it would have to be updated to
> go through the Android logging facility rather than print to console.
>
>
> Note that this patch is not yet ready to be applied, even if the
> hacks may be considered acceptable, since I haven't yet tested
> functionality, and I'd like to convert to an Android.bp instead of
> Android.mk.
>
>
> On Mon, Apr 1, 2019 at 10:48 PM Gary E. Miller <address@hidden> wrote:
>  
>> Yo Larry!
>>
>> On Mon, 1 Apr 2019 22:25:57 -0400
>> Larry Boyd <address@hidden> wrote:
>>  
>> > The problem is that scons is not part of the android build system,
>> > and as a result can't be used.  
>>
>> Sure it can.  Any OS that has Python installed can run scons.
>>  
>> > The entire build, every piece, has to be defined in an Android.mk
>> > or Android.bp file.  
>>
>> And your Android.mk can easily call scons.
>>  
>> > There is no user interaction during the build
>> > process.  
>>
>> And luckily scons requires no user interaction.
>>
>> Go ahead and solve the problem as you see fit, you are doing the
>> work.
>>
>> Whatever you come up with, document it, put it in a subdirectory, and
>> we'll see how to fit it into the gpsd git.
>>
>> RGDS
>> GARY
>>
>> ---------------------------------------------------------------------------
>> Gary E. Miller Rellim 109 NW Wilmington Ave., Suite E, Bend, OR 97703
>>         address@hidden  Tel:+1 541 382 8588
>>
>>             Veritas liberabit vos. -- Quid est veritas?
>>     "If you can’t measure it, you can’t improve it." - Lord Kelvin
>>  
>  



RGDS
GARY
---------------------------------------------------------------------------
Gary E. Miller Rellim 109 NW Wilmington Ave., Suite E, Bend, OR 97703
        address@hidden  Tel:+1 541 382 8588

            Veritas liberabit vos. -- Quid est veritas?
    "If you can’t measure it, you can’t improve it." - Lord Kelvin
Here's a better one. It accomplishes the same thing, but uses the new build system.

On Tue, Apr 2, 2019 at 9:42 AM Larry Boyd <address@hidden> wrote:
I'm looking for comments on the attached patch as an approach to this.

Instead of using the config file, I have the variables set as compiler flags. The majority of the changes in this patch are to disinclude gpsd_config.h when it is being built on Android. This is done by wrapping the include lines with #ifndef __ANDROID__ #endif so that it doesn't impact any other platform.

I've also disabled AIS to remove that dependency.

The "big hack" in this commit, is to replace gps_maskdump with a macro that returns an empty string. It allows it to compile, but doesn't return anything useful at this point. Most of the debugging won't work on Android as it is anyway, since it would have to be updated to go through the Android logging facility rather than print to console.


Note that this patch is not yet ready to be applied, even if the hacks may be considered acceptable, since I haven't yet tested functionality, and I'd like to convert to an Android.bp instead of Android.mk.


On Mon, Apr 1, 2019 at 10:48 PM Gary E. Miller <address@hidden> wrote:
Yo Larry!

On Mon, 1 Apr 2019 22:25:57 -0400
Larry Boyd <address@hidden> wrote:

> The problem is that scons is not part of the android build system,
> and as a result can't be used.

Sure it can.  Any OS that has Python installed can run scons.

> The entire build, every piece, has to be defined in an Android.mk or
> Android.bp file.

And your Android.mk can easily call scons.

> There is no user interaction during the build
> process.

And luckily scons requires no user interaction.

Go ahead and solve the problem as you see fit, you are doing the work.

Whatever you come up with, document it, put it in a subdirectory, and
we'll see how to fit it into the gpsd git.

RGDS
GARY
---------------------------------------------------------------------------
Gary E. Miller Rellim 109 NW Wilmington Ave., Suite E, Bend, OR 97703
        address@hidden  Tel:+1 541 382 8588

            Veritas liberabit vos. -- Quid est veritas?
    "If you can’t measure it, you can’t improve it." - Lord Kelvin

Attachment: gpsd_aosp.patch
Description: Text Data

Attachment: pgpc8ldio0HrR.pgp
Description: OpenPGP digital signature


reply via email to

[Prev in Thread] Current Thread [Next in Thread]