[Piuparts-devel] Bug#740386: Bug#740386: Bug#740386: More robust piuparts results reporting format

[Holger Levsen]

control: tags -1 + pending

Hi Dave,

thanks for working on this!

On Sonntag, 2. März 2014, Dave Steele wrote:
> Another tack. I've removed the PackagesDB prefix commit. There is no
> linkage between the logdir traceback issue and summary-json.
> 
> summary-json (11)
> 
> 247ac53 piuparts-report: Minor rework of main loop
> 54d116c README_server.txt: Add reporting-section

+ ...The names
+ "unstable", "testing", "stable", "oldstable", and "experimental"
+ have special meaning. 

yet this text lacks an explaination what the special meaning is. can you 
explain and add that please?!?

> 94e8539 pkgsummary.py: Initial checkin for pkg summaries
> b00a6ea piuparts-report.py: Section-level summary support
> d442180 pkgsummary.py: Add support for global merge

typo in commit msg: "ohter"

Also, this commit made me think whether its really clever to shorten summary 
to "summ": it doesnt get much shorter, but adds confusion.

> 9bb8332 piuparts-report.py: generate_global_summary()
> 4fd7512 Add blocking count to the JSON summary format

typo in commit msg: "numb4er"

/me likes how you decided to count the blocking count! :)

> 6a1c8f1 pkgsummary.py: tooltip() routing for DDPO
> e57fdcf changelog: Entry for summary.json

This misses the (Closes: #740386) entry.

> 35f473b index.tpl: Placekeeper announce of summary.json
> 5892fca piuparts.conf.pejacevic: Add reporting-sections

I don't like this patch as it hardcodes info which is also in the distro-info 
packages. Plus: why is it untested?

I've merged them all into develop now, except 5892fca. Plus I have pushed the 
branch but not yet deployed it.


cheers,
	Holger
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 828 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.alioth.debian.org/pipermail/piuparts-devel/attachments/20140302/77dcfb52/attachment.sig>