GNU bug report logs -
#60864
[PATCH] gnu: Unify webkitgtk and webkitgtk-next.
Previous Next
To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 60864 in the body.
You can then email your comments to 60864 AT debbugs.gnu.org in the normal way.
Toggle the display of automated, internal messages from the tracker.
Report forwarded
to
guix-patches <at> gnu.org
:
bug#60864
; Package
guix-patches
.
(Mon, 16 Jan 2023 17:18:02 GMT)
Full text and
rfc822 format available.
Acknowledgement sent
to
Liliana Marie Prikler <liliana.prikler <at> gmail.com>
:
New bug report received and forwarded. Copy sent to
guix-patches <at> gnu.org
.
(Mon, 16 Jan 2023 17:18:02 GMT)
Full text and
rfc822 format available.
Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):
* gnu/packages/webkit.scm (%webkit-version): Update to 2.38.3.
(webkitgtk)[source]: Update hash.
[arguments]<#:phases>: Add ‘set-CC’. Remove ‘patch-gtk-doc-scan’.
[native-inputs]: Remove gtk-doc/stable and docbook-xml. Add gi-docgen.
(webkitgtk-next)[name]: Rename to ‘webkitgtk-next’.
[source, native-inputs]: Inherit fully.
[arguments]<#:phases>: Inherit fully.
---
Hi Guix,
our Webkit packages diverged a little with the introduction of webkitgtk-next.
Since the only difference between it and regular webkitgtk is the use of
gtk4 as toolkit, however, I think that resynchronizing them might be better
overall.
Cheers
gnu/packages/webkit.scm | 45 ++++++++++-------------------------------
1 file changed, 11 insertions(+), 34 deletions(-)
diff --git a/gnu/packages/webkit.scm b/gnu/packages/webkit.scm
index 29f89721b7..400cf8372c 100644
--- a/gnu/packages/webkit.scm
+++ b/gnu/packages/webkit.scm
@@ -124,18 +124,18 @@ (define-public wpebackend-fdo
engine that uses Wayland for graphics output.")
(license license:bsd-2)))
-(define %webkit-version "2.36.8") ;webkit2gtk4
+(define %webkit-version "2.38.3")
(define-public webkitgtk
(package
- (name "webkitgtk")
+ (name "webkitgtk") ; webkit2gtk4
(version %webkit-version)
(source (origin
(method url-fetch)
(uri (string-append "https://www.webkitgtk.org/releases/"
name "-" version ".tar.xz"))
(sha256
- (base32 "0dq4s0rw3cmsxlv22pc38qdsq4wx2yyq9wgsi4wgw243y9mzpn8a"))
+ (base32 "0njac0878ksh53gn50phly1vzvj08q7g5aclncv6k324xp8h3w21"))
(patches (search-patches
"webkitgtk-adjust-bubblewrap-paths.patch"))))
(build-system cmake-build-system)
@@ -166,6 +166,11 @@ (define-public webkitgtk
#:make-flags #~(list "-j" (number->string (max 2 (parallel-job-count))))
#:phases
#~(modify-phases %standard-phases
+ (add-before 'build 'set-CC
+ (lambda _
+ ;; Some Perl scripts check for the CC environment variable, else
+ ;; use /usr/bin/gcc.
+ (setenv "CC" #$(cc-for-target))))
(add-after 'unpack 'configure-bubblewrap-store-directory
(lambda _
;; This phase works in tandem with
@@ -189,12 +194,6 @@ (define-public webkitgtk
;; the pkg-config search to locate headers.
(("pkg_check_modules\\(PC_SYSTEMD QUIET libsystemd")
"pkg_check_modules(PC_SYSTEMD QUIET libelogind"))))
- (add-after 'unpack 'patch-gtk-doc-scan
- (lambda* (#:key native-inputs inputs #:allow-other-keys)
- (substitute* (find-files "Source" "\\.sgml$")
- (("http://www.oasis-open.org/docbook/xml/4.1.2/docbookx.dtd")
- (search-input-file (or native-inputs inputs)
- "xml/dtd/docbook/docbookx.dtd")))))
(add-after 'unpack 'embed-absolute-wpebackend-reference
(lambda* (#:key inputs #:allow-other-keys)
(let ((wpebackend-fdo (assoc-ref inputs "wpebackend-fdo")))
@@ -223,9 +222,7 @@ (define-public webkitgtk
perl
pkg-config
python-wrapper
- ;; These are required to build the documentation.
- gtk-doc/stable
- docbook-xml
+ gi-docgen
ruby))
(propagated-inputs
(list gtk+ libsoup))
@@ -282,33 +279,13 @@ (define-public webkitgtk
(define-public webkitgtk-next
(package
(inherit webkitgtk)
- (name "webkitgtk")
- (version "2.38.3") ;webkit2gtk5
- (source (origin
- (inherit (package-source webkitgtk))
- (method url-fetch)
- (uri (string-append "https://www.webkitgtk.org/releases/"
- name "-" version ".tar.xz"))
- (sha256
- (base32 "0njac0878ksh53gn50phly1vzvj08q7g5aclncv6k324xp8h3w21"))))
- (build-system cmake-build-system)
+ (name "webkitgtk-next") ; webkit2gtk5
(arguments
(substitute-keyword-arguments (package-arguments webkitgtk)
((#:configure-flags flags)
#~(cons* "-DENABLE_INTROSPECTION=ON"
"-DUSE_GTK4=ON"
- (delete "-DENABLE_GTKDOC=ON" #$flags)))
- ((#:phases phases)
- #~(modify-phases #$phases
- (add-before 'build 'set-CC
- (lambda _
- ;; Some Perl scripts check for the CC environment variable, else
- ;; use /usr/bin/gcc.
- (setenv "CC" "gcc")))))))
- (native-inputs
- (modify-inputs (package-native-inputs webkitgtk)
- (delete "docbook-xml" "gtk-doc")
- (append gi-docgen)))
+ (delete "-DENABLE_GTKDOC=ON" #$flags)))))
(propagated-inputs
(modify-inputs (package-propagated-inputs webkitgtk)
(replace "gtk+" gtk)))
--
2.38.1
Information forwarded
to
guix-patches <at> gnu.org
:
bug#60864
; Package
guix-patches
.
(Mon, 16 Jan 2023 21:44:02 GMT)
Full text and
rfc822 format available.
Message #8 received at 60864 <at> debbugs.gnu.org (full text, mbox):
Hi Liliana,
Liliana Marie Prikler <liliana.prikler <at> gmail.com> writes:
> * gnu/packages/webkit.scm (%webkit-version): Update to 2.38.3.
> (webkitgtk)[source]: Update hash.
> [arguments]<#:phases>: Add ‘set-CC’. Remove ‘patch-gtk-doc-scan’.
> [native-inputs]: Remove gtk-doc/stable and docbook-xml. Add gi-docgen.
> (webkitgtk-next)[name]: Rename to ‘webkitgtk-next’.
> [source, native-inputs]: Inherit fully.
> [arguments]<#:phases>: Inherit fully.
> ---
> Hi Guix,
>
> our Webkit packages diverged a little with the introduction of webkitgtk-next.
> Since the only difference between it and regular webkitgtk is the use of
> gtk4 as toolkit, however, I think that resynchronizing them might be better
> overall.
LGTM!
--
Thanks,
Maxim
Information forwarded
to
guix-patches <at> gnu.org
:
bug#60864
; Package
guix-patches
.
(Tue, 17 Jan 2023 20:37:02 GMT)
Full text and
rfc822 format available.
Message #11 received at 60864 <at> debbugs.gnu.org (full text, mbox):
* gnu/packages/webkit.scm (%webkit-version): Update to 2.38.3.
(webkitgtk)[source]: Update hash.
[arguments]<#:phases>: Add ‘set-CC’. Remove ‘patch-gtk-doc-scan’.
[native-inputs]: Remove gtk-doc/stable and docbook-xml. Add gi-docgen.
(webkitgtk-next)[name]: Rename to ‘webkitgtk-next’.
[source, native-inputs]: Inherit fully.
[arguments]<#:phases>: Inherit fully.
(wpewebkit)[source]: Update hash.
---
In the previous version, I forgot to update teh hash of WPEWebkit.
This is now done.
Sadly, I've already had two builds failing due to the compiler getting
killed near the end. I hope this is just a limitation on my end and not
a general feature.
Cheers
gnu/packages/webkit.scm | 47 +++++++++++------------------------------
1 file changed, 12 insertions(+), 35 deletions(-)
diff --git a/gnu/packages/webkit.scm b/gnu/packages/webkit.scm
index 29f89721b7..35fb5926a3 100644
--- a/gnu/packages/webkit.scm
+++ b/gnu/packages/webkit.scm
@@ -124,18 +124,18 @@ (define-public wpebackend-fdo
engine that uses Wayland for graphics output.")
(license license:bsd-2)))
-(define %webkit-version "2.36.8") ;webkit2gtk4
+(define %webkit-version "2.38.3")
(define-public webkitgtk
(package
- (name "webkitgtk")
+ (name "webkitgtk") ; webkit2gtk4
(version %webkit-version)
(source (origin
(method url-fetch)
(uri (string-append "https://www.webkitgtk.org/releases/"
name "-" version ".tar.xz"))
(sha256
- (base32 "0dq4s0rw3cmsxlv22pc38qdsq4wx2yyq9wgsi4wgw243y9mzpn8a"))
+ (base32 "0njac0878ksh53gn50phly1vzvj08q7g5aclncv6k324xp8h3w21"))
(patches (search-patches
"webkitgtk-adjust-bubblewrap-paths.patch"))))
(build-system cmake-build-system)
@@ -166,6 +166,11 @@ (define-public webkitgtk
#:make-flags #~(list "-j" (number->string (max 2 (parallel-job-count))))
#:phases
#~(modify-phases %standard-phases
+ (add-before 'build 'set-CC
+ (lambda _
+ ;; Some Perl scripts check for the CC environment variable, else
+ ;; use /usr/bin/gcc.
+ (setenv "CC" #$(cc-for-target))))
(add-after 'unpack 'configure-bubblewrap-store-directory
(lambda _
;; This phase works in tandem with
@@ -189,12 +194,6 @@ (define-public webkitgtk
;; the pkg-config search to locate headers.
(("pkg_check_modules\\(PC_SYSTEMD QUIET libsystemd")
"pkg_check_modules(PC_SYSTEMD QUIET libelogind"))))
- (add-after 'unpack 'patch-gtk-doc-scan
- (lambda* (#:key native-inputs inputs #:allow-other-keys)
- (substitute* (find-files "Source" "\\.sgml$")
- (("http://www.oasis-open.org/docbook/xml/4.1.2/docbookx.dtd")
- (search-input-file (or native-inputs inputs)
- "xml/dtd/docbook/docbookx.dtd")))))
(add-after 'unpack 'embed-absolute-wpebackend-reference
(lambda* (#:key inputs #:allow-other-keys)
(let ((wpebackend-fdo (assoc-ref inputs "wpebackend-fdo")))
@@ -223,9 +222,7 @@ (define-public webkitgtk
perl
pkg-config
python-wrapper
- ;; These are required to build the documentation.
- gtk-doc/stable
- docbook-xml
+ gi-docgen
ruby))
(propagated-inputs
(list gtk+ libsoup))
@@ -282,33 +279,13 @@ (define-public webkitgtk
(define-public webkitgtk-next
(package
(inherit webkitgtk)
- (name "webkitgtk")
- (version "2.38.3") ;webkit2gtk5
- (source (origin
- (inherit (package-source webkitgtk))
- (method url-fetch)
- (uri (string-append "https://www.webkitgtk.org/releases/"
- name "-" version ".tar.xz"))
- (sha256
- (base32 "0njac0878ksh53gn50phly1vzvj08q7g5aclncv6k324xp8h3w21"))))
- (build-system cmake-build-system)
+ (name "webkitgtk-next") ; webkit2gtk5
(arguments
(substitute-keyword-arguments (package-arguments webkitgtk)
((#:configure-flags flags)
#~(cons* "-DENABLE_INTROSPECTION=ON"
"-DUSE_GTK4=ON"
- (delete "-DENABLE_GTKDOC=ON" #$flags)))
- ((#:phases phases)
- #~(modify-phases #$phases
- (add-before 'build 'set-CC
- (lambda _
- ;; Some Perl scripts check for the CC environment variable, else
- ;; use /usr/bin/gcc.
- (setenv "CC" "gcc")))))))
- (native-inputs
- (modify-inputs (package-native-inputs webkitgtk)
- (delete "docbook-xml" "gtk-doc")
- (append gi-docgen)))
+ (delete "-DENABLE_GTKDOC=ON" #$flags)))))
(propagated-inputs
(modify-inputs (package-propagated-inputs webkitgtk)
(replace "gtk+" gtk)))
@@ -340,7 +317,7 @@ (define-public wpewebkit
(uri (string-append "https://wpewebkit.org/releases/"
name "-" version ".tar.xz"))
(sha256
- (base32 "1svmvj96c0lhdhs7fndgwchkvv4wyb7gwd4d3fbd1chhr54s6hld"))))
+ (base32 "160456k4yiml0zn2fxba9qwp94dvvn93z9rqs2qa2lvjxig0gn8x"))))
(arguments
(substitute-keyword-arguments (package-arguments webkitgtk)
((#:configure-flags flags)
--
2.38.1
Information forwarded
to
guix-patches <at> gnu.org
:
bug#60864
; Package
guix-patches
.
(Mon, 23 Jan 2023 13:46:02 GMT)
Full text and
rfc822 format available.
Message #14 received at 60864 <at> debbugs.gnu.org (full text, mbox):
Hi Liliana,
Liliana Marie Prikler <liliana.prikler <at> gmail.com> writes:
> * gnu/packages/webkit.scm (%webkit-version): Update to 2.38.3.
> (webkitgtk)[source]: Update hash.
> [arguments]<#:phases>: Add ‘set-CC’. Remove ‘patch-gtk-doc-scan’.
> [native-inputs]: Remove gtk-doc/stable and docbook-xml. Add gi-docgen.
> (webkitgtk-next)[name]: Rename to ‘webkitgtk-next’.
> [source, native-inputs]: Inherit fully.
> [arguments]<#:phases>: Inherit fully.
> (wpewebkit)[source]: Update hash.
> ---
> In the previous version, I forgot to update teh hash of WPEWebkit.
> This is now done.
>
> Sadly, I've already had two builds failing due to the compiler getting
> killed near the end. I hope this is just a limitation on my end and not
> a general feature.
I just built both successfully here (x86_64); LGTM.
--
Thanks,
Maxim
Reply sent
to
Liliana Marie Prikler <liliana.prikler <at> gmail.com>
:
You have taken responsibility.
(Sun, 29 Jan 2023 07:29:02 GMT)
Full text and
rfc822 format available.
Notification sent
to
Liliana Marie Prikler <liliana.prikler <at> gmail.com>
:
bug acknowledged by developer.
(Sun, 29 Jan 2023 07:29:02 GMT)
Full text and
rfc822 format available.
Message #19 received at 60864-done <at> debbugs.gnu.org (full text, mbox):
Am Montag, dem 23.01.2023 um 08:44 -0500 schrieb Maxim Cournoyer:
> Hi Liliana,
>
> Liliana Marie Prikler <liliana.prikler <at> gmail.com> writes:
>
> > * gnu/packages/webkit.scm (%webkit-version): Update to 2.38.3.
> > (webkitgtk)[source]: Update hash.
> > [arguments]<#:phases>: Add ‘set-CC’. Remove ‘patch-gtk-doc-scan’.
> > [native-inputs]: Remove gtk-doc/stable and docbook-xml. Add gi-
> > docgen.
> > (webkitgtk-next)[name]: Rename to ‘webkitgtk-next’.
> > [source, native-inputs]: Inherit fully.
> > [arguments]<#:phases>: Inherit fully.
> > (wpewebkit)[source]: Update hash.
> > ---
> > In the previous version, I forgot to update teh hash of WPEWebkit.
> > This is now done.
> >
> > Sadly, I've already had two builds failing due to the compiler
> > getting killed near the end. I hope this is just a limitation on
> > my end and not a general feature.
>
> I just built both successfully here (x86_64); LGTM.
Thanks for the review. Pushed now.
bug archived.
Request was from
Debbugs Internal Request <help-debbugs <at> gnu.org>
to
internal_control <at> debbugs.gnu.org
.
(Sun, 26 Feb 2023 12:24:08 GMT)
Full text and
rfc822 format available.
This bug report was last modified 1 year and 31 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.