Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#789 closed defect (fixed)

bind egg: unable to parse command-line parameters

Reported by: kristianlm Owned by: felix winkelmann
Priority: major Milestone: 4.9.0
Component: extensions Version: 4.7.x
Keywords: bind Cc:
Estimated difficulty:

Description

After installing chicken-bind, and supplying certain command-line options, {{{chicken-bind}} fails during command-line parsing:

$ chicken-bind -export-constants

Error: bad argument count - received 2 but expected 1: #<procedure>

This small patch should fix the problem:

diff --git a/bind-translator.scm b/bind-translator.scm
index 31cf9d2..6796063 100644
--- a/bind-translator.scm
+++ b/bind-translator.scm
@@ -1382,7 +1382,7 @@ EOF
             (loop (cdr pstack) ppstack) ] 
            [else (loop pstack ppstack)] ) ) ) ) ) )
 
-(define (set-bind-options opts)
+(define (set-bind-options . opts)
   (let loop ((opts opts))
     (match opts
       (() #f)

Attachments (1)

bind-fix-for-fix.diff (397 bytes) - added by Christian Kellermann 12 years ago.
diff against trunk to really fix the issue (hopefully)

Download all attachments as: .zip

Change History (6)

comment:1 Changed 12 years ago by felix winkelmann

Resolution: fixed
Status: newclosed

Thanks for reporting this - the patch has been applied and a new version (0.99) been tagged.

comment:2 Changed 12 years ago by Christian Kellermann

Resolution: fixed
Status: closedreopened

This is only part of the fix, as the bind tests fail when you use the set-bind-options in the macro bind-options, see http://tests.call-cc.org/master/linux/x86/2012/02/21/salmonella-report/test/bind.html.

Attached is a patch that fixes *that* issue as well.

Changed 12 years ago by Christian Kellermann

Attachment: bind-fix-for-fix.diff added

diff against trunk to really fix the issue (hopefully)

comment:3 Changed 12 years ago by Christian Kellermann

Owner: set to felix winkelmann
Status: reopenedassigned

comment:4 Changed 12 years ago by felix winkelmann

Resolution: fixed
Status: assignedclosed

Thanks, sett bind 0.991

comment:5 Changed 12 years ago by felix winkelmann

Milestone: 4.8.04.9.0

Milestone 4.8.0 deleted

Note: See TracTickets for help on using tickets.