On 02/15/18 23:01, Björn Ketelaars wrote:
On Fri 09/02/2018 18:29, Björn Ketelaars wrote:
On Fri 09/02/2018 17:09, Stuart Henderson wrote:
On 2018/02/09 07:55, Björn Ketelaars wrote:
Included a diff for bringing devel/msgpack to 2.1.5. From CHANGELOG:

- Improve cross platform configuration
- Add boost asio examples (including zlib)
- Remove some warnings
- Fix unpack visitor to treat float32/64 correctly
- Improve documents
- Fix alignment problem on some platform
- Fix conflict std::tuple, std::pair, and boost::fusion::sequence problem

While here drop GH_ in favour of MASTER_SITES, and switch HOMEPAGE to
https. No need to bump SHARED_LIBS.
GH_* is correct here.

/archive  -> GH_*
/releases -> MASTER_SITES
New diff, which includes sthen@'s comment, and excludes the reordering of
Makefile. Output make test:

100% tests passed, 0 tests failed out of 30

Build tested both consumers (editors/neovim and sysutils/tmate). Run
tested with editors/neovim.

OK?

Sorry to make you do more work, but:
1. Could you eliminate -Werror in all the patches? It's in one but not the other.

2. There are some more -O3 floating around. For example:
[6/104] /usr/ports/pobj/msgpack-2.1.5/bin/c++ -I/usr/ports/pobj/msgpack-2.1.5/msgpack-c-cpp-2.1.5/include -Iinclude -DMSGPACK_DEFAULT_API_VERSION=2 -std=c++98 -O2 -pipe -DNDEBUG   -Wall -Wextra -Werror -g -O3 -MD -MT example/cpp03/CMakeFiles/protocol_new.dir/protocol_new.cpp.o -MF example/cpp03/CMakeFiles/protocol_new.dir/protocol_new.cpp.o.d -o example/cpp03/CMakeFiles/protocol_new.dir/protocol_new.cpp.o -c /usr/ports/pobj/msgpack-2.1.5/msgpack-c-cpp-2.1.5/example/cpp03/protocol_new.cpp

~Brian


Index: Makefile
===================================================================
RCS file: /cvs/ports/devel/msgpack/Makefile,v
retrieving revision 1.8
diff -u -p -r1.8 Makefile
--- Makefile    20 Jan 2018 14:03:39 -0000      1.8
+++ Makefile    15 Feb 2018 20:56:24 -0000
@@ -2,13 +2,13 @@
COMMENT = MessagePack implementation for C and C++ -V = 2.1.4
+V =                    2.1.5
  DISTNAME =            msgpack-${V}
  SHARED_LIBS +=        msgpackc                  1.0 # 2.0
CATEGORIES = devel -HOMEPAGE = http://msgpack.org/
+HOMEPAGE =             https://msgpack.org/
# Boost license
  PERMIT_PACKAGE_CDROM =        Yes
Index: distinfo
===================================================================
RCS file: /cvs/ports/devel/msgpack/distinfo,v
retrieving revision 1.4
diff -u -p -r1.4 distinfo
--- distinfo    9 Aug 2017 23:02:22 -0000       1.4
+++ distinfo    15 Feb 2018 20:56:24 -0000
@@ -1,2 +1,2 @@
-SHA256 (msgpack-2.1.4.tar.gz) = DjufGzKWf4m0FuAor+LH98wI4middd4i8HrZtNsU9a4=
-SIZE (msgpack-2.1.4.tar.gz) = 460891
+SHA256 (msgpack-2.1.5.tar.gz) = nIf4D8ZRuQB3LervCrFUtjFgx00pJSm1vm0G1khdRkA=
+SIZE (msgpack-2.1.5.tar.gz) = 461179


Reply via email to