Ubuntu
php-apigen package

Update d/rules UPSTREAM regex

Bug #1563098 reported by Nish Aravamudan
8
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.

Revision history for this message
Steve Langasek (vorlon) wrote :

+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/-[^)-]\+).*//')

Steve Langasek (vorlon)
Changed in php-apigen (Ubuntu):
status: New → Fix Committed
Changed in phing (Ubuntu):
status: New → Fix Committed
Nish Aravamudan (nacc)
Changed in phing (Ubuntu):
status: Fix Committed → New
Changed in php-apigen (Ubuntu):
status: Fix Committed → New
Revision history for this message
Nish Aravamudan (nacc) wrote :
Revision history for this message
Nish Aravamudan (nacc) wrote :
Revision history for this message
Nish Aravamudan (nacc) wrote :

Thanks for the help on IRC, Steve, I believe these regex are correct, and match what some other packages do.

Steve Langasek (vorlon)
Changed in phing (Ubuntu):
status: New → Fix Committed
Changed in php-apigen (Ubuntu):
status: New → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package phing - 2.13.0-2ubuntu2

---------------
phing (2.13.0-2ubuntu2) xenial; urgency=medium

  * Correctly update regex (LP: #1563098).

 -- Nishanth Aravamudan <email address hidden> Tue, 29 Mar 2016 13:20:00 -0700

Changed in phing (Ubuntu):
status: Fix Committed → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package php-apigen - 4.1.2-1ubuntu2

---------------
php-apigen (4.1.2-1ubuntu2) xenial; urgency=medium

  * Correctly update regex (LP: #1563098).

 -- Nishanth Aravamudan <email address hidden> Tue, 29 Mar 2016 13:20:40 -0700

Changed in php-apigen (Ubuntu):
status: Fix Committed → Fix Released
Bug Watch Updater (bug-watch-updater)
Changed in phing (Debian):
status: Unknown → New
Bug Watch Updater (bug-watch-updater)
Changed in phing (Debian):
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.