This is an archive of the discontinued LLVM Phabricator instance.

Clean up docs references to './configure' in preparation for deprecating in-source builds
ClosedPublic

Authored by jroelofs on Apr 1 2015, 3:17 PM.

Details

Summary

There's a couple of remaining ones that I didn't touch:

./docs/GoldPlugin.rst:163:     % ./configure && make && make check
./docs/LibFuzzer.rst:88:  (cd pcre; ./autogen.sh; CC="clang -fsanitize=address $COV_FLAGS" ./configure --prefix=`pwd`/../inst && make -j && make install)
./llvm.spec.in:28:./configure \

The first is for building Gold, which I'm not familiar enough with to make the change (AFAICT it's referring to a different configure file anyway).
The second I believe is referring to a different configure file. Not sure though... @kcc?
I don't know what the third file is.

Diff Detail

Event Timeline

jroelofs updated this revision to Diff 23091.Apr 1 2015, 3:17 PM
jroelofs retitled this revision from to Clean up docs references to './configure' in preparation for deprecating in-source builds.
jroelofs updated this object.
jroelofs edited the test plan for this revision. (Show Details)
jroelofs added reviewers: echristo, nicholas.
jroelofs added subscribers: Unknown Object (MLST), kcc.
kcc added a comment.Apr 1 2015, 3:26 PM

The second I believe is referring to a different configure file. Not sure though... @kcc?

Yes, this ./configure is from a totally different package and is here just as an example.

echristo edited edge metadata.Apr 1 2015, 11:26 PM

Inline question, otherwise feel free to have at with the changes for this.

-eric

docs/FAQ.rst
105

Can we use a $LLVM_SRC_DIR here as well?

jroelofs accepted this revision.Apr 29 2015, 1:10 PM
jroelofs added a reviewer: jroelofs.

r236144

This revision is now accepted and ready to land.Apr 29 2015, 1:10 PM
jroelofs closed this revision.Apr 29 2015, 1:10 PM