Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 1/1] package/bsdiff: drop package
@ 2024-10-31  8:48 Dario Binacchi
  2024-10-31 12:50 ` Thomas Petazzoni via buildroot
  0 siblings, 1 reply; 2+ messages in thread
From: Dario Binacchi @ 2024-10-31  8:48 UTC (permalink / raw)
  To: buildroot; +Cc: linux-amarula, Thomas Petazzoni, Dario Binacchi

As advocated by Thomas Petazzoni [1], drop bsdiff as it is not maintained
anymore.

Meanwhile, autobuild also detected that the package was no longer
downloadable from the URL, and indeed, Colin Percival, the maintainer,
confirmed that he intentionally disabled the package download due to the
presence of some security bugs that he doesn't have time to fix.

[1] https://patchwork.ozlabs.org/project/buildroot/patch/20241004215451.1198861-2-dario.binacchi@amarulasolutions.com/

Fixes:
- http://autobuild.buildroot.org/results/21bb827d3dbf553c79f49d7c184099c75df3d45d

Suggested-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
Signed-off-by: Dario Binacchi <dario.binacchi@amarulasolutions.com>
---
 .checkpackageignore                           |  1 -
 Config.in.legacy                              |  7 +++++
 package/Config.in                             |  1 -
 .../0001-Add-missing-header-for-u_char.patch  | 17 ------------
 package/bsdiff/Config.in                      |  8 ------
 package/bsdiff/bsdiff.hash                    |  4 ---
 package/bsdiff/bsdiff.mk                      | 26 -------------------
 7 files changed, 7 insertions(+), 57 deletions(-)
 delete mode 100644 package/bsdiff/0001-Add-missing-header-for-u_char.patch
 delete mode 100644 package/bsdiff/Config.in
 delete mode 100644 package/bsdiff/bsdiff.hash
 delete mode 100644 package/bsdiff/bsdiff.mk

diff --git a/.checkpackageignore b/.checkpackageignore
index 91e31eebc58b..eceea28f0c54 100644
--- a/.checkpackageignore
+++ b/.checkpackageignore
@@ -335,7 +335,6 @@ package/brickd/S70brickd Shellcheck lib_sysv.Indent lib_sysv.Variables
 package/bridge-utils/0001-fix-build-on-musl.patch lib_patch.Upstream
 package/brltty/0001-Fix-linking-error-on-mips64el.patch lib_patch.Upstream
 package/brltty/S10brltty Shellcheck lib_sysv.Indent lib_sysv.Variables
-package/bsdiff/0001-Add-missing-header-for-u_char.patch lib_patch.Upstream
 package/bustle/0001-Makefile-fix-pcap-config-call.patch lib_patch.Upstream
 package/busybox/0001-networking-libiproute-use-linux-if_packet.h-instead-.patch lib_patch.Upstream
 package/busybox/0002-Makefile.flags-strip-non-l-arguments-returned-by-pkg.patch lib_patch.Upstream
diff --git a/Config.in.legacy b/Config.in.legacy
index 30179add233b..63360b42b878 100644
--- a/Config.in.legacy
+++ b/Config.in.legacy
@@ -146,6 +146,13 @@ endif
 
 comment "Legacy options removed in 2024.11"
 
+config BR2_PACKAGE_BSDIFF
+	bool "bsdiff has been removed"
+	select BR2_LEGACY
+	help
+	  The bsdiff package was removed as it is not maintained
+	  anymore (no commit since 2014).
+
 config BR2_PACKAGE_PROCPS_NS_ORIGINAL_TOP
 	bool "BR2_PACKAGE_PROCPS_NS_ORIGINAL_TOP has been renamed"
 	select BR2_LEGACY
diff --git a/package/Config.in b/package/Config.in
index b1e999a63b18..876b340a8a67 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -172,7 +172,6 @@ menu "Development tools"
 	source "package/bats-core/Config.in"
 	source "package/binutils/Config.in"
 	source "package/bitwise/Config.in"
-	source "package/bsdiff/Config.in"
 	source "package/bustle/Config.in"
 	source "package/check/Config.in"
 	source "package/cmake/Config.in"
diff --git a/package/bsdiff/0001-Add-missing-header-for-u_char.patch b/package/bsdiff/0001-Add-missing-header-for-u_char.patch
deleted file mode 100644
index 56affb4102ce..000000000000
--- a/package/bsdiff/0001-Add-missing-header-for-u_char.patch
+++ /dev/null
@@ -1,17 +0,0 @@
-bspatch: Fix missing header <sys/types.h> for u_char
-
-Fixes http://autobuild.buildroot.net/results/31a/31a837cf6e34b02dce498f2b12e40d6d16a5a8e6/
-
-Signed-off-by: Jörg Krause <joerg.krause@embedded.rocks>
-
-diff -purN bsdiff-4.3.orig/bspatch.c bsdiff-4.3/bspatch.c
---- bsdiff-4.3.orig/bspatch.c	2015-04-30 13:47:26.485903359 +0200
-+++ bsdiff-4.3/bspatch.c	2015-04-30 13:48:14.808908672 +0200
-@@ -35,6 +35,7 @@ __FBSDID("$FreeBSD: src/usr.bin/bsdiff/b
- #include <err.h>
- #include <unistd.h>
- #include <fcntl.h>
-+#include <sys/types.h>
-
- static off_t offtin(u_char *buf)
- {
diff --git a/package/bsdiff/Config.in b/package/bsdiff/Config.in
deleted file mode 100644
index 11fd8b9d3523..000000000000
--- a/package/bsdiff/Config.in
+++ /dev/null
@@ -1,8 +0,0 @@
-config BR2_PACKAGE_BSDIFF
-	bool "bsdiff"
-	select BR2_PACKAGE_BZIP2
-	help
-	  Binary patch/diff like xdelta but creates smaller diffs.
-	  Needs bzip2 support.
-
-	  http://www.daemonology.net/bsdiff/
diff --git a/package/bsdiff/bsdiff.hash b/package/bsdiff/bsdiff.hash
deleted file mode 100644
index 102086b9fc04..000000000000
--- a/package/bsdiff/bsdiff.hash
+++ /dev/null
@@ -1,4 +0,0 @@
-# From http://www.daemonology.net/bsdiff/:
-md5  e6d812394f0e0ecc8d5df255aa1db22a  bsdiff-4.3.tar.gz
-# locally computed
-sha256  fc0a6e634ef77dcf14bf36c7b6d1e57ba1ac5c4809073dfaacb3b5f7ab277eb7  bsdiff.c
diff --git a/package/bsdiff/bsdiff.mk b/package/bsdiff/bsdiff.mk
deleted file mode 100644
index 97a1b3921df4..000000000000
--- a/package/bsdiff/bsdiff.mk
+++ /dev/null
@@ -1,26 +0,0 @@
-################################################################################
-#
-# bsdiff
-#
-################################################################################
-
-BSDIFF_VERSION = 4.3
-BSDIFF_SITE = http://www.daemonology.net/bsdiff
-BSDIFF_DEPENDENCIES = bzip2
-BSDIFF_LICENSE = BSD-2-Clause
-BSDIFF_LICENSE_FILES = bsdiff.c
-BSDIFF_CPE_ID_VENDOR = daemonology
-
-define BSDIFF_BUILD_CMDS
-	$(TARGET_MAKE_ENV) $(TARGET_CC) $(TARGET_CFLAGS) $(TARGET_LDFLAGS) \
-		$(@D)/bsdiff.c -lbz2 -o $(@D)/bsdiff
-	$(TARGET_MAKE_ENV) $(TARGET_CC) $(TARGET_CFLAGS) $(TARGET_LDFLAGS) \
-		$(@D)/bspatch.c -lbz2 -o $(@D)/bspatch
-endef
-
-define BSDIFF_INSTALL_TARGET_CMDS
-	$(INSTALL) -D -m 755 $(@D)/bsdiff $(TARGET_DIR)/usr/bin/bsdiff
-	$(INSTALL) -D -m 755 $(@D)/bspatch $(TARGET_DIR)/usr/bin/bspatch
-endef
-
-$(eval $(generic-package))
-- 
2.43.0

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [Buildroot] [PATCH 1/1] package/bsdiff: drop package
  2024-10-31  8:48 [Buildroot] [PATCH 1/1] package/bsdiff: drop package Dario Binacchi
@ 2024-10-31 12:50 ` Thomas Petazzoni via buildroot
  0 siblings, 0 replies; 2+ messages in thread
From: Thomas Petazzoni via buildroot @ 2024-10-31 12:50 UTC (permalink / raw)
  To: Dario Binacchi; +Cc: buildroot, linux-amarula

On Thu, 31 Oct 2024 09:48:40 +0100
Dario Binacchi <dario.binacchi@amarulasolutions.com> wrote:

> As advocated by Thomas Petazzoni [1], drop bsdiff as it is not maintained
> anymore.
> 
> Meanwhile, autobuild also detected that the package was no longer
> downloadable from the URL, and indeed, Colin Percival, the maintainer,
> confirmed that he intentionally disabled the package download due to the
> presence of some security bugs that he doesn't have time to fix.
> 
> [1] https://patchwork.ozlabs.org/project/buildroot/patch/20241004215451.1198861-2-dario.binacchi@amarulasolutions.com/
> 
> Fixes:
> - http://autobuild.buildroot.org/results/21bb827d3dbf553c79f49d7c184099c75df3d45d
> 
> Suggested-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
> Signed-off-by: Dario Binacchi <dario.binacchi@amarulasolutions.com>
> ---
>  .checkpackageignore                           |  1 -
>  Config.in.legacy                              |  7 +++++
>  package/Config.in                             |  1 -
>  .../0001-Add-missing-header-for-u_char.patch  | 17 ------------
>  package/bsdiff/Config.in                      |  8 ------
>  package/bsdiff/bsdiff.hash                    |  4 ---
>  package/bsdiff/bsdiff.mk                      | 26 -------------------
>  7 files changed, 7 insertions(+), 57 deletions(-)
>  delete mode 100644 package/bsdiff/0001-Add-missing-header-for-u_char.patch
>  delete mode 100644 package/bsdiff/Config.in
>  delete mode 100644 package/bsdiff/bsdiff.hash
>  delete mode 100644 package/bsdiff/bsdiff.mk

Applied to master, thanks.

Thomas
-- 
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2024-10-31 12:50 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-10-31  8:48 [Buildroot] [PATCH 1/1] package/bsdiff: drop package Dario Binacchi
2024-10-31 12:50 ` Thomas Petazzoni via buildroot

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox