search

ianfhunter / TeamCrab

Group Project for CS4098
MIT License
0 stars 1 forks source link

Fix makefile for running without installing #37

Closed scottcunningham closed 10 years ago

scottcunningham commented 10 years ago

Bash was trying to interpret the pyc file, we should change it to either do: ./bin/SESimulator.py or python ./bin/SESimulator.pyc

Thoughts?

ianfhunter commented 10 years ago

Maybe just write in readme, if x fails , do y?

-----Original Message----- From: "Scott Cunningham" notifications@github.com Sent: ‎08/‎03/‎2014 15:05 To: "ianfhunter/TeamCrab" TeamCrab@noreply.github.com Subject: [TeamCrab] Fix makefile for running without installing (#37)

Bash was trying to interpret the pyc file, we should change it to either do: ./bin/SESimulator.py or python ./bin/SESimulator.pyc Thoughts? — Reply to this email directly or view it on GitHub.

scottcunningham commented 10 years ago

That seems pretty confusing. Running python bin/SESimulator.pyc will work every time. Why not just use that?

On 8 March 2014 16:39, Ian Hunter notifications@github.com wrote:

Maybe just write in readme, if x fails , do y?

-----Original Message----- From: "Scott Cunningham" notifications@github.com Sent: 08/03/2014 15:05 To: "ianfhunter/TeamCrab" TeamCrab@noreply.github.com Subject: [TeamCrab] Fix makefile for running without installing (#37)

Bash was trying to interpret the pyc file, we should change it to either do: ./bin/SESimulator.py or python ./bin/SESimulator.pyc

Thoughts?

Reply to this email directly or view it on GitHub.

Reply to this email directly or view it on GitHubhttps://github.com/ianfhunter/TeamCrab/issues/37#issuecomment-37102352 .

ianfhunter commented 10 years ago

Oh yeah that makes a lot more sense

On Sat, Mar 8, 2014 at 7:09 PM, Scott Cunningham notifications@github.comwrote:

That seems pretty confusing. Running python bin/SESimulator.pyc will work every time. Why not just use that?

On 8 March 2014 16:39, Ian Hunter notifications@github.com wrote:

Maybe just write in readme, if x fails , do y?

-----Original Message----- From: "Scott Cunningham" notifications@github.com Sent: 08/03/2014 15:05 To: "ianfhunter/TeamCrab" TeamCrab@noreply.github.com Subject: [TeamCrab] Fix makefile for running without installing (#37)

Bash was trying to interpret the pyc file, we should change it to either do: ./bin/SESimulator.py or python ./bin/SESimulator.pyc

Thoughts?

Reply to this email directly or view it on GitHub.

Reply to this email directly or view it on GitHub< https://github.com/ianfhunter/TeamCrab/issues/37#issuecomment-37102352>

.

Reply to this email directly or view it on GitHubhttps://github.com/ianfhunter/TeamCrab/issues/37#issuecomment-37106569 .

ianhunter.ie

scottcunningham commented 10 years ago

did someone do this? looks done to me

ianfhunter commented 10 years ago

Looks like it was done in 67799aa16604be41e0245fb44b60d078013ab08c , but this was in iteration 3's submission - the current master branch - So there's something weird going on if it wasn't working in the meeting. :/

ianfhunter commented 10 years ago

ah. The instructions to use that command weren't put in the readme. Shall fix.

ianfhunter commented 10 years ago

Would like someone to verify that it reads correctly though