Ticket #789 (closed defect: fixed)

Opened 3 months ago

Last modified 3 months ago

bind egg: unable to parse command-line parameters

Reported by: kristianlm Owned by: felix
Priority: major Milestone: 4.8.0
Component: extensions Version: 4.7.x
Keywords: bind Cc:

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

bind-fix-for-fix.diff (397 bytes) - added by ckeen 3 months ago.
diff against trunk to really fix the issue (hopefully)

Change History

Changed 3 months ago by felix

  • status changed from new to closed
  • resolution set to fixed

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

Changed 3 months ago by ckeen

  • status changed from closed to reopened
  • resolution fixed deleted

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 3 months ago by ckeen

diff against trunk to really fix the issue (hopefully)

Changed 3 months ago by ckeen

  • owner set to felix
  • status changed from reopened to assigned

Changed 3 months ago by felix

  • status changed from assigned to closed
  • resolution set to fixed

Thanks, sett bind 0.991

Note: See TracTickets for help on using tickets.