From mboxrd@z Thu Jan 1 00:00:00 1970 From: ng0 Subject: Re: [PATCH] gnu: Add libosinfo. Date: Mon, 01 Aug 2016 14:50:18 +0000 Message-ID: <87bn1cr1at.fsf@we.make.ritual.n0.is> References: <6fad26579ed8d83868da31c85a816d1d@openmailbox.org> <51e7032bbd0d6d0273dc033ad4a71da9@openmailbox.org> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:35594) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bUEXy-0000xG-Qs for guix-devel@gnu.org; Mon, 01 Aug 2016 10:50:28 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bUEXt-0008W6-S4 for guix-devel@gnu.org; Mon, 01 Aug 2016 10:50:25 -0400 Received: from mithlond.libertad.in-berlin.de ([2001:67c:1400:2490::1]:37689 helo=beleriand.n0.is) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bUEXt-0008W0-H6 for guix-devel@gnu.org; Mon, 01 Aug 2016 10:50:21 -0400 In-Reply-To: <51e7032bbd0d6d0273dc033ad4a71da9@openmailbox.org> 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" To: rennes@openmailbox.org, Ricardo Wurmus Cc: guix-devel@gnu.org Hi Rene and Ricardo, I found this contribution on my search for forgotten packages/contributions. Rene, was this simply forgotten or left due to whatever reasons, or do you want someone else to move this over the finnishing line? Thanks, ng0 rennes@openmailbox.org writes: > On 2016-02-16 07:24, Ricardo Wurmus wrote: >> rennes@openmailbox.org writes: >> >>> i attached libosinfo patch required for GNOME Boxes. >> >> Thank you very much! Below I’ll add a couple of comments. I’m not >> sure >> about a couple of things and I hope you can shed some light on these >> issues. >> >>> Considerations: >>> >>> a) In the source i used >>> "https://fedorahosted.org/releases/l/i/libosinfo" instead >>> "mirror://gnome/sources/". >> >> The project home page points to fedorahosted.org for downloading >> releases. I don’t even find the libosinfo sources on ftp.gnome.org, so >> I think your choice is completely fine. >> >>> b) In native inputs, i used "vala" instead "glib:bin"; i follow the >>> README. >> >> I don’t understand what you mean. Vala and glib:bin are not the same >> thing. Following the README is a good idea. If we don’t need >> “glib:bin” then it’s the right thing not to add it. >> >>> From 073a183499bd764b0b0efc246748638c6e4d3aeb Mon Sep 17 00:00:00 2001 >>> From: Rene Saavedra >>> Date: Sat, 13 Feb 2016 16:23:10 -0600 >>> Subject: [PATCH] gnu: Add libosinfo. >> >>> * gnu/packages/gnome.scm (libosinfo): New variable. >> >> OK. >> >>> + >>> +(define-public libosinfo >>> + (package >>> + (name "libosinfo") >>> + (version "0.3.0") >>> + (source >>> + (origin >>> + (method url-fetch) >>> + (uri (string-append >>> "https://fedorahosted.org/releases/l/i/libosinfo/" >>> + name "-" version ".tar.gz")) >>> + (sha256 >>> + (base32 >>> + "1g7g5hc4lhi4y0j3mbcj19hawlqkflni1zk4aggrx49fg5l392jk")))) >> >> The indentation of the “(base32” expression is wrong; the opening >> parenthesis should be aligned with the ‘s’ of “(sha256”. (Don’t worry >> about this too much — I can fix this before pushing if you don’t get it >> right.) >> >>> + (build-system glib-or-gtk-build-system) >> >> Is this necessary or can we use the simpler “gnu-build-system” instead? >> >>> + (native-inputs >>> + `(("check" ,check) >>> + ("intltool" ,intltool) >>> + ("libsoup" ,libsoup) >>> + ("pkg-config" ,pkg-config) >>> + ("vala" ,vala) >>> + ("wget" ,wget))) >> >> Why are “wget” and “libsoup” native inputs? In the build environment >> there is no network access, so wget cannot be used to download >> anything. >> Is libosinfo linked with libsoup? If so, it should be a regular input, >> not among the native-inputs. >> >>> + (inputs >>> + `(("libxslt" ,libxslt))) >> >> Does it link with libxslt? Or is it used at build time only (e.g. for >> building manuals from XML sources)? >> >>> + (home-page "https://libosinfo.org") >>> + (synopsis "Library for managing information about operating >>> systems") >>> + (description >>> + "libosinfo is a GObject based library API for managing >>> information about >>> +operating systems, hypervisors and the (virtual) hardware devices >>> they can >>> +support. It includes a database containing device metadata and >>> provides APIs >>> +to match/identify optimal devices for deploying an operating system >>> on a >>> +hypervisor. Via the magic of GObject Introspection, the API is >>> available in all >>> +common programming languages with demos for javascript (GJS/Seed) and >>> python >>> +(PyGObject). Also provided are Vala bindings.") >> >> I would cut “the magic of”. Please also replace “javascript” with >> “JavaScript” and “python” with “Python”. Are the examples and the Vala >> bindings actually installed? I see that gobject-introspection is not >> among the inputs, so I wonder if >> >>> + (license license:lgpl2.1+))) >> >> At least “tools/osinfo-query.c” has a license header that says it’s >> released under GPLv2+. It is better to make the license field hold a >> list of “(license:gpl2+ license:lgpl2.1+)” with a comment above it that >> explains what files are under GPL (the tools) and what files are LGPL >> (the library). >> >> Could you please send an updated patch (after clarifying the points >> above if they are confusing/contentious)? >> >> Thanks again for the patch! >> >> ~~ Ricardo > > > Hi Ricardo, > > I made changes to the patch. I generated the patch for the file > "gnu/packages/gnome.scm". > > Explanations/ questions: > > a) I replace "build-system glib-or-gtk-build-system", in this case is a > library(not GNOME applications); then i think is convenient use > "build-system gnu-build-system". > b) I don't know exactly the function in the package "libxslt", but is > required(README). I can see the package "libxslt" in the output of the > command "./pre-inst-env guix size libosinfo". I will look for more > documentation. > c) The examples are not installed, sorry for my mistake in the > description. I removed that part. > > > Thanks > > From 23b88e547c0950f0706dd240071f5953eb86ea00 Mon Sep 17 00:00:00 2001 > From: Rene Saavedra > Date: Thu, 18 Feb 2016 08:22:04 -0600 > Subject: [PATCH] gnu: Add libosinfo. > > --- > gnu/packages/gnome.scm | 35 +++++++++++++++++++++++++++++++++++ > 1 file changed, 35 insertions(+) > > diff --git a/gnu/packages/gnome.scm b/gnu/packages/gnome.scm > index 363d920..d24fd41 100644 > --- a/gnu/packages/gnome.scm > +++ b/gnu/packages/gnome.scm > @@ -4711,6 +4711,41 @@ can add your own files to the collection.") > license:cc-by-sa2.0 > license:cc-by-sa3.0)))) > > +(define-public libosinfo > + (package > + (name "libosinfo") > + (version "0.3.0") > + (source > + (origin > + (method url-fetch) > + (uri (string-append "https://fedorahosted.org/releases/l/i/libosinfo/" > + name "-" version ".tar.gz")) > + (sha256 > + (base32 > + "1g7g5hc4lhi4y0j3mbcj19hawlqkflni1zk4aggrx49fg5l392jk")))) > + (build-system gnu-build-system) > + (native-inputs > + `(("check" ,check) > + ("glib:bin" ,glib "bin") ;; XXX: Required for glib-mkenums. > + ("intltool" ,intltool) > + ("pkg-config" ,pkg-config))) > + (inputs > + `(("gobject-introspection" ,gobject-introspection) > + ("libsoup" ,libsoup) > + ("libxslt" ,libxslt) > + ("wget" ,wget))) ;; XXX: Required during make install. > + (home-page "https://libosinfo.org") > + (synopsis "Library for managing information about operating systems") > + (description > + "Libosinfo is a GObject based library API for managing information about > +operating systems, hypervisors and the (virtual) hardware devices they can > +support. It includes a database containing device metadata and provides APIs > +to match/identify optimal devices for deploying an operating system on a > +hypervisor. Via GObject Introspection, the API is available in all common > +programming languages.") > + (license (list license:gpl2+ > + license:lgpl2.1+)))) ;; XXX: for libraries(COPYING.LIB). > + > (define-public gnome > (package > (name "gnome") > -- > 2.6.3 > -- ♥Ⓐ ng0 Current Keys: https://we.make.ritual.n0.is/ng0.txt For non-prism friendly talk find me on http://www.psyced.org