bug patterns need to escape parentheses
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Apport |
Invalid
|
Undecided
|
Unassigned | ||
apport (Ubuntu) |
Invalid
|
Low
|
Unassigned |
Bug Description
We've discovered that when writing bug patterns it isn't necessary to escape parens "()" in the Title and OriginalTitle key of bug patterns. For example:
<pattern url="https:/
<re key="Package"
<re key="OriginalTi
<re key="Traceback"
</pattern>
The pattern above matches when used with test-local from the bugpatterns branch (rev 329) and so does:
<pattern url="https:/
<re key="Package"
<re key="OriginalTi
<re key="Traceback"
</pattern>
However, if one were to remove the escaping of parens in the Traceback key of the pattern the pattern would not match. This makes little sense and inspecting the code of both test-local and the "_check_
ProblemType: Bug
DistroRelease: Ubuntu 11.10
Package: apport 1.22.1-0ubuntu2
ProcVersionSign
Uname: Linux 3.0.0-10-generic x86_64
ApportLog:
ApportVersion: 1.22.1-0ubuntu2
Architecture: amd64
CheckboxSubmission: fee5e196cb921cb
CheckboxSystem: 2a6f54df59af338
Date: Fri Sep 9 08:26:57 2011
PackageArchitec
ProcEnviron:
PATH=(custom, user)
LANG=en_US.UTF-8
SHELL=/bin/zsh
SourcePackage: apport
UpgradeStatus: Upgraded to oneiric on 2011-09-06 (2 days ago)
Changed in apport (Ubuntu): | |
status: | New → Confirmed |
summary: |
- TItle and OriginalTitle field of bugpatterns behaves oddly + Title and OriginalTitle field of bugpatterns behaves oddly |
Oh bug 832745 is actually a bad test with test-local as it doesn't have an OriginalTitle - you'll want to use a duplicate like bug 845654 to really test the pattern.