Skip to content

buildflags: fix missing cache type error typo#3969

Merged
thaJeztah merged 1 commit into
docker:masterfrom
crazy-max:fix-cache-type-error-message
Jul 23, 2026
Merged

buildflags: fix missing cache type error typo#3969
thaJeztah merged 1 commit into
docker:masterfrom
crazy-max:fix-cache-type-error-message

Conversation

@crazy-max

Copy link
Copy Markdown
Member

The cache flag parser reported form> instead of for when type= was omitted from cache options.

Signed-off-by: CrazyMax <1951866+crazy-max@users.noreply.github.com>

@thaJeztah thaJeztah left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Comment thread util/buildflags/cache.go
default:
text, _ = json.Marshal(gv)
}
return errors.Errorf("type required form> %q", string(text))

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are you sure it wasn't meant to be <form>? 🤣

@thaJeztah
thaJeztah merged commit 05a1121 into docker:master Jul 23, 2026
161 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants