[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