Re: [PR PATCH] [Updated] Fluxbox switch source

[zlice <zlice@users.noreply.github.com>]

[-- Attachment #1: Type: text/plain, Size: 967 bytes --]

There is an updated pull request by zlice against master on the void-packages repository

https://github.com/zlice/void-packages fluxbox_switch_source
https://github.com/void-linux/void-packages/pull/36863

Fluxbox switch source
#### Testing the changes
- I tested the changes in this PR: **YES**

#### Local build testing
- I built this PR locally for my native architecture, x86_64

#### Comments

Ran into issues with the sourceforge tar using pregenerated aclocal.m4/make files. Saw that github is getting commits here and there and the source has autogen.sh so I built from there.

There are a few file differences from the sourceforge tar. From what I can tell:

 - a `defaults.cc` generated by a makefile?
 - some `tests/` are in github that aren't in sourceforge
 - already made languages files in sourceforge
 - updated docs that aren't in sourceforge

A patch file from https://github.com/void-linux/void-packages/pull/36863.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-fluxbox_switch_source-36863.patch --]
[-- Type: text/x-diff, Size: 1993 bytes --]

From 6c7bd4af3f7442fa50166cf7f75e07db918d2e68 Mon Sep 17 00:00:00 2001
From: zlice <zlice555@gmail.com>
Date: Mon, 25 Apr 2022 22:41:33 -0400
Subject: [PATCH] fluxbox: build from github

---
 srcpkgs/fluxbox/template | 16 +++++++++++-----
 1 file changed, 11 insertions(+), 5 deletions(-)

diff --git a/srcpkgs/fluxbox/template b/srcpkgs/fluxbox/template
index 3328421d91ac..db5c61f17e95 100644
--- a/srcpkgs/fluxbox/template
+++ b/srcpkgs/fluxbox/template
@@ -1,26 +1,32 @@
 # Template file for 'fluxbox'
 pkgname=fluxbox
 version=1.3.7
-revision=3
+revision=4
+wrksrc=fluxbox-Release-${version//./_}
 build_style=gnu-configure
 configure_args="--enable-nls
  ac_cv_lib_X11_XOpenDisplay=yes ac_cv_lib_Xft_XftFontOpen=yes
  ac_cv_lib_Xrender_XRenderCreatePicture=yes ac_cv_lib_Xpm_XpmReadFileToPixmap=yes
  ac_cv_lib_Xinerama_XineramaQueryScreens=yes ac_cv_lib_Xext_XShapeCombineShape=yes
  ac_cv_lib_Xrandr_XRRQueryExtension=yes ac_cv_lib_fribidi_fribidi_version_info=yes"
-hostmakedepends="pkg-config"
+hostmakedepends="pkg-config automake libtool gettext gettext-devel"
 makedepends="libSM-devel libXrender-devel libXft-devel libXpm-devel imlib2-devel
  libXext-devel libXinerama-devel libXrandr-devel fribidi-devel"
 short_desc="Highly configurable and low resource X11 Window manager"
 maintainer="Orphaned <orphan@voidlinux.org>"
-homepage="http://www.fluxbox.org"
 license="MIT"
-distfiles="${SOURCEFORGE_SITE}/${pkgname}/${pkgname}-${version}.tar.xz"
-checksum=fc8c75fe94c54ed5a5dd3fd4a752109f8949d6df67a48e5b11a261403c382ec0
+homepage="http://www.fluxbox.org"
+distfiles="https://github.com/fluxbox/fluxbox/archive/refs/tags/Release-${version//./_}.tar.gz"
+checksum=990414ae342a35c5703e5025a8bf019b0b4dada55d57071c51d3d498abb5cd36
 
 case "$XBPS_TARGET_MACHINE" in
 	*-musl) configure_args="${configure_args/enable-nls/disable-nls}" ;;
 esac
+
+pre_configure() {
+	./autogen.sh
+}
+
 post_install() {
 	vinstall ${FILESDIR}/fluxbox.desktop 644 usr/share/xsessions
 	vlicense COPYING