Skip to content
This repository has been archived by the owner on Dec 29, 2021. It is now read-only.

Reduce needless recompiles #40

Merged
merged 1 commit into from
Sep 24, 2017
Merged

Reduce needless recompiles #40

merged 1 commit into from
Sep 24, 2017

Conversation

epage
Copy link
Collaborator

@epage epage commented Sep 24, 2017

Build script calls skeptic which emits information asking cargo to
monitor content of README.md.skt.md for changes. If changes to that file
are detected cargo should rebuild this crate. In case of this project
the files does not exist. This causes cargo to trigger a rebuild every
time in hope that it will get created by the build script.

The workaround is to create the file that skeptic asks to monitor which
stops cargo from triggering the rebuild.

For details consult this issues:

Build script calls skeptic which emits information asking cargo to
monitor content of README.md.skt.md for changes. If changes to that file
are detected cargo should rebuild this crate.  In case of this project
the files does not exist. This causes cargo to trigger a rebuild every
time in hope that it will get created by the build script.

The workaround is to create the file that skeptic asks to monitor which
stops cargo from triggering the rebuild.

For details consult this issues:

* budziq/rust-skeptic#25
* rust-lang/cargo#4213
@epage
Copy link
Collaborator Author

epage commented Sep 24, 2017

bors: +r

@bors
Copy link
Contributor

bors bot commented Sep 24, 2017

Did you mean "r+"?

@epage
Copy link
Collaborator Author

epage commented Sep 24, 2017

bors: r+

bors bot added a commit that referenced this pull request Sep 24, 2017
40: Reduce needless recompiles r=epage a=epage

Build script calls skeptic which emits information asking cargo to
monitor content of README.md.skt.md for changes. If changes to that file
are detected cargo should rebuild this crate.  In case of this project
the files does not exist. This causes cargo to trigger a rebuild every
time in hope that it will get created by the build script.

The workaround is to create the file that skeptic asks to monitor which
stops cargo from triggering the rebuild.

For details consult this issues:

* budziq/rust-skeptic#25
* rust-lang/cargo#4213
@bors
Copy link
Contributor

bors bot commented Sep 24, 2017

Build succeeded

@bors bors bot merged commit ccc8af5 into assert-rs:master Sep 24, 2017
@epage epage deleted the skeptic branch September 24, 2017 03:34
@killercup
Copy link
Collaborator

Let me highjack this merged PR to thank you for releasing 0.5 and 0.5.1! <3

@epage
Copy link
Collaborator Author

epage commented Sep 25, 2017

You're welcome!

I'm curious, what do you see needing done before moving this to 1.0? While I should get back to my regularly schedule project, I would like to also help work towards more crates hitting 1.0.

@killercup
Copy link
Collaborator

killercup commented Sep 25, 2017

@killercup killercup mentioned this pull request Sep 25, 2017
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