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

	F19506682: D7439.id16806.diff
	Fri, Jan 9, 4:50 PM

	F19053503: D7439.id16761.diff
	Nov 28 2025, 9:10 AM

	F18816867: D7439.id.diff
	Oct 21 2025, 10:01 AM

	F18806895: D7439.diff
	Oct 18 2025, 9:57 PM

	F18804259: D7439.diff
	Oct 18 2025, 7:03 AM

	F18672801: D7439.id16806.diff
	Sep 25 2025, 8:16 AM

	F18672549: D7439.id.diff
	Sep 25 2025, 7:44 AM

	F18671801: D7439.diff
	Sep 25 2025, 5:21 AM

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