This is an archive of the discontinued LLVM Phabricator instance.

[git-llvm] Only use --force-interactive when supported
ClosedPublic

Authored by smeenai on Mar 8 2019, 3:29 PM.

Details

Summary

The --force-interactive option was introduced in SVN 1.8, and trying to
pass it to older SVN clients causes an error; CentOS 7 includes SVN 1.7,
for example, so this makes git llvm not usable out of the box. Older
clients would be interactive by default anyway [1], so just don't pass
the option if it's not supported.

An alternative would be to check the version instead of checking the
help text, but I think directly detecting the presence of the option is
more direct.

[1] http://svn.apache.org/viewvc?view=revision&revision=1424037

Diff Detail

Repository
rL LLVM

Event Timeline

smeenai created this revision.Mar 8 2019, 3:29 PM
Herald added a project: Restricted Project. · View Herald TranscriptMar 8 2019, 3:29 PM
mehdi_amini accepted this revision.Mar 9 2019, 4:44 PM
This revision is now accepted and ready to land.Mar 9 2019, 4:44 PM
This revision was automatically updated to reflect the committed changes.