Page MenuHomePhabricator
Differential D21730

Remove "bin/celerity" dependency on database setup
ClosedPublic
Actions

Authored by epriestley on Nov 19 2021, 5:48 PM.
Tags
None
Referenced Files
F14091043: D21730.diff
Sun, Nov 24, 9:56 PM
F14090556: D21730.diff
Sun, Nov 24, 6:47 PM
Unknown Object (File)
Thu, Nov 21, 6:34 AM
Unknown Object (File)
Sun, Nov 17, 2:02 AM
Unknown Object (File)
Thu, Nov 14, 5:30 AM
Unknown Object (File)
Thu, Nov 14, 1:17 AM
Unknown Object (File)
Wed, Nov 13, 5:58 AM
Unknown Object (File)
Sat, Nov 9, 4:48 PM
View All Files
Subscribers
None

Details

Reviewers
None
Commits
rP8103d361774c: Remove "bin/celerity" dependency on database setup
Summary

Ref PHI2157. Like other low-level tools, "bin/celerity" does not need databases configured in order to execute.

Test Plan

Ran bin/celerity map with and without the database available.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Nov 19 2021, 5:48 PM
Harbormaster returned this revision to the author for changes because remote builds failed.Nov 19 2021, 5:48 PM
Harbormaster failed remote builds in B25580: Diff 51785!
Harbormaster completed remote builds in B25580: Diff 51785.Nov 19 2021, 5:58 PM
epriestley requested review of this revision.Nov 19 2021, 5:58 PM
epriestley added a comment.Nov 19 2021, 6:10 PM

See also D19941, which removed a dependency that did previously exist.

This revision was not accepted when it landed; it landed in state Needs Review.Nov 19 2021, 6:11 PM
Closed by commit rP8103d361774c: Remove "bin/celerity" dependency on database setup (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.
epriestley added a commit: rP8103d361774c: Remove "bin/celerity" dependency on database setup.

Revision Contents
Changeset List

PathSize
scripts/
setup/
2 lines

Diff 51786

View Options

scripts/setup/manage_celerity.php

Loading...
Phabricator · Built by Phacility