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.

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

2htdp/planetcute ambiguous binding error #1099

Closed
tonyg opened this issue Oct 22, 2015 · 1 comment
Closed

2htdp/planetcute ambiguous binding error #1099

tonyg opened this issue Oct 22, 2015 · 1 comment

Comments

@tonyg
Copy link
Member

tonyg commented Oct 22, 2015

Program:

#lang racket
(require 2htdp/planetcute)
character-cat-girl

Expected outcome: Successful compilation; single image result (from the value of character-cat-girl)

Actual outcome:

character-cat-girl: identifier's binding is ambiguous
  context...:
   #(1972 module) #(1973 module t 0) #(4134 local) #(4135 intdef) #(4136 macro)
   #(4137 macro)
  matching binding...:
   #<module-path-index:((lib "character-cat-girl.rkt" "2htdp" "planetcute"))>
   #(1973 module t 0) #(4137 macro)
  matching binding...:
   #<module-path-index:(2htdp/planetcute)>
   #(1972 module) #(1973 module t 0)
  in: character-cat-girl
  context...:
   standard-module-name-resolver

It also produces an error in DrRacket's BSL and ASL (with the #lang line removed, of course).

Racket version:

~$ racket --version
Welcome to Racket v6.3.0.2.
@tonyg
Copy link
Member Author

tonyg commented Oct 22, 2015

However, it works to do this, and use character-cat-girl* uniformly in place of character-cat-girl:

(require (rename-in 2htdp/planetcute [character-cat-girl character-cat-girl*]))

samth added a commit to samth/htdp that referenced this issue Oct 23, 2015
This avoids an ambigous binding error created by the new macro system.
Fixes racket/racket#1099.
samth added a commit to samth/htdp that referenced this issue Oct 23, 2015
This avoids an ambigous binding error created by the new macro system.
Fixes racket/racket#1099.

Add a test file that will be run in DrDr.
samth added a commit to samth/htdp that referenced this issue Oct 23, 2015
This avoids an ambigous binding error created by the new macro system.
Fixes racket/racket#1099.

Add a test file that will be run in DrDr.
samth added a commit to racket/htdp that referenced this issue Oct 26, 2015
This avoids an ambigous binding error created by the new macro system.
Fixes racket/racket#1099.

Add a test file that will be run in DrDr.

(cherry picked from commit b936b54)
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

No branches or pull requests

1 participant