Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 834758 - [TRACKER] "go mod tidy" became fatal in go-module.eclass
Summary: [TRACKER] "go mod tidy" became fatal in go-module.eclass
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Eclasses (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: William Hubbs
URL:
Whiteboard:
Keywords: PullRequest
Depends on:
Blocks: 834603 834607 834608 834610 834625 834626 834630 834669 834695 834711
  Show dependency tree
 
Reported: 2022-03-08 08:17 UTC by Florian Schmaus
Modified: 2022-03-08 13:27 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments
go-module.eclass-invoke-ego-mod-tidy-with-nonfatal.patch (0001-go-module.eclass-invoke-ego-mod-tidy-with-nonfatal.patch,1.55 KB, patch)
2022-03-08 08:18 UTC, Florian Schmaus
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Florian Schmaus gentoo-dev 2022-03-08 08:17:02 UTC
With f9ee55e698f8 ("go-module.eclass: use ego helper function") the invocation if "go mod tidy" become fatal. This causes package build failures for previously building packages.
Comment 1 Florian Schmaus gentoo-dev 2022-03-08 08:18:47 UTC
Created attachment 766531 [details, diff]
go-module.eclass-invoke-ego-mod-tidy-with-nonfatal.patch
Comment 2 Larry the Git Cow gentoo-dev 2022-03-08 13:27:34 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=2ad9100728886b7a73ed2eba0710e3ab29622f52

commit 2ad9100728886b7a73ed2eba0710e3ab29622f52
Author:     Florian Schmaus <flow@gentoo.org>
AuthorDate: 2022-03-07 08:43:52 +0000
Commit:     Sam James <sam@gentoo.org>
CommitDate: 2022-03-08 13:27:18 +0000

    go-module.eclass: invoke "ego mod tidy" with nonfatal
    
    We previously invoked "mod tidy" without the ego helper function in a
    non-fatal way. This was switched to using the ego helper in
    f9ee55e698f8 ("go-module.eclass: use ego helper function") but without
    prefixing the call with nonfatal. Hence the semantic of the eclass
    changes: a previously non-fatal invocation became fatal, causing e.g.,
    bug #834695.
    
    [sam: fixes semantic change which wasn't advertised/seemingly intentional,
    so let's get this in now to avoid a significant number of ebuilds being broken.
    
    We can introduce this change if desired at a later point.
    
    Fixes: f9ee55e698f8 ("go-module.eclass: use ego helper function")
    Closes: https://bugs.gentoo.org/834758
    Closes: https://bugs.gentoo.org/834762
    Closes: https://bugs.gentoo.org/834607
    Closes: https://bugs.gentoo.org/834608
    Closes: https://bugs.gentoo.org/834625
    Closes: https://bugs.gentoo.org/834626
    Closes: https://bugs.gentoo.org/834630
    Closes: https://bugs.gentoo.org/834669
    Closes: https://bugs.gentoo.org/834603
    Closes: https://bugs.gentoo.org/834610
    Closes: https://bugs.gentoo.org/834711
    Closes: https://bugs.gentoo.org/834763
    Signed-off-by: Florian Schmaus <flow@gentoo.org>
    Closes: https://github.com/gentoo/gentoo/pull/24437
    Signed-off-by: Sam James <sam@gentoo.org>

 eclass/go-module.eclass | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)