This is an archive of the discontinued LLVM Phabricator instance.

cmake: Remove check for files not in build list.
ClosedPublic

Authored by arsenm on Jul 17 2014, 11:58 PM.

Details

Reviewers
chandlerc
Summary

I don't see what value this provides. It does globbing for
files to build, and still requires manually listing the files.
emacs by default creates backup files in the same directory if you
make changes without saving, resulting in really irritating
build errors from the "found unknown source file .#Foo.cpp"

I also don't think it's consistently used, since this
doesn't seems to happen for some subset of directories.

Diff Detail

Event Timeline

arsenm updated this revision to Diff 11635.Jul 17 2014, 11:58 PM
arsenm retitled this revision from to cmake: Remove check for files not in build list. .
arsenm updated this object.
arsenm edited the test plan for this revision. (Show Details)
arsenm added a subscriber: Unknown Object (MLST).
arsenm updated this revision to Diff 12218.Aug 5 2014, 6:06 PM

Insert a check for backup files instead. Skip files that begin with the normal "." for unix hidden files

This looks like it matches our consensus, but I'm not entirely
comfortable signing off on cmake stuff. Can any build gurus take a look
at this and confirm it does what the comment says?

chandlerc accepted this revision.Sep 2 2014, 12:52 PM
chandlerc added a reviewer: chandlerc.
chandlerc added a subscriber: chandlerc.

Please commit with the condition rewritten as suggested or something similar (provided that you test it works, I didn't try out my cmake scribble here).

cmake/modules/LLVMProcessSources.cmake
66

While regexs are a bit heavy handed, i think this is simpler as:

if (NOT "${fn}" MATCHES "^\.")

Otherwise, this looks fine.

This revision is now accepted and ready to land.Sep 2 2014, 12:52 PM
arsenm closed this revision.Sep 2 2014, 1:30 PM

r216941 with the regex (which needed another backslash)