Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix: Missing arguments on "main" function call introduced in 5f7e2d51… #38

Closed
wants to merge 1 commit into from
Closed

Fix: Missing arguments on "main" function call introduced in 5f7e2d51… #38

wants to merge 1 commit into from

Conversation

desyncr
Copy link
Contributor

@desyncr desyncr commented Sep 28, 2016 •
edited

As reported in #37 changes introduced in 5f7e2d5 broke script.

This small fix addressed the main issue which turns the script unusable. There is no further looking into other issues.

Fixes #37

jdloft
jdloft approved these changes Oct 14, 2016
View reviewed changes
Copy link
Contributor

@jdloft jdloft left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixes it for me. Without this fix a blank program is passed to the handler function which reports it as being missing.

@mchav
Copy link
Owner

mchav commented Oct 19, 2016

Closed in favour of the more recent one.

@mchav mchav closed this Oct 19, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jdloft jdloft jdloft approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Program not installed
3 participants
@desyncr @mchav @jdloft