-
-
Notifications
You must be signed in to change notification settings - Fork 12.4k
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
crossplane 1.17.1 (new formula) #192300
base: master
Are you sure you want to change the base?
crossplane 1.17.1 (new formula) #192300
Conversation
9fb390a
to
de1b848
Compare
de1b848
to
b15e86e
Compare
b15e86e
to
3908675
Compare
Formula/c/crossplane.rb
Outdated
depends_on "go" => :build | ||
|
||
def install | ||
system "go", "build", *std_go_args(ldflags: "-s -w -X=github.com/crossplane/crossplane/internal/version.version=v#{version}", output: bin/"crossplane"), "./cmd/crank" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why just build crank, but not crossplane as well?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't quite understand what's the use of crossplane binary for normal users. Following https://docs.crossplane.io/latest/cli/ the shell script referenced installs only the crank
binary
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Crank is the crossplane cli, crossplane is the actual crossplane binary, so in this context only crank makes sense indeed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the crossplane
binary runs in an init container to install some crossplane CRDs to a cluster where someone is installing Crossplane via Helm. It does not seems to be intended for running on local machines.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Both as init container and main container. I confirm it's not meant to be run locally.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sounds good then, just checking 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(related, would it make more sense to have the formula as crank, rather than crossplane then?)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Crank is just an internal name, the cli should be called crossplane, so crossplane is correct 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So brew install crossplane
will mean that there is no crossplane
binary, just a crank
one?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
brew install crossplane
will install the crank
binary as crossplane
.
b966d9e
to
b1e9575
Compare
b1d397b
to
2df5389
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
need to squash the commits
Signed-off-by: Moulick Aggarwal <[email protected]>
eab5eae
to
ac09bc9
Compare
done! |
HOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>
, where<formula>
is the name of the formula you're submitting?brew test <formula>
, where<formula>
is the name of the formula you're submitting?brew audit --strict <formula>
(after doingHOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>
)? If this is a new formula, does it passbrew audit --new <formula>
?crossplane/crossplane#4899