Github user justinleet commented on the issue:
https://github.com/apache/metron/pull/816
I agree we leave it as-is.
---
Github user mattf-horton commented on the issue:
https://github.com/apache/metron/pull/816
Agree with @mmiklavc comment.
---
Github user mmiklavc commented on the issue:
https://github.com/apache/metron/pull/816
@cestella @mattf-horton - I don't think the commit hurts anything, so no
point in removing it now that it's in. But I do agree with otto that it would
be better to use $METRON_HOME and that this
Github user cestella commented on the issue:
https://github.com/apache/metron/pull/816
Yeah, I agree that it doesn't materially affect the example. I decided to
adjust it anyway excepting the names of the rpms. Perhaps I should revert the
commit. Thoughts?
---
Github user mattf-horton commented on the issue:
https://github.com/apache/metron/pull/816
@mmiklavc and @cestella regarding "0.4.1" references in README.md files,
please see https://github.com/apache/metron/pull/767 , specifically Otto's
comment
Github user mmiklavc commented on the issue:
https://github.com/apache/metron/pull/816
I just found a few more references to 0.4.1. We should probably change
paths from explicit version paths to $METRON_HOME in README's. We probably
don't want to tackle this here. We should fix the
Github user cestella commented on the issue:
https://github.com/apache/metron/pull/816
I found a couple of refs to 0.4.1 in the profiler READMEs. I went ahead
and corrected them while I was in there.
---
Github user mmiklavc commented on the issue:
https://github.com/apache/metron/pull/816
+1 by inspection
---
Github user mattf-horton commented on the issue:
https://github.com/apache/metron/pull/816
+1 by inspection, pending Travis
---
Github user justinleet commented on the issue:
https://github.com/apache/metron/pull/816
+1 by inspection, pending Travis
---
10 matches
Mail list logo