Skip to content
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

Change steps.Git(… method="fresh") to method="clean" #611

Merged
merged 1 commit into from Mar 3, 2017
Merged

Conversation

@SimonSapin
Copy link
Member

SimonSapin commented Mar 3, 2017

http://docs.buildbot.net/latest/manual/cfg-buildsteps.html#git

This will keep the target directory between builds, and hopefully reduce build times since not every build will need to recompile everything.


This change is Reviewable

http://docs.buildbot.net/latest/manual/cfg-buildsteps.html#git

This will keep the `target` directory between builds, and hopefully reduce build times since not every build will need to recompile everything.
@highfive
Copy link

highfive commented Mar 3, 2017

Thanks for the pull request, and welcome! The Servo team is excited to review your changes, and you should hear from @aneeshusa (or someone else) soon.

@metajack
Copy link
Contributor

metajack commented Mar 3, 2017

@bors-servo
Copy link
Contributor

bors-servo commented Mar 3, 2017

📌 Commit 37f65ca has been approved by metajack

@bors-servo
Copy link
Contributor

bors-servo commented Mar 3, 2017

Testing commit 37f65ca with merge a51202b...

bors-servo added a commit that referenced this pull request Mar 3, 2017
Change steps.Git(… method="fresh") to method="clean"

http://docs.buildbot.net/latest/manual/cfg-buildsteps.html#git

This will keep the `target` directory between builds, and hopefully reduce build times since not every build will need to recompile everything.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/saltfs/611)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Mar 3, 2017

☀️ Test successful - status-travis
Approved by: metajack
Pushing a51202b to master...

@bors-servo bors-servo merged commit 37f65ca into master Mar 3, 2017
2 of 3 checks passed
2 of 3 checks passed
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
homu Test successful
Details
@SimonSapin SimonSapin deleted the git-clean branch Mar 4, 2017
@SimonSapin
Copy link
Member Author

SimonSapin commented Mar 4, 2017

@metajack Was this deployed? Recent builds are still running git clean -f -f -d -x in their "git update" step. For example: http://build.servo.org/builders/mac-rel-css/builds/5996/steps/git/logs/stdio

@metajack
Copy link
Contributor

metajack commented Mar 6, 2017

I did the deploy, but perhaps the buildbots don't restart automatically? I will try and confirm.

@SimonSapin
Copy link
Member Author

SimonSapin commented Mar 6, 2017

Looks good now.

@SimonSapin SimonSapin restored the git-clean branch Mar 6, 2017
@SimonSapin SimonSapin deleted the git-clean branch Mar 31, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

5 participants
You can’t perform that action at this time.