This is an archive of the discontinued LLVM Phabricator instance.

[cmake] Remove deprecated GetSVN.cmake parameters
Changes PlannedPublic

Authored by modocache on Jul 7 2017, 11:32 AM.

Details

Summary

Depends on https://reviews.llvm.org/D35133.

As of https://reviews.llvm.org/D35133, Clang will have migrated to the
new parameter lists in GetSVN.cmake: SOURCE_DIRS and NAMES. Remove
the old parameters, FIRST_SOURCE_DIR/FIRST_NAME and
SECOND_SOURCE_DIR/SECOND_NAME.

Other external projects using LLVM's GetSVN.cmake, such as Apple's
Swift, will also be forced to migrate.

Test Plan:

  1. Perform a clean build of Clang, verify that clang --version still prints the correct LLVM and Clang revision information.
  2. Apply this patch, then perform another clean build. clang --version should still print the correct revision information.

Event Timeline

modocache created this revision.Jul 7 2017, 11:32 AM
jordan_rose edited edge metadata.Jul 18 2017, 2:41 PM

Rather than delete these outright, it's probably polite to error on the presence of FIRST_SOURCE_DIR. There might be other out-of-tree projects that need to update.

modocache planned changes to this revision.Jul 19 2017, 9:48 AM

Good idea, thanks! I'll do so.