From: Andreas Enge <andreas@enge.fr>
To: Maxim Cournoyer <maxim.cournoyer@gmail.com>
Subject: Re: gstreamer 2.22, webkitgtk 2.40.0, qt 5.15.8 and ffmpeg 6 on staging
Date: Wed, 29 Mar 2023 14:58:25 +0200 [thread overview]
Message-ID: <ZCQ18VcjNAhYEA9Y@jurong> (raw)
In-Reply-To: <87jzyz3fhz.fsf@gmail.com>
[-- Attachment #1: Type: text/plain, Size: 845 bytes --]
Hello Maxim!
Am Wed, Mar 29, 2023 at 08:32:08AM -0400 schrieb Maxim Cournoyer:
> I'll
> gladly volunteer to do the tricky merge after staging is merged into
> master (and removed).
Great, thanks! It should be quite feasible when paying attention to this
special case, but I only did a merge once and do not feel confident about it.
Concerning kcodecs, attached is a patch that disables the test (but keeps
the class in that is supposed to work around the fixed Qt bug; so it is
quite possible that this class is wrong currently). However I did compile
a KDE program with it and launched it successfully, so I would be in favour
of applying the patch to staging (and thus later merge it to core-updates).
What do you think?
Apart from this, all of my profile is built (and available through CI),
so I would be happy with the merge.
Andreas
[-- Attachment #2: 0001-gnu-kcodecs-Disable-obsolete-test.patch --]
[-- Type: text/plain, Size: 3157 bytes --]
From f8694bf4f32ea8f817fbc32abe28de52472b61f4 Mon Sep 17 00:00:00 2001
Message-Id: <f8694bf4f32ea8f817fbc32abe28de52472b61f4.1680094311.git.andreas@enge.fr>
From: Andreas Enge <andreas@enge.fr>
Date: Wed, 29 Mar 2023 14:50:49 +0200
Subject: [PATCH] gnu: kcodecs: Disable obsolete test.
* gnu/packages/patches/kcodecs-5.98-drop-test.patch: New file.
* gnu/packages/kde-frameworks.scm (kcodecs): Add patch.
* gnu/local.mk (dist_patch_DATA): Register patch.
---
gnu/local.mk | 1 +
gnu/packages/kde-frameworks.scm | 3 ++-
.../patches/kcodecs-5.98-drop-test.patch | 20 +++++++++++++++++++
3 files changed, 23 insertions(+), 1 deletion(-)
create mode 100644 gnu/packages/patches/kcodecs-5.98-drop-test.patch
diff --git a/gnu/local.mk b/gnu/local.mk
index 65662f8667..45c786f136 100644
--- a/gnu/local.mk
+++ b/gnu/local.mk
@@ -1397,6 +1397,7 @@ dist_patch_DATA = \
%D%/packages/patches/lightdm-vncserver-check.patch \
%D%/packages/patches/lightdm-vnc-color-depth.patch \
%D%/packages/patches/localed-xorg-keyboard.patch \
+ %D%/packages/patches/kcodecs-5.98-drop-test.patch \
%D%/packages/patches/kcontacts-incorrect-country-name.patch \
%D%/packages/patches/kde-cli-tools-delay-mime-db.patch \
%D%/packages/patches/kdiagram-Fix-missing-link-libraries.patch \
diff --git a/gnu/packages/kde-frameworks.scm b/gnu/packages/kde-frameworks.scm
index 10b8ac0134..65c4ccb4f4 100644
--- a/gnu/packages/kde-frameworks.scm
+++ b/gnu/packages/kde-frameworks.scm
@@ -532,7 +532,8 @@ (define-public kcodecs
name "-" version ".tar.xz"))
(sha256
(base32
- "0n10r7s9r25xp7vlym41qi421kld00niig73yark7yghj0r41jcz"))))
+ "0n10r7s9r25xp7vlym41qi421kld00niig73yark7yghj0r41jcz"))
+ (patches (search-patches "kcodecs-5.98-drop-test.patch"))))
(build-system cmake-build-system)
(native-inputs
(list extra-cmake-modules gperf qttools-5))
diff --git a/gnu/packages/patches/kcodecs-5.98-drop-test.patch b/gnu/packages/patches/kcodecs-5.98-drop-test.patch
new file mode 100644
index 0000000000..fbb9bf42bc
--- /dev/null
+++ b/gnu/packages/patches/kcodecs-5.98-drop-test.patch
@@ -0,0 +1,20 @@
+diff -u -r kcodecs-5.98.0.alt/autotests/CMakeLists.txt kcodecs-5.98.0/autotests/CMakeLists.txt
+--- kcodecs-5.98.0.alt/autotests/CMakeLists.txt 2023-03-29 12:40:12.904883966 +0200
++++ kcodecs-5.98.0/autotests/CMakeLists.txt 2023-03-29 12:40:28.952883548 +0200
+@@ -25,16 +25,6 @@
+ LINK_LIBRARIES KF5::Codecs Qt${QT_MAJOR_VERSION}::Test ${_qt5_compat_libs}
+ )
+
+-ecm_add_test(
+- ${CMAKE_SOURCE_DIR}/src/kusasciitextcodec.cpp
+- kusasciitextcodectest.cpp
+- TEST_NAME kusasciitextcodectest
+- LINK_LIBRARIES Qt${QT_MAJOR_VERSION}::Test ${_qt5_compat_libs}
+-)
+-target_include_directories(kusasciitextcodectest
+- PRIVATE "${CMAKE_SOURCE_DIR}/src"
+-)
+-
+ # Benchmark, compiled, but not run automatically with ctest
+ add_executable(base64benchmark base64benchmark.cpp)
+ target_link_libraries(base64benchmark KF5::Codecs Qt${QT_MAJOR_VERSION}::Test)
--
2.39.2
next prev parent reply other threads:[~2023-03-29 13:48 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-03-29 3:10 gstreamer 2.22, webkitgtk 2.40.0, qt 5.15.8 and ffmpeg 6 on staging Maxim Cournoyer
2023-03-29 9:17 ` Andreas Enge
2023-03-29 12:32 ` Maxim Cournoyer
2023-03-29 12:58 ` Andreas Enge [this message]
2023-03-29 9:35 ` Andreas Enge
2023-03-29 9:39 ` Andreas Enge
2023-03-29 9:53 ` Andreas Enge
2023-03-29 9:58 ` Andreas Enge
2023-04-03 11:45 ` Andreas Enge
2023-04-07 21:24 ` Maxim Cournoyer
2023-04-08 9:37 ` Andreas Enge
2023-04-08 10:22 ` Josselin Poiret
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=ZCQ18VcjNAhYEA9Y@jurong \
--to=andreas@enge.fr \
--cc=maxim.cournoyer@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/guix.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.