Skip Menu |

This queue is for tickets about the SVN-Notify-Config CPAN distribution.

Report information
The Basics
Id: 38421
Status: resolved
Priority: 0/
Queue: SVN-Notify-Config

Owner: Nobody in particular
Requestors: grousse [...]

Bug Information
Severity: Normal
Broken in: 0.0911
Fixed in: (no value)

Subject: unable to use svnnotify --add-header option
Download (untitled) / with headers
text/plain 461b
This simple configuration file doesn't pass the 'add-header' option to svnnotify: #!/usr/bin/perl -MSVN::Notify::Config=$0 --- #YAML:1.0 '': PATH: "/usr/bin:/usr/local/bin" '/cfengine': handler: HTML::ColorDiff with-diff: 0 add-header: X-Notifier-Id=svnmi to: - Calling svnnotify directly works OK. Here are the involved versions: perl-SVN-Notify-Config-0.09.11-1mdv2008.0 perl-SVN-Notify-2.75-1mdv2008.0
Download (untitled) / with headers
text/plain 1.7k
On Tue Aug 12 09:38:21 2008, GROUSSE wrote: Show quoted text
> This simple configuration file doesn't pass the 'add-header' option to > svnnotify: > add-header: X-Notifier-Id=svnmi
The add-header commandline option to svnnotify can appear multiple times, so it has to be added to the YAML file as an hash of headers. Additionally, since each header can added multiple times, they need to listed as an array. The short answer is that this will work: add-headers: X-Notifier-Id: - svnmi If you want to test out something, you can use the SVN::Notify::Dummy handler (in the t/libs directory), which simply prints out the SVN::Notify command structure as a YAML file. So this commandline: $ perl -I t/lib /usr/bin/svnnotify --repos-path t/test-repos --handler Dummy --revision 12 --add-header X-Notifier-Id=svnmi --to dummy@localhost prints: --- YAML --- !!perl/hash:SVN::Notify::Dummy add_headers: X-Notifier-Id: - svnmi attach_diff: ~ author_url: ~ date: '2005-12-26 12:38:22 -0500 (Mon, 26 Dec 2005)' diff_encoding: UTF-8 diff_switches: ~ encoding: UTF-8 files: A: - project2/tags/TRUNK-1135539568/ filters: {} footer: ~ from: jpeacock header: ~ language: ~ max_diff_length: ~ max_sub_length: ~ message: - tag trunk after merging from branch message_size: 35 no_first_line: ~ reply_to: ~ repos_path: t/test-repos revision: 12 revision_url: ~ sendmail: /usr/sbin/sendmail set_sender: ~ smtp: ~ smtp_authtype: PLAIN smtp_pass: ~ smtp_user: ~ strip_cx_regex: ~ subject: '[12] tag trunk after merging from branch' subject_cx: ~ subject_prefix: ~ svn_encoding: UTF-8 svnlook: /usr/local/bin/svnlook ticket_map: ~ ticket_regex: ~ ticket_url: ~ to: - dummy@localhost to_email_map: ~ to_regex_map: ~ user: jpeacock user_domain: ~ verbose: 0 with_diff: ~ HTH John

This service is sponsored and maintained by Best Practical Solutions and runs on infrastructure.

Please report any issues with to