From: Andrew Cagney <ac131313@redhat.com>
To: overseers@sources.redhat.com
Subject: [rfa] %&^%*()) MOZILLA
Date: Sat, 01 Feb 2003 01:08:00 -0000 [thread overview]
Message-ID: <3E3B1DF6.1040104@redhat.com> (raw)
[-- Attachment #1: Type: text/plain, Size: 305 bytes --]
Hello,
The attached path to gnatsweb.pl:
- gets around a bug in mozilla where the list of interested parties
e-mailed to included half the subject line
- filters out /*-prs@/ from the list of interested parties (can't mail
that anyway).
Can I commit it?
you can test it using xgnatsweb.pl.
Andrew
[-- Attachment #2: diffs --]
[-- Type: text/plain, Size: 2387 bytes --]
? diffs
Index: ChangeLog
===================================================================
RCS file: /cvs/sourceware/cgi-bin/ChangeLog,v
retrieving revision 1.36
diff -u -r1.36 ChangeLog
--- ChangeLog 10 Jan 2002 02:07:52 -0000 1.36
+++ ChangeLog 1 Feb 2003 01:03:40 -0000
@@ -1,3 +1,10 @@
+2003-01-31 Andrew Cagney <ac131313@redhat.com>
+
+ * gnatsweb.pl (get_mailto_link): Do not escape the e-mail list.
+ Work around mozilla bug.
+ (interested_parties): Strip out /*-prs@/ as that doesn't accept
+ e-mail. Don't include spaces in list of e-mail addresses.
+
2002-01-09 Christopher Faylor <cgf@redhat.com>
* gnatsweb.pl: Updated from gcc.gnu.org version.
Index: gnatsweb.pl
===================================================================
RCS file: /cvs/sourceware/cgi-bin/gnatsweb.pl,v
retrieving revision 1.38
diff -u -r1.38 gnatsweb.pl
--- gnatsweb.pl 10 Jan 2002 02:11:00 -0000 1.38
+++ gnatsweb.pl 1 Feb 2003 01:03:41 -0000
@@ -843,7 +843,14 @@
sub get_mailto_link
{
my($pr,%fields) = @_;
- my $mailto = $q->escape(scalar(interested_parties($pr, 1, %fields)));
+
+ # NOTE: cagney/2003-01-31: Don't escape the interested parties
+ # e-mail list. MOZILLA has a nasty bug were it doesn't re-adjust
+ # the e-mail list length after de-escaping it. This causes MOZILLA
+ # to use "?Sub..." in the list of e-mail addresses.
+
+ # my $mailto = $q->escape(scalar(interested_parties($pr, 1, %fields)));
+ my $mailto = interested_parties($pr, 1, %fields);
my $subject = $q->escape("Re: $fields{'Category'}/$pr: $fields{'Synopsis'}");
my $body = $q->escape(get_viewpr_url($pr));
@@ -3182,7 +3189,15 @@
foreach $list (@prospect_list) {
if (defined($list)) {
foreach $person (split_csl ($list)) {
- push(@people, $person) if $person;
+
+ # Weed out *-prs. On sware the gdb-prs@ address is read only.
+ # Besides, when gdb-gnats@ gets the PR, it will quickly
+ # forward it onto the Submitter-Id's submitter contact
+ # gdb-prs@.
+
+ if ($person !~ /-prs@/) {
+ push(@people, $person) if $person;
+ }
}
}
}
@@ -3195,7 +3210,9 @@
$addr = praddr($person) || $person;
$addrs{$addr} = 1;
}
- return wantarray ? %addrs : join(', ', keys(%addrs));
+
+ # Do not include spaces. Keeps the mailto: line simpler.
+ return wantarray ? %addrs : join(',', keys(%addrs));
}
# Split comma-separated list.
next reply other threads:[~2003-02-01 1:08 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2003-02-01 1:08 Andrew Cagney [this message]
2003-02-01 2:05 ` Christopher Faylor
2003-02-01 7:44 ` Andrew Cagney
2003-02-01 18:53 ` Christopher Faylor
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=3E3B1DF6.1040104@redhat.com \
--to=ac131313@redhat.com \
--cc=overseers@sources.redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).