[pca] confirming the validiting of a home grown patch_order file

[Thomas Gouverneur]

Read my comments below ;)


On Mon, 30 Jan 2012 15:19:46 +0100
Martin Paul <martin at par.univie.ac.at> wrote:

> Thomas Gouverneur wrote:
> >  - I've fixed the header for PCA to admit the date parsing (i hope
> > at least ;))
> 
> Nearly :) You should replace month numbers by their
> 3-letter-abbreviations in the date (01 -> Jan, 02 ->
> Feb,Mar,Apr,May,Jun,Jul,Aug,Sep,Oct,Nov,Dec).

Right! Done!

> 
> >  - I've also fixed the multiple-release present, it now take the
> >    biggest release of a patch present into every dependancy.
> 
> Yes, the number of patches in the generated xref file has now been
> reduced.
> 
> > So, whenever there is a bad patch, should I look at the first upper
> > revision which is not a bad patch ? is it safe to do that ?
> 
> Yes, that's what I'd do.

-> Implemented

> 
> > Any other remark ?
> 
> I still see "empty" patch lines. E.g. in the xref file generated for
> a list with only patch 148301-01 on it, the resulting xref file
> contains:
> 
>    125369|05|Jan/01/70| | | |  ||||
>    125547|01|Jan/01/70| | | |  ||||

For theses one I don't know again how to threat them... They are
referenced as dependencies over some other patch but looks unresolvable
(maybe they are internal release of theses patches....). The first
release that I can see downloaded for 125369 is the -10.. How to handle
that while generating the patchdiag.xref ?

> 
> There's also:
> 
>    123839|01|Jan/01/70| | | |  ||||
> 
> which wouldn't have to be included anyway, as it is obsoleted by
> 126897-02, which is included due to some other dependency requirement.

Dependency over other patch id are now implemented and you shouldn't
get that anymore...

> 
> Martin.
> 


Kind regards :)


-- 
Thomas Gouverneur
 _____           _      
| ____|___ _ __ (_)_  __
|  _| / __| '_ \| \ \/ /
| |___\__ \ |_) | |>  < 
|_____|___/ .__/|_/_/\_\
 Network  |_|       SPRL
   TVA: BE6836018011

T: +32 498 23 00 40
W: http://espix.net
M: <thomas at espix.net>