mirror of
https://github.com/ko-build/ko.git
synced 2024-12-03 08:35:34 +02:00
ff61ea330c
This change more or less completely changes how `ko://` is handled internally to `ko`, but the user-facing changes should only be net-positive. `ko://` was previously stripped at the highest level, and the build logic was unaware, which had some undesirable diagnostic/functional implications that are collectively addressed in this change. With this change, the `ko://` prefix is preserved and passed to the build logic, which internally parses a new `reference` type (this was useful to have Go's type checker find all of the places that needed fixing). The main functional differences are: 1. If a reference is prefixed with `ko://` we will now fail fast in `IsSupportedReference` regardless of whether `--strict` is passed. 2. If a reference is prefixed with `ko://` it will bypass the prefix check, which allows the use of `ko://github.com/another/repo` that references a vendored binary package. For `2.` the absence of the module prefix causes the filtering logic Jon introduced to avoid the reference. This was critical for efficiency when `ko://` isn't around because we feed every string in the yaml through it, but when the user has explicitly decorated things it's the perfect thing to be sensitive to. Fixes: https://github.com/google/ko/issues/146 Fixes: https://github.com/google/ko/issues/152
50 lines
1.2 KiB
Go
50 lines
1.2 KiB
Go
// Copyright 2020 Google LLC All Rights Reserved.
|
|
//
|
|
// Licensed under the Apache License, Version 2.0 (the "License");
|
|
// you may not use this file except in compliance with the License.
|
|
// You may obtain a copy of the License at
|
|
//
|
|
// http://www.apache.org/licenses/LICENSE-2.0
|
|
//
|
|
// Unless required by applicable law or agreed to in writing, software
|
|
// distributed under the License is distributed on an "AS IS" BASIS,
|
|
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
// See the License for the specific language governing permissions and
|
|
// limitations under the License.
|
|
|
|
package build
|
|
|
|
import "strings"
|
|
|
|
// StrictScheme is a prefix that can be placed on import paths that users
|
|
// think MUST be supported references.
|
|
const StrictScheme = "ko://"
|
|
|
|
type reference struct {
|
|
strict bool
|
|
path string
|
|
}
|
|
|
|
func newRef(s string) reference {
|
|
return reference{
|
|
strict: strings.HasPrefix(s, StrictScheme),
|
|
path: strings.TrimPrefix(s, StrictScheme),
|
|
}
|
|
}
|
|
|
|
func (r reference) IsStrict() bool {
|
|
return r.strict
|
|
}
|
|
|
|
func (r reference) Path() string {
|
|
return r.path
|
|
}
|
|
|
|
func (r reference) String() string {
|
|
if r.IsStrict() {
|
|
return StrictScheme + r.Path()
|
|
} else {
|
|
return r.Path()
|
|
}
|
|
}
|