Github user lewismc commented on the pull request:
https://github.com/apache/climate/pull/29#issuecomment-43571116
Hey Mike. I fetched your PR locally and ran the easy-ocw script. First and
foremost... good work, this quickens installation to a fraction of the time it
was taking before and is exactly what climate needed.
Please see my comments below regarding my initial observations of the log
output. I've attached my log output to JIRA Issues CLIMATE-370 so you can see
it there.
First though, for others, Mikes patch can be fetched and applied locally as
follows
https://help.github.com/articles/checking-out-pull-requests-locally
*1*
```
Installing collected packages: conda, pycosat, pyyaml
Running setup.py install for conda
changing mode of build/scripts-2.7/conda from 644 to 755
error: could not create '/Library/Python/2.7/site-packages/conda':
Permission denied
```
Is it a prerequisite to be logged in as sudo to successfully access
offending Python directory and install? If so then we need to document this...
possibly within the initial message prompt provided when one invokes the script
(I've only run the macosx script so far Mike).
*2*
```
error: /Library/Python/2.7/site-packages/bottle.py: Permission denied
```
Same as above?
*3*
Many instances of the following
```
warning: no previously-included files matching '*' found under
directory 'docs/_build/_sources'
warning: no files found matching '*.js' under directory 'paste'
warning: no files found matching '*.jpg' under directory 'paste'
```
I don not actually think that these are problems as they are warnings. If
this is the case however can we suppress them somehow?
*4*
```
clang: warning: unknown argument: '-mno-fused-madd'
[-Wunused-command-line-argument-hard-error-in-future]
clang: note: this will be a hard error (cannot be downgraded to a warning)
in the future
clang: warning: argument unused during compilation: '-mno-fused-madd'
```
I am out of suggestions for this one... I do not have a shovel large enough
to dig this up right now. Any ideas?
*5*
```
error: could not create '/Library/Python/2.7/site-packages/webtest':
Permission denied
```
Same as 1 & 2 ?
*6*
```
no previously-included directories found matching 'doc/.build'
Installing collected packages: nose
Running setup.py install for nose
error: could not create '/Library/Python/2.7/site-packages/nose':
Permission denied
```
... and again
*7*
```
error: could not create '/Library/Python/2.7/site-packages/pylint':
Permission denied
```
enough already ;)
*8* The install script does not specify whether the install process was
successful. It would be really nice to see some kind of verification that a
successful (or unsuccessful) build took place.
Thanks Mike very much for working on this. Great job.
---
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.
---
