[ 
https://issues.apache.org/jira/browse/THRIFT-5358?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17287397#comment-17287397
 ] 

Yuxuan Wang commented on THRIFT-5358:
-------------------------------------

To clarify, the lack of go.mod files does not break our users importing the go 
library, it only breaks/blocks developers working on the library code.

As a result, combining with that the 2nd step is actually a lot of work, I 
slightly prefer to skip this in 0.14.1.

But only adding the first one (lib/go/thrift/go.mod) into 0.14.1 will help 
users importing the master branch version after that, because if we also do the 
release process change in 0.14.1, they will get a pseudo v0.14.2-timestamp-sha1 
version in their go.mod file, instead of v0.0.0-timestamp-sha1 version.

> Add go.mod file(s)
> ------------------
>
>                 Key: THRIFT-5358
>                 URL: https://issues.apache.org/jira/browse/THRIFT-5358
>             Project: Thrift
>          Issue Type: Task
>          Components: Go - Library
>    Affects Versions: 0.14.0
>            Reporter: Yuxuan Wang
>            Priority: Major
>
> Go 1.16 already disallowed building without go.mod file by default (still 
> override-able with GO111MODULES=auto environment variable), and Go 1.17 will 
> remove that override option, so we need to add go.mod file(s) to unblock 
> developers working on tip and using Go 1.16+.
> Based on the current discussion on https://github.com/golang/go/issues/34055, 
> we probably will need to add 2 go.mod files:
> # one under lib/go/thrift for our users to use. A release process change is 
> also required because of this file: for all the future releases, we would 
> need to double tag with "lib/go/thrift/" prefix (e.g. for 0.15.0 release we 
> would need to git tag "v0.15.0" and "lib/go/thrift/v0.15.0" on the same 
> commit)
> # another under root for the tests to work on Go 1.16+



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to