Skip to content
This repository was archived by the owner on Dec 4, 2023. It is now read-only.

fix checkout and clean tasks#168

Merged
cowboyd merged 1 commit intomasterfrom
fix-clean-and-checkout
Apr 10, 2015
Merged

fix checkout and clean tasks#168
cowboyd merged 1 commit intomasterfrom
fix-clean-and-checkout

Conversation

@cowboyd
Copy link
Copy Markdown
Collaborator

@cowboyd cowboyd commented Apr 9, 2015

  • for some reason, the git clean task inside the gyp directory was
    returning an exit status of 1 even though it was cleaning out
    files. Just ignore the exit status
  • chomp the output of the git-svn invocation. At least on OSX, it was
    throwing a specious warning about "-f" not being a valid command, and
    if the force checkout was actually necessary, it was failing

* for some reason, the git clean task inside the gyp directory was
  returning an exit status of `1'` even though it was cleaning out
  files. Just ignore the exit status
* chomp the output of the `git-svn` invocation. At least on OSX, it was
  throwing a specious warning about "-f" not being a valid command, and
  if the force checkout was actually necessary, it was failing
@ignisf
Copy link
Copy Markdown
Collaborator

ignisf commented Apr 9, 2015

I've entirely removed the checkout logic from trunk and thus removed the dependency on git-svn and svn... Have to do it here too sometime. Looks good for merging.

@cowboyd
Copy link
Copy Markdown
Collaborator Author

cowboyd commented Apr 10, 2015

Alright, thanks for the heads up.

One of the reason's I wanted to get our builds sorted is so that we can automatically build and store artifacts on all the different platforms so that we can really reduce the effort involved to generate a release.

cowboyd added a commit that referenced this pull request Apr 10, 2015
@cowboyd cowboyd merged commit 46132e8 into master Apr 10, 2015
@ignisf ignisf deleted the fix-clean-and-checkout branch May 13, 2016 00:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants