This is an archive of the discontinued LLVM Phabricator instance.

Discourage in-source autoconf builds (as we already do for the cmake build)
ClosedPublic

Authored by jroelofs on Feb 27 2015, 2:22 PM.

Details

Reviewers
joey
echristo

Diff Detail

Event Timeline

jroelofs updated this revision to Diff 20901.Feb 27 2015, 2:22 PM
jroelofs retitled this revision from to Discourage in-source autoconf builds (as we already do for the cmake build).
jroelofs updated this object.
jroelofs edited the test plan for this revision. (Show Details)
jroelofs added a reviewer: joey.
jroelofs added a subscriber: Unknown Object (MLST).
jroelofs updated this revision to Diff 20905.Feb 27 2015, 3:16 PM
jroelofs added a reviewer: echristo.

Now barfs earlier. Needs autoconf to be run on it, though.

Any benefit to having this in both the configure and the makefile, other than to make it slightly harder to do the wrong thing?

echristo edited edge metadata.Feb 27 2015, 3:17 PM

None that I can think of.

-eric

jroelofs updated this revision to Diff 20907.Feb 27 2015, 3:34 PM
jroelofs edited edge metadata.

@echristo twisted my arm... so I built the appropriate autoconf/automake, and generated a new configure. Also discarded the Makefile.config.in change.

echristo accepted this revision.Feb 27 2015, 3:35 PM
echristo edited edge metadata.

LGTM. Thanks!

-eric

This revision is now accepted and ready to land.Feb 27 2015, 3:35 PM
jroelofs closed this revision.Feb 27 2015, 3:38 PM

r230812