Github user MJJoyce commented on the pull request:
https://github.com/apache/climate/pull/29#issuecomment-43632364
@lewismc, if you don't run this script with -e (per the recommendation in
the initial header) or you don't run it in a virtualenv environment then you
will run need to run it as sudo on OS X. Of course, that's assuming that you
don't have any other Python around that isn't the system Python. You shouldn't
have any problems if you run it in a virtualenv on OS X without sudo, but if
you run into any let me know. Either way, that will be made explicit in the
wiki page that I'm writing.
Re. 3, I don't see the point in suppressing what's output to the log. The
log is mostly for debugging. The average user wont ever need to look at it. If
that stuff is getting dumped to stdout, then yes I think it should be
suppressed (mostly by being sent to the log).
4 you can ignore. It's a issue with clang on OS X and the warnings are an
artifact of the config flag set before installation. If you aren't seeing those
you should be concerned, namely because clang will then elevate them to errors
and all your builds will fail =)
Re. 8, we could print some message about the installation completing. But
I'm not a fan of printing that it was successful unless we actually validate
the install. There's nothing more annoying than seeing the message when
something actually failed. I'll make a note of it and add it to the list of
JIRAs I'm going to make once I merge this.
---
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.
---
