Skip to content

Conversation

@QuentinPerez
Copy link
Contributor

fix #210

(buildpath/"src/github.com/scaleway/scaleway-cli").install Dir["*"]

system "go", "build", "-o", "#{bin}/scw", "-v", "-ldflags", "-X github.com/scaleway/scaleway-cli/pkg/scwversion.GITCOMMIT=homebrew", "github.com/scaleway/scaleway-cli/cmd/scw/"
system "rm", "-f", "~/.scw-cache.db"

Choose a reason for hiding this comment

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

Inconsistent indentation detected.
Tab detected.

(buildpath/"src/github.com/scaleway/scaleway-cli").install Dir["*"]

system "go", "build", "-o", "#{bin}/scw", "-v", "-ldflags", "-X github.com/scaleway/scaleway-cli/pkg/scwversion.GITCOMMIT=homebrew", "github.com/scaleway/scaleway-cli/cmd/scw/"
system "rm", "-f", "~/.scw-cache.db"
Copy link
Contributor

Choose a reason for hiding this comment

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

add a comment, so the homebrew team won't be afraid by this addition :)


system "go", "build", "-o", "#{bin}/scw", "-v", "-ldflags", "-X github.com/scaleway/scaleway-cli/pkg/scwversion.GITCOMMIT=homebrew", "github.com/scaleway/scaleway-cli/cmd/scw/"
# we remove our .scw-cache.db file, to clean the cache of scw
# this file is created and handled by scw

Choose a reason for hiding this comment

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

Incorrect indentation detected (column 1 instead of 4).
Tab detected.

@QuentinPerez
Copy link
Contributor Author

Done

@moul
Copy link
Contributor

moul commented Mar 18, 2016

LGTM

moul added a commit that referenced this pull request Mar 18, 2016
@moul moul merged commit ae75f89 into scaleway:master Mar 18, 2016
@moul moul removed the in progress label Mar 18, 2016
@QuentinPerez QuentinPerez deleted the brew branch October 27, 2016 13:43
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.

Update brew script to _flush-cache after install

3 participants