* Re: mkvtoolnix: update to 56.0.0
2021-04-07 21:17 [PR PATCH] mkvtoolnix: update to 56.0.0 mvf
@ 2021-04-07 21:29 ` ericonr
2021-04-11 3:11 ` ericonr
` (5 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: ericonr @ 2021-04-07 21:29 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 233 bytes --]
New comment by ericonr on void-packages repository
https://github.com/void-linux/void-packages/pull/30078#issuecomment-815277446
Comment:
Do you know if they ship some sort of test suite? Would be nice to have a `do_check` to run.
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: mkvtoolnix: update to 56.0.0
2021-04-07 21:17 [PR PATCH] mkvtoolnix: update to 56.0.0 mvf
2021-04-07 21:29 ` ericonr
@ 2021-04-11 3:11 ` ericonr
2021-04-13 20:20 ` [PR PATCH] [Updated] " mvf
` (4 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: ericonr @ 2021-04-11 3:11 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 160 bytes --]
New comment by ericonr on void-packages repository
https://github.com/void-linux/void-packages/pull/30078#issuecomment-817239663
Comment:
Also, 56.1.0 is out
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PR PATCH] [Updated] mkvtoolnix: update to 56.0.0
2021-04-07 21:17 [PR PATCH] mkvtoolnix: update to 56.0.0 mvf
2021-04-07 21:29 ` ericonr
2021-04-11 3:11 ` ericonr
@ 2021-04-13 20:20 ` mvf
2021-04-13 20:20 ` mkvtoolnix: update to 56.1.0 mvf
` (3 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: mvf @ 2021-04-13 20:20 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 1593 bytes --]
There is an updated pull request by mvf against master on the void-packages repository
https://github.com/mvf/void-packages pr/mkvtoolnix
https://github.com/void-linux/void-packages/pull/30078
mkvtoolnix: update to 56.0.0
<!-- 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/30078.patch is attached
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-pr/mkvtoolnix-30078.patch --]
[-- Type: text/x-diff, Size: 1960 bytes --]
From ed70286fb9658a315806738e5384b106976f09ea Mon Sep 17 00:00:00 2001
From: Matthias von Faber <mvf@gmx.eu>
Date: Tue, 13 Apr 2021 22:15:10 +0200
Subject: [PATCH] mkvtoolnix: update to 56.1.0, enable unit tests
---
srcpkgs/mkvtoolnix/template | 13 ++++++++-----
1 file changed, 8 insertions(+), 5 deletions(-)
diff --git a/srcpkgs/mkvtoolnix/template b/srcpkgs/mkvtoolnix/template
index a6606d0adca1..6704d111631e 100644
--- a/srcpkgs/mkvtoolnix/template
+++ b/srcpkgs/mkvtoolnix/template
@@ -1,22 +1,22 @@
# Template file for 'mkvtoolnix'
pkgname=mkvtoolnix
-version=55.0.0
+version=56.1.0
revision=1
build_style=gnu-configure
build_helper=qmake
configure_args="--with-docbook-xsl-root=/usr/share/xsl/docbook --enable-qt
--disable-update-check"
hostmakedepends="autoconf docbook-xsl gettext libxslt pkg-config qt5-tools-devel ruby"
-makedepends="boost-devel cmark-devel file-devel fmt-devel json-c++ libdvdread-devel
- libflac-devel libmatroska-devel libvorbis-devel pcre2-devel pugixml-devel
- qt5-multimedia-devel qt5-tools-devel"
+makedepends="boost-devel cmark-devel file-devel fmt-devel gtest-devel json-c++
+ libdvdread-devel libflac-devel libmatroska-devel libvorbis-devel pcre2-devel
+ pugixml-devel qt5-multimedia-devel qt5-tools-devel"
short_desc="Create, alter and inspect Matroska videos"
maintainer="Matthias von Faber <mvf@gmx.eu>"
license="GPL-2.0-only"
homepage="https://mkvtoolnix.download"
changelog="https://mkvtoolnix.download/doc/NEWS.md"
distfiles="https://mkvtoolnix.download/sources/${pkgname}-${version}.tar.xz"
-checksum=6061639bda18adbf5c49264d5498fd2abab420113ba680d0eb441a9aa02afba6
+checksum=75ff6476e7c4eab10c315af4e2dd78f1399d35da7c4611a8f93f4c3c0b475f72
if [ "$CROSS_BUILD" ]; then
configure_args+=" --with-boost=${XBPS_CROSS_BASE}/usr"
@@ -28,6 +28,9 @@ pre_configure() {
do_build() {
rake ${makejobs}
}
+do_check() {
+ rake tests:run_unit
+}
do_install() {
rake DESTDIR=${DESTDIR} install
}
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: mkvtoolnix: update to 56.1.0
2021-04-07 21:17 [PR PATCH] mkvtoolnix: update to 56.0.0 mvf
` (2 preceding siblings ...)
2021-04-13 20:20 ` [PR PATCH] [Updated] " mvf
@ 2021-04-13 20:20 ` mvf
2021-04-13 20:34 ` [PR REVIEW] " ericonr
` (2 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: mvf @ 2021-04-13 20:20 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 202 bytes --]
New comment by mvf on void-packages repository
https://github.com/void-linux/void-packages/pull/30078#issuecomment-819026745
Comment:
Thanks, updated to 56.1.0 and `do_check` now runs the unit tests.
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PR REVIEW] mkvtoolnix: update to 56.1.0
2021-04-07 21:17 [PR PATCH] mkvtoolnix: update to 56.0.0 mvf
` (3 preceding siblings ...)
2021-04-13 20:20 ` mkvtoolnix: update to 56.1.0 mvf
@ 2021-04-13 20:34 ` ericonr
2021-04-14 6:28 ` [PR PATCH] [Updated] " mvf
2021-04-14 11:59 ` [PR PATCH] [Merged]: " ericonr
6 siblings, 0 replies; 8+ messages in thread
From: ericonr @ 2021-04-13 20:34 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 191 bytes --]
New review comment by ericonr on void-packages repository
https://github.com/void-linux/void-packages/pull/30078#discussion_r612757824
Comment:
nit: include `gtest-devel` in `checkdepends`
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PR PATCH] [Updated] mkvtoolnix: update to 56.1.0
2021-04-07 21:17 [PR PATCH] mkvtoolnix: update to 56.0.0 mvf
` (4 preceding siblings ...)
2021-04-13 20:34 ` [PR REVIEW] " ericonr
@ 2021-04-14 6:28 ` mvf
2021-04-14 11:59 ` [PR PATCH] [Merged]: " ericonr
6 siblings, 0 replies; 8+ messages in thread
From: mvf @ 2021-04-14 6:28 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 1593 bytes --]
There is an updated pull request by mvf against master on the void-packages repository
https://github.com/mvf/void-packages pr/mkvtoolnix
https://github.com/void-linux/void-packages/pull/30078
mkvtoolnix: update to 56.1.0
<!-- 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/30078.patch is attached
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-pr/mkvtoolnix-30078.patch --]
[-- Type: text/x-diff, Size: 1678 bytes --]
From 4e2f2f26c4bbb0bfe2a7280016cbda1cc4f8635c Mon Sep 17 00:00:00 2001
From: Matthias von Faber <mvf@gmx.eu>
Date: Tue, 13 Apr 2021 22:15:10 +0200
Subject: [PATCH] mkvtoolnix: update to 56.1.0, enable unit tests
---
srcpkgs/mkvtoolnix/template | 8 ++++++--
1 file changed, 6 insertions(+), 2 deletions(-)
diff --git a/srcpkgs/mkvtoolnix/template b/srcpkgs/mkvtoolnix/template
index a6606d0adca1..efce272baacb 100644
--- a/srcpkgs/mkvtoolnix/template
+++ b/srcpkgs/mkvtoolnix/template
@@ -1,6 +1,6 @@
# Template file for 'mkvtoolnix'
pkgname=mkvtoolnix
-version=55.0.0
+version=56.1.0
revision=1
build_style=gnu-configure
build_helper=qmake
@@ -10,13 +10,14 @@ hostmakedepends="autoconf docbook-xsl gettext libxslt pkg-config qt5-tools-devel
makedepends="boost-devel cmark-devel file-devel fmt-devel json-c++ libdvdread-devel
libflac-devel libmatroska-devel libvorbis-devel pcre2-devel pugixml-devel
qt5-multimedia-devel qt5-tools-devel"
+checkdepends="gtest-devel"
short_desc="Create, alter and inspect Matroska videos"
maintainer="Matthias von Faber <mvf@gmx.eu>"
license="GPL-2.0-only"
homepage="https://mkvtoolnix.download"
changelog="https://mkvtoolnix.download/doc/NEWS.md"
distfiles="https://mkvtoolnix.download/sources/${pkgname}-${version}.tar.xz"
-checksum=6061639bda18adbf5c49264d5498fd2abab420113ba680d0eb441a9aa02afba6
+checksum=75ff6476e7c4eab10c315af4e2dd78f1399d35da7c4611a8f93f4c3c0b475f72
if [ "$CROSS_BUILD" ]; then
configure_args+=" --with-boost=${XBPS_CROSS_BASE}/usr"
@@ -28,6 +29,9 @@ pre_configure() {
do_build() {
rake ${makejobs}
}
+do_check() {
+ rake tests:run_unit
+}
do_install() {
rake DESTDIR=${DESTDIR} install
}
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PR PATCH] [Merged]: mkvtoolnix: update to 56.1.0
2021-04-07 21:17 [PR PATCH] mkvtoolnix: update to 56.0.0 mvf
` (5 preceding siblings ...)
2021-04-14 6:28 ` [PR PATCH] [Updated] " mvf
@ 2021-04-14 11:59 ` ericonr
6 siblings, 0 replies; 8+ messages in thread
From: ericonr @ 2021-04-14 11:59 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 1439 bytes --]
There's a merged pull request on the void-packages repository
mkvtoolnix: update to 56.1.0
https://github.com/void-linux/void-packages/pull/30078
Description:
<!-- 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