Bug 191064 - Improvements to ports notify_bz post-commit hook
Summary: Improvements to ports notify_bz post-commit hook
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Ports Framework (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Many People
Assignee: Bryan Drewery
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-06-15 15:59 UTC by Alex Kozlov
Modified: 2014-09-11 00:24 UTC (History)
3 users (show)

See Also:

Attachments
improvements to ports notify_bz hook (1020 bytes, patch)
2014-06-15 16:00 UTC, Alex Kozlov 		no flags Details | Diff
slightly improved version. (1013 bytes, patch)
2014-06-18 19:30 UTC, Alex Kozlov 		no flags Details | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Alex Kozlov freebsd_committer freebsd_triage 2014-06-15 15:59:25 UTC 
Make some improvements to ports notify_bz hook:

- Simplify regexp
- Relax match pattern, to restore 'PR: 123456 [1]', 'PR: 123456 (based on)', etc forms
- Process multiple PR: lines
Comment 1 Alex Kozlov freebsd_committer freebsd_triage 2014-06-15 16:00:46 UTC 
Created attachment 143810 [details]
improvements to ports notify_bz hook
Comment 2 Alex Kozlov freebsd_committer freebsd_triage 2014-06-18 19:30:51 UTC 
Created attachment 143909 [details]
slightly improved version.
Comment 3 Bryan Drewery freebsd_committer freebsd_triage 2014-06-19 13:51:48 UTC 
There's also a patch in Bug 190866
Comment 4 commit-hook freebsd_committer freebsd_triage 2014-09-08 16:10:24 UTC 
A commit references this bug:

Author: bdrewery
Date: Mon Sep  8 16:09:47 UTC 2014
New revision: 367646
URL: http://svnweb.freebsd.org/changeset/ports/367646

Log:
  Fix bugzilla dfilter script to handle:
    1. Multiple PR per line
    2. Multiple PR
    3. PR lines with extra stuff after them such as references and (notes)

  PR:		ports/190866 [1]
  PR:		ports/191064 [2]
  Reported by:	many
  Submitted by:	ak, bdrewery

Changes:
  svnadmin/hooks/scripts/notify_bz.sh
Comment 5 commit-hook freebsd_committer freebsd_triage 2014-09-11 00:11:03 UTC 
A commit references this bug:

Author: bdrewery
Date: Thu Sep 11 00:10:55 UTC 2014
New revision: 271412
URL: http://svnweb.freebsd.org/changeset/ports/271412

Log:
  Fix bugzilla dfilter script to handle:
    1. Multiple PR per line
    2. Multiple PR
    3. PR lines with extra stuff after them such as references and (notes)

  PR:		ports/190866 [1]
  PR:		ports/191064 [2]
  Reported by:	many
  Submitted by:	ak, bdrewery
  Acked by:	peter

Changes:
  svnadmin/hooks/scripts/notify_bz.sh
Comment 6 commit-hook freebsd_committer freebsd_triage 2014-09-11 00:20:07 UTC 
A commit references this bug:

Author: bdrewery
Date: Thu Sep 11 00:19:38 UTC 2014
New revision: 45589
URL: http://svnweb.freebsd.org/changeset/ports/45589

Log:
  Fix bugzilla dfilter script to handle:
    1. Multiple PR per line
    2. Multiple PR
    3. PR lines with extra stuff after them such as references and (notes)

  PR:		190866 [1]
  PR:		191064 [2]
  Reported by:	many
  Submitted by:	ak, bdrewery
  Approved by:	doceng (gjb)

Changes:
  svnadmin/hooks/scripts/notify_bz.sh
Comment 7 Bryan Drewery freebsd_committer freebsd_triage 2014-09-11 00:24:21 UTC 
Fixed in ports/base/doc.