Github messages for voidlinux
 help / color / mirror / Atom feed
* [PR PATCH] ardour: update to 6.6.
@ 2021-02-28 14:45 tibequadorian
  2021-02-28 20:42 ` [PR REVIEW] " ericonr
                   ` (41 more replies)
  0 siblings, 42 replies; 43+ messages in thread
From: tibequadorian @ 2021-02-28 14:45 UTC (permalink / raw)
  To: ml

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

There is a new pull request by tibequadorian against master on the void-packages repository

https://github.com/tibequadorian/void-packages ardour
https://github.com/void-linux/void-packages/pull/29120

ardour: update to 6.6.
adopt package
fetch source from github mirror because upstream deletes older source archives

<!-- Mark items with [x] where applicable -->

#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)

#### Have the results of the proposed changes been tested?
- [x] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR

<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->
<!-- 
#### Does it build and run successfully? 
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [ ] I built this PR locally for my native architecture, (ARCH-LIBC)
- [ ] I built this PR locally for these architectures (if supported. mark crossbuilds):
  - [ ] aarch64-musl
  - [ ] armv7l
  - [ ] armv6l-musl
-->


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

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

From be5c7189c15f34ddfc0f0661df731e5d1d8579f4 Mon Sep 17 00:00:00 2001
From: tibequadorian <tibequadorian@posteo.de>
Date: Sun, 28 Feb 2021 15:07:24 +0100
Subject: [PATCH] ardour: update to 6.6.

adopt package
fetch source from github mirror because upstream deletes older source archives
---
 .../patches/fix-musl-platform-detection.patch | 13 ------------
 srcpkgs/ardour/template                       | 20 ++++++++++---------
 2 files changed, 11 insertions(+), 22 deletions(-)
 delete mode 100644 srcpkgs/ardour/patches/fix-musl-platform-detection.patch

diff --git a/srcpkgs/ardour/patches/fix-musl-platform-detection.patch b/srcpkgs/ardour/patches/fix-musl-platform-detection.patch
deleted file mode 100644
index 8f2d29cc423..00000000000
--- a/srcpkgs/ardour/patches/fix-musl-platform-detection.patch
+++ /dev/null
@@ -1,13 +0,0 @@
-On musl libc only __linux__ is defined
-
---- libs/vst3/pluginterfaces/base/fplatform.h
-+++ libs/vst3/pluginterfaces/base/fplatform.h
-@@ -86,7 +86,7 @@
- //-----------------------------------------------------------------------------
- // LINUX
- //-----------------------------------------------------------------------------
--#elif __gnu_linux__
-+#elif __linux__
- 	#define SMTG_OS_LINUX	1
- 	#define SMTG_OS_MACOS	0
- 	#define SMTG_OS_WINDOWS	0
diff --git a/srcpkgs/ardour/template b/srcpkgs/ardour/template
index 7fb37a36769..47a2ef8c646 100644
--- a/srcpkgs/ardour/template
+++ b/srcpkgs/ardour/template
@@ -1,12 +1,11 @@
 # Template file for 'ardour'
 pkgname=ardour
-version=6.5.0
+version=6.6
 revision=1
-wrksrc="Ardour-${version}"
 build_style=waf3
 configure_args="--cxx11 --no-phone-home --with-backends=jack,alsa,dummy
  --libjack=weak --optimize --docs --use-external-libs --freedesktop"
-hostmakedepends="python3 graphviz doxygen pkg-config clang perl
+hostmakedepends="git python3 graphviz doxygen pkg-config clang perl
  itstool gettext tar readline-devel"
 makedepends="boost-devel liblrdf-devel liblo-devel lilv-devel suil-devel
  taglib-devel aubio-devel rubberband-devel alsa-lib-devel libsndfile-devel
@@ -15,11 +14,9 @@ makedepends="boost-devel liblrdf-devel liblo-devel lilv-devel suil-devel
  libltc-devel glibmm-devel"
 depends="desktop-file-utils"
 short_desc="Professional-grade digital audio workstation"
-maintainer="Orphaned <orphan@voidlinux.org>"
+maintainer="tibequadorian <tibequadorian@posteo.de>"
 license="GPL-2.0-or-later"
 homepage="http://ardour.org"
-distfiles="https://community.ardour.org/src/Ardour-${version}.tar.bz2"
-checksum=ac981ec7565fa8a171116bc2fbeb3198a01aad4b7d7c846f965e074a8a162bdb
 
 case "$XBPS_TARGET_MACHINE" in
 	x86_64*)  configure_args+=" --dist-target=x86_64" ;;
@@ -31,14 +28,19 @@ case "$XBPS_TARGET_MACHINE" in
 	*)        broken="Don't know build target" ;;
 esac
 
+do_fetch() {
+	git clone --branch ${version} --depth=1 \
+	 git://github.com/Ardour/ardour ${pkgname}-${version}
+}
+
 post_install() {
 	vinstall build/gtk2_ardour/ardour.xml 644 usr/share/mime/packages
 	vinstall "build/gtk2_ardour/ardour${version%%.*}.desktop" \
-		644 usr/share/applications
+	 644 usr/share/applications
 	for size in 16 22 32 48 256 512; do
 		vinstall gtk2_ardour/resources/Ardour-icon_${size}px.png 644 \
-			usr/share/icons/hicolor/${size}x${size}/apps \
-			ardour${version%%.*}.png
+		 usr/share/icons/hicolor/${size}x${size}/apps \
+		 ardour${version%%.*}.png
 	done
 	vman ardour.1
 }

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

* Re: [PR REVIEW] ardour: update to 6.6.
  2021-02-28 14:45 [PR PATCH] ardour: update to 6.6 tibequadorian
  2021-02-28 20:42 ` [PR REVIEW] " ericonr
@ 2021-02-28 20:42 ` ericonr
  2021-02-28 22:09 ` prez
                   ` (39 subsequent siblings)
  41 siblings, 0 replies; 43+ messages in thread
From: ericonr @ 2021-02-28 20:42 UTC (permalink / raw)
  To: ml

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

New review comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/29120#discussion_r584357334

Comment:
This needs a comment on why, though it is kinda acceptable, given that they delete old tarballs :(

Maybe we could hardcode the commit for the given tag? Instead of pulling the tag directly. Not sure that's supported or if it'd make the clone too big.

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

* Re: [PR REVIEW] ardour: update to 6.6.
  2021-02-28 14:45 [PR PATCH] ardour: update to 6.6 tibequadorian
@ 2021-02-28 20:42 ` ericonr
  2021-02-28 20:42 ` ericonr
                   ` (40 subsequent siblings)
  41 siblings, 0 replies; 43+ messages in thread
From: ericonr @ 2021-02-28 20:42 UTC (permalink / raw)
  To: ml

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

New review comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/29120#discussion_r584356926

Comment:
These should be using tabs.

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

* Re: ardour: update to 6.6.
  2021-02-28 14:45 [PR PATCH] ardour: update to 6.6 tibequadorian
  2021-02-28 20:42 ` [PR REVIEW] " ericonr
  2021-02-28 20:42 ` ericonr
@ 2021-02-28 22:09 ` prez
  2021-02-28 22:10 ` prez
                   ` (38 subsequent siblings)
  41 siblings, 0 replies; 43+ messages in thread
From: prez @ 2021-02-28 22:09 UTC (permalink / raw)
  To: ml

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

New comment by prez on void-packages repository

https://github.com/void-linux/void-packages/pull/29120#issuecomment-787532280

Comment:
Could I ask you to enable to portaudio backend while you're at it please?

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

* Re: ardour: update to 6.6.
  2021-02-28 14:45 [PR PATCH] ardour: update to 6.6 tibequadorian
                   ` (2 preceding siblings ...)
  2021-02-28 22:09 ` prez
@ 2021-02-28 22:10 ` prez
  2021-03-01  5:48 ` [PR REVIEW] " tibequadorian
                   ` (37 subsequent siblings)
  41 siblings, 0 replies; 43+ messages in thread
From: prez @ 2021-02-28 22:10 UTC (permalink / raw)
  To: ml

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

New comment by prez on void-packages repository

https://github.com/void-linux/void-packages/pull/29120#issuecomment-787532280

Comment:
Could I ask you to enable to portaudio backend while you're at it please? That should work with sndio.

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

* Re: [PR REVIEW] ardour: update to 6.6.
  2021-02-28 14:45 [PR PATCH] ardour: update to 6.6 tibequadorian
                   ` (3 preceding siblings ...)
  2021-02-28 22:10 ` prez
@ 2021-03-01  5:48 ` tibequadorian
  2021-03-01  5:49 ` tibequadorian
                   ` (36 subsequent siblings)
  41 siblings, 0 replies; 43+ messages in thread
From: tibequadorian @ 2021-03-01  5:48 UTC (permalink / raw)
  To: ml

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

New review comment by tibequadorian on void-packages repository

https://github.com/void-linux/void-packages/pull/29120#discussion_r584459627

Comment:
oops sry

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

* Re: [PR REVIEW] ardour: update to 6.6.
  2021-02-28 14:45 [PR PATCH] ardour: update to 6.6 tibequadorian
                   ` (4 preceding siblings ...)
  2021-03-01  5:48 ` [PR REVIEW] " tibequadorian
@ 2021-03-01  5:49 ` tibequadorian
  2021-03-01  5:53 ` [PR PATCH] [Updated] " tibequadorian
                   ` (35 subsequent siblings)
  41 siblings, 0 replies; 43+ messages in thread
From: tibequadorian @ 2021-03-01  5:49 UTC (permalink / raw)
  To: ml

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

New review comment by tibequadorian on void-packages repository

https://github.com/void-linux/void-packages/pull/29120#discussion_r584460112

Comment:
>Maybe we could hardcode the commit for the given tag? Instead of pulling the tag directly. Not sure that's supported or if it'd make the clone too big.

Why? Whats wrong with using the tag directly?

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

* Re: [PR PATCH] [Updated] ardour: update to 6.6.
  2021-02-28 14:45 [PR PATCH] ardour: update to 6.6 tibequadorian
                   ` (5 preceding siblings ...)
  2021-03-01  5:49 ` tibequadorian
@ 2021-03-01  5:53 ` tibequadorian
  2021-03-01  6:07 ` tibequadorian
                   ` (34 subsequent siblings)
  41 siblings, 0 replies; 43+ messages in thread
From: tibequadorian @ 2021-03-01  5:53 UTC (permalink / raw)
  To: ml

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

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

https://github.com/tibequadorian/void-packages ardour
https://github.com/void-linux/void-packages/pull/29120

ardour: update to 6.6.
adopt package
fetch source from github mirror because upstream deletes older source archives

<!-- Mark items with [x] where applicable -->

#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)

#### Have the results of the proposed changes been tested?
- [x] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR

<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->
<!-- 
#### Does it build and run successfully? 
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [ ] I built this PR locally for my native architecture, (ARCH-LIBC)
- [ ] I built this PR locally for these architectures (if supported. mark crossbuilds):
  - [ ] aarch64-musl
  - [ ] armv7l
  - [ ] armv6l-musl
-->


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

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

From f6feb0bfffbef839dce955840a515905f180845e Mon Sep 17 00:00:00 2001
From: tibequadorian <tibequadorian@posteo.de>
Date: Sun, 28 Feb 2021 15:07:24 +0100
Subject: [PATCH] ardour: update to 6.6.

adopt package
fetch source from github mirror because upstream deletes older source archives
---
 .../patches/fix-musl-platform-detection.patch    | 13 -------------
 srcpkgs/ardour/template                          | 16 ++++++++++------
 2 files changed, 10 insertions(+), 19 deletions(-)
 delete mode 100644 srcpkgs/ardour/patches/fix-musl-platform-detection.patch

diff --git a/srcpkgs/ardour/patches/fix-musl-platform-detection.patch b/srcpkgs/ardour/patches/fix-musl-platform-detection.patch
deleted file mode 100644
index 8f2d29cc423..00000000000
--- a/srcpkgs/ardour/patches/fix-musl-platform-detection.patch
+++ /dev/null
@@ -1,13 +0,0 @@
-On musl libc only __linux__ is defined
-
---- libs/vst3/pluginterfaces/base/fplatform.h
-+++ libs/vst3/pluginterfaces/base/fplatform.h
-@@ -86,7 +86,7 @@
- //-----------------------------------------------------------------------------
- // LINUX
- //-----------------------------------------------------------------------------
--#elif __gnu_linux__
-+#elif __linux__
- 	#define SMTG_OS_LINUX	1
- 	#define SMTG_OS_MACOS	0
- 	#define SMTG_OS_WINDOWS	0
diff --git a/srcpkgs/ardour/template b/srcpkgs/ardour/template
index 7fb37a36769..fb1e52fab4e 100644
--- a/srcpkgs/ardour/template
+++ b/srcpkgs/ardour/template
@@ -1,12 +1,11 @@
 # Template file for 'ardour'
 pkgname=ardour
-version=6.5.0
+version=6.6
 revision=1
-wrksrc="Ardour-${version}"
 build_style=waf3
 configure_args="--cxx11 --no-phone-home --with-backends=jack,alsa,dummy
  --libjack=weak --optimize --docs --use-external-libs --freedesktop"
-hostmakedepends="python3 graphviz doxygen pkg-config clang perl
+hostmakedepends="git python3 graphviz doxygen pkg-config clang perl
  itstool gettext tar readline-devel"
 makedepends="boost-devel liblrdf-devel liblo-devel lilv-devel suil-devel
  taglib-devel aubio-devel rubberband-devel alsa-lib-devel libsndfile-devel
@@ -15,11 +14,9 @@ makedepends="boost-devel liblrdf-devel liblo-devel lilv-devel suil-devel
  libltc-devel glibmm-devel"
 depends="desktop-file-utils"
 short_desc="Professional-grade digital audio workstation"
-maintainer="Orphaned <orphan@voidlinux.org>"
+maintainer="tibequadorian <tibequadorian@posteo.de>"
 license="GPL-2.0-or-later"
 homepage="http://ardour.org"
-distfiles="https://community.ardour.org/src/Ardour-${version}.tar.bz2"
-checksum=ac981ec7565fa8a171116bc2fbeb3198a01aad4b7d7c846f965e074a8a162bdb
 
 case "$XBPS_TARGET_MACHINE" in
 	x86_64*)  configure_args+=" --dist-target=x86_64" ;;
@@ -31,6 +28,13 @@ case "$XBPS_TARGET_MACHINE" in
 	*)        broken="Don't know build target" ;;
 esac
 
+# Upstream deletes older release tarballs from their official source and
+# GitHub tarballs are empty: https://tracker.ardour.org/view.php?id=7328
+do_fetch() {
+	git clone --branch ${version} --depth=1 \
+		git://github.com/Ardour/ardour ${pkgname}-${version}
+}
+
 post_install() {
 	vinstall build/gtk2_ardour/ardour.xml 644 usr/share/mime/packages
 	vinstall "build/gtk2_ardour/ardour${version%%.*}.desktop" \

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

* Re: ardour: update to 6.6.
  2021-02-28 14:45 [PR PATCH] ardour: update to 6.6 tibequadorian
                   ` (6 preceding siblings ...)
  2021-03-01  5:53 ` [PR PATCH] [Updated] " tibequadorian
@ 2021-03-01  6:07 ` tibequadorian
  2021-03-01  6:11 ` [PR PATCH] [Updated] " tibequadorian
                   ` (33 subsequent siblings)
  41 siblings, 0 replies; 43+ messages in thread
From: tibequadorian @ 2021-03-01  6:07 UTC (permalink / raw)
  To: ml

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

New comment by tibequadorian on void-packages repository

https://github.com/void-linux/void-packages/pull/29120#issuecomment-787672380

Comment:
> Could I ask you to enable to portaudio backend while you're at it please? That should work with sndio.

Not sure if this should be enabled by default, I could add a build option though.

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

* Re: [PR PATCH] [Updated] ardour: update to 6.6.
  2021-02-28 14:45 [PR PATCH] ardour: update to 6.6 tibequadorian
                   ` (7 preceding siblings ...)
  2021-03-01  6:07 ` tibequadorian
@ 2021-03-01  6:11 ` tibequadorian
  2021-03-01 13:08 ` prez
                   ` (32 subsequent siblings)
  41 siblings, 0 replies; 43+ messages in thread
From: tibequadorian @ 2021-03-01  6:11 UTC (permalink / raw)
  To: ml

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

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

https://github.com/tibequadorian/void-packages ardour
https://github.com/void-linux/void-packages/pull/29120

ardour: update to 6.6.
adopt package
fetch source from github mirror because upstream deletes older source archives

<!-- Mark items with [x] where applicable -->

#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)

#### Have the results of the proposed changes been tested?
- [x] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR

<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->
<!-- 
#### Does it build and run successfully? 
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [ ] I built this PR locally for my native architecture, (ARCH-LIBC)
- [ ] I built this PR locally for these architectures (if supported. mark crossbuilds):
  - [ ] aarch64-musl
  - [ ] armv7l
  - [ ] armv6l-musl
-->


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

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

From e4a69664ece646bd4ff86fdeb0fa1c083749f058 Mon Sep 17 00:00:00 2001
From: tibequadorian <tibequadorian@posteo.de>
Date: Sun, 28 Feb 2021 15:07:24 +0100
Subject: [PATCH] ardour: update to 6.6.

adopt package
fetch source from github mirror because upstream deletes older source archives
---
 .../patches/fix-musl-platform-detection.patch    | 13 -------------
 srcpkgs/ardour/template                          | 16 ++++++++++------
 2 files changed, 10 insertions(+), 19 deletions(-)
 delete mode 100644 srcpkgs/ardour/patches/fix-musl-platform-detection.patch

diff --git a/srcpkgs/ardour/patches/fix-musl-platform-detection.patch b/srcpkgs/ardour/patches/fix-musl-platform-detection.patch
deleted file mode 100644
index 8f2d29cc423..00000000000
--- a/srcpkgs/ardour/patches/fix-musl-platform-detection.patch
+++ /dev/null
@@ -1,13 +0,0 @@
-On musl libc only __linux__ is defined
-
---- libs/vst3/pluginterfaces/base/fplatform.h
-+++ libs/vst3/pluginterfaces/base/fplatform.h
-@@ -86,7 +86,7 @@
- //-----------------------------------------------------------------------------
- // LINUX
- //-----------------------------------------------------------------------------
--#elif __gnu_linux__
-+#elif __linux__
- 	#define SMTG_OS_LINUX	1
- 	#define SMTG_OS_MACOS	0
- 	#define SMTG_OS_WINDOWS	0
diff --git a/srcpkgs/ardour/template b/srcpkgs/ardour/template
index 7fb37a36769..a62f41544dc 100644
--- a/srcpkgs/ardour/template
+++ b/srcpkgs/ardour/template
@@ -1,12 +1,11 @@
 # Template file for 'ardour'
 pkgname=ardour
-version=6.5.0
+version=6.6
 revision=1
-wrksrc="Ardour-${version}"
 build_style=waf3
 configure_args="--cxx11 --no-phone-home --with-backends=jack,alsa,dummy
  --libjack=weak --optimize --docs --use-external-libs --freedesktop"
-hostmakedepends="python3 graphviz doxygen pkg-config clang perl
+hostmakedepends="git python3 graphviz doxygen pkg-config clang perl
  itstool gettext tar readline-devel"
 makedepends="boost-devel liblrdf-devel liblo-devel lilv-devel suil-devel
  taglib-devel aubio-devel rubberband-devel alsa-lib-devel libsndfile-devel
@@ -15,11 +14,9 @@ makedepends="boost-devel liblrdf-devel liblo-devel lilv-devel suil-devel
  libltc-devel glibmm-devel"
 depends="desktop-file-utils"
 short_desc="Professional-grade digital audio workstation"
-maintainer="Orphaned <orphan@voidlinux.org>"
+maintainer="tibequadorian <tibequadorian@posteo.de>"
 license="GPL-2.0-or-later"
 homepage="http://ardour.org"
-distfiles="https://community.ardour.org/src/Ardour-${version}.tar.bz2"
-checksum=ac981ec7565fa8a171116bc2fbeb3198a01aad4b7d7c846f965e074a8a162bdb
 
 case "$XBPS_TARGET_MACHINE" in
 	x86_64*)  configure_args+=" --dist-target=x86_64" ;;
@@ -31,6 +28,13 @@ case "$XBPS_TARGET_MACHINE" in
 	*)        broken="Don't know build target" ;;
 esac
 
+# Upstream deletes older release tarballs from their site and
+# GitHub tarballs are empty: https://tracker.ardour.org/view.php?id=7328
+do_fetch() {
+	git clone --branch ${version} --depth=1 \
+		git://github.com/Ardour/ardour ${pkgname}-${version}
+}
+
 post_install() {
 	vinstall build/gtk2_ardour/ardour.xml 644 usr/share/mime/packages
 	vinstall "build/gtk2_ardour/ardour${version%%.*}.desktop" \

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

* Re: ardour: update to 6.6.
  2021-02-28 14:45 [PR PATCH] ardour: update to 6.6 tibequadorian
                   ` (8 preceding siblings ...)
  2021-03-01  6:11 ` [PR PATCH] [Updated] " tibequadorian
@ 2021-03-01 13:08 ` prez
  2021-03-01 13:09 ` prez
                   ` (31 subsequent siblings)
  41 siblings, 0 replies; 43+ messages in thread
From: prez @ 2021-03-01 13:08 UTC (permalink / raw)
  To: ml

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

New comment by prez on void-packages repository

https://github.com/void-linux/void-packages/pull/29120#issuecomment-787936413

Comment:
> Not sure if this should be enabled by default, I could add a build option though.

I would appreciate it. There is precedence for it in https://github.com/void-linux/void-packages/pull/26252 and it's not like there is a patch required.

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

* Re: ardour: update to 6.6.
  2021-02-28 14:45 [PR PATCH] ardour: update to 6.6 tibequadorian
                   ` (9 preceding siblings ...)
  2021-03-01 13:08 ` prez
@ 2021-03-01 13:09 ` prez
  2021-03-01 13:11 ` tibequadorian
                   ` (30 subsequent siblings)
  41 siblings, 0 replies; 43+ messages in thread
From: prez @ 2021-03-01 13:09 UTC (permalink / raw)
  To: ml

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

New comment by prez on void-packages repository

https://github.com/void-linux/void-packages/pull/29120#issuecomment-787936413

Comment:
> Not sure if this should be enabled by default, I could add a build option though.

I would appreciate it (enabled by default). There is precedence for it in https://github.com/void-linux/void-packages/pull/26252 and it's not like there is a patch required.

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

* Re: ardour: update to 6.6.
  2021-02-28 14:45 [PR PATCH] ardour: update to 6.6 tibequadorian
                   ` (10 preceding siblings ...)
  2021-03-01 13:09 ` prez
@ 2021-03-01 13:11 ` tibequadorian
  2021-03-01 13:11 ` tibequadorian
                   ` (29 subsequent siblings)
  41 siblings, 0 replies; 43+ messages in thread
From: tibequadorian @ 2021-03-01 13:11 UTC (permalink / raw)
  To: ml

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

New comment by tibequadorian on void-packages repository

https://github.com/void-linux/void-packages/pull/29120#issuecomment-787938202

Comment:
Did you check if the portaudio really works with sndio? Look at https://github.com/void-linux/void-packages/commit/f7f83416453f008b6fbfc3811d6fa8ca8338b6b5

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

* Re: ardour: update to 6.6.
  2021-02-28 14:45 [PR PATCH] ardour: update to 6.6 tibequadorian
                   ` (11 preceding siblings ...)
  2021-03-01 13:11 ` tibequadorian
@ 2021-03-01 13:11 ` tibequadorian
  2021-03-01 13:12 ` tibequadorian
                   ` (28 subsequent siblings)
  41 siblings, 0 replies; 43+ messages in thread
From: tibequadorian @ 2021-03-01 13:11 UTC (permalink / raw)
  To: ml

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

New comment by tibequadorian on void-packages repository

https://github.com/void-linux/void-packages/pull/29120#issuecomment-787938202

Comment:
@prez Did you check if the portaudio really works with sndio? Look at https://github.com/void-linux/void-packages/commit/f7f83416453f008b6fbfc3811d6fa8ca8338b6b5

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

* Re: ardour: update to 6.6.
  2021-02-28 14:45 [PR PATCH] ardour: update to 6.6 tibequadorian
                   ` (12 preceding siblings ...)
  2021-03-01 13:11 ` tibequadorian
@ 2021-03-01 13:12 ` tibequadorian
  2021-03-01 13:14 ` prez
                   ` (27 subsequent siblings)
  41 siblings, 0 replies; 43+ messages in thread
From: tibequadorian @ 2021-03-01 13:12 UTC (permalink / raw)
  To: ml

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

New comment by tibequadorian on void-packages repository

https://github.com/void-linux/void-packages/pull/29120#issuecomment-787938202

Comment:
@prez Did you check if the portaudio backend really works with sndio? Look at https://github.com/void-linux/void-packages/commit/f7f83416453f008b6fbfc3811d6fa8ca8338b6b5

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

* Re: ardour: update to 6.6.
  2021-02-28 14:45 [PR PATCH] ardour: update to 6.6 tibequadorian
                   ` (13 preceding siblings ...)
  2021-03-01 13:12 ` tibequadorian
@ 2021-03-01 13:14 ` prez
  2021-03-01 14:33 ` tibequadorian
                   ` (26 subsequent siblings)
  41 siblings, 0 replies; 43+ messages in thread
From: prez @ 2021-03-01 13:14 UTC (permalink / raw)
  To: ml

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

New comment by prez on void-packages repository

https://github.com/void-linux/void-packages/pull/29120#issuecomment-787939997

Comment:
I have the package on hold because of that and hope it gets upstreamed one day. In the meantime I think a fourth audio backend would not really hurt.

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

* Re: ardour: update to 6.6.
  2021-02-28 14:45 [PR PATCH] ardour: update to 6.6 tibequadorian
                   ` (14 preceding siblings ...)
  2021-03-01 13:14 ` prez
@ 2021-03-01 14:33 ` tibequadorian
  2021-03-01 14:33 ` [PR PATCH] [Closed]: " tibequadorian
                   ` (25 subsequent siblings)
  41 siblings, 0 replies; 43+ messages in thread
From: tibequadorian @ 2021-03-01 14:33 UTC (permalink / raw)
  To: ml

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

New comment by tibequadorian on void-packages repository

https://github.com/void-linux/void-packages/pull/29120#issuecomment-787994224

Comment:
I don't agree with that. Currently enabled backends are recommended by upstream and JACK is de facto standard for audio production on Linux.

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

* Re: [PR PATCH] [Closed]: ardour: update to 6.6.
  2021-02-28 14:45 [PR PATCH] ardour: update to 6.6 tibequadorian
                   ` (15 preceding siblings ...)
  2021-03-01 14:33 ` tibequadorian
@ 2021-03-01 14:33 ` tibequadorian
  2021-03-01 14:33 ` tibequadorian
                   ` (24 subsequent siblings)
  41 siblings, 0 replies; 43+ messages in thread
From: tibequadorian @ 2021-03-01 14:33 UTC (permalink / raw)
  To: ml

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

There's a closed pull request on the void-packages repository

ardour: update to 6.6.
https://github.com/void-linux/void-packages/pull/29120

Description:
adopt package
fetch source from github mirror because upstream deletes older source archives

<!-- Mark items with [x] where applicable -->

#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)

#### Have the results of the proposed changes been tested?
- [x] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR

<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->
<!-- 
#### Does it build and run successfully? 
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [ ] I built this PR locally for my native architecture, (ARCH-LIBC)
- [ ] I built this PR locally for these architectures (if supported. mark crossbuilds):
  - [ ] aarch64-musl
  - [ ] armv7l
  - [ ] armv6l-musl
-->


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

* Re: ardour: update to 6.6.
  2021-02-28 14:45 [PR PATCH] ardour: update to 6.6 tibequadorian
                   ` (16 preceding siblings ...)
  2021-03-01 14:33 ` [PR PATCH] [Closed]: " tibequadorian
@ 2021-03-01 14:33 ` tibequadorian
  2021-03-01 15:51 ` [PR PATCH] [Updated] " tibequadorian
                   ` (23 subsequent siblings)
  41 siblings, 0 replies; 43+ messages in thread
From: tibequadorian @ 2021-03-01 14:33 UTC (permalink / raw)
  To: ml

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

New comment by tibequadorian on void-packages repository

https://github.com/void-linux/void-packages/pull/29120#issuecomment-787994224

Comment:
I don't agree with that. Currently enabled backends are recommended by upstream and JACK is de facto standard for audio production on Linux.

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

* Re: [PR PATCH] [Updated] ardour: update to 6.6.
  2021-02-28 14:45 [PR PATCH] ardour: update to 6.6 tibequadorian
                   ` (17 preceding siblings ...)
  2021-03-01 14:33 ` tibequadorian
@ 2021-03-01 15:51 ` tibequadorian
  2021-03-01 15:52 ` tibequadorian
                   ` (22 subsequent siblings)
  41 siblings, 0 replies; 43+ messages in thread
From: tibequadorian @ 2021-03-01 15:51 UTC (permalink / raw)
  To: ml

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

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

https://github.com/tibequadorian/void-packages ardour
https://github.com/void-linux/void-packages/pull/29120

ardour: update to 6.6.
adopt package
fetch source from github mirror because upstream deletes older source archives

<!-- Mark items with [x] where applicable -->

#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)

#### Have the results of the proposed changes been tested?
- [x] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR

<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->
<!-- 
#### Does it build and run successfully? 
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [ ] I built this PR locally for my native architecture, (ARCH-LIBC)
- [ ] I built this PR locally for these architectures (if supported. mark crossbuilds):
  - [ ] aarch64-musl
  - [ ] armv7l
  - [ ] armv6l-musl
-->


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

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

From 82f7476f347ac299f04c405dc65236a79f7319a8 Mon Sep 17 00:00:00 2001
From: tibequadorian <tibequadorian@posteo.de>
Date: Sun, 28 Feb 2021 15:07:24 +0100
Subject: [PATCH] ardour: update to 6.6.

adopt package
enable portaudio backend
fetch source from github mirror because upstream deletes older source archives
---
 .../patches/fix-musl-platform-detection.patch  | 13 -------------
 srcpkgs/ardour/template                        | 18 +++++++++++-------
 2 files changed, 11 insertions(+), 20 deletions(-)
 delete mode 100644 srcpkgs/ardour/patches/fix-musl-platform-detection.patch

diff --git a/srcpkgs/ardour/patches/fix-musl-platform-detection.patch b/srcpkgs/ardour/patches/fix-musl-platform-detection.patch
deleted file mode 100644
index 8f2d29cc423..00000000000
--- a/srcpkgs/ardour/patches/fix-musl-platform-detection.patch
+++ /dev/null
@@ -1,13 +0,0 @@
-On musl libc only __linux__ is defined
-
---- libs/vst3/pluginterfaces/base/fplatform.h
-+++ libs/vst3/pluginterfaces/base/fplatform.h
-@@ -86,7 +86,7 @@
- //-----------------------------------------------------------------------------
- // LINUX
- //-----------------------------------------------------------------------------
--#elif __gnu_linux__
-+#elif __linux__
- 	#define SMTG_OS_LINUX	1
- 	#define SMTG_OS_MACOS	0
- 	#define SMTG_OS_WINDOWS	0
diff --git a/srcpkgs/ardour/template b/srcpkgs/ardour/template
index 7fb37a36769..ec1e8cce220 100644
--- a/srcpkgs/ardour/template
+++ b/srcpkgs/ardour/template
@@ -1,12 +1,11 @@
 # Template file for 'ardour'
 pkgname=ardour
-version=6.5.0
+version=6.6
 revision=1
-wrksrc="Ardour-${version}"
 build_style=waf3
-configure_args="--cxx11 --no-phone-home --with-backends=jack,alsa,dummy
+configure_args="--cxx11 --no-phone-home --with-backends=jack,alsa,dummy,portaudio
  --libjack=weak --optimize --docs --use-external-libs --freedesktop"
-hostmakedepends="python3 graphviz doxygen pkg-config clang perl
+hostmakedepends="git python3 graphviz doxygen pkg-config clang perl
  itstool gettext tar readline-devel"
 makedepends="boost-devel liblrdf-devel liblo-devel lilv-devel suil-devel
  taglib-devel aubio-devel rubberband-devel alsa-lib-devel libsndfile-devel
@@ -15,11 +14,9 @@ makedepends="boost-devel liblrdf-devel liblo-devel lilv-devel suil-devel
  libltc-devel glibmm-devel"
 depends="desktop-file-utils"
 short_desc="Professional-grade digital audio workstation"
-maintainer="Orphaned <orphan@voidlinux.org>"
+maintainer="tibequadorian <tibequadorian@posteo.de>"
 license="GPL-2.0-or-later"
 homepage="http://ardour.org"
-distfiles="https://community.ardour.org/src/Ardour-${version}.tar.bz2"
-checksum=ac981ec7565fa8a171116bc2fbeb3198a01aad4b7d7c846f965e074a8a162bdb
 
 case "$XBPS_TARGET_MACHINE" in
 	x86_64*)  configure_args+=" --dist-target=x86_64" ;;
@@ -31,6 +28,13 @@ case "$XBPS_TARGET_MACHINE" in
 	*)        broken="Don't know build target" ;;
 esac
 
+# Upstream deletes older release tarballs from their site and
+# GitHub tarballs are empty: https://tracker.ardour.org/view.php?id=7328
+do_fetch() {
+	git clone --branch ${version} --depth=1 \
+		git://github.com/Ardour/ardour ${pkgname}-${version}
+}
+
 post_install() {
 	vinstall build/gtk2_ardour/ardour.xml 644 usr/share/mime/packages
 	vinstall "build/gtk2_ardour/ardour${version%%.*}.desktop" \

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

* Re: [PR PATCH] [Updated] ardour: update to 6.6.
  2021-02-28 14:45 [PR PATCH] ardour: update to 6.6 tibequadorian
                   ` (18 preceding siblings ...)
  2021-03-01 15:51 ` [PR PATCH] [Updated] " tibequadorian
@ 2021-03-01 15:52 ` tibequadorian
  2021-03-01 16:00 ` tibequadorian
                   ` (21 subsequent siblings)
  41 siblings, 0 replies; 43+ messages in thread
From: tibequadorian @ 2021-03-01 15:52 UTC (permalink / raw)
  To: ml

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

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

https://github.com/tibequadorian/void-packages ardour
https://github.com/void-linux/void-packages/pull/29120

ardour: update to 6.6.
adopt package
fetch source from github mirror because upstream deletes older source archives

<!-- Mark items with [x] where applicable -->

#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)

#### Have the results of the proposed changes been tested?
- [x] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR

<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->
<!-- 
#### Does it build and run successfully? 
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [ ] I built this PR locally for my native architecture, (ARCH-LIBC)
- [ ] I built this PR locally for these architectures (if supported. mark crossbuilds):
  - [ ] aarch64-musl
  - [ ] armv7l
  - [ ] armv6l-musl
-->


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

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

From a4fb46aad7b95b28dc02d6c5ddec1bce2567517f Mon Sep 17 00:00:00 2001
From: tibequadorian <tibequadorian@posteo.de>
Date: Sun, 28 Feb 2021 15:07:24 +0100
Subject: [PATCH] ardour: update to 6.6.

adopt package
enable portaudio backend
fetch source from github mirror because upstream deletes older source archives
---
 .../patches/fix-musl-platform-detection.patch | 13 ------------
 srcpkgs/ardour/template                       | 20 +++++++++++--------
 2 files changed, 12 insertions(+), 21 deletions(-)
 delete mode 100644 srcpkgs/ardour/patches/fix-musl-platform-detection.patch

diff --git a/srcpkgs/ardour/patches/fix-musl-platform-detection.patch b/srcpkgs/ardour/patches/fix-musl-platform-detection.patch
deleted file mode 100644
index 8f2d29cc423..00000000000
--- a/srcpkgs/ardour/patches/fix-musl-platform-detection.patch
+++ /dev/null
@@ -1,13 +0,0 @@
-On musl libc only __linux__ is defined
-
---- libs/vst3/pluginterfaces/base/fplatform.h
-+++ libs/vst3/pluginterfaces/base/fplatform.h
-@@ -86,7 +86,7 @@
- //-----------------------------------------------------------------------------
- // LINUX
- //-----------------------------------------------------------------------------
--#elif __gnu_linux__
-+#elif __linux__
- 	#define SMTG_OS_LINUX	1
- 	#define SMTG_OS_MACOS	0
- 	#define SMTG_OS_WINDOWS	0
diff --git a/srcpkgs/ardour/template b/srcpkgs/ardour/template
index 7fb37a36769..67a92e23ee2 100644
--- a/srcpkgs/ardour/template
+++ b/srcpkgs/ardour/template
@@ -1,25 +1,22 @@
 # Template file for 'ardour'
 pkgname=ardour
-version=6.5.0
+version=6.6
 revision=1
-wrksrc="Ardour-${version}"
 build_style=waf3
-configure_args="--cxx11 --no-phone-home --with-backends=jack,alsa,dummy
+configure_args="--cxx11 --no-phone-home --with-backends=jack,alsa,dummy,portaudio
  --libjack=weak --optimize --docs --use-external-libs --freedesktop"
-hostmakedepends="python3 graphviz doxygen pkg-config clang perl
+hostmakedepends="git python3 graphviz doxygen pkg-config clang perl
  itstool gettext tar readline-devel"
 makedepends="boost-devel liblrdf-devel liblo-devel lilv-devel suil-devel
  taglib-devel aubio-devel rubberband-devel alsa-lib-devel libsndfile-devel
  vamp-plugin-sdk-devel fftw-devel jack-devel libsamplerate-devel lv2
  sratom-devel gtkmm2-devel libarchive-devel fluidsynth-devel hidapi-devel
- libltc-devel glibmm-devel"
+ libltc-devel glibmm-devel portaudio-devel"
 depends="desktop-file-utils"
 short_desc="Professional-grade digital audio workstation"
-maintainer="Orphaned <orphan@voidlinux.org>"
+maintainer="tibequadorian <tibequadorian@posteo.de>"
 license="GPL-2.0-or-later"
 homepage="http://ardour.org"
-distfiles="https://community.ardour.org/src/Ardour-${version}.tar.bz2"
-checksum=ac981ec7565fa8a171116bc2fbeb3198a01aad4b7d7c846f965e074a8a162bdb
 
 case "$XBPS_TARGET_MACHINE" in
 	x86_64*)  configure_args+=" --dist-target=x86_64" ;;
@@ -31,6 +28,13 @@ case "$XBPS_TARGET_MACHINE" in
 	*)        broken="Don't know build target" ;;
 esac
 
+# Upstream deletes older release tarballs from their site and
+# GitHub tarballs are empty: https://tracker.ardour.org/view.php?id=7328
+do_fetch() {
+	git clone --branch ${version} --depth=1 \
+		git://github.com/Ardour/ardour ${pkgname}-${version}
+}
+
 post_install() {
 	vinstall build/gtk2_ardour/ardour.xml 644 usr/share/mime/packages
 	vinstall "build/gtk2_ardour/ardour${version%%.*}.desktop" \

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

* Re: ardour: update to 6.6.
  2021-02-28 14:45 [PR PATCH] ardour: update to 6.6 tibequadorian
                   ` (19 preceding siblings ...)
  2021-03-01 15:52 ` tibequadorian
@ 2021-03-01 16:00 ` tibequadorian
  2021-03-01 16:01 ` [PR PATCH] [Updated] " tibequadorian
                   ` (20 subsequent siblings)
  41 siblings, 0 replies; 43+ messages in thread
From: tibequadorian @ 2021-03-01 16:00 UTC (permalink / raw)
  To: ml

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

New comment by tibequadorian on void-packages repository

https://github.com/void-linux/void-packages/pull/29120#issuecomment-788063257

Comment:
build fails: PortAudio Backend is not for Linux

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

* Re: [PR PATCH] [Updated] ardour: update to 6.6.
  2021-02-28 14:45 [PR PATCH] ardour: update to 6.6 tibequadorian
                   ` (20 preceding siblings ...)
  2021-03-01 16:00 ` tibequadorian
@ 2021-03-01 16:01 ` tibequadorian
  2021-03-08  4:12 ` [PR REVIEW] " ericonr
                   ` (19 subsequent siblings)
  41 siblings, 0 replies; 43+ messages in thread
From: tibequadorian @ 2021-03-01 16:01 UTC (permalink / raw)
  To: ml

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

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

https://github.com/tibequadorian/void-packages ardour
https://github.com/void-linux/void-packages/pull/29120

ardour: update to 6.6.
adopt package
fetch source from github mirror because upstream deletes older source archives

<!-- Mark items with [x] where applicable -->

#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)

#### Have the results of the proposed changes been tested?
- [x] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR

<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->
<!-- 
#### Does it build and run successfully? 
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [ ] I built this PR locally for my native architecture, (ARCH-LIBC)
- [ ] I built this PR locally for these architectures (if supported. mark crossbuilds):
  - [ ] aarch64-musl
  - [ ] armv7l
  - [ ] armv6l-musl
-->


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

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

From 9f30462e958b048d2cc6df10c89a8634fa77f1e4 Mon Sep 17 00:00:00 2001
From: tibequadorian <tibequadorian@posteo.de>
Date: Sun, 28 Feb 2021 15:07:24 +0100
Subject: [PATCH] ardour: update to 6.6.

adopt package
fetch source from github mirror because upstream deletes older source archives
---
 .../patches/fix-musl-platform-detection.patch    | 13 -------------
 srcpkgs/ardour/template                          | 16 ++++++++++------
 2 files changed, 10 insertions(+), 19 deletions(-)
 delete mode 100644 srcpkgs/ardour/patches/fix-musl-platform-detection.patch

diff --git a/srcpkgs/ardour/patches/fix-musl-platform-detection.patch b/srcpkgs/ardour/patches/fix-musl-platform-detection.patch
deleted file mode 100644
index 8f2d29cc423..00000000000
--- a/srcpkgs/ardour/patches/fix-musl-platform-detection.patch
+++ /dev/null
@@ -1,13 +0,0 @@
-On musl libc only __linux__ is defined
-
---- libs/vst3/pluginterfaces/base/fplatform.h
-+++ libs/vst3/pluginterfaces/base/fplatform.h
-@@ -86,7 +86,7 @@
- //-----------------------------------------------------------------------------
- // LINUX
- //-----------------------------------------------------------------------------
--#elif __gnu_linux__
-+#elif __linux__
- 	#define SMTG_OS_LINUX	1
- 	#define SMTG_OS_MACOS	0
- 	#define SMTG_OS_WINDOWS	0
diff --git a/srcpkgs/ardour/template b/srcpkgs/ardour/template
index 7fb37a36769..a62f41544dc 100644
--- a/srcpkgs/ardour/template
+++ b/srcpkgs/ardour/template
@@ -1,12 +1,11 @@
 # Template file for 'ardour'
 pkgname=ardour
-version=6.5.0
+version=6.6
 revision=1
-wrksrc="Ardour-${version}"
 build_style=waf3
 configure_args="--cxx11 --no-phone-home --with-backends=jack,alsa,dummy
  --libjack=weak --optimize --docs --use-external-libs --freedesktop"
-hostmakedepends="python3 graphviz doxygen pkg-config clang perl
+hostmakedepends="git python3 graphviz doxygen pkg-config clang perl
  itstool gettext tar readline-devel"
 makedepends="boost-devel liblrdf-devel liblo-devel lilv-devel suil-devel
  taglib-devel aubio-devel rubberband-devel alsa-lib-devel libsndfile-devel
@@ -15,11 +14,9 @@ makedepends="boost-devel liblrdf-devel liblo-devel lilv-devel suil-devel
  libltc-devel glibmm-devel"
 depends="desktop-file-utils"
 short_desc="Professional-grade digital audio workstation"
-maintainer="Orphaned <orphan@voidlinux.org>"
+maintainer="tibequadorian <tibequadorian@posteo.de>"
 license="GPL-2.0-or-later"
 homepage="http://ardour.org"
-distfiles="https://community.ardour.org/src/Ardour-${version}.tar.bz2"
-checksum=ac981ec7565fa8a171116bc2fbeb3198a01aad4b7d7c846f965e074a8a162bdb
 
 case "$XBPS_TARGET_MACHINE" in
 	x86_64*)  configure_args+=" --dist-target=x86_64" ;;
@@ -31,6 +28,13 @@ case "$XBPS_TARGET_MACHINE" in
 	*)        broken="Don't know build target" ;;
 esac
 
+# Upstream deletes older release tarballs from their site and
+# GitHub tarballs are empty: https://tracker.ardour.org/view.php?id=7328
+do_fetch() {
+	git clone --branch ${version} --depth=1 \
+		git://github.com/Ardour/ardour ${pkgname}-${version}
+}
+
 post_install() {
 	vinstall build/gtk2_ardour/ardour.xml 644 usr/share/mime/packages
 	vinstall "build/gtk2_ardour/ardour${version%%.*}.desktop" \

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

* Re: [PR REVIEW] ardour: update to 6.6.
  2021-02-28 14:45 [PR PATCH] ardour: update to 6.6 tibequadorian
                   ` (21 preceding siblings ...)
  2021-03-01 16:01 ` [PR PATCH] [Updated] " tibequadorian
@ 2021-03-08  4:12 ` ericonr
  2021-03-08  9:01 ` tibequadorian
                   ` (18 subsequent siblings)
  41 siblings, 0 replies; 43+ messages in thread
From: ericonr @ 2021-03-08  4:12 UTC (permalink / raw)
  To: ml

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

New review comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/29120#discussion_r589159233

Comment:
Did you manage to clone a specific hash? If that doesn't work but the repository isn't huge, we could probably clone the entire thing and then check it out. Since retagging is possible, I'd prefer to have a unique identifier.

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

* Re: [PR REVIEW] ardour: update to 6.6.
  2021-02-28 14:45 [PR PATCH] ardour: update to 6.6 tibequadorian
                   ` (22 preceding siblings ...)
  2021-03-08  4:12 ` [PR REVIEW] " ericonr
@ 2021-03-08  9:01 ` tibequadorian
  2021-03-08  9:04 ` tibequadorian
                   ` (17 subsequent siblings)
  41 siblings, 0 replies; 43+ messages in thread
From: tibequadorian @ 2021-03-08  9:01 UTC (permalink / raw)
  To: ml

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

New review comment by tibequadorian on void-packages repository

https://github.com/void-linux/void-packages/pull/29120#discussion_r589262888

Comment:
>Did you manage to clone a specific hash?

No. Whats wrong with using the tag directly?

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

* Re: [PR REVIEW] ardour: update to 6.6.
  2021-02-28 14:45 [PR PATCH] ardour: update to 6.6 tibequadorian
                   ` (23 preceding siblings ...)
  2021-03-08  9:01 ` tibequadorian
@ 2021-03-08  9:04 ` tibequadorian
  2021-03-08  9:05 ` tibequadorian
                   ` (16 subsequent siblings)
  41 siblings, 0 replies; 43+ messages in thread
From: tibequadorian @ 2021-03-08  9:04 UTC (permalink / raw)
  To: ml

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

New review comment by tibequadorian on void-packages repository

https://github.com/void-linux/void-packages/pull/29120#discussion_r589262888

Comment:
>Did you manage to clone a specific hash?

No.

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

* Re: [PR REVIEW] ardour: update to 6.6.
  2021-02-28 14:45 [PR PATCH] ardour: update to 6.6 tibequadorian
                   ` (24 preceding siblings ...)
  2021-03-08  9:04 ` tibequadorian
@ 2021-03-08  9:05 ` tibequadorian
  2021-03-08  9:09 ` tibequadorian
                   ` (15 subsequent siblings)
  41 siblings, 0 replies; 43+ messages in thread
From: tibequadorian @ 2021-03-08  9:05 UTC (permalink / raw)
  To: ml

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

New review comment by tibequadorian on void-packages repository

https://github.com/void-linux/void-packages/pull/29120#discussion_r589265218

Comment:
But they won't retag. From https://tracker.ardour.org/view.php?id=7328:

>git-tag is the very first step in the release procedure. If some issues or regressions are found during the release process, the tag is skipped (e.g. there was no Ardour 5.7 release; 5.7-2 became 5.8)

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

* Re: [PR REVIEW] ardour: update to 6.6.
  2021-02-28 14:45 [PR PATCH] ardour: update to 6.6 tibequadorian
                   ` (25 preceding siblings ...)
  2021-03-08  9:05 ` tibequadorian
@ 2021-03-08  9:09 ` tibequadorian
  2021-03-08  9:17 ` tibequadorian
                   ` (14 subsequent siblings)
  41 siblings, 0 replies; 43+ messages in thread
From: tibequadorian @ 2021-03-08  9:09 UTC (permalink / raw)
  To: ml

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

New review comment by tibequadorian on void-packages repository

https://github.com/void-linux/void-packages/pull/29120#discussion_r589267566

Comment:
I mean the repository isn't huge, but ... large? Takes around a minute to clone on my computer...

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

* Re: [PR REVIEW] ardour: update to 6.6.
  2021-02-28 14:45 [PR PATCH] ardour: update to 6.6 tibequadorian
                   ` (26 preceding siblings ...)
  2021-03-08  9:09 ` tibequadorian
@ 2021-03-08  9:17 ` tibequadorian
  2021-03-08  9:44 ` [PR PATCH] [Updated] " tibequadorian
                   ` (13 subsequent siblings)
  41 siblings, 0 replies; 43+ messages in thread
From: tibequadorian @ 2021-03-08  9:17 UTC (permalink / raw)
  To: ml

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

New review comment by tibequadorian on void-packages repository

https://github.com/void-linux/void-packages/pull/29120#discussion_r589271985

Comment:
I'd rather check if the commit hash is correct?

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

* Re: [PR PATCH] [Updated] ardour: update to 6.6.
  2021-02-28 14:45 [PR PATCH] ardour: update to 6.6 tibequadorian
                   ` (27 preceding siblings ...)
  2021-03-08  9:17 ` tibequadorian
@ 2021-03-08  9:44 ` tibequadorian
  2021-03-08  9:47 ` tibequadorian
                   ` (12 subsequent siblings)
  41 siblings, 0 replies; 43+ messages in thread
From: tibequadorian @ 2021-03-08  9:44 UTC (permalink / raw)
  To: ml

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

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

https://github.com/tibequadorian/void-packages ardour
https://github.com/void-linux/void-packages/pull/29120

ardour: update to 6.6.
adopt package
fetch source from github mirror because upstream deletes older source archives

<!-- Mark items with [x] where applicable -->

#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)

#### Have the results of the proposed changes been tested?
- [x] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR

<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->
<!-- 
#### Does it build and run successfully? 
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [ ] I built this PR locally for my native architecture, (ARCH-LIBC)
- [ ] I built this PR locally for these architectures (if supported. mark crossbuilds):
  - [ ] aarch64-musl
  - [ ] armv7l
  - [ ] armv6l-musl
-->


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

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

From 0316048e4912150c0d320f17f9d9868c3c0f092b Mon Sep 17 00:00:00 2001
From: tibequadorian <tibequadorian@posteo.de>
Date: Mon, 8 Mar 2021 10:27:01 +0100
Subject: [PATCH] ardour: update to 6.6.

---
 .../patches/fix-musl-platform-detection.patch | 13 ------------
 srcpkgs/ardour/template                       | 21 +++++++++++++------
 2 files changed, 15 insertions(+), 19 deletions(-)
 delete mode 100644 srcpkgs/ardour/patches/fix-musl-platform-detection.patch

diff --git a/srcpkgs/ardour/patches/fix-musl-platform-detection.patch b/srcpkgs/ardour/patches/fix-musl-platform-detection.patch
deleted file mode 100644
index 8f2d29cc423..00000000000
--- a/srcpkgs/ardour/patches/fix-musl-platform-detection.patch
+++ /dev/null
@@ -1,13 +0,0 @@
-On musl libc only __linux__ is defined
-
---- libs/vst3/pluginterfaces/base/fplatform.h
-+++ libs/vst3/pluginterfaces/base/fplatform.h
-@@ -86,7 +86,7 @@
- //-----------------------------------------------------------------------------
- // LINUX
- //-----------------------------------------------------------------------------
--#elif __gnu_linux__
-+#elif __linux__
- 	#define SMTG_OS_LINUX	1
- 	#define SMTG_OS_MACOS	0
- 	#define SMTG_OS_WINDOWS	0
diff --git a/srcpkgs/ardour/template b/srcpkgs/ardour/template
index 7fb37a36769..d1c1fdee2f1 100644
--- a/srcpkgs/ardour/template
+++ b/srcpkgs/ardour/template
@@ -1,12 +1,12 @@
 # Template file for 'ardour'
 pkgname=ardour
-version=6.5.0
+version=6.6
+_commit="e4e21f4d073ab00b1a0bb6ff6ca49f28b02fd68a"
 revision=1
-wrksrc="Ardour-${version}"
 build_style=waf3
 configure_args="--cxx11 --no-phone-home --with-backends=jack,alsa,dummy
  --libjack=weak --optimize --docs --use-external-libs --freedesktop"
-hostmakedepends="python3 graphviz doxygen pkg-config clang perl
+hostmakedepends="git python3 graphviz doxygen pkg-config clang perl
  itstool gettext tar readline-devel"
 makedepends="boost-devel liblrdf-devel liblo-devel lilv-devel suil-devel
  taglib-devel aubio-devel rubberband-devel alsa-lib-devel libsndfile-devel
@@ -15,11 +15,9 @@ makedepends="boost-devel liblrdf-devel liblo-devel lilv-devel suil-devel
  libltc-devel glibmm-devel"
 depends="desktop-file-utils"
 short_desc="Professional-grade digital audio workstation"
-maintainer="Orphaned <orphan@voidlinux.org>"
+maintainer="tibequadorian <tibequadorian@posteo.de>"
 license="GPL-2.0-or-later"
 homepage="http://ardour.org"
-distfiles="https://community.ardour.org/src/Ardour-${version}.tar.bz2"
-checksum=ac981ec7565fa8a171116bc2fbeb3198a01aad4b7d7c846f965e074a8a162bdb
 
 case "$XBPS_TARGET_MACHINE" in
 	x86_64*)  configure_args+=" --dist-target=x86_64" ;;
@@ -31,6 +29,17 @@ case "$XBPS_TARGET_MACHINE" in
 	*)        broken="Don't know build target" ;;
 esac
 
+# Upstream deletes older release tarballs from their site and
+# GitHub tarballs are empty: https://tracker.ardour.org/view.php?id=7328
+do_fetch() {
+	git clone --branch ${version} --depth=1 \
+		git://github.com/Ardour/ardour ${pkgname}-${version}
+	if [ "$(git rev-parse HEAD)" != "${_commit}" ]; then
+		rm -rf ${pkgname}-${version}
+		msg_error "Veryfing commit hash failed\n"
+	fi
+}
+
 post_install() {
 	vinstall build/gtk2_ardour/ardour.xml 644 usr/share/mime/packages
 	vinstall "build/gtk2_ardour/ardour${version%%.*}.desktop" \

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

* Re: [PR PATCH] [Updated] ardour: update to 6.6.
  2021-02-28 14:45 [PR PATCH] ardour: update to 6.6 tibequadorian
                   ` (28 preceding siblings ...)
  2021-03-08  9:44 ` [PR PATCH] [Updated] " tibequadorian
@ 2021-03-08  9:47 ` tibequadorian
  2021-03-08  9:54 ` tibequadorian
                   ` (11 subsequent siblings)
  41 siblings, 0 replies; 43+ messages in thread
From: tibequadorian @ 2021-03-08  9:47 UTC (permalink / raw)
  To: ml

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

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

https://github.com/tibequadorian/void-packages ardour
https://github.com/void-linux/void-packages/pull/29120

ardour: update to 6.6.
adopt package
fetch source from github mirror because upstream deletes older source archives

<!-- Mark items with [x] where applicable -->

#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)

#### Have the results of the proposed changes been tested?
- [x] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR

<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->
<!-- 
#### Does it build and run successfully? 
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [ ] I built this PR locally for my native architecture, (ARCH-LIBC)
- [ ] I built this PR locally for these architectures (if supported. mark crossbuilds):
  - [ ] aarch64-musl
  - [ ] armv7l
  - [ ] armv6l-musl
-->


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

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

From f3a9c0a51a4a535efcc4bc44cd163d9d3e11cdee Mon Sep 17 00:00:00 2001
From: tibequadorian <tibequadorian@posteo.de>
Date: Mon, 8 Mar 2021 10:27:01 +0100
Subject: [PATCH] ardour: update to 6.6.

---
 .../patches/fix-musl-platform-detection.patch | 13 ------------
 srcpkgs/ardour/template                       | 21 +++++++++++++------
 2 files changed, 15 insertions(+), 19 deletions(-)
 delete mode 100644 srcpkgs/ardour/patches/fix-musl-platform-detection.patch

diff --git a/srcpkgs/ardour/patches/fix-musl-platform-detection.patch b/srcpkgs/ardour/patches/fix-musl-platform-detection.patch
deleted file mode 100644
index 8f2d29cc423..00000000000
--- a/srcpkgs/ardour/patches/fix-musl-platform-detection.patch
+++ /dev/null
@@ -1,13 +0,0 @@
-On musl libc only __linux__ is defined
-
---- libs/vst3/pluginterfaces/base/fplatform.h
-+++ libs/vst3/pluginterfaces/base/fplatform.h
-@@ -86,7 +86,7 @@
- //-----------------------------------------------------------------------------
- // LINUX
- //-----------------------------------------------------------------------------
--#elif __gnu_linux__
-+#elif __linux__
- 	#define SMTG_OS_LINUX	1
- 	#define SMTG_OS_MACOS	0
- 	#define SMTG_OS_WINDOWS	0
diff --git a/srcpkgs/ardour/template b/srcpkgs/ardour/template
index 7fb37a36769..6ed5e785ae0 100644
--- a/srcpkgs/ardour/template
+++ b/srcpkgs/ardour/template
@@ -1,12 +1,12 @@
 # Template file for 'ardour'
 pkgname=ardour
-version=6.5.0
+version=6.6
 revision=1
-wrksrc="Ardour-${version}"
+_commit="e4e21f4d073ab00b1a0bb6ff6ca49f28b02fd68a"
 build_style=waf3
 configure_args="--cxx11 --no-phone-home --with-backends=jack,alsa,dummy
  --libjack=weak --optimize --docs --use-external-libs --freedesktop"
-hostmakedepends="python3 graphviz doxygen pkg-config clang perl
+hostmakedepends="git python3 graphviz doxygen pkg-config clang perl
  itstool gettext tar readline-devel"
 makedepends="boost-devel liblrdf-devel liblo-devel lilv-devel suil-devel
  taglib-devel aubio-devel rubberband-devel alsa-lib-devel libsndfile-devel
@@ -15,11 +15,9 @@ makedepends="boost-devel liblrdf-devel liblo-devel lilv-devel suil-devel
  libltc-devel glibmm-devel"
 depends="desktop-file-utils"
 short_desc="Professional-grade digital audio workstation"
-maintainer="Orphaned <orphan@voidlinux.org>"
+maintainer="tibequadorian <tibequadorian@posteo.de>"
 license="GPL-2.0-or-later"
 homepage="http://ardour.org"
-distfiles="https://community.ardour.org/src/Ardour-${version}.tar.bz2"
-checksum=ac981ec7565fa8a171116bc2fbeb3198a01aad4b7d7c846f965e074a8a162bdb
 
 case "$XBPS_TARGET_MACHINE" in
 	x86_64*)  configure_args+=" --dist-target=x86_64" ;;
@@ -31,6 +29,17 @@ case "$XBPS_TARGET_MACHINE" in
 	*)        broken="Don't know build target" ;;
 esac
 
+# Upstream deletes older release tarballs from their site and
+# GitHub tarballs are empty: https://tracker.ardour.org/view.php?id=7328
+do_fetch() {
+	git clone --branch ${version} --depth=1 \
+		git://github.com/Ardour/ardour ${pkgname}-${version}
+	if [ "$(git rev-parse HEAD)" != "${_commit}" ]; then
+		rm -rf ${pkgname}-${version}
+		msg_error "Veryfing commit hash failed\n"
+	fi
+}
+
 post_install() {
 	vinstall build/gtk2_ardour/ardour.xml 644 usr/share/mime/packages
 	vinstall "build/gtk2_ardour/ardour${version%%.*}.desktop" \

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

* Re: [PR PATCH] [Updated] ardour: update to 6.6.
  2021-02-28 14:45 [PR PATCH] ardour: update to 6.6 tibequadorian
                   ` (29 preceding siblings ...)
  2021-03-08  9:47 ` tibequadorian
@ 2021-03-08  9:54 ` tibequadorian
  2021-03-08  9:55 ` [PR REVIEW] " tibequadorian
                   ` (10 subsequent siblings)
  41 siblings, 0 replies; 43+ messages in thread
From: tibequadorian @ 2021-03-08  9:54 UTC (permalink / raw)
  To: ml

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

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

https://github.com/tibequadorian/void-packages ardour
https://github.com/void-linux/void-packages/pull/29120

ardour: update to 6.6.
adopt package
fetch source from github mirror because upstream deletes older source archives

<!-- Mark items with [x] where applicable -->

#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)

#### Have the results of the proposed changes been tested?
- [x] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR

<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->
<!-- 
#### Does it build and run successfully? 
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [ ] I built this PR locally for my native architecture, (ARCH-LIBC)
- [ ] I built this PR locally for these architectures (if supported. mark crossbuilds):
  - [ ] aarch64-musl
  - [ ] armv7l
  - [ ] armv6l-musl
-->


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

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

From 09799eadcf8794b0cf4e75b0414fa69193704a17 Mon Sep 17 00:00:00 2001
From: tibequadorian <tibequadorian@posteo.de>
Date: Mon, 8 Mar 2021 10:27:01 +0100
Subject: [PATCH] ardour: update to 6.6.

---
 .../patches/fix-musl-platform-detection.patch    | 13 -------------
 srcpkgs/ardour/template                          | 16 ++++++++++------
 2 files changed, 10 insertions(+), 19 deletions(-)
 delete mode 100644 srcpkgs/ardour/patches/fix-musl-platform-detection.patch

diff --git a/srcpkgs/ardour/patches/fix-musl-platform-detection.patch b/srcpkgs/ardour/patches/fix-musl-platform-detection.patch
deleted file mode 100644
index 8f2d29cc423..00000000000
--- a/srcpkgs/ardour/patches/fix-musl-platform-detection.patch
+++ /dev/null
@@ -1,13 +0,0 @@
-On musl libc only __linux__ is defined
-
---- libs/vst3/pluginterfaces/base/fplatform.h
-+++ libs/vst3/pluginterfaces/base/fplatform.h
-@@ -86,7 +86,7 @@
- //-----------------------------------------------------------------------------
- // LINUX
- //-----------------------------------------------------------------------------
--#elif __gnu_linux__
-+#elif __linux__
- 	#define SMTG_OS_LINUX	1
- 	#define SMTG_OS_MACOS	0
- 	#define SMTG_OS_WINDOWS	0
diff --git a/srcpkgs/ardour/template b/srcpkgs/ardour/template
index 7fb37a36769..a62f41544dc 100644
--- a/srcpkgs/ardour/template
+++ b/srcpkgs/ardour/template
@@ -1,12 +1,11 @@
 # Template file for 'ardour'
 pkgname=ardour
-version=6.5.0
+version=6.6
 revision=1
-wrksrc="Ardour-${version}"
 build_style=waf3
 configure_args="--cxx11 --no-phone-home --with-backends=jack,alsa,dummy
  --libjack=weak --optimize --docs --use-external-libs --freedesktop"
-hostmakedepends="python3 graphviz doxygen pkg-config clang perl
+hostmakedepends="git python3 graphviz doxygen pkg-config clang perl
  itstool gettext tar readline-devel"
 makedepends="boost-devel liblrdf-devel liblo-devel lilv-devel suil-devel
  taglib-devel aubio-devel rubberband-devel alsa-lib-devel libsndfile-devel
@@ -15,11 +14,9 @@ makedepends="boost-devel liblrdf-devel liblo-devel lilv-devel suil-devel
  libltc-devel glibmm-devel"
 depends="desktop-file-utils"
 short_desc="Professional-grade digital audio workstation"
-maintainer="Orphaned <orphan@voidlinux.org>"
+maintainer="tibequadorian <tibequadorian@posteo.de>"
 license="GPL-2.0-or-later"
 homepage="http://ardour.org"
-distfiles="https://community.ardour.org/src/Ardour-${version}.tar.bz2"
-checksum=ac981ec7565fa8a171116bc2fbeb3198a01aad4b7d7c846f965e074a8a162bdb
 
 case "$XBPS_TARGET_MACHINE" in
 	x86_64*)  configure_args+=" --dist-target=x86_64" ;;
@@ -31,6 +28,13 @@ case "$XBPS_TARGET_MACHINE" in
 	*)        broken="Don't know build target" ;;
 esac
 
+# Upstream deletes older release tarballs from their site and
+# GitHub tarballs are empty: https://tracker.ardour.org/view.php?id=7328
+do_fetch() {
+	git clone --branch ${version} --depth=1 \
+		git://github.com/Ardour/ardour ${pkgname}-${version}
+}
+
 post_install() {
 	vinstall build/gtk2_ardour/ardour.xml 644 usr/share/mime/packages
 	vinstall "build/gtk2_ardour/ardour${version%%.*}.desktop" \

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

* Re: [PR REVIEW] ardour: update to 6.6.
  2021-02-28 14:45 [PR PATCH] ardour: update to 6.6 tibequadorian
                   ` (30 preceding siblings ...)
  2021-03-08  9:54 ` tibequadorian
@ 2021-03-08  9:55 ` tibequadorian
  2021-03-08  9:57 ` [PR PATCH] [Updated] " tibequadorian
                   ` (9 subsequent siblings)
  41 siblings, 0 replies; 43+ messages in thread
From: tibequadorian @ 2021-03-08  9:55 UTC (permalink / raw)
  To: ml

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

New review comment by tibequadorian on void-packages repository

https://github.com/void-linux/void-packages/pull/29120#discussion_r589271985

Comment:
I'd rather check if the commit hash is correct?

EDIT: doesn't work :/

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

* Re: [PR PATCH] [Updated] ardour: update to 6.6.
  2021-02-28 14:45 [PR PATCH] ardour: update to 6.6 tibequadorian
                   ` (31 preceding siblings ...)
  2021-03-08  9:55 ` [PR REVIEW] " tibequadorian
@ 2021-03-08  9:57 ` tibequadorian
  2021-03-08 10:01 ` tibequadorian
                   ` (8 subsequent siblings)
  41 siblings, 0 replies; 43+ messages in thread
From: tibequadorian @ 2021-03-08  9:57 UTC (permalink / raw)
  To: ml

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

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

https://github.com/tibequadorian/void-packages ardour
https://github.com/void-linux/void-packages/pull/29120

ardour: update to 6.6.
adopt package
fetch source from github mirror because upstream deletes older source archives

<!-- Mark items with [x] where applicable -->

#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)

#### Have the results of the proposed changes been tested?
- [x] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR

<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->
<!-- 
#### Does it build and run successfully? 
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [ ] I built this PR locally for my native architecture, (ARCH-LIBC)
- [ ] I built this PR locally for these architectures (if supported. mark crossbuilds):
  - [ ] aarch64-musl
  - [ ] armv7l
  - [ ] armv6l-musl
-->


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

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

From 119823d0e72845539dc5cea5ab83f16d7294f322 Mon Sep 17 00:00:00 2001
From: tibequadorian <tibequadorian@posteo.de>
Date: Mon, 8 Mar 2021 10:27:01 +0100
Subject: [PATCH] ardour: update to 6.6.

---
 .../patches/fix-musl-platform-detection.patch   | 13 -------------
 srcpkgs/ardour/template                         | 17 +++++++++++------
 2 files changed, 11 insertions(+), 19 deletions(-)
 delete mode 100644 srcpkgs/ardour/patches/fix-musl-platform-detection.patch

diff --git a/srcpkgs/ardour/patches/fix-musl-platform-detection.patch b/srcpkgs/ardour/patches/fix-musl-platform-detection.patch
deleted file mode 100644
index 8f2d29cc423..00000000000
--- a/srcpkgs/ardour/patches/fix-musl-platform-detection.patch
+++ /dev/null
@@ -1,13 +0,0 @@
-On musl libc only __linux__ is defined
-
---- libs/vst3/pluginterfaces/base/fplatform.h
-+++ libs/vst3/pluginterfaces/base/fplatform.h
-@@ -86,7 +86,7 @@
- //-----------------------------------------------------------------------------
- // LINUX
- //-----------------------------------------------------------------------------
--#elif __gnu_linux__
-+#elif __linux__
- 	#define SMTG_OS_LINUX	1
- 	#define SMTG_OS_MACOS	0
- 	#define SMTG_OS_WINDOWS	0
diff --git a/srcpkgs/ardour/template b/srcpkgs/ardour/template
index 7fb37a36769..725d6a92fbf 100644
--- a/srcpkgs/ardour/template
+++ b/srcpkgs/ardour/template
@@ -1,12 +1,12 @@
 # Template file for 'ardour'
 pkgname=ardour
-version=6.5.0
+version=6.6
 revision=1
-wrksrc="Ardour-${version}"
+_commit="e4e21f4d073ab00b1a0bb6ff6ca49f28b02fd68a"
 build_style=waf3
 configure_args="--cxx11 --no-phone-home --with-backends=jack,alsa,dummy
  --libjack=weak --optimize --docs --use-external-libs --freedesktop"
-hostmakedepends="python3 graphviz doxygen pkg-config clang perl
+hostmakedepends="git python3 graphviz doxygen pkg-config clang perl
  itstool gettext tar readline-devel"
 makedepends="boost-devel liblrdf-devel liblo-devel lilv-devel suil-devel
  taglib-devel aubio-devel rubberband-devel alsa-lib-devel libsndfile-devel
@@ -15,11 +15,9 @@ makedepends="boost-devel liblrdf-devel liblo-devel lilv-devel suil-devel
  libltc-devel glibmm-devel"
 depends="desktop-file-utils"
 short_desc="Professional-grade digital audio workstation"
-maintainer="Orphaned <orphan@voidlinux.org>"
+maintainer="tibequadorian <tibequadorian@posteo.de>"
 license="GPL-2.0-or-later"
 homepage="http://ardour.org"
-distfiles="https://community.ardour.org/src/Ardour-${version}.tar.bz2"
-checksum=ac981ec7565fa8a171116bc2fbeb3198a01aad4b7d7c846f965e074a8a162bdb
 
 case "$XBPS_TARGET_MACHINE" in
 	x86_64*)  configure_args+=" --dist-target=x86_64" ;;
@@ -31,6 +29,13 @@ case "$XBPS_TARGET_MACHINE" in
 	*)        broken="Don't know build target" ;;
 esac
 
+# Upstream deletes older release tarballs from their site and
+# GitHub tarballs are empty: https://tracker.ardour.org/view.php?id=7328
+do_fetch() {
+	git clone git://github.com/Ardour/ardour ${pkgname}-${version}
+	git checkout ${_commit}
+}
+
 post_install() {
 	vinstall build/gtk2_ardour/ardour.xml 644 usr/share/mime/packages
 	vinstall "build/gtk2_ardour/ardour${version%%.*}.desktop" \

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

* Re: [PR PATCH] [Updated] ardour: update to 6.6.
  2021-02-28 14:45 [PR PATCH] ardour: update to 6.6 tibequadorian
                   ` (32 preceding siblings ...)
  2021-03-08  9:57 ` [PR PATCH] [Updated] " tibequadorian
@ 2021-03-08 10:01 ` tibequadorian
  2021-03-08 10:04 ` [PR REVIEW] " tibequadorian
                   ` (7 subsequent siblings)
  41 siblings, 0 replies; 43+ messages in thread
From: tibequadorian @ 2021-03-08 10:01 UTC (permalink / raw)
  To: ml

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

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

https://github.com/tibequadorian/void-packages ardour
https://github.com/void-linux/void-packages/pull/29120

ardour: update to 6.6.
adopt package
fetch source from github mirror because upstream deletes older source archives

<!-- Mark items with [x] where applicable -->

#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)

#### Have the results of the proposed changes been tested?
- [x] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR

<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->
<!-- 
#### Does it build and run successfully? 
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [ ] I built this PR locally for my native architecture, (ARCH-LIBC)
- [ ] I built this PR locally for these architectures (if supported. mark crossbuilds):
  - [ ] aarch64-musl
  - [ ] armv7l
  - [ ] armv6l-musl
-->


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

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

From d7ac197d537e73e3427859f1f4f6a35ba205243c Mon Sep 17 00:00:00 2001
From: tibequadorian <tibequadorian@posteo.de>
Date: Mon, 8 Mar 2021 10:27:01 +0100
Subject: [PATCH] ardour: update to 6.6.

---
 .../patches/fix-musl-platform-detection.patch  | 13 -------------
 srcpkgs/ardour/template                        | 18 ++++++++++++------
 2 files changed, 12 insertions(+), 19 deletions(-)
 delete mode 100644 srcpkgs/ardour/patches/fix-musl-platform-detection.patch

diff --git a/srcpkgs/ardour/patches/fix-musl-platform-detection.patch b/srcpkgs/ardour/patches/fix-musl-platform-detection.patch
deleted file mode 100644
index 8f2d29cc423..00000000000
--- a/srcpkgs/ardour/patches/fix-musl-platform-detection.patch
+++ /dev/null
@@ -1,13 +0,0 @@
-On musl libc only __linux__ is defined
-
---- libs/vst3/pluginterfaces/base/fplatform.h
-+++ libs/vst3/pluginterfaces/base/fplatform.h
-@@ -86,7 +86,7 @@
- //-----------------------------------------------------------------------------
- // LINUX
- //-----------------------------------------------------------------------------
--#elif __gnu_linux__
-+#elif __linux__
- 	#define SMTG_OS_LINUX	1
- 	#define SMTG_OS_MACOS	0
- 	#define SMTG_OS_WINDOWS	0
diff --git a/srcpkgs/ardour/template b/srcpkgs/ardour/template
index 7fb37a36769..abcb18f819f 100644
--- a/srcpkgs/ardour/template
+++ b/srcpkgs/ardour/template
@@ -1,12 +1,12 @@
 # Template file for 'ardour'
 pkgname=ardour
-version=6.5.0
+version=6.6
 revision=1
-wrksrc="Ardour-${version}"
+_commit="e4e21f4d073ab00b1a0bb6ff6ca49f28b02fd68a"
 build_style=waf3
 configure_args="--cxx11 --no-phone-home --with-backends=jack,alsa,dummy
  --libjack=weak --optimize --docs --use-external-libs --freedesktop"
-hostmakedepends="python3 graphviz doxygen pkg-config clang perl
+hostmakedepends="git python3 graphviz doxygen pkg-config clang perl
  itstool gettext tar readline-devel"
 makedepends="boost-devel liblrdf-devel liblo-devel lilv-devel suil-devel
  taglib-devel aubio-devel rubberband-devel alsa-lib-devel libsndfile-devel
@@ -15,11 +15,9 @@ makedepends="boost-devel liblrdf-devel liblo-devel lilv-devel suil-devel
  libltc-devel glibmm-devel"
 depends="desktop-file-utils"
 short_desc="Professional-grade digital audio workstation"
-maintainer="Orphaned <orphan@voidlinux.org>"
+maintainer="tibequadorian <tibequadorian@posteo.de>"
 license="GPL-2.0-or-later"
 homepage="http://ardour.org"
-distfiles="https://community.ardour.org/src/Ardour-${version}.tar.bz2"
-checksum=ac981ec7565fa8a171116bc2fbeb3198a01aad4b7d7c846f965e074a8a162bdb
 
 case "$XBPS_TARGET_MACHINE" in
 	x86_64*)  configure_args+=" --dist-target=x86_64" ;;
@@ -31,6 +29,14 @@ case "$XBPS_TARGET_MACHINE" in
 	*)        broken="Don't know build target" ;;
 esac
 
+# Upstream deletes older release tarballs from their site and
+# GitHub tarballs are empty: https://tracker.ardour.org/view.php?id=7328
+do_fetch() {
+	git clone git://github.com/Ardour/ardour ${pkgname}-${version}
+	cd ${pkgname}-${version}
+	git checkout ${_commit}
+}
+
 post_install() {
 	vinstall build/gtk2_ardour/ardour.xml 644 usr/share/mime/packages
 	vinstall "build/gtk2_ardour/ardour${version%%.*}.desktop" \

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

* Re: [PR REVIEW] ardour: update to 6.6.
  2021-02-28 14:45 [PR PATCH] ardour: update to 6.6 tibequadorian
                   ` (33 preceding siblings ...)
  2021-03-08 10:01 ` tibequadorian
@ 2021-03-08 10:04 ` tibequadorian
  2021-03-08 10:08 ` tibequadorian
                   ` (6 subsequent siblings)
  41 siblings, 0 replies; 43+ messages in thread
From: tibequadorian @ 2021-03-08 10:04 UTC (permalink / raw)
  To: ml

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

New review comment by tibequadorian on void-packages repository

https://github.com/void-linux/void-packages/pull/29120#discussion_r589303959

Comment:
I did it with the clone and checkout now. But *if* they retag, the build will probably fail because their build relies on `git describe`, see https://tracker.ardour.org/view.php?id=7328

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

* Re: [PR REVIEW] ardour: update to 6.6.
  2021-02-28 14:45 [PR PATCH] ardour: update to 6.6 tibequadorian
                   ` (34 preceding siblings ...)
  2021-03-08 10:04 ` [PR REVIEW] " tibequadorian
@ 2021-03-08 10:08 ` tibequadorian
  2021-03-08 10:40 ` tibequadorian
                   ` (5 subsequent siblings)
  41 siblings, 0 replies; 43+ messages in thread
From: tibequadorian @ 2021-03-08 10:08 UTC (permalink / raw)
  To: ml

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

New review comment by tibequadorian on void-packages repository

https://github.com/void-linux/void-packages/pull/29120#discussion_r589303959

Comment:
I did it with the clone and checkout now. But *if* they retag, the build will probably fail because it relies on `git describe`, see https://tracker.ardour.org/view.php?id=7328

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

* Re: [PR REVIEW] ardour: update to 6.6.
  2021-02-28 14:45 [PR PATCH] ardour: update to 6.6 tibequadorian
                   ` (35 preceding siblings ...)
  2021-03-08 10:08 ` tibequadorian
@ 2021-03-08 10:40 ` tibequadorian
  2021-03-08 15:38 ` ericonr
                   ` (4 subsequent siblings)
  41 siblings, 0 replies; 43+ messages in thread
From: tibequadorian @ 2021-03-08 10:40 UTC (permalink / raw)
  To: ml

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

New review comment by tibequadorian on void-packages repository

https://github.com/void-linux/void-packages/pull/29120#discussion_r589271985

Comment:
I'd rather check if the commit hash is correct?

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

* Re: [PR REVIEW] ardour: update to 6.6.
  2021-02-28 14:45 [PR PATCH] ardour: update to 6.6 tibequadorian
                   ` (36 preceding siblings ...)
  2021-03-08 10:40 ` tibequadorian
@ 2021-03-08 15:38 ` ericonr
  2021-03-08 15:40 ` [PR PATCH] [Updated] " tibequadorian
                   ` (3 subsequent siblings)
  41 siblings, 0 replies; 43+ messages in thread
From: ericonr @ 2021-03-08 15:38 UTC (permalink / raw)
  To: ml

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

New review comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/29120#discussion_r589518907

Comment:
You can use `${wrksrc}` instead of `${pkgname}-${version}`. Then it's good to merge.

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

* Re: [PR PATCH] [Updated] ardour: update to 6.6.
  2021-02-28 14:45 [PR PATCH] ardour: update to 6.6 tibequadorian
                   ` (37 preceding siblings ...)
  2021-03-08 15:38 ` ericonr
@ 2021-03-08 15:40 ` tibequadorian
  2021-03-08 15:41 ` tibequadorian
                   ` (2 subsequent siblings)
  41 siblings, 0 replies; 43+ messages in thread
From: tibequadorian @ 2021-03-08 15:40 UTC (permalink / raw)
  To: ml

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

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

https://github.com/tibequadorian/void-packages ardour
https://github.com/void-linux/void-packages/pull/29120

ardour: update to 6.6.
adopt package
fetch source from github mirror because upstream deletes older source archives

<!-- Mark items with [x] where applicable -->

#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)

#### Have the results of the proposed changes been tested?
- [x] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR

<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->
<!-- 
#### Does it build and run successfully? 
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [ ] I built this PR locally for my native architecture, (ARCH-LIBC)
- [ ] I built this PR locally for these architectures (if supported. mark crossbuilds):
  - [ ] aarch64-musl
  - [ ] armv7l
  - [ ] armv6l-musl
-->


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

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

From 3a6ac4db4ef62aff5dba61dbf2f713d7a25182f5 Mon Sep 17 00:00:00 2001
From: tibequadorian <tibequadorian@posteo.de>
Date: Mon, 8 Mar 2021 10:27:01 +0100
Subject: [PATCH] ardour: update to 6.6.

---
 .../patches/fix-musl-platform-detection.patch  | 13 -------------
 srcpkgs/ardour/template                        | 18 ++++++++++++------
 2 files changed, 12 insertions(+), 19 deletions(-)
 delete mode 100644 srcpkgs/ardour/patches/fix-musl-platform-detection.patch

diff --git a/srcpkgs/ardour/patches/fix-musl-platform-detection.patch b/srcpkgs/ardour/patches/fix-musl-platform-detection.patch
deleted file mode 100644
index 8f2d29cc423..00000000000
--- a/srcpkgs/ardour/patches/fix-musl-platform-detection.patch
+++ /dev/null
@@ -1,13 +0,0 @@
-On musl libc only __linux__ is defined
-
---- libs/vst3/pluginterfaces/base/fplatform.h
-+++ libs/vst3/pluginterfaces/base/fplatform.h
-@@ -86,7 +86,7 @@
- //-----------------------------------------------------------------------------
- // LINUX
- //-----------------------------------------------------------------------------
--#elif __gnu_linux__
-+#elif __linux__
- 	#define SMTG_OS_LINUX	1
- 	#define SMTG_OS_MACOS	0
- 	#define SMTG_OS_WINDOWS	0
diff --git a/srcpkgs/ardour/template b/srcpkgs/ardour/template
index 7fb37a36769..ff1f6ba7f69 100644
--- a/srcpkgs/ardour/template
+++ b/srcpkgs/ardour/template
@@ -1,12 +1,12 @@
 # Template file for 'ardour'
 pkgname=ardour
-version=6.5.0
+version=6.6
 revision=1
-wrksrc="Ardour-${version}"
+_commit="e4e21f4d073ab00b1a0bb6ff6ca49f28b02fd68a"
 build_style=waf3
 configure_args="--cxx11 --no-phone-home --with-backends=jack,alsa,dummy
  --libjack=weak --optimize --docs --use-external-libs --freedesktop"
-hostmakedepends="python3 graphviz doxygen pkg-config clang perl
+hostmakedepends="git python3 graphviz doxygen pkg-config clang perl
  itstool gettext tar readline-devel"
 makedepends="boost-devel liblrdf-devel liblo-devel lilv-devel suil-devel
  taglib-devel aubio-devel rubberband-devel alsa-lib-devel libsndfile-devel
@@ -15,11 +15,9 @@ makedepends="boost-devel liblrdf-devel liblo-devel lilv-devel suil-devel
  libltc-devel glibmm-devel"
 depends="desktop-file-utils"
 short_desc="Professional-grade digital audio workstation"
-maintainer="Orphaned <orphan@voidlinux.org>"
+maintainer="tibequadorian <tibequadorian@posteo.de>"
 license="GPL-2.0-or-later"
 homepage="http://ardour.org"
-distfiles="https://community.ardour.org/src/Ardour-${version}.tar.bz2"
-checksum=ac981ec7565fa8a171116bc2fbeb3198a01aad4b7d7c846f965e074a8a162bdb
 
 case "$XBPS_TARGET_MACHINE" in
 	x86_64*)  configure_args+=" --dist-target=x86_64" ;;
@@ -31,6 +29,14 @@ case "$XBPS_TARGET_MACHINE" in
 	*)        broken="Don't know build target" ;;
 esac
 
+# Upstream deletes older release tarballs from their site and
+# GitHub tarballs are empty: https://tracker.ardour.org/view.php?id=7328
+do_fetch() {
+	git clone git://github.com/Ardour/ardour ${pkgname}-${version}
+	cd ${wrksrc}
+	git checkout ${_commit}
+}
+
 post_install() {
 	vinstall build/gtk2_ardour/ardour.xml 644 usr/share/mime/packages
 	vinstall "build/gtk2_ardour/ardour${version%%.*}.desktop" \

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

* Re: [PR PATCH] [Updated] ardour: update to 6.6.
  2021-02-28 14:45 [PR PATCH] ardour: update to 6.6 tibequadorian
                   ` (38 preceding siblings ...)
  2021-03-08 15:40 ` [PR PATCH] [Updated] " tibequadorian
@ 2021-03-08 15:41 ` tibequadorian
  2021-03-08 15:56 ` [PR REVIEW] " tibequadorian
  2021-03-09  6:36 ` [PR PATCH] [Merged]: " ericonr
  41 siblings, 0 replies; 43+ messages in thread
From: tibequadorian @ 2021-03-08 15:41 UTC (permalink / raw)
  To: ml

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

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

https://github.com/tibequadorian/void-packages ardour
https://github.com/void-linux/void-packages/pull/29120

ardour: update to 6.6.
adopt package
fetch source from github mirror because upstream deletes older source archives

<!-- Mark items with [x] where applicable -->

#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)

#### Have the results of the proposed changes been tested?
- [x] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR

<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->
<!-- 
#### Does it build and run successfully? 
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [ ] I built this PR locally for my native architecture, (ARCH-LIBC)
- [ ] I built this PR locally for these architectures (if supported. mark crossbuilds):
  - [ ] aarch64-musl
  - [ ] armv7l
  - [ ] armv6l-musl
-->


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

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

From c7b96241f0b58a96304731e1fa3cb16c87ca3345 Mon Sep 17 00:00:00 2001
From: tibequadorian <tibequadorian@posteo.de>
Date: Mon, 8 Mar 2021 10:27:01 +0100
Subject: [PATCH] ardour: update to 6.6.

---
 .../patches/fix-musl-platform-detection.patch  | 13 -------------
 srcpkgs/ardour/template                        | 18 ++++++++++++------
 2 files changed, 12 insertions(+), 19 deletions(-)
 delete mode 100644 srcpkgs/ardour/patches/fix-musl-platform-detection.patch

diff --git a/srcpkgs/ardour/patches/fix-musl-platform-detection.patch b/srcpkgs/ardour/patches/fix-musl-platform-detection.patch
deleted file mode 100644
index 8f2d29cc423..00000000000
--- a/srcpkgs/ardour/patches/fix-musl-platform-detection.patch
+++ /dev/null
@@ -1,13 +0,0 @@
-On musl libc only __linux__ is defined
-
---- libs/vst3/pluginterfaces/base/fplatform.h
-+++ libs/vst3/pluginterfaces/base/fplatform.h
-@@ -86,7 +86,7 @@
- //-----------------------------------------------------------------------------
- // LINUX
- //-----------------------------------------------------------------------------
--#elif __gnu_linux__
-+#elif __linux__
- 	#define SMTG_OS_LINUX	1
- 	#define SMTG_OS_MACOS	0
- 	#define SMTG_OS_WINDOWS	0
diff --git a/srcpkgs/ardour/template b/srcpkgs/ardour/template
index 7fb37a36769..6be4257f233 100644
--- a/srcpkgs/ardour/template
+++ b/srcpkgs/ardour/template
@@ -1,12 +1,12 @@
 # Template file for 'ardour'
 pkgname=ardour
-version=6.5.0
+version=6.6
 revision=1
-wrksrc="Ardour-${version}"
+_commit="e4e21f4d073ab00b1a0bb6ff6ca49f28b02fd68a"
 build_style=waf3
 configure_args="--cxx11 --no-phone-home --with-backends=jack,alsa,dummy
  --libjack=weak --optimize --docs --use-external-libs --freedesktop"
-hostmakedepends="python3 graphviz doxygen pkg-config clang perl
+hostmakedepends="git python3 graphviz doxygen pkg-config clang perl
  itstool gettext tar readline-devel"
 makedepends="boost-devel liblrdf-devel liblo-devel lilv-devel suil-devel
  taglib-devel aubio-devel rubberband-devel alsa-lib-devel libsndfile-devel
@@ -15,11 +15,9 @@ makedepends="boost-devel liblrdf-devel liblo-devel lilv-devel suil-devel
  libltc-devel glibmm-devel"
 depends="desktop-file-utils"
 short_desc="Professional-grade digital audio workstation"
-maintainer="Orphaned <orphan@voidlinux.org>"
+maintainer="tibequadorian <tibequadorian@posteo.de>"
 license="GPL-2.0-or-later"
 homepage="http://ardour.org"
-distfiles="https://community.ardour.org/src/Ardour-${version}.tar.bz2"
-checksum=ac981ec7565fa8a171116bc2fbeb3198a01aad4b7d7c846f965e074a8a162bdb
 
 case "$XBPS_TARGET_MACHINE" in
 	x86_64*)  configure_args+=" --dist-target=x86_64" ;;
@@ -31,6 +29,14 @@ case "$XBPS_TARGET_MACHINE" in
 	*)        broken="Don't know build target" ;;
 esac
 
+# Upstream deletes older release tarballs from their site and
+# GitHub tarballs are empty: https://tracker.ardour.org/view.php?id=7328
+do_fetch() {
+	git clone git://github.com/Ardour/ardour ${wrksrc}
+	cd ${wrksrc}
+	git checkout ${_commit}
+}
+
 post_install() {
 	vinstall build/gtk2_ardour/ardour.xml 644 usr/share/mime/packages
 	vinstall "build/gtk2_ardour/ardour${version%%.*}.desktop" \

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

* Re: [PR REVIEW] ardour: update to 6.6.
  2021-02-28 14:45 [PR PATCH] ardour: update to 6.6 tibequadorian
                   ` (39 preceding siblings ...)
  2021-03-08 15:41 ` tibequadorian
@ 2021-03-08 15:56 ` tibequadorian
  2021-03-09  6:36 ` [PR PATCH] [Merged]: " ericonr
  41 siblings, 0 replies; 43+ messages in thread
From: tibequadorian @ 2021-03-08 15:56 UTC (permalink / raw)
  To: ml

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

New review comment by tibequadorian on void-packages repository

https://github.com/void-linux/void-packages/pull/29120#discussion_r589532880

Comment:
done :)

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

* Re: [PR PATCH] [Merged]: ardour: update to 6.6.
  2021-02-28 14:45 [PR PATCH] ardour: update to 6.6 tibequadorian
                   ` (40 preceding siblings ...)
  2021-03-08 15:56 ` [PR REVIEW] " tibequadorian
@ 2021-03-09  6:36 ` ericonr
  41 siblings, 0 replies; 43+ messages in thread
From: ericonr @ 2021-03-09  6:36 UTC (permalink / raw)
  To: ml

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

There's a merged pull request on the void-packages repository

ardour: update to 6.6.
https://github.com/void-linux/void-packages/pull/29120

Description:
adopt package
fetch source from github mirror because upstream deletes older source archives

<!-- Mark items with [x] where applicable -->

#### General
- [ ] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)

#### Have the results of the proposed changes been tested?
- [x] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR

<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->
<!-- 
#### Does it build and run successfully? 
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [ ] I built this PR locally for my native architecture, (ARCH-LIBC)
- [ ] I built this PR locally for these architectures (if supported. mark crossbuilds):
  - [ ] aarch64-musl
  - [ ] armv7l
  - [ ] armv6l-musl
-->


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

end of thread, other threads:[~2021-03-09  6:36 UTC | newest]

Thread overview: 43+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-02-28 14:45 [PR PATCH] ardour: update to 6.6 tibequadorian
2021-02-28 20:42 ` [PR REVIEW] " ericonr
2021-02-28 20:42 ` ericonr
2021-02-28 22:09 ` prez
2021-02-28 22:10 ` prez
2021-03-01  5:48 ` [PR REVIEW] " tibequadorian
2021-03-01  5:49 ` tibequadorian
2021-03-01  5:53 ` [PR PATCH] [Updated] " tibequadorian
2021-03-01  6:07 ` tibequadorian
2021-03-01  6:11 ` [PR PATCH] [Updated] " tibequadorian
2021-03-01 13:08 ` prez
2021-03-01 13:09 ` prez
2021-03-01 13:11 ` tibequadorian
2021-03-01 13:11 ` tibequadorian
2021-03-01 13:12 ` tibequadorian
2021-03-01 13:14 ` prez
2021-03-01 14:33 ` tibequadorian
2021-03-01 14:33 ` [PR PATCH] [Closed]: " tibequadorian
2021-03-01 14:33 ` tibequadorian
2021-03-01 15:51 ` [PR PATCH] [Updated] " tibequadorian
2021-03-01 15:52 ` tibequadorian
2021-03-01 16:00 ` tibequadorian
2021-03-01 16:01 ` [PR PATCH] [Updated] " tibequadorian
2021-03-08  4:12 ` [PR REVIEW] " ericonr
2021-03-08  9:01 ` tibequadorian
2021-03-08  9:04 ` tibequadorian
2021-03-08  9:05 ` tibequadorian
2021-03-08  9:09 ` tibequadorian
2021-03-08  9:17 ` tibequadorian
2021-03-08  9:44 ` [PR PATCH] [Updated] " tibequadorian
2021-03-08  9:47 ` tibequadorian
2021-03-08  9:54 ` tibequadorian
2021-03-08  9:55 ` [PR REVIEW] " tibequadorian
2021-03-08  9:57 ` [PR PATCH] [Updated] " tibequadorian
2021-03-08 10:01 ` tibequadorian
2021-03-08 10:04 ` [PR REVIEW] " tibequadorian
2021-03-08 10:08 ` tibequadorian
2021-03-08 10:40 ` tibequadorian
2021-03-08 15:38 ` ericonr
2021-03-08 15:40 ` [PR PATCH] [Updated] " tibequadorian
2021-03-08 15:41 ` tibequadorian
2021-03-08 15:56 ` [PR REVIEW] " tibequadorian
2021-03-09  6:36 ` [PR PATCH] [Merged]: " ericonr

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).