Bug 229128

Summary: [maintainer] sysutils/boxbackup: pet pkg-fallout, and portlint
Product: Ports & Packages Reporter: Chris Hutchinson <portmaster>
Component: Individual Port(s)Assignee: Torsten Zuehlsdorff <tz>
Status: Closed FIXED    
Severity: Affects Some People CC: riggs, tz
Priority: --- Keywords: needs-patch, needs-qa
Version: LatestFlags: portmaster: maintainer-feedback+
Hardware: Any   
OS: Any   
Description Flags
svn diff for sysutils/boxbackup
portmaster: maintainer-approval+
QA LOG for sysutils/boxbackup none

Description Chris Hutchinson 2018-06-18 21:37:37 UTC
Created attachment 194375 [details]
svn diff for sysutils/boxbackup

pkg-fallout complains that sysutils/boxbackup doesn't build.
This pr(1) corrects that problem. While here, pet portlint
as well.


That's it!

patch, and QA LOG attached.


Comment 1 Chris Hutchinson 2018-06-18 21:41:37 UTC
Created attachment 194376 [details]
QA LOG for sysutils/boxbackup


works/tested on 12-CURRENT

There are still several warnings, that would be fairly simple to
fix. But ${WORK} && ${LIFE} have me pretty busy right now.
But if you, or someone you know wants to submit any patches,
they will be happily accepted. ;-)

Comment 2 Thomas Zander freebsd_committer 2018-06-24 13:09:56 UTC
This does not build on 11:

BackupStoreInfo.cpp:162:3: error: non-constant-expression cannot be narrowed from type 'unsigned long' to 'int64_t' (aka 'long') in initializer list [-Wc++11-narrowing]
                box_hton64(BlockHardLimit), // mBlocksHardLimit
Comment 3 commit-hook freebsd_committer 2018-06-28 14:15:36 UTC
A commit references this bug:

Author: tz
Date: Thu Jun 28 14:15:01 UTC 2018
New revision: 473497
URL: https://svnweb.freebsd.org/changeset/ports/473497

  sysutils/boxbackup: pet pkg-fallout

  Fix build by using -std=c++98

  Also regenerate patch with make makepatch

  PR:		229128
  Submitted by:	Chris Hutchinson <portmaster@bsdforge.com>

Comment 4 Torsten Zuehlsdorff freebsd_committer 2018-06-28 14:16:29 UTC
Committed, thanks! :)