* Kawa invoke branch merged into master
@ 2017-05-01 16:17 Per Bothner
2017-05-03 5:41 ` Kay Zheng
0 siblings, 1 reply; 6+ messages in thread
From: Per Bothner @ 2017-05-01 16:17 UTC (permalink / raw)
To: Kawa mailing list
See http://per.bothner.com/kawa/invoke/news.html
Please let me know of any problems.
--
--Per Bothner
per@bothner.com http://per.bothner.com/
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: Kawa invoke branch merged into master
2017-05-01 16:17 Kawa invoke branch merged into master Per Bothner
@ 2017-05-03 5:41 ` Kay Zheng
2017-05-03 5:59 ` Per Bothner
0 siblings, 1 reply; 6+ messages in thread
From: Kay Zheng @ 2017-05-03 5:41 UTC (permalink / raw)
To: Per Bothner; +Cc: Kawa mailing list
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.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: Kawa invoke branch merged into master
2017-05-03 5:41 ` Kay Zheng
@ 2017-05-03 5:59 ` Per Bothner
2017-05-03 6:22 ` Kay Zheng
0 siblings, 1 reply; 6+ messages in thread
From: Per Bothner @ 2017-05-03 5:59 UTC (permalink / raw)
To: Kay Zheng; +Cc: Kawa mailing list
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 <per@bothner.com>
* ClassMethods.java: Accept any CharSequence as a string.
* CompileBuildObject.java: Likewise.
* Invoke.java: Likewise.
--
--Per Bothner
per@bothner.com http://per.bothner.com/
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: Kawa invoke branch merged into master
2017-05-03 5:59 ` Per Bothner
@ 2017-05-03 6:22 ` Kay Zheng
2017-05-03 20:20 ` Per Bothner
0 siblings, 1 reply; 6+ messages in thread
From: Kay Zheng @ 2017-05-03 6:22 UTC (permalink / raw)
To: Per Bothner; +Cc: Kawa mailing list
> 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 <per@bothner.com>
>
> * 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.
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: Kawa invoke branch merged into master
2017-05-03 6:22 ` Kay Zheng
@ 2017-05-03 20:20 ` Per Bothner
2017-05-04 1:06 ` Kay Zheng
0 siblings, 1 reply; 6+ messages in thread
From: Per Bothner @ 2017-05-03 20:20 UTC (permalink / raw)
To: Kay Zheng; +Cc: Kawa mailing list
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 <per@bothner.com>
>>
>> * 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
per@bothner.com http://per.bothner.com/
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: Kawa invoke branch merged into master
2017-05-03 20:20 ` Per Bothner
@ 2017-05-04 1:06 ` Kay Zheng
0 siblings, 0 replies; 6+ messages in thread
From: Kay Zheng @ 2017-05-04 1:06 UTC (permalink / raw)
To: Per Bothner; +Cc: Kawa mailing list
2017-05-04 4:19 GMT+08:00 Per Bothner <per@bothner.com>:
>
> 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.
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2017-05-04 1:06 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-05-01 16:17 Kawa invoke branch merged into master Per Bothner
2017-05-03 5:41 ` Kay Zheng
2017-05-03 5:59 ` Per Bothner
2017-05-03 6:22 ` Kay Zheng
2017-05-03 20:20 ` Per Bothner
2017-05-04 1:06 ` Kay Zheng
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).