This is an archive of the discontinued LLVM Phabricator instance.

[lldb] Ignore various test artifacts (NFCI)
ClosedPublic

Authored by vsk on Jul 29 2016, 5:17 PM.

Reviewers

Commits

vsk retitled this revision from to [lldb] Ignore various test artifacts (NFCI).

vsk updated this object.

vsk added a reviewer: tfiala.

vsk added a subscriber: llvm-commits.

tfiala requested changes to this revision.Jul 29 2016, 5:34 PM

tfiala edited edge metadata.

tfiala added inline comments.

.gitignore
38 ↗	(On Diff #66206)	Fine
39 ↗	(On Diff #66206)	Fine.
40 ↗	(On Diff #66206)	We don't want this one. Even though some tests are being bad (eh, hem, test writers) and leaving their junk around (typically the auto-generated tests like inlines), we do have plenty of Makefile files in the test subdirectory that are part of the repo and are modified. Putting this here will create some level of chaos when changes are made to those Makefiles locally and are masked out by this .gitignore. If you don't plan on changing Makefile content yourself, you could go ahead and put this line in your user's global gitignore file if you set one up.

This revision now requires changes to proceed.Jul 29 2016, 5:34 PM

tfiala added a subscriber: lldb-commits.

Looks good!

We're working on cleaning up the makefiles. Thanks for pointing that out.

This revision is now accepted and ready to land.Jul 29 2016, 9:14 PM

This revision was automatically updated to reflect the committed changes.

Path

Size

lldb/

trunk/

2 lines

	Show All 29 Lines

	build/			build/
	pyproj/			pyproj/
	llvm-build/			llvm-build/
	ninja/			ninja/
	*xcuserdata			*xcuserdata
	test/20*			test/20*
	__pycache__/			__pycache__/
				*.lock
				*.so

	clang-module-cache			clang-module-cache

	# We should ignore Xcode-style embedding of llvm/ at lldb root dir.			# We should ignore Xcode-style embedding of llvm/ at lldb root dir.
	# Do not add trailing '/'s, they skip symlinks.			# Do not add trailing '/'s, they skip symlinks.
	/llvm			/llvm
	/DerivedData			/DerivedData