- From: Ville Skyttä <ville.skytta@iki.fi>
- Date: 01 Apr 2002 00:40:38 +0300
- To: www-validator@w3.org
- Cc: link@pobox.com
- Message-Id: <1017610838.18308.100.camel@bobcat.ods.org>
Terje, thanks for looking into my patches again; here's some comments and a new set of them. > * base-css.patch > - Applied (apart from the style for "label" that I forgot, > but will be in the next checkin). Not necessary, I think, see the updated HTML patch below. > * check-cfg.patch: > - Applied and expanded on. Ok. There may be something weird in the CVS, check.cfg and tips.cfg are shown in the Attic? However, checkout of validator-0_6_0 works ok and fetches these files too. http://dev.w3.org/cvsweb/validator/htdocs/config/?hideattic=0&only_with_tag=validator-0_6_0 > * check-protocols.patch: > - Applied. > - Also led to some infrastructure work that was much needed. > I reworked your patch slightly to fit into this new scheme. I realized that this patch was pretty ugly and insufficient, and accidentally found out that the same functionality is actually in libwww-perl 5.53_94 and newer. The new version below still takes advantage of the new "exception framework", but also adds a configuration option for allowed protocols, defaulting to HTTP and HTTPS. Also, a similar implementation for checklink.pl, using $ua->protocols_allowed() would clean it up somewhat. Quick comments about the new patches: - check-html.patch: HTML related fixes. - check-paths.patch: Portability patch. - check-protocols.patch: Cleaner and better supported/allowed protocol handling. - checklink-options.patch: Improved command line option handling. - checklink-cleanup.patch: General cleanups and HTML fixes. - also available from http://cachalot.ods.org/patches/w3c-validator/ and http://koti.welho.com/vskytta/patches/w3c-validator/ Then, a few comments to the TODO list at http://validator.w3.org:8001/todo.html [0.6.0, 1.] - I tried the absolute URI fixup a few months ago to make validator easier to install locally, and found it generally possible, but the currently used SSI's are a problem. Also the http://foo/check/referer needs to be taken care of (at least if there is a "Revalidate" link on the page). [Misc, 25,33.] - SSL/TLS and the error count seem to be already covered by recent updates. [Misc, new entry] - Check if this needs a fix: http://lists.w3.org/Archives/Public/www-validator/2002Mar/0055.html Cheers, -- Ville Skyttä ville.skytta@iki.fi
Attachments
- text/plain attachment: check-html.patch
- text/plain attachment: check-paths.patch
- text/plain attachment: check-protocols.patch
- text/plain attachment: checklink-cleanup.patch
- text/plain attachment: checklink-options.patch
Received on Sunday, 31 March 2002 16:41:03 UTC