Re: Adding build status to the package list table

[ludo@gnu.org (Ludovic Courtès)]

Roel Janssen <roel@gnu.org> skribis:

> I've attached a patch that adds status icons to the links in the
> expanded view. It only loads when the user clicks on "expand".

This is great!

Minor things: I would move the patch list above the status, and remove
the “status:” heading.

> We might need to add a HTTP header from the GNU server to allow
> loading data from external sources:
>
>   https://en.wikipedia.org/wiki/Cross-origin_resource_sharing

Indeed, because here the requests get blocked.  Could you get in touch
with webmasters@gnu.org and report back?

> I hope my commit message is OK this way.

It’s almost perfect.  ;-)  The only missing bits are the changes to
packages.css that are not mentioned, and the added files that must be
listed.

> For fetching all statuses in one API call, I would like to propose
> adding a function to Hydra that can return the status in an array like
> the following JSON code:
>
> [
>   {"timestamp":1447890755,"buildstatus":0,"job":"a2ps-4.14.x86_64-linux","jobset":"master","project":"gnu","system":"x86_64-linux","nixname":"a2ps-4.14","finished":1,"id":812555},
>   {"finished":1,"id":811961,"system":"i686-linux","nixname":"a2ps-4.14","project":"gnu","jobset":"master","job":"a2ps-4.14.i686-linux","timestamp":1447890647,"buildstatus":0},
>   {"timestamp":1447890485,"buildstatus":0,"job":"a2ps-4.14.armhf-linux","nixname":"a2ps-4.14","system":"armhf-linux","id":811309,"finished":1,"jobset":"master","project":"gnu"},
>   ...
> ]
>
> Then I could add columns to the table with build status icons. I don't
> know the Hydra code (or Perl), so I wonder if anyone else has the time
> and wants to add the API call.

I’d rather avoid hacking it, to be honest ;-), but I don’t mind if
someone else does, of course.

Thank you!

Ludo’.