Paths

Table of Contentst

Update path in startup error
ClosedPublic
Actions

Authored by fgaz on Oct 3 2023, 12:51.

Tags

None

Referenced Files

	F2963412: D25445.1739204360.diff
	Sun, Feb 9, 16:19

	F2961362: D25445.1739162657.diff
	Sun, Feb 9, 04:44

	F2961360: D25445.1739162650.diff
	Sun, Feb 9, 04:44

	F2961354: D25445.1739162646.diff
	Sun, Feb 9, 04:44

	F2949619: D25445.1738642596.diff
	Mon, Feb 3, 04:16

	F2948271: D25445.1738541564.diff
	Sun, Feb 2, 00:12

	F2941069: D25445.1738087718.diff
	Mon, Jan 27, 18:08

	F2933374: D25445.1737764926.diff
	Fri, Jan 24, 00:28

Subscribers

Details

Reviewers

valerio.bozzolan

Group Reviewers

O1: Blessed Committers

Commits

rP71b273a62281: Update path in startup error

Summary

The error mentioned a "phabricator" directory, but according to the docs it has been migrated to "phorge".

Test Plan

Move the "arcanist" directory somewhere else, run something from phorge/bin/.

Diff Detail

Repository

rP Phorge

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

fgaz created this revision.Oct 3 2023, 12:51

Owners added a reviewer: O1: Blessed Committers.Oct 3 2023, 12:51

Herald added subscribers: Cigaryno, Matthew, valerio.bozzolan, tobiaswiese. · View Herald TranscriptOct 3 2023, 12:51

fgaz requested review of this revision.Oct 3 2023, 12:51

Harbormaster completed remote builds in B858: Diff 1406.Oct 3 2023, 12:51

Thanks!

I'm inclined to accept this, premising that usually we adopt PlatformSymbols::getPlatformServerPath() but in this case that will not work since this error is very low-level about the load of Arcanist itself.

Let's wait for more eyes on this :)

I accept but please wait at least one day before landing :) Thanks for your code contribution!

This revision is now accepted and ready to land.Oct 3 2023, 12:55

Closed by commit rP71b273a62281: Update path in startup error. · Explain WhyOct 5 2023, 08:36

This revision was automatically updated to reflect the committed changes.

fgaz added a commit: rP71b273a62281: Update path in startup error.

Revision Contents
Changeset List

Path

Size

Packages

support/

startup/

PhabricatorStartup.php

2 lines

O1: Blessed Committers

Diff 1407

View Options

support/startup/PhabricatorStartup.php

Update path in startup errorClosedPublicActions