#64206 [PATCH] gnu: webview: Tiny cross-platform webview library.

Message #8 received at 64206@debbugs.gnu.org (full text, mbox, reply):

Received: (at 64206) by debbugs.gnu.org; 22 Jun 2023 18:29:34 +0000
From debbugs-submit-bounces@debbugs.gnu.org Thu Jun 22 14:29:34 2023
Received: from localhost ([127.0.0.1]:36706 helo=debbugs.gnu.org)
	by debbugs.gnu.org with esmtp (Exim 4.84_2)
	(envelope-from <debbugs-submit-bounces@debbugs.gnu.org>)
	id 1qCP3l-0007lh-O7
	for submit@debbugs.gnu.org; Thu, 22 Jun 2023 14:29:34 -0400
Received: from mail-ej1-f68.google.com ([209.85.218.68]:46418)
 by debbugs.gnu.org with esmtp (Exim 4.84_2)
 (envelope-from <liliana.prikler@gmail.com>) id 1qCP3i-0007lT-11
 for 64206@debbugs.gnu.org; Thu, 22 Jun 2023 14:29:31 -0400
Received: by mail-ej1-f68.google.com with SMTP id
 a640c23a62f3a-98bcc5338d8so271914766b.1
 for <64206@debbugs.gnu.org>; Thu, 22 Jun 2023 11:29:29 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
 d=gmail.com; s=20221208; t=1687458564; x=1690050564;
 h=mime-version:user-agent:content-transfer-encoding:references
 :in-reply-to:date:to:from:subject:message-id:from:to:cc:subject:date
 :message-id:reply-to;
 bh=uXTQk12v+bsqZmPNPMRfWzXIx4dn9vdHWDJAYGNpreo=;
 b=oUOLTQL5bc2cHVrjq4W6Z2EUIoPNU9qovcwFPzRbo42yX/x1+kpt9FVKUbpQvCzXtB
 oqFjQRDNXHQBf8Ql/H67r1IApMcksPJIA1KyquffGWe/A1weI0EgMVe5Sf1ghVnCFSXX
 qlbY9TTkwt6KYwLT6ohiqc4cX9FhU0zbkKvDXK+aLAVnfCsTHT7hCVhjny8UdDJfumNZ
 t05SWCkfsXWZTJZ2P1mNRfsex+OFbtKK6skxHgkieO2PokLH0/I7RxKKkv4OsV5JHAmU
 alq+b7TSmdL+yNB9AyC1zLpvKMZlBAU00FRDagf1BEPeVy4n3+BIdatspSTSo1DlKZYF
 B6dQ==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
 d=1e100.net; s=20221208; t=1687458564; x=1690050564;
 h=mime-version:user-agent:content-transfer-encoding:references
 :in-reply-to:date:to:from:subject:message-id:x-gm-message-state:from
 :to:cc:subject:date:message-id:reply-to;
 bh=uXTQk12v+bsqZmPNPMRfWzXIx4dn9vdHWDJAYGNpreo=;
 b=B8lwapbJWdrLutEf+0HlIBmf3tk2X09MMfAB+J+qfgZ2qekn2Xu4x8gxj3+vGed5zs
 WUAJNx+kJn0Rj3QHMozdP+AzxkFcoC9sGDkSmPkHEJs4E6GO2bidQD1b1b722zl1+JiK
 NWp8T5S0dBus1yKIDTB0zhLxRgGcBnhrkQnyBI5uWJjF7Z7f/xN6U3gUCdF+/cBg+zXp
 3L8is3ngjlHz8bSWcdOem5p69U992Mk42omxQYhGmeQXDfbuo8Xyn5Akw9zuWMYDuKlz
 N1O1IqXD8NeJpIPXBP0JHYpyWIlnWeKMSPFpAPwJm1uYh5dyAQitF1mafj5m+Zwt9sVE
 Ja3A==
X-Gm-Message-State: AC+VfDw1NuT0No3ISg/pT/mXVIz8mQ7D0RqD2qvroNfdH+inJv4YJqNj
 ZaA1cyp9HGgXUBPMntWJzsY=
X-Google-Smtp-Source: ACHHUZ7FrJMKPBaieMsp5a9+Mc6imxQcExY/+yr+1YTKPWCRbKCx+bTDefzgAbFeK1ndPNpA7DRINg==
X-Received: by 2002:a17:906:fd89:b0:988:8be0:3077 with SMTP id
 xa9-20020a170906fd8900b009888be03077mr11174383ejb.31.1687458563970; 
 Thu, 22 Jun 2023 11:29:23 -0700 (PDT)
Received: from lumine.fritz.box (85-127-52-93.dsl.dynamic.surfer.at.
 [85.127.52.93]) by smtp.gmail.com with ESMTPSA id
 n22-20020a170906b31600b00982c0ac984asm5011116ejz.176.2023.06.22.11.29.23
 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256);
 Thu, 22 Jun 2023 11:29:23 -0700 (PDT)
Message-ID: <aae5bc57be98e24397af0d1ddd2380b66daa05ae.camel@gmail.com>
Subject: Re: [PATCH] gnu: webview: Tiny cross-platform webview library.
From: Liliana Marie Prikler <liliana.prikler@gmail.com>
To: Jose Ortiz <kotshie@gmail.com>, 64206@debbugs.gnu.org
Date: Thu, 22 Jun 2023 20:29:22 +0200
In-Reply-To: <b954cdeac839e861108cce23f29127905589e547.1687310903.git.kotshie@gmail.com>
References: <b954cdeac839e861108cce23f29127905589e547.1687310903.git.kotshie@gmail.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: base64
User-Agent: Evolution 3.46.4 
MIME-Version: 1.0
X-Spam-Score: 0.0 (/)
X-Debbugs-Envelope-To: 64206
X-BeenThere: debbugs-submit@debbugs.gnu.org
X-Mailman-Version: 2.1.18
Precedence: list
List-Id: <debbugs-submit.debbugs.gnu.org>
List-Unsubscribe: <https://debbugs.gnu.org/cgi-bin/mailman/options/debbugs-submit>, 
 <mailto:debbugs-submit-request@debbugs.gnu.org?subject=unsubscribe>
List-Archive: <https://debbugs.gnu.org/cgi-bin/mailman/private/debbugs-submit/>
List-Post: <mailto:debbugs-submit@debbugs.gnu.org>
List-Help: <mailto:debbugs-submit-request@debbugs.gnu.org?subject=help>
List-Subscribe: <https://debbugs.gnu.org/cgi-bin/mailman/listinfo/debbugs-submit>, 
 <mailto:debbugs-submit-request@debbugs.gnu.org?subject=subscribe>
Errors-To: debbugs-submit-bounces@debbugs.gnu.org
Sender: "Debbugs-submit" <debbugs-submit-bounces@debbugs.gnu.org>
X-Spam-Score: -1.0 (-)

Hi, Jose

Am Dienstag, dem 20.06.2023 um 18:28 -0700 schrieb Jose Ortiz:
> * gnu/packages/webview.scm (libwebview): Tiny cross-platform webview
> library for C/C++/Golang. Uses Webkit(Gtk/Cocoa) and Edge(Windows).
Please refer to section 22.6 of the Guix manual and 6.8.2 of the GNU
style manual on how to write commit messages :)

> ---
>  gnu/packages/webview.scm | 81
> ++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 81 insertions(+)
>  create mode 100644 gnu/packages/webview.scm
> 
> diff --git a/gnu/packages/webview.scm b/gnu/packages/webview.scm
> new file mode 100644
> index 0000000000..ee3977ba6a
> --- /dev/null
> +++ b/gnu/packages/webview.scm
Creating a new file appears to me as somewhat overkill.  We do have
web.scm FWIW and it already pulls in Rust and Go if you need those for
future libraries on top of webview.

> @@ -0,0 +1,81 @@
> +;;; GNU Guix --- Functional package management for GNU
> +;;; Copyright © 2023 Jose Ortiz <kotshie@gmail.com>
> +;;;
> +;;; This file is part of GNU Guix.
> +;;;
> +;;; GNU Guix is free software; you can redistribute it and/or modify
> it
> +;;; under the terms of the GNU General Public License as published
> by
> +;;; the Free Software Foundation; either version 3 of the License,
> or (at
> +;;; your option) any later version.
> +;;;
> +;;; GNU Guix is distributed in the hope that it will be useful, but
> +;;; WITHOUT ANY WARRANTY; without even the implied warranty of
> +;;; MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> +;;; GNU General Public License for more details.
> +;;;
> +;;; You should have received a copy of the GNU General Public
> License
> +;;; along with GNU Guix.  If not, see
> <http://www.gnu.org/licenses/>.
> +
> +(define-module (gnu packages webview)
> +  #:use-module (gnu packages)
> +  #:use-module (guix packages)
> +  #:use-module (guix build-system gnu)
> +  #:use-module (guix git-download)
> +  #:use-module (gnu packages pkg-config)
> +  #:use-module (gnu packages gtk)
> +  #:use-module (gnu packages webkit)
> +  #:use-module ((guix licenses) #:prefix license:))
> +
> +
> +(define-public webview
> +  (package
> +    (name "webview")
> +    (version "899018a")
A commit hash is not a version.  It turns out there was a release back
in 2020, though I don't know how reliable that is.
> +    (source (origin
> +              (method git-fetch)
> +              (uri (git-reference
> +                    (url "https://github.com/webview/webview")
> +                    (commit
> "899018ad0e5cc22a18cd734393ccae4d55e3b2b4")))
> +              (file-name (git-file-name name version))
> +              (sha256
> +               (base32
> +               
> "008bk2l6spsixsyk0dssnhnwdhmhwbm3qypk1rrz02rza28dzigb"))))
> +    (build-system gnu-build-system)
> +    (arguments
> +     `(#:tests? #f
Never #:tests? #f without an appropriate comment.
> +       #:phases (modify-phases %standard-phases
> +                  (delete 'configure)
> +                  (replace 'build
> +                    (lambda _
> +                      (use-modules (ice-9 popen)
> +                                   (ice-9 rdelim))
> +                      (define (webview-flags)
> +                        (let* ((port (open-input-pipe (string-append
> +                                                       "pkg-config -
> -cflags --libs "
> +                                                       "gtk+-3.0
> webkit2gtk-4.0")))
> +                               (results (string-split (read-line
> port) #\space)))
> +                          results))
> +                      (apply invoke
> +                             `("g++" "webview.cc"
> +                               "-fPIC"
> +                               "-shared"
> +                               "-DWEBVIEW_GTK"
> +                               "-std=c++11"
> +                               ,@(webview-flags)
> +                               "-o"
> +                               "libwebview.so"))))
> +                  (replace 'install
> +                    (lambda* (#:key outputs #:allow-other-keys)
> +                      (let* ((output (assoc-ref outputs "out"))
> +                             (lib (string-append output "/lib"))
> +                             (include (string-append output
> "/include")))
> +                        (install-file "libwebview.so" lib)
> +                        (mkdir-p include)
> +                        (install-file "webview.h" include)))))))
This looks like it could more correctly be implemented via copy-build-
system with a pre-install build phase.
> +    (native-inputs (list pkg-config))
> +    (inputs (list gtk+ webkitgtk-with-libsoup2))
> +    (home-page "https://github.com/webview/webview")
> +    (synopsis "Tiny cross-platform webview library")
> +    (description "Tiny cross-platform webview library for
> C/C++/Golang.
> +Uses WebKit (Gtk/Cocoa) and Edge (Windows).")
> +    (license license:expat)))
> 
> base-commit: d884fc9e2efecfba09af4694f5a13ad7fc6f704f

Cheers

Send a report that this bug log contains spam.