Update antlr4 version to 4.13.0 #81
No reviewers
Labels
No labels
P0
P1
P2
P3
good first issue
pool
Infrastructure
blocked
bug
config
discussion
documentation
duplicate
enhancement
go
help wanted
internal
invalid
kludge
observability
perfomance
question
refactoring
wontfix
No milestone
No project
No assignees
3 participants
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: TrueCloudLab/frostfs-sdk-go#81
Loading…
Reference in a new issue
No description provided.
Delete branch "acid-ant/frostfs-sdk-go:bugfix/75-antlr4-in-docker"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Add
make
targetspolicy
anddocker/%
.Close #75
Signed-off-by: Anton Nikiforov an.nikiforov@yadro.com
@ -121,6 +121,24 @@ func placementNodes(addr *object.Address, p *netmap.PlacementPolicy, frostfsNode
}
```
#### antlr commands to update parsers
What about writing a Make target for this?
I mean we also have a specific version and dockerfile is rather simple, so storing just dockerfile is an option.
We seem to download everything in Docker anyway.
Maybe we even can add this to
go:generate
: it's just docker docker build and docker run.Added two targets
antlr-image
andantlr-run
. @realloc that is what you need?I think the intention was to have
make policy
andmake docker/policy
, where the first one runs the first target in the docker container.On a side-note, I think it makes sense to build in the same target, I can't imagine the use-case for a separate target here.
Added, please review.
Can we make it similar to this one?
docker/%:
Done.
b54f3461b7
to3f6c44f13d
3f6c44f13d
to3f3a65be53
3f3a65be53
to29fc6288fe
29fc6288fe
to86f5468506
Update antlr4 version to 4.13.0to WIP: Update antlr4 version to 4.13.086f5468506
to4592f7f24d
4592f7f24d
to857c4bfda4
WIP: Update antlr4 version to 4.13.0to Update antlr4 version to 4.13.0@ -32,0 +36,4 @@
@java -Xmx500M -cp "`pwd`/antlr4-tool.jar" "org.antlr.v4.Tool" -o `pwd`/netmap/parser/ -Dlanguage=Go -no-listener -visitor `pwd`/netmap/parser/Query.g4 `pwd`/netmap/parser/QueryLexer.g4
docker/policy:
@docker build -q -t antlr/antlr4 --platform linux/amd64 . > /dev/null
Do we need to specify platform?