* [gentoo-commits] proj/kde:master commit in: kde-apps/k3b/files/, kde-apps/k3b/
@ 2017-08-18 20:55 Andreas Sturmlechner
0 siblings, 0 replies; 2+ messages in thread
From: Andreas Sturmlechner @ 2017-08-18 20:55 UTC (permalink / raw
To: gentoo-commits
commit: ea71cd8b2e75b469241f06749fe6f4161f2c1c4d
Author: Andreas Sturmlechner <asturm <AT> gentoo <DOT> org>
AuthorDate: Fri Aug 18 20:54:38 2017 +0000
Commit: Andreas Sturmlechner <asturm <AT> gentoo <DOT> org>
CommitDate: Fri Aug 18 20:54:38 2017 +0000
URL: https://gitweb.gentoo.org/proj/kde.git/commit/?id=ea71cd8b
kde-apps/k3b: Disable broken test
Gentoo-bug: 628166
Package-Manager: Portage-2.3.8, Repoman-2.3.3
kde-apps/k3b/files/k3b-17.04.3-out-of-bounds.patch | 75 ----------------------
kde-apps/k3b/k3b-17.08.49.9999.ebuild | 11 +++-
2 files changed, 9 insertions(+), 77 deletions(-)
diff --git a/kde-apps/k3b/files/k3b-17.04.3-out-of-bounds.patch b/kde-apps/k3b/files/k3b-17.04.3-out-of-bounds.patch
deleted file mode 100644
index 83034e7481..0000000000
--- a/kde-apps/k3b/files/k3b-17.04.3-out-of-bounds.patch
+++ /dev/null
@@ -1,75 +0,0 @@
-From 7f0be6a33b8260f7789c6aeed58be8d1c844229a Mon Sep 17 00:00:00 2001
-From: Leslie Zhai <lesliezhai@llvm.org.cn>
-Date: Tue, 1 Aug 2017 14:13:05 +0800
-Subject: Fix K3b::Device::from2Byte out-of-bounds issue.
-
-A great bug report by Mark!
-
-BUG: 382941
----
- libk3bdevice/k3bdeviceglobals.cpp | 24 ++++++++++++------------
- tests/k3bdeviceglobalstest.cpp | 4 +++-
- 2 files changed, 15 insertions(+), 13 deletions(-)
-
-diff --git a/libk3bdevice/k3bdeviceglobals.cpp b/libk3bdevice/k3bdeviceglobals.cpp
-index 090ed29..c016f59 100644
---- a/libk3bdevice/k3bdeviceglobals.cpp
-+++ b/libk3bdevice/k3bdeviceglobals.cpp
-@@ -212,27 +212,27 @@ void K3b::Device::debugBitfield( unsigned char* data, long len )
- }
-
-
--quint16 K3b::Device::from2Byte( const unsigned char* d )
-+quint16 K3b::Device::from2Byte(const unsigned char* d)
- {
-- if (d == NULL) {
-- qWarning() << "Invalid nullptr!";
-+ if (d == NULL || strlen((const char *) d) < 2) {
-+ qWarning() << "Invalid Byte!";
- return 0;
- }
-- return ( (d[0] << 8 & 0xFF00) |
-- (d[1] & 0xFF) );
-+ return ((d[0] << 8 & 0xFF00) |
-+ (d[1] & 0xFF));
- }
-
-
--quint32 K3b::Device::from4Byte( const unsigned char* d )
-+quint32 K3b::Device::from4Byte(const unsigned char* d)
- {
-- if (d == NULL) {
-- qWarning() << "Invalid nullptr!";
-+ if (d == NULL || strlen((const char *) d) < 4) {
-+ qWarning() << "Invalid Byte!";
- return 0;
- }
-- return ( (d[0] << 24 & 0xFF000000) |
-- (d[1] << 16 & 0xFF0000) |
-- (d[2] << 8 & 0xFF00) |
-- (d[3] & 0xFF) );
-+ return ((d[0] << 24 & 0xFF000000) |
-+ (d[1] << 16 & 0xFF0000) |
-+ (d[2] << 8 & 0xFF00) |
-+ (d[3] & 0xFF));
- }
-
-
-diff --git a/tests/k3bdeviceglobalstest.cpp b/tests/k3bdeviceglobalstest.cpp
-index 307b772..635ee39 100644
---- a/tests/k3bdeviceglobalstest.cpp
-+++ b/tests/k3bdeviceglobalstest.cpp
-@@ -23,8 +23,10 @@ DeviceGlobalsTest::DeviceGlobalsTest()
-
- void DeviceGlobalsTest::testFrom2Byte()
- {
-- const unsigned char* d = NULL;
-+ unsigned char* d = NULL;
- QCOMPARE(K3b::Device::from2Byte(d), (quint16)0);
-+ unsigned char buf[1] = { '\0' };
-+ QCOMPARE(K3b::Device::from2Byte(buf), (quint16)0);
- }
-
- void DeviceGlobalsTest::testFrom4Byte()
---
-cgit v0.11.2
diff --git a/kde-apps/k3b/k3b-17.08.49.9999.ebuild b/kde-apps/k3b/k3b-17.08.49.9999.ebuild
index 70aec75bf0..6e1046bf8d 100644
--- a/kde-apps/k3b/k3b-17.08.49.9999.ebuild
+++ b/kde-apps/k3b/k3b-17.08.49.9999.ebuild
@@ -77,8 +77,6 @@ REQUIRED_USE="
DOCS+=( ChangeLog {FAQ,PERMISSIONS,README}.txt )
-PATCHES=( "${FILESDIR}/${PN}-17.04.3-out-of-bounds.patch" )
-
src_configure() {
local mycmakeargs=(
-DK3B_BUILD_API_DOCS=OFF
@@ -104,6 +102,15 @@ src_configure() {
kde5_src_configure
}
+src_test() {
+ # 628166
+ local myctestargs=(
+ -E "(k3bdeviceglobalstest)"
+ )
+
+ kde5_src_test
+}
+
pkg_postinst() {
kde5_pkg_postinst
^ permalink raw reply related [flat|nested] 2+ messages in thread
* [gentoo-commits] proj/kde:master commit in: kde-apps/k3b/files/, kde-apps/k3b/
@ 2023-07-06 12:00 Andreas Sturmlechner
0 siblings, 0 replies; 2+ messages in thread
From: Andreas Sturmlechner @ 2023-07-06 12:00 UTC (permalink / raw
To: gentoo-commits
commit: a108fdfe12ad87b18b63991d86009ac01b41bf2d
Author: Andreas Sturmlechner <asturm <AT> gentoo <DOT> org>
AuthorDate: Thu Jul 6 11:56:10 2023 +0000
Commit: Andreas Sturmlechner <asturm <AT> gentoo <DOT> org>
CommitDate: Thu Jul 6 11:57:40 2023 +0000
URL: https://gitweb.gentoo.org/proj/kde.git/commit/?id=a108fdfe
kde-apps/k3b: Re-enable external encoder plugin
Hard-disable transcode in code by reverting upstream commit
f7d385a40cc062fa4db46c165f82f437dea82c58.
Bug: https://bugs.gentoo.org/891585
Bug: https://bugs.gentoo.org/906526
Signed-off-by: Andreas Sturmlechner <asturm <AT> gentoo.org>
.../k3b/files/k3b-23.04.2-disable-transcode.patch | 38 ++++++++++++++++++++++
kde-apps/k3b/k3b-23.04.49.9999.ebuild | 4 ++-
kde-apps/k3b/k3b-9999.ebuild | 4 ++-
3 files changed, 44 insertions(+), 2 deletions(-)
diff --git a/kde-apps/k3b/files/k3b-23.04.2-disable-transcode.patch b/kde-apps/k3b/files/k3b-23.04.2-disable-transcode.patch
new file mode 100644
index 0000000000..209c13cd45
--- /dev/null
+++ b/kde-apps/k3b/files/k3b-23.04.2-disable-transcode.patch
@@ -0,0 +1,38 @@
+From cc1e0b76f080ed95c1f6e6d66ad806427f373909 Mon Sep 17 00:00:00 2001
+From: Andreas Sturmlechner <asturm@gentoo.org>
+Date: Thu, 6 Jul 2023 13:52:04 +0200
+Subject: [PATCH] Revert "Keep transcode until a replacement is implemented."
+
+Make sure users are not presented with a completely broken option.
+
+This reverts commit f7d385a40cc062fa4db46c165f82f437dea82c58.
+
+Signed-off-by: Andreas Sturmlechner <asturm@gentoo.org>
+---
+ libk3b/core/k3bdefaultexternalprograms.cpp | 3 +++
+ 1 file changed, 3 insertions(+)
+
+diff --git a/libk3b/core/k3bdefaultexternalprograms.cpp b/libk3b/core/k3bdefaultexternalprograms.cpp
+index e659fc1f8..2ebf88367 100644
+--- a/libk3b/core/k3bdefaultexternalprograms.cpp
++++ b/libk3b/core/k3bdefaultexternalprograms.cpp
+@@ -37,6 +37,8 @@ void K3b::addDefaultPrograms( K3b::ExternalBinManager* m )
+ // https://bugs.kde.org/show_bug.cgi?id=381131
+ void K3b::addTranscodePrograms(K3b::ExternalBinManager* m)
+ {
++ Q_UNUSED(m);
++ /* Deprecated transcode
+ static const char* const transcodeTools[] = {"transcode",
+ 0, // K3b 1.0 only uses the transcode binary
+ "tcprobe",
+@@ -48,6 +50,7 @@ void K3b::addTranscodePrograms(K3b::ExternalBinManager* m)
+
+ for (int i = 0; transcodeTools[i]; ++i)
+ m->addProgram(new K3b::TranscodeProgram(transcodeTools[i]));
++ */
+ }
+
+
+--
+2.41.0
+
diff --git a/kde-apps/k3b/k3b-23.04.49.9999.ebuild b/kde-apps/k3b/k3b-23.04.49.9999.ebuild
index f51f2af3a3..15d27d3eba 100644
--- a/kde-apps/k3b/k3b-23.04.49.9999.ebuild
+++ b/kde-apps/k3b/k3b-23.04.49.9999.ebuild
@@ -72,10 +72,12 @@ RDEPEND="${DEPEND}
DOCS+=( ChangeLog {FAQ,PERMISSIONS,README}.txt )
+PATCHES=( "${FILESDIR}/${PN}-23.04.2-disable-transcode.patch" ) # bugs 891585, 906526
+
src_configure() {
local mycmakeargs=(
-DK3B_BUILD_API_DOCS=OFF
- -DK3B_BUILD_EXTERNAL_ENCODER_PLUGIN=OFF
+ -DK3B_BUILD_EXTERNAL_ENCODER_PLUGIN=ON
-DK3B_BUILD_WAVE_DECODER_PLUGIN=ON
-DK3B_ENABLE_HAL_SUPPORT=OFF
-DK3B_ENABLE_MUSICBRAINZ=OFF
diff --git a/kde-apps/k3b/k3b-9999.ebuild b/kde-apps/k3b/k3b-9999.ebuild
index f51f2af3a3..15d27d3eba 100644
--- a/kde-apps/k3b/k3b-9999.ebuild
+++ b/kde-apps/k3b/k3b-9999.ebuild
@@ -72,10 +72,12 @@ RDEPEND="${DEPEND}
DOCS+=( ChangeLog {FAQ,PERMISSIONS,README}.txt )
+PATCHES=( "${FILESDIR}/${PN}-23.04.2-disable-transcode.patch" ) # bugs 891585, 906526
+
src_configure() {
local mycmakeargs=(
-DK3B_BUILD_API_DOCS=OFF
- -DK3B_BUILD_EXTERNAL_ENCODER_PLUGIN=OFF
+ -DK3B_BUILD_EXTERNAL_ENCODER_PLUGIN=ON
-DK3B_BUILD_WAVE_DECODER_PLUGIN=ON
-DK3B_ENABLE_HAL_SUPPORT=OFF
-DK3B_ENABLE_MUSICBRAINZ=OFF
^ permalink raw reply related [flat|nested] 2+ messages in thread
end of thread, other threads:[~2023-07-06 12:00 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-07-06 12:00 [gentoo-commits] proj/kde:master commit in: kde-apps/k3b/files/, kde-apps/k3b/ Andreas Sturmlechner
-- strict thread matches above, loose matches on Subject: below --
2017-08-18 20:55 Andreas Sturmlechner
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox