Every required change should be in now. Please try again.
On Friday, January 6, 2017 at 8:09:22 PM UTC+1, glenford williams wrote:
>
> thanks. any idea when this will merged?
>
> On Sunday, 1 January 2017 17:28:01 UTC-5, Elias Naur wrote:
>>
>> That was more difficult than expected. The CL series
thanks. any idea when this will merged?
On Sunday, 1 January 2017 17:28:01 UTC-5, Elias Naur wrote:
>
> That was more difficult than expected. The CL series ending with
>
> https://go-review.googlesource.com/c/34777/
>
> should fix both problems. A backwards incompatible change had to be made
> t
That was more difficult than expected. The CL series ending with
https://go-review.googlesource.com/c/34777/
should fix both problems. A backwards incompatible change had to be made to
implicit "this" types. Instead of using a super class, you now use the
wrapped type for the generated class. R
I am attaching a simple reproduction of the package name crash (using
reverse.go). There is no need to modify the android manifest file for this
to trigger. The error I see is given below.
I'm also attaching a proposed fix for the renaming clash which works for my
case. I can submit it as a CL or
here is the diff for reproducing the JNI issue (I sent the previous
email without it). my guess is that "context" needs to be resolved to
the java side for this to work:
$ git diff .
diff --git a/example/reverse/reverse/reverse.go
b/example/reverse/reverse/reverse.go
index c4c3377..4ccf1e7 100644
12-25 09:52:45.244 10998 10998 E art : JNI ERROR (app bug): attempt to
pass an instance of go.Seq$Ref as argument 1 to void
android.app.AlertDialog$Builder.(android.content.Context)
go version devel +5bfba30 Sat Dec 24 22:37:11 2016 + darwin/amd64
my gomobile version is as of last night,
for me running the following
package reverse
import (
"Java/android/databinding/DataBindingUtil"
"Java/android/os"
sapp "Java/android/support/v7/app"
rlayout "Java/go/reverse/R/layout"
"Java/go/reverse/databinding/ActivityMainBinding"
"fmt"
)
type MainActivity struct {
Hi,
Can I persuade you to send a complete example that demonstrates the package
name clash problem and one that demonstrates the runtime JNI crash? Thanks.
- elias
On Saturday, December 24, 2016 at 11:27:56 PM UTC+1, andrey mirtchovski
wrote:
>
> Apologies for the digression.
>
>
>> This su
You're right, I failed to consider the constants on Service. I thought I
saw something about this recently but it just turned out to be you five
days ago :)
On Sat, Dec 24, 2016 at 4:27 PM andrey mirtchovski
wrote:
> Apologies for the digression.
>
>
> This suggests that you still have multiple
Apologies for the digression.
> This suggests that you still have multiple `app` imports without each
> having a unique name. Double check your imports in all source files being
> compiled.
>
I have renamed both imports to something else and ensured there is nothing
importing or using "app", how
> app redeclared as imported package name
This suggests that you still have multiple `app` imports without each
having a unique name. Double check your imports in all source files being
compiled.
> import "Java/android/app/Service"
I'd imagine `Service` is not a package and not something you can
> I haven't used these bindings but wouldn't you rename the v7 import as
>
> sapp "Java/android/support/v7/app"
>
> and then import
>
> "Java/android/app"
>
I'm still struggling with this. In my case I need v7.app as well as
android.app and android.app.Service, so my imports are (as per your
sugg
I haven't used these bindings but wouldn't you rename the v7 import as
sapp "Java/android/support/v7/app"
and then import
"Java/android/app"
then reference `app.AlertDialog` as appropriate? You'll also need to call
`dialog.Show()` in `showDialog`.
On Sat, Dec 24, 2016 at 8:35 AM glenford willi
thanks that worked.
the basic example works but i tried modifying to show a dialog but got
error
/home/kingwill101/go/bin/gomobile: loadExportData failed go install -pkgdir=
/home/kingwill101/go/pkg/gomobile/pkg_android_386 -tags="" -gcflags=-shared
-ldflags=-shared gitlab.com/kingwill101/reve
14 matches
Mail list logo
