Homec4science

(stable) Fix two issues with repository monogram regular expressions

Authored by epriestley <git@epriestley.com> on Jan 18 2016, 18:32.

Description

(stable) Fix two issues with repository monogram regular expressions

Summary: Ref T4245. Fixes T10172. These regular expressions were simply incorrect: they intend <start> (form one | form two) <end> but were written as (<start> form one) | (form two <end>) which allowed stuff like "R2/R13" to be interpreted as a monogram because it matches (<start> form one).

Test Plan: Parsed commit ba46ffa6169c from RTEMS repository, see T10172. Before patch, got an identical trace; after patch, clean import.

Reviewers: chad, avivey

Reviewed By: avivey

Maniphest Tasks: T4245, T10172

Differential Revision: https://secure.phabricator.com/D15049

Details

Committed
epriestley <git@epriestley.com>Jan 20 2016, 13:26
Pushed
aubortJan 31 2017, 17:16
Parents
rPHeac1124bd954: (stable) Promote 2016 Week 3
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPHba2215986639: (stable) Fix two issues with repository monogram regular expressions (authored by epriestley <git@epriestley.com>).Jan 20 2016, 13:26