Bug 224187 - net-p2p/zetacoin: fails to build with boost 1.66
Summary: net-p2p/zetacoin: fails to build with boost 1.66
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Jan Beich
Keywords: needs-patch
Depends on:
Blocks: 223922
  Show dependency treegraph
Reported: 2017-12-08 16:15 UTC by Jan Beich
Modified: 2018-01-18 04:18 UTC (History)
1 user (show)

See Also:
bugzilla: maintainer-feedback? (daniel)

patch for bitcoin-cli.cpp (681 bytes, patch)
2018-01-13 05:07 UTC, Daniel Morante
no flags Details | Diff
patch for rpcserver.cpp (2.93 KB, patch)
2018-01-13 05:08 UTC, Daniel Morante
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jan Beich freebsd_committer 2017-12-08 16:15:54 UTC
As part of exp-run in bug 223922 this port was found to break on Boost update. If the issue won't disappear on its own by Boost release I plan to mark the port as BROKEN. Can you check with upstream maintainer or help with a fix?

$ fetch -qo- 'https://reviews.freebsd.org/D13279?download=true' | patch -Efsp0 -d /usr/ports
$ make all deinstall install clean -C/usr/ports/devel/boost-libs
$ make -C/usr/ports/net-p2p/zetacoin
bitcoin-cli.cpp:108:31: error: no matching constructor for initialization of 'boost::asio::ssl::context'
    boost::asio::ssl::context context(io_service, boost::asio::ssl::context::sslv23);
                              ^       ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
/usr/local/include/boost/asio/ssl/impl/context.ipp:63:10: note: candidate constructor not viable: requires single argument 'm', but 2 arguments were provided
context::context(context::method m)
/usr/local/include/boost/asio/ssl/context.hpp:36:7: note: candidate constructor (the implicit copy constructor) not viable: requires 1 argument, but 2 were provided
class context


Comment 1 Jan Beich freebsd_committer 2017-12-08 16:16:18 UTC
Maybe you can replace io_service with io_context similar to https://github.com/arvidn/libtorrent/commit/f42b63c7ea82
Comment 2 Daniel Morante 2017-12-08 22:50:53 UTC
I don't know how to test with boost 1.66.  Grabbing the latest ports tree with portsnap still gives me 1.65.

Anyway, does the problem still occur if you try it with this version?: https://github.com/tuaris/FreeBSD-Coin-Ports/tree/master/ports/zetacoin
Comment 3 Daniel Morante 2017-12-08 22:52:27 UTC
Sorry, I just re-read your initial comment.  I'll try that.
Comment 4 Daniel Morante 2017-12-09 02:14:06 UTC
I've filed the issue with the upstream project.  The suggestion to io_context in place of io_service didn't work or the changes required are above my understanding of C++ code.

Comment 5 Daniel Morante 2018-01-13 05:07:44 UTC
Created attachment 189677 [details]
patch for bitcoin-cli.cpp
Comment 6 Daniel Morante 2018-01-13 05:08:18 UTC
Created attachment 189678 [details]
patch for rpcserver.cpp
Comment 7 Daniel Morante 2018-01-13 05:08:42 UTC
These patches will allow the build to succeed.  They will be included in the next port update to
Comment 8 commit-hook freebsd_committer 2018-01-18 04:01:52 UTC
A commit references this bug:

Author: jbeich
Date: Thu Jan 18 04:01:01 UTC 2018
New revision: 459313
URL: https://svnweb.freebsd.org/changeset/ports/459313

  net-p2p/zetacoin: unbreak with boost 1.66

  PR:		224187
  Submitted by:	Daniel Morante (maintainer)

Comment 9 Jan Beich freebsd_committer 2018-01-18 04:18:24 UTC
Thanks. Landed.