[GitHub] nifi-minifi-cpp issue #84: MINIFI-274 Use external project with existing sou...

2017-05-02 Thread brosander
Github user brosander commented on the issue: https://github.com/apache/nifi-minifi-cpp/pull/84 +1 works for me on bare metal Ubuntu 16.04, OSx 10.12.4, in docker container on both platforms as well --- If your project is set up for it, you can reply to this email and have your

[GitHub] nifi-minifi-cpp issue #84: MINIFI-274 Use external project with existing sou...

2017-05-02 Thread apiri
Github user apiri commented on the issue: https://github.com/apache/nifi-minifi-cpp/pull/84 @brosander Scope out @phrocker's work and merged into my branch which should address the build issues as per MINIFI-283. Updated my branch/PR to include those changes. Let us know how things

[GitHub] nifi-minifi-cpp issue #84: MINIFI-274 Use external project with existing sou...

2017-05-02 Thread phrocker
Github user phrocker commented on the issue: https://github.com/apache/nifi-minifi-cpp/pull/84 @apiri Issues experience by @brosander appear to be solved with https://github.com/phrocker/nifi-minifi-cpp/tree/minifi-274-jsoncpp. The top two commits are the necessary commits. Feel free

[GitHub] nifi-minifi-cpp issue #84: MINIFI-274 Use external project with existing sou...

2017-05-02 Thread kevdoran
Github user kevdoran commented on the issue: https://github.com/apache/nifi-minifi-cpp/pull/84 I was able to reproduce the same behavior using the steps @brosander provided in his comment. MacOS 10.12.4 and Docker CE v 10.03.1-ce-mac5 (16048), stable, b18e2a50cc --- If your project

[GitHub] nifi-minifi-cpp issue #84: MINIFI-274 Use external project with existing sou...

2017-05-01 Thread brosander
Github user brosander commented on the issue: https://github.com/apache/nifi-minifi-cpp/pull/84 The following docker image repros it for me: https://github.com/brosander/minifi-cpp-tooling/tree/9e5b1031d1772ccbee3ec8844a3decf232185284/Dockerfiles/ubuntu/xenial/build ```

[GitHub] nifi-minifi-cpp issue #84: MINIFI-274 Use external project with existing sou...

2017-05-01 Thread brosander
Github user brosander commented on the issue: https://github.com/apache/nifi-minifi-cpp/pull/84 Ubuntu 16.04 64 bit (latest LTS release of Ubuntu) I'll craft a dockerfile for easier testing/validation. --- If your project is set up for it, you can reply to this email and have your

[GitHub] nifi-minifi-cpp issue #84: MINIFI-274 Use external project with existing sou...

2017-05-01 Thread apiri
Github user apiri commented on the issue: https://github.com/apache/nifi-minifi-cpp/pull/84 @brosander could you list your environment? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] nifi-minifi-cpp issue #84: MINIFI-274 Use external project with existing sou...

2017-05-01 Thread brosander
Github user brosander commented on the issue: https://github.com/apache/nifi-minifi-cpp/pull/84 ``` TestSiteToSiteVerifySend ---

[GitHub] nifi-minifi-cpp issue #84: MINIFI-274 Use external project with existing sou...

2017-05-01 Thread brosander
Github user brosander commented on the issue: https://github.com/apache/nifi-minifi-cpp/pull/84 definitely better but I still seem to be getting a segfault on the s2s test --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] nifi-minifi-cpp issue #84: MINIFI-274 Use external project with existing sou...

2017-05-01 Thread brosander
Github user brosander commented on the issue: https://github.com/apache/nifi-minifi-cpp/pull/84 @apiri I'll take a look, thanks! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature

[GitHub] nifi-minifi-cpp issue #84: MINIFI-274 Use external project with existing sou...

2017-05-01 Thread apiri
Github user apiri commented on the issue: https://github.com/apache/nifi-minifi-cpp/pull/84 @brosander looks like some of the environment issues have been ironed out and things look good in Travis, OSX 10.11, & Ubuntu 17.04. --- If your project is set up for it, you can reply to

[GitHub] nifi-minifi-cpp issue #84: MINIFI-274 Use external project with existing sou...

2017-04-28 Thread apiri
Github user apiri commented on the issue: https://github.com/apache/nifi-minifi-cpp/pull/84 @brosander Okay, rebased on latest master and have pushed to update. Thanks for scoping out. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] nifi-minifi-cpp issue #84: MINIFI-274 Use external project with existing sou...

2017-04-28 Thread brosander
Github user brosander commented on the issue: https://github.com/apache/nifi-minifi-cpp/pull/84 +1 merging --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so,

[GitHub] nifi-minifi-cpp issue #84: MINIFI-274 Use external project with existing sou...

2017-04-28 Thread apiri
Github user apiri commented on the issue: https://github.com/apache/nifi-minifi-cpp/pull/84 @brosander just noticed there was a merge conflict from one of the prior merges I performed. will fix up quickly. --- If your project is set up for it, you can reply to this email and have

[GitHub] nifi-minifi-cpp issue #84: MINIFI-274 Use external project with existing sou...

2017-04-28 Thread brosander
Github user brosander commented on the issue: https://github.com/apache/nifi-minifi-cpp/pull/84 Reviewing --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so,