Update d/rules UPSTREAM regex
Bug #1563098 reported by
Nish Aravamudan
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
phing (Debian) |
Fix Released
|
Unknown
|
|||
phing (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned | ||
php-apigen (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
The UPSTREAM regex in d/rules does not work for longer suffixes.
Changed in php-apigen (Ubuntu): | |
status: | New → Fix Committed |
Changed in phing (Ubuntu): | |
status: | New → Fix Committed |
Changed in phing (Ubuntu): | |
status: | Fix Committed → New |
Changed in php-apigen (Ubuntu): | |
status: | Fix Committed → New |
Changed in phing (Ubuntu): | |
status: | New → Fix Committed |
Changed in php-apigen (Ubuntu): | |
status: | New → Fix Committed |
Changed in phing (Debian): | |
status: | Unknown → New |
Changed in phing (Debian): | |
status: | New → Fix Released |
To post a comment you must log in.
+UPSTREAM := $(shell head -1 debian/changelog | sed 's/.*(/ /;s/-.* ?*).*// ')
That works, but isn't a great fix. The '.?*' from the original, AFAICS, never should have been there, and .*?* is differently buggy... there shouldn't be a literal '?' in the string.
A better fix would be:
+UPSTREAM := $(shell head -1 debian/changelog | sed 's/.*(/ /;s/-[^ )-]\+). *//')