Skip to content
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

Warn instead of fail on invaild paths in tarball #39

Open
wants to merge 1 commit into
base: v1
Choose a base branch
from

Conversation

gwatts
Copy link

@gwatts gwatts commented Jan 25, 2019

The build of Go 1.11.5 has some harmless build artifacts in the tarball;
emit a warning while iterating over those, rather than failing
completely.

See
https://groups.google.com/d/msg/golang-announce/mVeX35iXuSw/HxYXaBnYEAAJ

The build of Go 1.11.5 has some harmless build artifacts in the tarball;
emit a warning while iterating over those, rather than failing
completely.

See
https://groups.google.com/d/msg/golang-announce/mVeX35iXuSw/HxYXaBnYEAAJ
@gebi
Copy link

gebi commented Feb 1, 2019

@gwatts thx for your patch :)!

I implemented it a bit differently in our fork of godeb, to not shadow potential errors in the future with warning messages with and an eye towards automated CI systems.

#40 - ( To not create warnings which could be overseen in automated CI pipelines, i created a patch to explicitely ignore the superfluous artifacts included in go1.11.5 release tarballs instead of just downgrading all errors from unknown content to warnings. )

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants