Closed Bug 2078366 Opened 11 days ago Closed 10 days ago

AttributeError fetching revision with Jujutsu

Categories

(Conduit :: moz-phab, defect)

defect

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

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.

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!

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.

Attachment

General

Created:
Updated:
Size: