Skip to content

Support Go 1.13 and Go 1.14 both #964

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 50 commits into from
Closed

Conversation

visualfc
Copy link
Contributor

@visualfc visualfc commented Feb 8, 2020

Build on Go 1.13 / Go 1.14 and one build support working on Go 1.12 Go 1.13 Go 1.14 three version.

cd goproj/src
git clone https://github.com/visualfc/gopherjs github.com/gopherjs/gopherjs
cd github.com/gopherjs/gopherjs
git checkout go1.13-dev
go install -v
  1. This version build and install on Go 1.13 or Go 1.14. And one build support working on Go 1.12 1.13 1.14 three version and dynamic select internal compiler/natives/src.
  2. This version build and install on Go 1.12 and working on Go 1.12 only.
  3. if change install go version, please use -a flags to force rebuild packages.
    gopherjs build -a -v or gopherjs test -a -v
  • add internal/reflectlite for Go 1.13 Go 1.14
  • update syscall/js API for Go 1.13 Go 1.14 changes
  • support Go Module, build go project check go.mod ( use go env)
  • add -a (--force) flags to force rebuild packages
  • check installed go version for build ReleaseTags, support working Go 1.12 Go 1.13 Go 1.14 three version

@dmitshur
Copy link
Member

dmitshur commented Feb 8, 2020

CI is failing because the circle.yml file in this PR doesn't include the fix for issue #937 that was applied in commit 1852f3a. @visualfc Can you include it?

@visualfc
Copy link
Contributor Author

visualfc commented Feb 9, 2020

CI is failing because the circle.yml file in this PR doesn't include the fix for issue #937 that was applied in commit 1852f3a. @visualfc Can you include it?

update, but tests failed for CI.

@dmitshur
Copy link
Member

dmitshur commented Feb 9, 2020

It's failing because the diff -u <(echo -n) <(go list ./compiler/natives/src/...) step is reporting a non-zero diff. You should add a // +build js constraint to the newly added files in that directory.

@nevkontakte
Copy link
Member

Any news? I have multiple reasons to prefer GopherJS over Go/WASM, but at some point being able to use an up to date version will outweigh the advantages of GopherJS and the headache of migration :-(

@pjebs
Copy link
Contributor

pjebs commented Sep 17, 2020

If someone hard forks, I'm happy to migrate over

@inliquid
Copy link

@nevkontakte this is from Slack:
изображение

@flimzy
Copy link
Member

flimzy commented Sep 18, 2020

:( Sounds like a fork is in order, then.

@neelance
Copy link
Member

Yes, feel free to create a fork. If it is well maintained for a while, then we can endorse it or maybe even consider a merge at some point. But first Dmitri and I would like to see that the maintenance situation turns out better than what we are currently able to provide. :)

@pjebs
Copy link
Contributor

pjebs commented Sep 18, 2020

what should it be called? jsgopher or gojs?

@flimzy
Copy link
Member

flimzy commented Sep 18, 2020

what should it be called? jsgopher or gojs?

I suggest we either open a new issue, and/or open a slack conversation about this, and other housekeeping details (which organization, who has commit access, etc, etc) so we don't clutter this PR, and so we have a canonical place to point people for posterity. WDYT?

@nevkontakte
Copy link
Member

@inliquid thanks for bringing the clarity.

@flimzy I suppose #894 is an appropriate place for this kind of discussion. Let's continue it there, shall we?

@visualfc
Copy link
Contributor Author

visualfc commented Oct 13, 2020

support work on Go1.15 https://github.com/visualfc/gopherjs/tree/go1.15-dev

@pjebs
Copy link
Contributor

pjebs commented Oct 13, 2020

@visualfc Can you merge it to master in your fork so I can use it? I think it's time to adopt yours as the "official" gopherjs.
If you want, you can create an organization. Perhaps called: "gojs". And then transfer the repo under that organization.

Thanks for the amazing work.

@visualfc
Copy link
Contributor Author

visualfc commented Oct 15, 2020

https://github.com/goplusjs/gopherjs for supported Go 1.12~1.15

go get -u github.com/goplusjs/gopherjs

all test on macOS
Go1.12
Go1.13
Go1.14
Go1.15

@pjebs
Copy link
Contributor

pjebs commented Oct 15, 2020

Does it pass all the go unit tests?

@visualfc
Copy link
Contributor Author

visualfc commented Oct 30, 2020

Does it pass all the go unit tests?

https://github.com/goplusjs/gopherjs
test pass on macOS Go1.12~Go1.15

@flimzy
Copy link
Member

flimzy commented Jan 31, 2021

@neelance thanks for the offer. The amount of spare time I have is also limited, but I do indeed have a few projects in production that rely on GopherJS. If there's no better volunteer available, I'm happy to join :)

@nevkontakte, are you still interested in helping to maintain this project? If so, would you mind adding your name to the document mentioned here? #964 (comment) And if not, that's of course perfectly understandable.

@nevkontakte
Copy link
Member

@flimzy I already have (the second entry) 😃

@flimzy
Copy link
Member

flimzy commented Jan 31, 2021

@flimzy I already have (the second entry)

Ha! Sorry I didn't make the connection. And happy you've expressed interest :)

@flimzy
Copy link
Member

flimzy commented Feb 2, 2021

@visualfc I should have pinged you, too. I know you're making pretty strong headway on your own fork now, so may not be interested in GopherJS any longer, but I want to make sure you have a chance to voice your interest, if you want. #964 (comment)

I'm going to try to schedule a call with those interested in helping, to discuss next steps, and if you are interested, of course I'd love to involve you.

@visualfc
Copy link
Contributor Author

visualfc commented Feb 2, 2021

@visualfc I should have pinged you, too. I know you're making pretty strong headway on your own fork now, so may not be interested in GopherJS any longer, but I want to make sure you have a chance to voice your interest, if you want. #964 (comment)

I'm going to try to schedule a call with those interested in helping, to discuss next steps, and if you are interested, of course I'd love to involve you.

I hope GopherJS can support more go versions. my fork and experiment can support go1.16rc1.

@flimzy
Copy link
Member

flimzy commented Feb 3, 2021

I hope GopherJS can support more go versions. my fork and experiment can support go1.16rc1.

Does this mean you're not interested in participating in the GopherJS planning call?

@visualfc
Copy link
Contributor Author

visualfc commented Feb 3, 2021

I hope GopherJS can support more go versions. my fork and experiment can support go1.16rc1.

Does this mean you're not interested in participating in the GopherJS planning call?

If gopherjs has plans to review PR, I will submit PR again.

@flimzy
Copy link
Member

flimzy commented Feb 3, 2021

If gopherjs has plans to review PR, I will submit PR again.

That's clear. Thank you for your response. Hopefully we can start reviewing PRs again soon!

@visualfc
Copy link
Contributor Author

visualfc commented Feb 6, 2021

https://github.com/goplusjs/gopherjs/releases/tag/v1.1.8

support go1.16rc1 go:embed

@pjebs
Copy link
Contributor

pjebs commented Feb 6, 2021

I want to make my https://github.com/rocketlaunchr/react package compatible with gopherjs and your package @visualfc . Do you know how I can do it?

@visualfc
Copy link
Contributor Author

visualfc commented Feb 6, 2021

I want to make my https://github.com/rocketlaunchr/react package compatible with gopherjs and your package @visualfc . Do you know how I can do it?

project keep stay import "github.com/gopherjs/gopherjs/js" or use "syscall/js" for compatible wasm
go get -u github.com/goplusjs/gopherjs for install gopherjs

@visualfc
Copy link
Contributor Author

https://github.com/goplusjs/gopherjs now support Go 1.12~1.16

Go1.12~1.15 test actions on macOS, Go1.16 test action on Ubuntu.

Go1.12
Go1.13
Go1.14
Go1.15
Go1.16

@pjebs
Copy link
Contributor

pjebs commented Feb 20, 2021

@visualfc I made a PR in your repo.

@flimzy
Copy link
Member

flimzy commented Apr 9, 2021

Now that #1015 has been merged, GopherJS is up to date with the latest upstream Go, so I'm going to close this PR.

@flimzy flimzy closed this Apr 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.