Closed
Bug 2078366
Opened 11 days ago
Closed 10 days ago
AttributeError fetching revision with Jujutsu
Categories
(Conduit :: moz-phab, defect)
Conduit
moz-phab
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: ahal, Assigned: ahal)
Details
Attachments
(1 file)
I hit an exception while running:
moz-phab patch D327415 --apply-to base --yes --name phab-D327415
| Assignee | ||
Comment 1•11 days ago
|
||
The wrapped Git repo reads args beyond what get_diff needs, e.g. upstream
when picking remotes in is_public and get_latest_landing_node. Until now
the args were only assigned to it inside get_diff, so calls made before
that (such as public-ancestor checks while validating the stack in submit)
ran without args and could fail or ignore --upstream.
Assign the args in set_args so the Git repo has them from the start, and
add a test covering it.
Updated•11 days ago
|
Attachment #9653201 -
Attachment description: WIP: Bug 2078366 - share parsed args with the wrapped Git repo in set_args → Bug 2078366 - share parsed args with the wrapped Git repo in set_args, r?sheehan!,shtrom!
Comment 2•10 days ago
|
||
Authored by https://github.com/ahal
https://github.com/mozilla-conduit/review/commit/45b85378c33b612a01164280d2e324283ae23cd9
[main] Bug 2078366 - share parsed args with the wrapped Git repo in set_args, r=sheehan
(via GitHub webhook, authenticated as shared-secret)
Status: ASSIGNED → RESOLVED
Closed: 10 days ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•