[PR PATCH] libmatroska: update to 1.6.3,

[PR PATCH] libmatroska: update to 1.6.3,

[mobinmob]

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

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

https://github.com/mobinmob/void-packages libmatroska
https://github.com/void-linux/void-packages/pull/30113

libmatroska: update to 1.6.3,
- change distfiles to github.
- change homepage,
- adjust wrksrc.

<!-- 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/30113.patch is attached

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

From dd4a034abb695dc5f1e90d12a80378a8aaaa2873 Mon Sep 17 00:00:00 2001
From: mobinmob <mobinmob@disroot.org>
Date: Fri, 9 Apr 2021 19:27:46 +0300
Subject: [PATCH] libmatroska: update to 1.6.3, - change distfiles to github. -
 change homepage, - adjust wrksrc.

---
 srcpkgs/libmatroska/template | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/srcpkgs/libmatroska/template b/srcpkgs/libmatroska/template
index b569bbd4dd01..5385183627d2 100644
--- a/srcpkgs/libmatroska/template
+++ b/srcpkgs/libmatroska/template
@@ -1,16 +1,17 @@
 # Template file for 'libmatroska'
 pkgname=libmatroska
-version=1.6.2
+version=1.6.3
 revision=1
+wrksrc="${pkgname}-release-${version}"
 build_style=cmake
 configure_args="-DBUILD_SHARED_LIBS=ON"
 makedepends="libebml-devel"
 short_desc="Extensible open standard Audio/Video container format"
 maintainer="Orphaned <orphan@voidlinux.org>"
 license="LGPL-2.1-or-later"
-homepage="https://dl.matroska.org/downloads/libmatroska/"
-distfiles="https://dl.matroska.org/downloads/libmatroska/libmatroska-${version}.tar.xz"
-checksum=bc4479aa8422ab07643df6a1fa5a19e4bed4badfd41ca77e081628620d1e1990
+homepage="https://www.matroska.org/downloads/libraries.html"
+distfiles="https://github.com/Matroska-Org/libmatroska/archive/refs/tags/release-${version}.tar.gz"
+checksum=0c8c875ae26ac69a722f7fd0f4a4fecb4fdff681f2a165f09c06a40cbf1d0de6
 
 libmatroska-devel_package() {
 	depends="${makedepends} ${sourcepkg}>=${version}_${revision}"

Re: libmatroska: update to 1.6.3,

[ericonr]

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

New comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/30113#issuecomment-816855352

Comment:
Can you explain why you switched to GH for the tarballs in the commit message? It's available in https://dl.matroska.org/downloads/libmatroska/ , after all.

Re: libmatroska: update to 1.6.3,

[mobinmob]

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

New comment by mobinmob on void-packages repository

https://github.com/void-linux/void-packages/pull/30113#issuecomment-816931574

Comment:
> Can you explain why you switched to GH for the tarballs in the commit message? It's available in https://dl.matroska.org/downloads/libmatroska/ , after all.

The project has [officially moved](https://github.com/void-linux/void-packages/pull/30113#issuecomment-816855352) its repositories to gh. Any reason not to take advantage of the free gh bandwidth?

Re: libmatroska: update to 1.6.3,

[mobinmob]

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

New comment by mobinmob on void-packages repository

https://github.com/void-linux/void-packages/pull/30113#issuecomment-816931574

Comment:
> Can you explain why you switched to GH for the tarballs in the commit message? It's available in https://dl.matroska.org/downloads/libmatroska/ , after all.

The project has [officially moved](https://www.matroska.org/source_code_repositories.html) its repositories to gh. Any reason not to take advantage of the free gh bandwidth?

Re: libmatroska: update to 1.6.3,

[ericonr]

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

New comment by ericonr on void-packages repository

https://github.com/void-linux/void-packages/pull/30113#issuecomment-816951439

Comment:
Fair enough :)

Just explain it in the commit message (so not only what's being done, but also why) and we should be good.

Re: [PR PATCH] [Updated] libmatroska: update to 1.6.3,

[mobinmob]

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

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

https://github.com/mobinmob/void-packages libmatroska
https://github.com/void-linux/void-packages/pull/30113

libmatroska: update to 1.6.3,
- change distfiles to github.
- change homepage,
- adjust wrksrc.

<!-- 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/30113.patch is attached

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

From a666fcbc16a20572e6517ff83c94a2f975645fbc Mon Sep 17 00:00:00 2001
From: mobinmob <mobinmob@disroot.org>
Date: Fri, 9 Apr 2021 19:27:46 +0300
Subject: [PATCH] libmatroska: update to 1.6.3, - project uses github for the
 official git repos, changing distfiles to github to take advantage, - change
 homepage, - adjust wrksrc.

---
 srcpkgs/libmatroska/template | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/srcpkgs/libmatroska/template b/srcpkgs/libmatroska/template
index b569bbd4dd01..5385183627d2 100644
--- a/srcpkgs/libmatroska/template
+++ b/srcpkgs/libmatroska/template
@@ -1,16 +1,17 @@
 # Template file for 'libmatroska'
 pkgname=libmatroska
-version=1.6.2
+version=1.6.3
 revision=1
+wrksrc="${pkgname}-release-${version}"
 build_style=cmake
 configure_args="-DBUILD_SHARED_LIBS=ON"
 makedepends="libebml-devel"
 short_desc="Extensible open standard Audio/Video container format"
 maintainer="Orphaned <orphan@voidlinux.org>"
 license="LGPL-2.1-or-later"
-homepage="https://dl.matroska.org/downloads/libmatroska/"
-distfiles="https://dl.matroska.org/downloads/libmatroska/libmatroska-${version}.tar.xz"
-checksum=bc4479aa8422ab07643df6a1fa5a19e4bed4badfd41ca77e081628620d1e1990
+homepage="https://www.matroska.org/downloads/libraries.html"
+distfiles="https://github.com/Matroska-Org/libmatroska/archive/refs/tags/release-${version}.tar.gz"
+checksum=0c8c875ae26ac69a722f7fd0f4a4fecb4fdff681f2a165f09c06a40cbf1d0de6
 
 libmatroska-devel_package() {
 	depends="${makedepends} ${sourcepkg}>=${version}_${revision}"

Re: libmatroska: update to 1.6.3,

[mobinmob]

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

New comment by mobinmob on void-packages repository

https://github.com/void-linux/void-packages/pull/30113#issuecomment-816973823

Comment:
> Fair enough :)
> 
> Just explain it in the commit message (so not only what's being done, but also why) and we should be good.

OK ;)

Re: [PR PATCH] [Merged]: libmatroska: update to 1.6.3,

[ericonr]

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

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

libmatroska: update to 1.6.3,
https://github.com/void-linux/void-packages/pull/30113

Description:
- change distfiles to github.
- change homepage,
- adjust wrksrc.

<!-- 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
-->