Bug 226232 - net/dbeacon: [maintainer] update to 0.3.9.2: adds C++11 support to permit building on 11 && CURRENT
Summary: net/dbeacon: [maintainer] update to 0.3.9.2: adds C++11 support to permit bui...
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: Tobias Kortkamp
URL:
Keywords: easy, patch, patch-ready
Depends on:
Blocks:
 
Reported: 2018-02-27 03:00 UTC by Chris Hutchinson
Modified: 2018-07-07 10:40 UTC (History)
1 user (show)

See Also:
portmaster: maintainer-feedback+


Attachments
svn diff for net/dbeacon (3.33 KB, patch)
2018-02-27 03:00 UTC, Chris Hutchinson
portmaster: maintainer-approval+
Details | Diff
QA log for net/dbeacon (2.04 KB, text/plain)
2018-02-27 03:01 UTC, Chris Hutchinson
no flags Details
svn diff for net/dbeacon (version 2) (1.13 KB, patch)
2018-06-16 07:45 UTC, Chris Hutchinson
portmaster: maintainer-approval+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Chris Hutchinson 2018-02-27 03:00:23 UTC
Created attachment 191044 [details]
svn diff for net/dbeacon

net/dbeacon doesn't build on 11 or CURRENT due to insufficient
support for C++11. This update brings this support to net/dbeacon
so as to build and operate correctly on 11 && CURRENT.

CHANGES
bump   PORTVERSION
convert LICENSE to MIT
add    USES tar:xz
remove files/patch-dbeacon.cpp
...
please see the attached svn diff for greater detail.
Tests good on CURRENT (see attached QA log for greater detail)

That's it!

Thanks!

--Chris
Comment 1 Chris Hutchinson 2018-02-27 03:01:23 UTC
Created attachment 191045 [details]
QA log for net/dbeacon

Tests good on CURRENT
Comment 2 Yuri Victorovich freebsd_committer freebsd_triage 2018-02-27 06:35:58 UTC
It failed on 12i386:
c++ -O2 -pipe -fstack-protector -fno-strict-aliasing  -g -Wall -O2   -c -o ssmping.o ssmping.cpp
dbeacon.cpp:419:78: error: invalid operands to binary expression ('__bind<int &, sockaddr *, int>' and 'int')
        if (bind(mcastSock, beaconUnicastAddr.saddr(), beaconUnicastAddr.addrlen()) != 0) {
            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ^  ~
/usr/include/c++/v1/system_error:606:1: note: candidate function not viable: no known conversion from '__bind<int &, sockaddr *, int>' to 'const std::__1::error_code' for 1st argument
operator!=(const error_code& __x, const error_code& __y) _NOEXCEPT
^
Comment 3 Yuri Victorovich freebsd_committer freebsd_triage 2018-03-11 21:09:45 UTC
(In reply to Yuri Victorovich from comment #2)

Ping Chris Hutchinson -)
Comment 4 Tobias Kortkamp freebsd_committer freebsd_triage 2018-06-16 07:12:51 UTC
Hi Chris,

it should be possible to fix the problem Yuri uncovered by adding
USE_CXXSTD=c++98 to the Makefile or by fixing it like we did in ports r471840.
Comment 5 Chris Hutchinson 2018-06-16 07:20:36 UTC
(In reply to Tobias Kortkamp from comment #4)
> Hi Chris,
> 
> it should be possible to fix the problem Yuri uncovered by adding
> USE_CXXSTD=c++98 to the Makefile or by fixing it like we did in ports
> r471840.

Right you are *again* Tobias! :-)
I've been buried lately. As I've just setup a CPAN mirror,
and am in the process of creating a replacement for the
soon-to-be unplugged search.cpan.org

Anyway. I'll post a patch that'll include that shortly.
Then I'll go back to it when I'm done with my current
events, and fix it *properly*.

Thanks!

--Chris
Comment 6 Chris Hutchinson 2018-06-16 07:45:13 UTC
Created attachment 194298 [details]
svn diff for net/dbeacon (version 2)

New patch, that includes all previous changes as well as;
and this is big; USE_CXXSTD=	c++98
;-)

That'll do it. I think we're *finally* done here! :-)

Thanks for all your time, and consideration!

--Chris
Comment 7 Tobias Kortkamp freebsd_committer freebsd_triage 2018-07-07 10:38:29 UTC
Assignee timeout. I'll take care of it.
Comment 8 commit-hook freebsd_committer freebsd_triage 2018-07-07 10:40:17 UTC
A commit references this bug:

Author: tobik
Date: Sat Jul  7 10:39:58 UTC 2018
New revision: 474070
URL: https://svnweb.freebsd.org/changeset/ports/474070

Log:
  net/dbeacon: Fix build with Clang 6

  dbeacon.cpp:415:78: error: invalid operands to binary expression ('__bind<int &,
        sockaddr *, int>' and 'int')
    ...(bind(mcastSock, beaconUnicastAddr.saddr(), beaconUnicastAddr.addrlen()) != 0) {
        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ^  ~

  - Follow MASTER_SITES redirect

  PR:		226232
  Submitted by:	maintainer

Changes:
  head/net/dbeacon/Makefile