golang July 2022

golang@lists.fedoraproject.org

8 participants
14 discussions

[go-rpm-macros] PR #47: Use Fedora's build flags for cgo.
by Maxwell G 13 Sep '22

13 Sep '22

gotmax23 opened a new pull-request against the project: `go-rpm-macros` that you are following: `` Use Fedora's build flags for cgo. `` To reply, visit the link below or just reply to this email https://pagure.io/go-rpm-macros/pull-request/47

2 6

Thread to send your Golang package reviews which are waiting
by Robert-André Mauchin 09 Sep '22

09 Sep '22

Thread to send your Golang package reviews which are waiting... reply to this thread so we can sync up our efforts. Best regards Robert-André

5 5

[go-rpm-macros] PR #48: Introduce %golang_arches_future and stop using %go_arches
by Maxwell G 07 Aug '22

07 Aug '22

gotmax23 opened a new pull-request against the project: `go-rpm-macros` that you are following: `` Introduce %golang_arches_future and stop using %go_arches `` To reply, visit the link below or just reply to this email https://pagure.io/go-rpm-macros/pull-request/48

2 6

cryptic build errors in syncthing with golang 1.19
by Fabio Valentini 29 Jul '22

29 Jul '22

Hi all, I've started to see very strange build failures of syncthing since the update from go 1.18 to 1.19: https://koschei.fedoraproject.org/package/syncthing?collection=f37 I've looked at build logs, but there's no obvious source of a problem that I can see in the usual "megabyte-sized wall of inscrutable build commands / output". Grepping for "error" does not help, the build just appears to "fail" at some point. Could somebody please take a look? I don't know what's going wrong. Thanks, Fabio

2 2

Please use and convert SPECs to SPDX
by Robert-André Mauchin 20 Jul '22

20 Jul '22

Hi, I'd like to recommend you to switch to SPDX for your license fields. Please intervene on any spec you create or edit. The license conversion should be disabled in go2rpm and then the rust2rpm dependencies should be dropped at a later date. Thank you. Robert-André

1 0

Re: Fedora Golang: SPEC and SRPM for ORAS (depends on itself?)
by Robert-André Mauchin 20 Jul '22

20 Jul '22

On 7/19/22 16:19, Mark E. Fuller wrote: > see https://copr.fedorainfracloud.org/coprs/fuller/test-builds/build/4642384/ > <https://copr.fedorainfracloud.org/coprs/fuller/test-builds/build/4642384/> > > Thanks > > Mark E. Fuller, Ph.D. > HaShikma 19, Apt. 24 > Nesher 3681219, Israel > +972 (0)53-872-6579 > mark.e.fuller(a)gmail.com <mailto:mark.e.fuller@gmail.com> > fuller(a)fedoraproject.org <mailto:fuller@fedoraproject.org> > @fuller:one.ems.host Ok, so the issue is, you don't have the correct default import path, you should check it in go.mod if you have a doubt, it starts with: > module oras.land/oras-go > > go 1.17 So you must use that import path when you create your SPEC: go2rpm -C -r oras.land/oras-go -f https://github.com/oras-project/oras-go You must specify the forge so the package can be downloaded correctly. Please note that the next version is a version 2 with a new import path: oras.land/oras-go/v2 You should then create a new package for this version specifically. But, the package still won't build because of Docker/Moby shenanigans: oras.land/oras-go/pkg/auth/docker # github.com/docker/docker/registry /usr/share/gocode/src/github.com/docker/docker/registry/auth.go:144:51: cannot use challengeManager (variable of type "github.com/docker/distribution/registry/client/auth/challenge".Manager) as type "github.com/distribution/distribution/v3/registry/client/auth/challenge".Manager in argument to auth.NewAuthorizer: "github.com/docker/distribution/registry/client/auth/challenge".Manager does not implement "github.com/distribution/distribution/v3/registry/client/auth/challenge".Manager (wrong type for GetChallenges method) have GetChallenges(endpoint url.URL) ([]"github.com/docker/distribution/registry/client/auth/challenge".Challenge, error) want GetChallenges(endpoint url.URL) ([]"github.com/distribution/distribution/v3/registry/client/auth/challenge".Challenge, error) FAIL oras.land/oras-go/pkg/auth/docker [build failed] Here the full list of Docker related dependencies according to go.mod: github.com/docker/cli v20.10.17+incompatible github.com/docker/distribution v2.7.1+incompatible github.com/docker/docker v20.10.17+incompatible github.com/docker/go-connections v0.4.0 I haven't checked Docker in a while. I think gotmax did a ticket on it for the next meeting. I'll have a look. Best regards, Robert-André

1 0

Golang Mass Rebuild
by Maxwell G 19 Jul '22

19 Jul '22

Hi Fedorians and Gophers, golang 1.18.4 was released a couple days ago. This release has fixes for 9 medium (rated by Red Hat Product Security) CVEs, so I will preform a rebuild in `rawhide` and `f36` to mitigate them[^0]. See https://groups.google.com/g/golang-dev/c/frczlF8OFQ0/m/4lrZh5BHDgAJ for more information about the specific vulnerabilities. [^0]: The golang version is Fedora 35 is EOL upstream, and the maintainers have not yet had a chance to backport the changes. Only packages that provide binaries need to be rebuilt, which will make this rebuild less disruptive. These packages were determined by querying for source packages that BuildRequire `golang` or `go-rpm-macros` and then checking if they provide any binary RPMS that install files in `/usr/bin`, `/usr/sbin`, or `/usr/libexec`. No action will be required from you, unless you'd like your package to receive special treatment regarding merging `rawhide` into `f36`. I plan to handle this rebuild later this week (the week of the 17th). In light of the recent discussion about large updates, I will most likely split this into 4 Bodhi updates per branch (a total of 8; each containing ~100 packages). ## rawhide Here[1] is a list of the affected packages on `rawhide`. [1]: https://git.sr.ht/~gotmax23/fedora-scripts/tree/main/item/rebuilds/golang_1… ## f36 Here[2] is a list of all of the affected packages on `f36`. However, I have further split this list down into two subgroups. [2]: https://git.sr.ht/~gotmax23/fedora-scripts/tree/main/item/rebuilds/golang_1… ### Mergable from Rawhide For these packages[3], `rawhide` was determined to be mergable back to `f36`, as `f36` is up to date with `rawhide`. [3]: https://git.sr.ht/~gotmax23/fedora-scripts/tree/main/item/rebuilds/golang_1… ### Not mergable These packages were determined to not be mergable[4], as `rawhide` is ahead of (or has otherwise diverged from) `f36`. Therefore, I will create a new rebuild commit and bump the release on `f36`. This will likely cause merge conflicts if you try to merge `rawhide` back into `f36` after this change. Assuming the update would be compatible and comply with the Updates Policy, I can move your package into the other list and merge `rawhide` into `f36`. Please leave a comment on https://pagure.io/GoSIG/go-sig/issue/44 if you would like me to do so. Conversely, if you believe your package is incorrectly in the mergable list, also let me know in the aforementioned ticket. [4]: https://git.sr.ht/~gotmax23/fedora-scripts/tree/main/item/rebuilds/golang_1… -- Thanks, Maxwell G (@gotmax23) Pronouns: He/Him/His

2 3

Re: helm rpm package
by Maxwell G 19 Jul '22

19 Jul '22

Jul 19, 2022 9:56:02 AM Davide Cavalca <dcavalca(a)fb.com>: > The latest version of helm picked up a dependency > on oras.land/oras-go. That library isn't currently packaged, and fails > in %check on a docker-related test due to a conflict between > docker/distribution and distribution/v3. Does it work if you add # Use new docker/distribution import path sed \ -e 's|github.com/docker/distribution|github.com/distribution/distribution/v3|' \ -i $(find . -name '*.go') in %prep and use dynamic buildrequires? -- Thanks, Maxwell G (@gotmax23) Pronouns: He/Him/His

1 0

Re: helm rpm package
by Maxwell G 19 Jul '22

19 Jul '22

Hi Ali, On 22/07/19 02:37PM, Ali Yousefi Sabzevar wrote: > I saw that those bugs were related to golang-helm-3 package, so maybe not > directly related to the helm3 rpm package. helm is a subpackage of the golang-helm-3 source package. These bugs are very much related to helm. > > I checked on > https://packages.fedoraproject.org/pkgs/golang-helm-3/helm/ that the > maintainer is *dcavalca*, but an email to dcavalca(a)redhat.com failed > to deliver. That is not the correct way to reach out to a maintainer. Not everyone who works on Fedora is a Red Hat employee. USERNAME(a)fedoraproject.org is an alias to whatever the user's actual email is. You can look in the package %changelog or on accounts.fedoraproject.org for a user's email, as well. This email also should've reached Davide directly. > > Anyway, if you see the helm3 package is orphaned and has no active > maintainer, I will pick it up as a maintainer and make an up to date rpm > package out of it. The package is not orphaned. The maintainer just hasn't updated it in a little while. I'm sure they would appreciate a PR. -- Thanks, Maxwell G (@gotmax23) Pronouns: He/Him/His

1 0

Fedora 38 Change: Exclude golang from x86_32 (system wide change).
by Stephen Smoogen 19 Jul '22

19 Jul '22

There is a lot of work needed to keep golang libraries working in Fedora, and in order to cut down extra items that are taking time and debugging, the golang SIG would like to remove x86_32 from i686 starting in Fedora 38. -- Stephen Smoogen, Red Hat Automotive Let us be kind to one another, for most of us are fighting a hard battle. -- Ian MacClaren

2 2

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

golang July 2022