-
Notifications
You must be signed in to change notification settings - Fork 16
do not rerun configure if Makefile exists #2
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
Comments
If you want to add an option such as Checking for Makefile existence isn't enough sadly, you'd have to account for other changes such as different with/without options requested. |
What about checking |
Sounds good (also saving the command in the build dir and compare it would work), are you willing to give it a try? |
@Dushistov do you still have the need for this feature? |
long story short: I have to turn off usage of autotools-rs from build.rs and use prebuild binary instead. The more long variant of story: during CI (Continuous Integration) run time to time build failed, because of autoconf/automake build system decides that it need rerun of |
Sounds a bad quirk in rsync, but if you have a tree using it I can have a look. |
It is reproducible on master branch of rsync. For example I build rsync out of tree: $ git clone git://git.samba.org/rsync.git
$ mkdir build-rsync
$ cd build-rsync
$ ../rsync/configure && make then I touch $ touch ../rsync/configure.ac
$ make
autoconf -o configure.sh
autoconf -o configure.sh
autoconf: error: no input file
autoconf: error: no input file Obviously I do not touch any source code of rsync, |
Probably would be a good idea to report the issue to the |
#6 should solve your problem. |
I use simple
build.rs
:and after adding such
build.rs
build becomes really slow,because of it runs
configure
every build.It would be nice check if
Makefile
exists and if so just runmake
.As I know
make
automatically runsconfigure
if something changed.The text was updated successfully, but these errors were encountered: