Skip to content

Conversation

@winwaed
Copy link
Contributor

@winwaed winwaed commented Jun 21, 2013

The call to run_semaphore() on line 153 in semaphore(), used the incorrect parameter name "semaphore". This should be called "release". This parameter passes the path to the SEMAFOR installation.

The call to run_semaphore() on line 153 in semaphore(), used the incorrect parameter name "semaphore". This should be called "release". This parameter passes the path to the SEMAFOR installation.
@jac2130
Copy link
Owner

jac2130 commented Jun 21, 2013

Thank you very much for the fixes, I will look at them and pull the changes
as soon as I'm done with the workshop that I am currently participating in!
I greatly apprechiate your help and your improvements!!!

Johannes

On Thu, Jun 20, 2013 at 6:02 PM, winwaed notifications@github.com wrote:

The call to run_semaphore() on line 153 in semaphore(), used the incorrect
parameter name "semaphore". This should be called "release". This parameter

passes the path to the SEMAFOR installation.

You can merge this Pull Request by running

git pull https://github.com/winwaed/semaphore-python patch-2

Or view, comment on, or merge it at:

#2
Commit Summary

  • Invalid parameter in call to run_semaphore fixed

File Changes

  • M semaphore.py (2)

Patch Links:

Johannes http://www.columbia.edu/~jac2130


"I can calculate the motions of the heavenly bodies, but not the madness of
people."

  • Isaac Newton

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants