* Re: libmatroska: update to 1.6.3,
2021-04-09 16:31 [PR PATCH] libmatroska: update to 1.6.3, mobinmob
@ 2021-04-09 17:57 ` ericonr
2021-04-09 19:52 ` mobinmob
` (5 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: ericonr @ 2021-04-09 17:57 UTC (permalink / raw)
To: ml
[-- 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.
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: libmatroska: update to 1.6.3,
2021-04-09 16:31 [PR PATCH] libmatroska: update to 1.6.3, mobinmob
2021-04-09 17:57 ` ericonr
@ 2021-04-09 19:52 ` mobinmob
2021-04-09 19:59 ` mobinmob
` (4 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: mobinmob @ 2021-04-09 19:52 UTC (permalink / raw)
To: ml
[-- 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?
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: libmatroska: update to 1.6.3,
2021-04-09 16:31 [PR PATCH] libmatroska: update to 1.6.3, mobinmob
2021-04-09 17:57 ` ericonr
2021-04-09 19:52 ` mobinmob
@ 2021-04-09 19:59 ` mobinmob
2021-04-09 20:22 ` ericonr
` (3 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: mobinmob @ 2021-04-09 19:59 UTC (permalink / raw)
To: ml
[-- 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?
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: libmatroska: update to 1.6.3,
2021-04-09 16:31 [PR PATCH] libmatroska: update to 1.6.3, mobinmob
` (2 preceding siblings ...)
2021-04-09 19:59 ` mobinmob
@ 2021-04-09 20:22 ` ericonr
2021-04-09 21:08 ` [PR PATCH] [Updated] " mobinmob
` (2 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: ericonr @ 2021-04-09 20:22 UTC (permalink / raw)
To: ml
[-- 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.
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PR PATCH] [Updated] libmatroska: update to 1.6.3,
2021-04-09 16:31 [PR PATCH] libmatroska: update to 1.6.3, mobinmob
` (3 preceding siblings ...)
2021-04-09 20:22 ` ericonr
@ 2021-04-09 21:08 ` mobinmob
2021-04-09 21:10 ` mobinmob
2021-04-10 0:40 ` [PR PATCH] [Merged]: " ericonr
6 siblings, 0 replies; 8+ messages in thread
From: mobinmob @ 2021-04-09 21:08 UTC (permalink / raw)
To: ml
[-- 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}"
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: libmatroska: update to 1.6.3,
2021-04-09 16:31 [PR PATCH] libmatroska: update to 1.6.3, mobinmob
` (4 preceding siblings ...)
2021-04-09 21:08 ` [PR PATCH] [Updated] " mobinmob
@ 2021-04-09 21:10 ` mobinmob
2021-04-10 0:40 ` [PR PATCH] [Merged]: " ericonr
6 siblings, 0 replies; 8+ messages in thread
From: mobinmob @ 2021-04-09 21:10 UTC (permalink / raw)
To: ml
[-- 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 ;)
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PR PATCH] [Merged]: libmatroska: update to 1.6.3,
2021-04-09 16:31 [PR PATCH] libmatroska: update to 1.6.3, mobinmob
` (5 preceding siblings ...)
2021-04-09 21:10 ` mobinmob
@ 2021-04-10 0:40 ` ericonr
6 siblings, 0 replies; 8+ messages in thread
From: ericonr @ 2021-04-10 0:40 UTC (permalink / raw)
To: ml
[-- 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
-->
^ permalink raw reply [flat|nested] 8+ messages in thread