Raise a setup warning for missing or invalid local repository directory
ClosedPublic
Actions

Authored by epriestley on Oct 30 2013, 12:03 AM.

Tags

None

Referenced Files

	F14024749: D7439.diff
	Thu, Nov 7, 12:03 PM

	F13988848: D7439.id16806.diff
	Mon, Oct 21, 4:19 PM

	F13981948: D7439.id16761.diff
	Sat, Oct 19, 8:10 PM

	Unknown Object (File)
	Fri, Oct 11, 3:57 PM

	Unknown Object (File)
	Fri, Oct 11, 3:57 PM

	Unknown Object (File)
	Fri, Oct 11, 3:57 PM

	Unknown Object (File)
	Oct 3 2024, 5:27 PM

	Unknown Object (File)
	Oct 3 2024, 5:23 PM

View All Files

Subscribers

aran

Details

Reviewers

btrahan

Commits

Restricted Diffusion Commit
rPf08908ff352f: Raise a setup warning for missing or invalid local repository directory

Summary

I'm planning to add more detailed info to Diffusion itself, but catch the big issue here.

Test Plan

Hit config issue locally, then resolved it.

Diff Detail

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

btrahan accepted this revision.Oct 30 2013, 7:28 PM

btrahan added inline comments.

src/applications/config/check/PhabricatorSetupCheckRepositories.php
20–33	This block got me thinking it would be very nice to tell permissions vs non-existence, but I don't see a way to actually do that after a brief foray into the code. Oh wells.

We do have some code which does it in PhabricatorSetupCheckPath, but it's a lot of code and I think it's hard to generalize the behavior and error messages:

https://secure.phabricator.com/diffusion/P/browse/master/src/applications/config/check/PhabricatorSetupCheckPath.php;90123dd7392f673d$33-95