Downloaded, verified the GPG signature and checksums, checked the README, 
LICENSE, and NOTICE files. Verified the git commit ID.

Built with make (had to install boost using Homebrew first). I got a couple 
warnings & errors during the make process (documented below) — not sure if they 
are known issues but they did not stop the build from completing.

I followed Koji’s lead and used the sample README flow.yml and sent output to a 
local NiFi instance using S2S. Everything worked great.

+1 (binding)

Noted issues (let me know if you would like me to report these as Jiras):

During `make`:

gcc -g -Wall -rdynamic  -O0 -ggdb -g3 -fexceptions -Wno-write-strings 
-fvisibility=hidden -fPIC -I.. -c clear.c  -o clear.o
clang: warning: argument unused during compilation: '-rdynamic'
gcc -g -Wall -rdynamic  -O0 -ggdb -g3 -fexceptions -Wno-write-strings 
-fvisibility=hidden -fPIC -I.. -c compare.c  -o compare.o
clang: warning: argument unused during compilation: '-rdynamic'
gcc -g -Wall -rdynamic  -O0 -ggdb -g3 -fexceptions -Wno-write-strings 
-fvisibility=hidden -fPIC -I.. -c copy.c  -o copy.o
clang: warning: argument unused during compilation: '-rdynamic'
gcc -g -Wall -rdynamic  -O0 -ggdb -g3 -fexceptions -Wno-write-strings 
-fvisibility=hidden -fPIC -I.. -c gen_uuid.c  -o gen_uuid.o
clang: warning: argument unused during compilation: '-rdynamic'
gcc -g -Wall -rdynamic  -O0 -ggdb -g3 -fexceptions -Wno-write-strings 
-fvisibility=hidden -fPIC -I.. -c isnull.c  -o isnull.o
clang: warning: argument unused during compilation: '-rdynamic'
gcc -g -Wall -rdynamic  -O0 -ggdb -g3 -fexceptions -Wno-write-strings 
-fvisibility=hidden -fPIC -I.. -c pack.c  -o pack.o
clang: warning: argument unused during compilation: '-rdynamic'
gcc -g -Wall -rdynamic  -O0 -ggdb -g3 -fexceptions -Wno-write-strings 
-fvisibility=hidden -fPIC -I.. -c parse.c  -o parse.o
clang: warning: argument unused during compilation: '-rdynamic'
gcc -g -Wall -rdynamic  -O0 -ggdb -g3 -fexceptions -Wno-write-strings 
-fvisibility=hidden -fPIC -I.. -c unpack.c  -o unpack.o
clang: warning: argument unused during compilation: '-rdynamic'
gcc -g -Wall -rdynamic  -O0 -ggdb -g3 -fexceptions -Wno-write-strings 
-fvisibility=hidden -fPIC -I.. -c unparse.c  -o unparse.o
clang: warning: argument unused during compilation: '-rdynamic'
gcc -g -Wall -rdynamic  -O0 -ggdb -g3 -fexceptions -Wno-write-strings 
-fvisibility=hidden -fPIC -I.. -c uuid_time.c  -o uuid_time.o
clang: warning: argument unused during compilation: '-rdynamic'

...


src/FlowController.cpp:83:7: warning: unused variable 'flow_full_path' 
[-Wunused-variable]
        char flow_full_path[PATH_MAX];
             ^
src/FlowController.cpp:82:8: warning: unused variable 'flowPath' 
[-Wunused-variable]
        char *flowPath = NULL;
              ^
2 warnings generated.

...


g++ -Os -fexceptions -fpermissive -Wno-write-strings -std=c++11 -fPIC -Wall -g 
-Wno-unused-private-field -I./inc -I./src -I./test -I./thirdparty 
-I/usr/include/libxml2 -I./thirdparty/yaml-cpp-yaml-cpp-0.5.3/include -o 
./build/Server test/Server.cpp -L./build -L./thirdparty/uuid 
-L./thirdparty/yaml-cpp-yaml-cpp-0.5.3/lib/ -lminifi -lxml2 -pthread -luuid 
-lyaml-cpp
clang: error: no such file or directory: 'test/Server.cpp'
make: [tests] Error 1 (ignored)

In minifi.properties (is there a reason this is nifi.version instead of 
minifi.version?):

# Core Properties #
nifi.version=0.0.1

Andy LoPresto
alopre...@apache.org
alopresto.apa...@gmail.com
PGP Fingerprint: 70EC B3E5 98A6 5A3F D3C4  BACE 3C6E F65B 2F7D EF69

> On Aug 29, 2016, at 4:47 PM, Koji Kawamura <ijokaruma...@gmail.com> wrote:
> 
> +1 (non-binding)
> 
> Downloaded the source archive, verified signature, md5, sha1, and
> sha256 hashes are all good.
> LICENSE file looks good
> Built the source successfully with the steps written in README, on Mac
> OS X El Captain
> Used brew to install boost: $ brew install boost
> Created conf/flow.yml:
> - copy & paste from README
> - modified Input Ports id so that it matches with the remote NiFi's
> input port uuid
> - enabled Site-to-Site at the remote NiFi
> Started minifi, created a file under /tmp/getfile dir, then confirmed
> the file content was sent to the remote NiFi via Site-to-Site
> 
> 
> 
> On Mon, Aug 29, 2016 at 5:52 AM, Aldrin Piri <ald...@apache.org> wrote:
>> Hello Apache NiFi Community,
>> 
>> I am pleased to be calling this vote for the source release of Apache NiFi 
>> MiNiFi C++, nifi-minifi-cpp-0.0.1.
>> 
>> The source archive, signature, and digests can be located at:
>> 
>> Source Archive:
>>        
>> https://dist.apache.org/repos/dist/dev/nifi/nifi-minifi-cpp/0.0.1/nifi-minifi-cpp-0.0.1-source.tar.gz
>> GPG armored signature:
>>        
>> https://dist.apache.org/repos/dist/dev/nifi/nifi-minifi-cpp/0.0.1/nifi-minifi-cpp-0.0.1-source.tar.gz.asc
>> Source MD5:
>>        
>> https://dist.apache.org/repos/dist/dev/nifi/nifi-minifi-cpp/0.0.1/nifi-minifi-cpp-0.0.1-source.tar.gz.md5
>> Source SHA1:
>>        
>> https://dist.apache.org/repos/dist/dev/nifi/nifi-minifi-cpp/0.0.1/nifi-minifi-cpp-0.0.1-source.tar.gz.sha1
>> Source SHA256:
>>        
>> https://dist.apache.org/repos/dist/dev/nifi/nifi-minifi-cpp/0.0.1/nifi-minifi-cpp-0.0.1-source.tar.gz.sha256
>> 
>> The Git tag is minifi-cpp-0.0.1-RC1
>> The Git commit hash is 61f0e450ef6604e7028be38f0335e70713ebdea3
>> * 
>> https://git-wip-us.apache.org/repos/asf?p=nifi-minifi-cpp.git;a=commit;h=61f0e450ef6604e7028be38f0335e70713ebdea3
>> * 
>> https://github.com/apache/nifi-minifi-cpp/commit/61f0e450ef6604e7028be38f0335e70713ebdea3
>> 
>> Checksums of nifi-minifi-cpp-0.0.1-source.tar.gz:
>> MD5: 24121f1ebe3a678d5e2dc84fb8479ef6
>> SHA1: 32c1fb584c9bb484c456cdc4f5dfdd9e91e13364
>> SHA256: c72046d86a9dc34033fa40ad1b2300f1f7915acd369f763e7fc02095d09bb91e
>> 
>> Release artifacts are signed with the following key:
>> https://people.apache.org/keys/committer/aldrin
>> 
>> KEYS file available here:
>> https://dist.apache.org/repos/dist/release/nifi/KEYS
>> 
>> 11 issues were closed/resolved for this release:
>> https://issues.apache.org/jira/secure/ReleaseNote.jspa?version=12337990&projectId=12319921
>> 
>> Release note highlights can be found here:
>> https://cwiki.apache.org/confluence/display/MINIFI/Release+Notes#ReleaseNotes-Versioncpp-0.0.1
>> 
>> The vote will be open for 72 hours.
>> Please download the release candidate and evaluate the necessary items
>> including checking hashes, signatures, build from source, and test. Then
>> please vote:
>> 
>> [ ] +1 Release this package as nifi-minifi-cpp-0.0.1
>> [ ] +0 no opinion
>> [ ] -1 Do not release this package because...
>> 
>> Thanks!

Attachment: signature.asc
Description: Message signed with OpenPGP using GPGMail

Reply via email to