Open CarlFK opened 10 years ago
I think I cleaned up all the things you mentioned.
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.
@CarlFK poke?
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 notifications@github.com 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 notifications@github.com 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 notifications@github.com 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:
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 < notifications@github.com> 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 notifications@github.com 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 notifications@github.com 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
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.