Looks ok- can you commit it and I'll let you know if there's still a
problem after the next bulk. Thanks!
--
Sent from a phone, apologies for poor formatting.
On 19 January 2019 09:07:34 Rafael Sadowski <raf...@sizeofvoid.org> wrote:
On Fri Jan 18, 2019 at 08:10:58PM +0000, Stuart Henderson wrote:
On 2019/01/14 23:16, Rafael Sadowski wrote:
> CVSROOT: /cvs
> Module name: ports
> Changes by: rsadow...@cvs.openbsd.org 2019/01/14 23:16:04
>=20
> Modified files:
> multimedia/mkvtoolnix: Makefile distinfo=20
>=20
> Log message:
> Update mkvtoolnix to 30.1.0
>=20
Broken on 32-bit:
c++ -Wall -Wno-comment -Wfatal-errors -Wno-mismatched-tags -Wno-self-assign=
-Qunused-arguments -Wno-inconsistent-missing-override -Wno-potentially-eva=
luated-expression -D_FILE_OFFSET_BITS=3D64 -DMTX_LOCALE_DIR=3D\"/usr/local/=
share/locale\" -DMTX_PKG_DATA_DIR=3D\"/usr/local/share/mkvtoolnix\" -DMTX_D=
OC_DIR=3D\"/usr/local/share/doc/mkvtoolnix\" -fstack-protector-strong -Ilib=
/nlohmann-json/include -Ilib/fmt/include -I/usr/local/include -I/usr/local/=
include -I/usr/local/include -I/usr/local/include -I/usr/X11R6/include -fPI=
C -std=3Dgnu++14 -Woverloaded-virtual -Wnon-virtual-dtor -Wextra -Wno-missi=
ng-field-initializers -I/usr/local/include/X11/qt5/QtWidgets -I/usr/local/i=
nclude/X11/qt5 -I/usr/local/include/X11/qt5/QtMultimedia -I/usr/local/inclu=
de/X11/qt5/QtGui -I/usr/local/include/X11/qt5/QtNetwork -I/usr/local/includ=
e/X11/qt5/QtConcurrent -I/usr/local/include/X11/qt5/QtDBus -I/usr/local/inc=
lude/X11/qt5/QtCore -I/usr/local/include -O2 -pipe -include src/common/comm=
on_pch.h -I. -Ilib -Ilib/avilib-0.6.10 -Isrc -Ilib/utf8-cpp/source -Ilib/pu=
gixml/src -c -MMD -MF /usr/obj/ports/mkvtoolnix-30.1.0/mkvtoolnix-30.1.0/ra=
ke.d/dependency.d/tmp/src_common_mm_mem_io_o.d -o src/common/mm_mem_io.o -x=
c++ src/common/mm_mem_io.cpp
In file included from src/common/mm_mem_io.cpp:19:
src/common/mm_mem_io_p.h:31:16: fatal error: non-constant-expression cannot=
be narrowed from type 'uint64_t' (aka 'unsigned long long') to 'std::size_=
t' (aka 'unsigned long') in initializer list [-Wc++11-narrowing]
: mem_size{p_mem_size}
^~~~~~~~~~
src/common/mm_mem_io_p.h:31:16: note: insert an explicit cast to silence th=
is issue
: mem_size{p_mem_size}
^~~~~~~~~~
static_cast<size_t>( )
1 error generated.
*** Error 1 in multimedia/mkvtoolnix (Makefile:91 'do-build')
*** Error 1 in multimedia/mkvtoolnix (/usr/ports/infrastructure/mk/bsd.port=
=2Emk:2797 '/usr/obj/ports/mkvtoolnix-30.1.0/.build_done')
=3D=3D=3D> Exiting multimedia/mkvtoolnix with an error
The diff below should fix the 32bit issue. Tested on amd64 only.
OK?
RS
Index: Makefile
===================================================================
RCS file: /cvs/ports/multimedia/mkvtoolnix/Makefile,v
retrieving revision 1.85
diff -u -p -u -p -r1.85 Makefile
--- Makefile 15 Jan 2019 06:16:04 -0000 1.85
+++ Makefile 19 Jan 2019 09:04:47 -0000
@@ -3,6 +3,7 @@
COMMENT= create, alter and inspect Matroska files
DISTNAME= mkvtoolnix-30.1.0
+REVISION= 0
CATEGORIES= multimedia x11
Index: patches/patch-src_common_mm_mem_io_p_h
===================================================================
RCS file: patches/patch-src_common_mm_mem_io_p_h
diff -N patches/patch-src_common_mm_mem_io_p_h
--- /dev/null 1 Jan 1970 00:00:00 -0000
+++ patches/patch-src_common_mm_mem_io_p_h 19 Jan 2019 09:04:47 -0000
@@ -0,0 +1,30 @@
+$OpenBSD$
+
+fix implicit narrowing conversion in brace initializers
+https://gitlab.com/mbunkus/mkvtoolnix/commit/653f74b692d334ea4b3691f3b61c195f464c39d4?view=inline
+
+Index: src/common/mm_mem_io_p.h
+--- src/common/mm_mem_io_p.h.orig
++++ src/common/mm_mem_io_p.h
+@@ -28,8 +28,8 @@ class mm_mem_io_private_c : public mm_io_private_c { (
+ explicit mm_mem_io_private_c(unsigned char *p_mem,
+ uint64_t p_mem_size,
+ std::size_t p_increase)
+- : mem_size{p_mem_size}
+- , allocated{p_mem_size}
++ : mem_size{static_cast<std::size_t>(p_mem_size)}
++ , allocated{static_cast<std::size_t>(p_mem_size)}
+ , increase{p_increase}
+ , mem{p_mem}
+ {
+@@ -48,8 +48,8 @@ class mm_mem_io_private_c : public mm_io_private_c { (
+
+ explicit mm_mem_io_private_c(unsigned char const *p_mem,
+ uint64_t p_mem_size)
+- : mem_size{p_mem_size}
+- , allocated{p_mem_size}
++ : mem_size{static_cast<std::size_t>(p_mem_size)}
++ , allocated{static_cast<std::size_t>(p_mem_size)}
+ , ro_mem{p_mem}
+ , read_only{true}
+ {