camsas / firmament

The Firmament cluster scheduling platform
Apache License 2.0
412 stars 77 forks source link

Cosmetic #29

Closed pooya closed 9 years ago

ms705 commented 9 years ago

Hi Shayan,

Thanks a lot for this -- and well done spotting the (more than cosmetic!) README errors :)

I'll extend the Dockerfile a bit after merging this. If building Firmament automatically and with sudo privileges, one can set the NONINTERACTIVE=1 in the environment and the build system will install dependencies automatically.

GerritHub for some reason won't import this pull request, but I'll see if I can bring it in manually. Otherwise, I'll manually merge, since it's tiny anyway.

Cheers, M.

ms705 commented 9 years ago

Code reviews here.

ms705 commented 9 years ago

All code reviews approved (thanks @ICGog); merging.