[pca] pca html report
Martin Paul
martin at par.univie.ac.at
Mon Oct 17 11:48:19 CEST 2011
Asif Iqbal wrote:
> Thanks for the patch. I actually would prefer a variable instead of
> hard coding wesunsolve.net, like the the original $o{patchurl} or
> $o{ohost}
The problem is that you probably want different formats when using "pca
--readme" (in a terminal pure text is preferred) and in the README links in HTML
output (this can be HTML with extra links etc. as provided by wesunsolve). So
simply adding a new "readmeurl" option won't be enough. And adding two different
options to define README locations would be pretty confusing.
> ... so I rather have this line be like this
>
> printf "<td><a href=\"https://$o{ohost}/readme/README.$id-$crev\">%s</a>
> (<a href=\"http://wesunsolve.net/readme/id/$id-$crev\">more</a>)</tr>\n",
> $synopsis;
But where's the added value then? The "http://wesunsolve.net/readme/" link
provides the plain patch README in text format, exactly the same as the
"https://$o{ohost}/readme/" link. I don't see a big advantage here, especially
as access to the READMEs on Oracle's server is pretty stable and fast.
Martin.
More information about the pca
mailing list