On Fri, 2008-10-10 at 16:33 +1100, Ben Finney wrote:
> When a bug contains ‘merged-upstream’ information as a result of an 
> appropriate ‘forwarded’ command, the resulting bug report page 
> generated has an invalid URL in the page.
> 
> An example: bug#494058 <URL:http://bugs.debian.org/494058> was altered  
> by the control server in message from ‘bts-link-upstream’ 
> <URL:http://bugs.debian.org/cgi-bin/bugreport.cgi?msg=16;bug=494058>. 
> That control message contained the command:
> 
>     forwarded 494058 http://bugzilla.gnome.org/show_bug.cgi?id=547854, 
> merged-upstream: http://bugzilla.gnome.org/show_bug.cgi?id=535127
[...]
> However, the summary information on the bug then contains (HTML):
> 
>    <p>Forwarded to <a 
> href="http://bugzilla.gnome.org/show_bug.cgi?id=547854";>http://bugzilla.gnome.org/show_bug.cgi?id=547854</a>,
>  <a 
> href="merged-upstream:%20http://bugzilla.gnome.org/show_bug.cgi?id=535127";>merged-upstream:
>  http://bugzilla.gnome.org/show_bug.cgi?id=535127</a></p>

The documentation for 'forwarded' reads:

        forwarded /bugnumber/ /address/
        
        Notes that /bugnumber/ has been forwarded to the upstream
        maintainer at /address/.
        
I'm inclined to say that what bts-link is supplying is not simply
"address" and that whilst it would be nice if debbugs handled the above
as two separate links with text separating them, I don't think it's
unreasonable for the HTML generation code to assume that the content of
the forwarded address is actually an address (and therefore
representable as a hyperlink).

Just MHO,

Regards,

Adam



--
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to