timvideos
vagrant to demonstrate install
Can you fix the spelling / punctuation in the commit messages?
You can use git rebase --interactive and then mark the patch as "edit". You can also use that to squash patches together.
Hi @CarlFK
This pull request no longer merges. Can you please rebase onto the latest and then I'll try merging it.
Tim
Reopening pull request because it commits were never merged. Need to figure out if we should remove the commits from @CarlFK repo or do something more.
I did this so GSOC students could verify that the install process works. I don't see much value in it now, so just ignore it.
On Sat, Jan 31, 2015 at 8:58 AM, Tim Ansell [email protected] wrote:
@CarlFK https://github.com/CarlFK poke?
— Reply to this email directly or view it on GitHub https://github.com/timvideos/streaming-system/pull/57#issuecomment-72320859 .
Carl K
There is another set of GSoC students only a month or two away from now, will they find it useful?
There were also some other changes in here you made to make the install more automated which I think are worth rescusing.
On 1 February 2015 at 02:07, Carl Karsten [email protected] wrote:
I did this so GSOC students could verify that the install process works. I don't see much value in it now, so just ignore it.
On Sat, Jan 31, 2015 at 8:58 AM, Tim Ansell [email protected] wrote:
@CarlFK https://github.com/CarlFK poke?
— Reply to this email directly or view it on GitHub < https://github.com/timvideos/streaming-system/pull/57#issuecomment-72320859>
.
Carl K
— Reply to this email directly or view it on GitHub https://github.com/timvideos/streaming-system/pull/57#issuecomment-72321201 .
I suspect the install is more complicated than it needs to be.
There are 2 cases:
- install into production on a fresh ubunu 10.04 box.
- install for development on some screwy laptop.
Those don't need to be the same script. If someone wants to install into production on something else, they can make the changes needed for whatever it is they are doing.
given that, I think we need to refactor the install process. Last year I did my best not to touch it so I didn't introduce any problems which would scare off anyone evaluating the project.
You / apps should have a good idea what the requirements are. put them in a script.
On Sat, Jan 31, 2015 at 9:41 AM, timsvideo-portable < [email protected]> wrote:
There is another set of GSoC students only a month or two away from now, will they find it useful?
There were also some other changes in here you made to make the install more automated which I think are worth rescusing.
On 1 February 2015 at 02:07, Carl Karsten [email protected] wrote:
I did this so GSOC students could verify that the install process works. I don't see much value in it now, so just ignore it.
On Sat, Jan 31, 2015 at 8:58 AM, Tim Ansell [email protected] wrote:
@CarlFK https://github.com/CarlFK poke?
— Reply to this email directly or view it on GitHub <
https://github.com/timvideos/streaming-system/pull/57#issuecomment-72320859>
.
Carl K
— Reply to this email directly or view it on GitHub < https://github.com/timvideos/streaming-system/pull/57#issuecomment-72321201>
.
— Reply to this email directly or view it on GitHub https://github.com/timvideos/streaming-system/pull/57#issuecomment-72322580 .
Carl K