From mboxrd@z Thu Jan 1 00:00:00 1970 From: ludo@gnu.org (Ludovic =?utf-8?Q?Court=C3=A8s?=) Subject: Re: Adding build status to the package list table Date: Wed, 25 Nov 2015 14:56:56 +0100 Message-ID: <87oaeifahj.fsf@gnu.org> References: <87poz6jgqi.fsf@gnu.org> <877flchhzg.fsf@gnu.org> <87poz4k4eh.fsf@gnu.org> <87610we16l.fsf@gnu.org> <878u5rjry5.fsf@gnu.org> <87r3jj9wdu.fsf@gnu.org> <87fuzvo7i3.fsf@gnu.org> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:40156) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1a1aZC-0003ZF-HZ for guix-devel@gnu.org; Wed, 25 Nov 2015 08:57:03 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1a1aZ9-00089P-9V for guix-devel@gnu.org; Wed, 25 Nov 2015 08:57:02 -0500 Received: from fencepost.gnu.org ([2001:4830:134:3::e]:50320) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1a1aZ9-00089L-6Z for guix-devel@gnu.org; Wed, 25 Nov 2015 08:56:59 -0500 In-Reply-To: <87fuzvo7i3.fsf@gnu.org> (Roel Janssen's message of "Tue, 24 Nov 2015 14:24:20 +0100") List-Id: "Development of GNU Guix and the GNU System distribution." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org Sender: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org To: Roel Janssen Cc: guix-devel@gnu.org Roel Janssen 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 =E2=80=9Cstatus:=E2=80=9D 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=E2=80=99s 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":1= 447890647,"buildstatus":0}, > {"timestamp":1447890485,"buildstatus":0,"job":"a2ps-4.14.armhf-linux","= nixname":"a2ps-4.14","system":"armhf-linux","id":811309,"finished":1,"jobse= t":"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=E2=80=99d rather avoid hacking it, to be honest ;-), but I don=E2=80=99t = mind if someone else does, of course. Thank you! Ludo=E2=80=99.