-
Notifications
You must be signed in to change notification settings - Fork 3.4k
Upgrade to go 1.21, replace golang.org/x/exp/slices with slices #4688
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
base: dev
Are you sure you want to change the base?
Conversation
Signed-off-by: Tiscs <tiscs@live.com>
|
I have already done this. It does not need a pr
…On Fri, Oct 11, 2024 at 03:33 Stephen Kitt ***@***.***> wrote:
***@***.**** approved this pull request.
—
Reply to this email directly, view it on GitHub
<#4688 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAJ7TMDYUOOTDQRP7PF6YPDZ26LM5AVCNFSM6AAAAABNOUCLYGVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDGNRSGM3TSOBYGU>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
It’s just confusing because it’s done in https://github.com/antlr4-go/antlr but not in https://github.com/antlr/antlr4/tree/dev/runtime/Go/antlr/v4 and the former says it’s a copy of the latter. |
|
Yes, but it referee to the build and release. You only use tagged versions
…On Fri, Oct 11, 2024 at 05:49 Stephen Kitt ***@***.***> wrote:
I have already done this. It does not need a pr
It’s just confusing because it’s done in
https://github.com/antlr4-go/antlr but not in
https://github.com/antlr/antlr4/tree/dev/runtime/Go/antlr/v4 and the
former says it’s a copy of the latter.
—
Reply to this email directly, view it on GitHub
<#4688 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAJ7TMHOCVQPLV6YRUTQH7LZ263LDAVCNFSM6AAAAABNOUCLYGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMBXGI2DKNZWGE>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
OK, sorry for the noise. |
I have previously created a PR in the antlr4-go/antlr project. but as mentioned in the project README.md, |
@jimidle But |
|
I am confused, because the latest commit in https://github.com/antlr4-go/antlr is from May 2024 and does not contain this change. (Reason is that I really want to drop the x/exp dependency in our project https://github.com/DAMEDIC/fhir-toolbox-go/blob/main/go.mod) |
|
I am also confused. The old exp module is still used both in this repo and in https://github.com/antlr4-go/antlr (even in the |
|
@jimidle your changes seem to have been lost, I’m guessing they were deleted when https://github.com/antlr4-go/antlr was updated from this repository — I can’t find your equivalent changes any more. In any case, all the available releases of |
|
I previously created a fork to address this issue by using the replace directive in go.mod to eliminate the dependency on golang.org/x/exp/slices. Here's a minimal example: // git clone https://github.com/choral-io/antlr4-go.git ../antlr4-go
module github.com/choral-io/gommerce-server-core
go 1.25.0
replace github.com/antlr4-go/antlr/v4 v4.13.1 => ../antlr4-go
require (
github.com/antlr4-go/antlr/v4 v4.13.1
// other dependencies...
)This workaround worked well in our project, but I sincerely hope the official repository can resolve this cleanly. |
|
I will update it all shortly. I’ve just been too busy with work and moving
…On Thu, Sep 18, 2025 at 08:35 Galen Suen ***@***.***> wrote:
*Tiscs* left a comment (antlr/antlr4#4688)
<#4688 (comment)>
I previously created a fork <https://github.com/choral-io/antlr4-go> to
address this issue by using the replace directive in go.mod to eliminate
the dependency on golang.org/x/exp/slices. Here's a minimal example:
// git clone https://github.com/choral-io/antlr4-go.git ../antlr4-go
module github.com/choral-io/gommerce-server-core
go 1.25.0
replace github.com/antlr4-go/antlr/v4 v4.13.1 => ../antlr4-go
require (
github.com/antlr4-go/antlr/v4 v4.13.1
// other dependencies...
)
This workaround worked well in our project, but I sincerely hope the
official repository can resolve this cleanly.
—
Reply to this email directly, view it on GitHub
<#4688 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAJ7TMF5OC2NOBBHIJTGMGL3TK7JTAVCNFSM6AAAAABNOUCLYGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTGMBXHAZDAMZSGQ>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Upgrade to go 1.21 to use
slicespackage instead ofgolang.org/x/exp/slicespackage.In the antlr4-go/antlr project, the go version declared in go.mod is already 1.22.