Github user cestella commented on the pull request:
https://github.com/apache/incubator-metron/pull/76#issuecomment-210557717
Yeah the license is fine. We are going to have it installed in the vagrant
box as part of ansible, right?
On Fri, Apr 15, 2016 at 18:16 Nick Allen <[email protected]> wrote:
> Interesting. I manually deployed on CentOS 6 with the latest change and it
> worked. But it appears argparse was already installed by something else.
> This node went through a partial Metron install, but no pycapa.
>
> [vagrant@node1 pycapa]$ ls -l /usr/lib/python2.6/site-packages/argparse*
> -rw-r--r--. 1 root root 87791 Mar 31 2011
/usr/lib/python2.6/site-packages/argparse.py
> -rw-r--r--. 1 root root 66661 Jul 23 2015
/usr/lib/python2.6/site-packages/argparse.pyc
> -rw-r--r--. 1 root root 66491 Jul 23 2015
/usr/lib/python2.6/site-packages/argparse.pyo
>
> /usr/lib/python2.6/site-packages/argparse-1.2.1-py2.6.egg-info:
> total 16
> -rw-r--r--. 1 root root 1 Jul 23 2015 dependency_links.txt
> -rw-r--r--. 1 root root 2927 Jul 23 2015 PKG-INFO
> -rw-r--r--. 1 root root 250 Jul 23 2015 SOURCES.txt
> -rw-r--r--. 1 root root 9 Jul 23 2015 top_level.txt
>
> Whether it is standard lib or installed via pip, the license issue is moot
> now right? So I could just revert the last commit since I know the
original
> code worked on 2.6 and 2.7?
>
> â
> You are receiving this because you were mentioned.
> Reply to this email directly or view it on GitHub
>
<https://github.com/apache/incubator-metron/pull/76#issuecomment-210552058>
>
---
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.
---
