-
-
Notifications
You must be signed in to change notification settings - Fork 12.3k
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
modd: deprecate #84056
modd: deprecate #84056
Conversation
Formula/modd.rb
Outdated
def install | ||
ENV["GOPATH"] = buildpath | ||
ENV["GO111MODULE"] = "auto" | ||
(buildpath/"src/github.com/cortesi/modd").install buildpath.children | ||
cd "src/github.com/cortesi/modd" do | ||
system "go", "build", *std_go_args, "./cmd/modd" | ||
system "dep", "ensure", "-vendor-only" |
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 think we want to go back to dep
, I'd rather deprecate the formula and let it use an older go version until it's eventually removed.
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.
works for me, it's disappointing because master still sees commits just no tagged release in 2+ years. maybe that's common in the go community
🤖 A scheduled task has triggered a merge. |
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 doingbrew install --build-from-source <formula>
)? If this is a new formula, does it passbrew audit --new <formula>
?To use go 1.17 for #83413,
x/sys
needs to be upgraded (see cortesi/modd#103). However, the patch isn't directly usable as the latest release isn't using modules and we've had no response from cortesi/modd#96.I first tried applying cortesi/devd@4ab3fc9, but that fails due to a symlink:
any other alternatives here?