Kawa invoke branch merged into master

classic Classic list List threaded Threaded
6 messages Options
Reply | Threaded
Open this post in threaded view
|

Kawa invoke branch merged into master

Per Bothner
See http://per.bothner.com/kawa/invoke/news.html
Please let me know of any problems.
--
        --Per Bothner
[hidden email]   http://per.bothner.com/
Reply | Threaded
Open this post in threaded view
|

Re: Kawa invoke branch merged into master

Kay Zheng
Hi Per,

It seems the Android support is broken. I was using the configure
script like this:

    JAVACFLAGS="-g -source 1.7 -target 1.7" ./configure
--with-android=/path/to/android.jar --disable-xquery --disable-jemacs
--with-java-source=7

And the compiler complained about incompatible types in the Android code:

    (compiling ViewBuilder.scm to gnu.kawa.android.ViewBuilder)
    ViewBuilder.scm:8:5: type istring is incompatible with required
type java.lang.String


Regards,

Kay Z.
Reply | Threaded
Open this post in threaded view
|

Re: Kawa invoke branch merged into master

Per Bothner
On 05/02/2017 10:41 PM, Kay Zheng wrote:

> Hi Per,
>
> It seems the Android support is broken. I was using the configure
> script like this:
>
>      JAVACFLAGS="-g -source 1.7 -target 1.7" ./configure
> --with-android=/path/to/android.jar --disable-xquery --disable-jemacs
> --with-java-source=7
>
> And the compiler complained about incompatible types in the Android code:
>
>      (compiling ViewBuilder.scm to gnu.kawa.android.ViewBuilder)
>      ViewBuilder.scm:8:5: type istring is incompatible with required
> type java.lang.String

Hm.  Are you sure you have the current git sources?  I fixed a related bug
just a few days ago.  You should have this in gnu/kawa/reflect/ChangeLog:

2017-04-30  Per Bothner  <[hidden email]>

         * ClassMethods.java: Accept any CharSequence as a string.
         * CompileBuildObject.java: Likewise.
         * Invoke.java: Likewise.

--
        --Per Bothner
[hidden email]   http://per.bothner.com/
Reply | Threaded
Open this post in threaded view
|

Re: Kawa invoke branch merged into master

Kay Zheng
> Hm.  Are you sure you have the current git sources?  I fixed a related bug
> just a few days ago.  You should have this in gnu/kawa/reflect/ChangeLog:
>
> 2017-04-30  Per Bothner  <[hidden email]>
>
>         * ClassMethods.java: Accept any CharSequence as a string.
>         * CompileBuildObject.java: Likewise.
>         * Invoke.java: Likewise.

Yes, it's the latest master branch (commit b781076, tagged 2.93). The
ChangeLog you mentioned is up-to-date.

Regards,

Kay Z.
Reply | Threaded
Open this post in threaded view
|

Re: Kawa invoke branch merged into master

Per Bothner
On 05/02/2017 11:21 PM, Kay Zheng wrote:

>> Hm.  Are you sure you have the current git sources?  I fixed a related bug
>> just a few days ago.  You should have this in gnu/kawa/reflect/ChangeLog:
>>
>> 2017-04-30  Per Bothner  <[hidden email]>
>>
>>          * ClassMethods.java: Accept any CharSequence as a string.
>>          * CompileBuildObject.java: Likewise.
>>          * Invoke.java: Likewise.
>
> Yes, it's the latest master branch (commit b781076, tagged 2.93). The
> ChangeLog you mentioned is up-to-date.

I checked in a fix for this.  (It wasn't anything Android-specific,
but the lack of a coerscion from IString to String being used.)

I wouldn't expect --with-java-source=7 to work for "deployed" Android,
because Android doesn't yet support MethodsHandles.  It looks like the Android "O"
developer preview does support Methodhandles, though, which is good news.
--
        --Per Bothner
[hidden email]   http://per.bothner.com/
Reply | Threaded
Open this post in threaded view
|

Re: Kawa invoke branch merged into master

Kay Zheng
2017-05-04 4:19 GMT+08:00 Per Bothner <[hidden email]>:
>
> I checked in a fix for this.  (It wasn't anything Android-specific,
> but the lack of a coerscion from IString to String being used.)
>
> I wouldn't expect --with-java-source=7 to work for "deployed" Android,
> because Android doesn't yet support MethodsHandles.  It looks like the
> Android "O"
> developer preview does support Methodhandles, though, which is good news.
>

Thank you very much for fixing this.

And thank you for the heads-up about MethodsHandles. I'm new to Kawa
and Android, and was trying out things.


Regards,

Kay Z.