For whatever reasons, the perl script query-pr-summary.cgi is horrible slow
for large output. It take more than 4 CPU minutes to parse 100_000 lines text.
It also doesn't make sense to generate a 20MB large HTML table.
Content-Disposition header.
- When downloading/referring to a particular patch embed the patch
number with the PR instead of using two different CGI variables.
This means that there aren't "&" in the patch URL's anymore which
makes things simpler for sh(1) like shells, by not requiring quoting
of URLs when using a command line tool, such as fetch(1), to
download patches.
- In HTML-ifying of URL's, handle when a URL contains a PR like string
which can e.g. be seen in PR www/113071.
- Add a comment about how to test script via the command line.
Submitted by: edwin, shaun
PR: www/113071
where no page was returned to the submitter resulting in many
duplicate PR's.
The pipe to sendmail was just being closed without the nice ".\n" to
tell sendmail this is the end of the mail. Now we always send "\n.\n"
to be sure the is a standalone "." on a line.
Why this wasn't a problem before, but is now, I'm not really sure, but
it's probably a perl 5.0 -> 5.8 change which triggered this.
If there are still problems after this commit, please report including
PR number(s).
Prodded by: bugmeister(s)
Co-prodding and testing help: edwin
Not fixed for far too long by: simon
own CVSweb so there is not really any big reason for us to have CVSweb
for them as well. [1]
Remove the "learn" CVS repository. It hasn't changed in 9 years... [1]
Remove the German and San Marino CVSweb mirrors from the list since
neither one has a hostname which resolves.
Suggested by: wosch [1]
the relevant parts onto CVSweb 3.0.6 stock files, so it is not entirely
impossible that some local changes were lost, though I don't think/hope
so.
The use of cgi-style.pl was merged by danger@. He also, together with
Dan Langille, got CVSweb 3 tested against FreshPorts prior to commit, so
this update should not blow up FreshPorts. Thanks to both for the help.
Note that this now means that the FreeBSD CGI scripts more or less
require perl 5.8 (or at least something newer than 5.0), to run without
local modifications - but people really shouldn't be running perl 5.0
these days anyway... (no comments on the fact that www.FreeBSD.org did
this until ~2 hours ago... ).
which has more strict taint checks.
This is a temporary stopgap while upgrading to perl 5.8 and migrating
to cvsweb 3 to get around the catch-22 that cvsweb 3 requires perl
5.8, and perl can't be upgraded to 5.8 if it breaks cvsweb.
- /faq.html -> /help.html
- merge /copyright.html into help.html
- section title without link and logo
- no logo in footer
- minor bugfixes in layout & design
- remove 'Quick Reference Categories'
This work included:
- move a few functions out of query-pr-summary.cgi into a new file -
query-pr-lib.pl, which now provides a procedure to display forms
- include query-pr-lib.pl in both query-pr.cgi and query-pr-summary.cgi
scripts
PR: www/35647
Approved by: trhodes (mentor), keramida (mentor)
Silence from: www@
