#31884 closed defect (fixed)

Define ExecuteBash in the Appveyor error block

Reported by: teor Owned by: teor
Priority: Medium Milestone: Tor: 0.3.5.x-final
Component: Core Tor/Tor Version: Tor:
Severity: Normal Keywords: tor-ci-fail-rarely, 042-can, 035-backport, 040-backport, 041-backport, 042-backport, consider-backport-after-ci-passes, BugSmashFund
Cc: catalyst Actual Points: 0.1
Parent ID: Points: 0.1
Reviewer: asn Sponsor:


When Appveyor fails before ExecuteBash is defined, it skips straight to the error block. But the error block wants ExecuteBash. So we should define it twice.


Child Tickets

Change History (8)

comment:1 Changed 13 months ago by teor

Anyone can fix this bug, but I'll try to to it on Monday.

comment:2 Changed 13 months ago by teor

Actual Points: 0.1
Keywords: consider-backport-after-0423 BugSmashFund added
Status: assignedneeds_review
Version: Tor: unspecifiedTor:

See my PR:

The test merges are all clean, the branches are here:

comment:3 Changed 13 months ago by asn

Reviewer: asn

comment:4 Changed 13 months ago by asn

Status: needs_reviewmerge_ready


comment:5 Changed 13 months ago by nickm

Milestone: Tor: 0.4.2.x-finalTor: 0.4.1.x-final

Merged to master, marking for backport.

comment:6 Changed 13 months ago by teor

Keywords: consider-backport-after-ci-passes added; consider-backport-after-0423 removed

This is a CI issue, so we can backport it as soon as CI passes,

comment:7 Changed 12 months ago by nickm

Milestone: Tor: 0.4.1.x-finalTor: 0.3.5.x-final

Merged to 0.3.5 and forward.

comment:8 Changed 12 months ago by teor

Resolution: fixed
Status: merge_readyclosed
Note: See TracTickets for help on using tickets.