Github user jvwing commented on the issue:
https://github.com/apache/nifi/pull/929
Thanks, @jdye64, that got it. I was able to build just fine with
contrib-check and everything. On to new topics:
* **License/Notice** - We need a separate LICENSE and NOTICE files for
nifi-poi-nar. I was reminded by the recent LICENSE/NOTICE discussion on the
mailing list about what to include in NAR bundles, and it applies here. Good
news, you've already put together most of the text for the nifi-assembly
LICENSE/NOTICE, and there are many examples of the file layout in other NARs.
I think a reasonably comparable NAR referencing another Apache project is
[nifi-ignite-nar](https://github.com/apache/nifi/tree/master/nifi-nar-bundles/nifi-ignite-bundle/nifi-ignite-nar/src/main/resources/META-INF).
* **Sheet in Filename** - Output filenames all have the UNKNOWN sheet name,
like `35200238025098_UNKNOWN.csv`. How about adding a sheet name check to one
of the unit tests?
* **Error Cases** - What will make this fail?
---
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, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
