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

build order issues #20

Closed
slyrus opened this issue Oct 24, 2018 · 1 comment
Closed

build order issues #20

slyrus opened this issue Oct 24, 2018 · 1 comment

Comments

@slyrus
Copy link
Contributor

slyrus commented Oct 24, 2018

When asdf:load-ing fset I see the following:

; file: /home/sly/quicklisp/local-projects/fset/Code/fset.lisp
; in: DEFMETHOD FSET::SORT-AND-GROUP (FSET:SEQ T)
; (FSET:DO-SEQ (FSET::X FSET::SORTED)
; (IF (OR (FSET:EMPTY? FSET::GROUP)
; (NOT
; (IF FSET::KEY
; #
; #)))
; (FSET:PUSH-LAST FSET::GROUP FSET::X)
; (PROGN (FSET:PUSH-LAST FSET::RESULT FSET::GROUP) (SETQ FSET::GROUP #))))
;
; caught STYLE-WARNING:
; undefined function: FSET:DO-SEQ

; (FSET:SEQ)
;
; caught STYLE-WARNING:
; undefined function: FSET:SEQ

; (FUNCALL FSET::KEY FSET::X)
; ==>
; (SB-C::%FUNCALL (SB-KERNEL:%COERCE-CALLABLE-FOR-CALL FSET::KEY) FSET::X)
;
; caught WARNING:
; undefined variable: FSET::X

; (FSET::X FSET::SORTED)
;
; caught STYLE-WARNING:
; undefined function: FSET::X
;
; compilation unit finished
; Undefined functions:
; FSET:DO-SEQ FSET:SEQ FSET::X
; Undefined variable:
; FSET::X
; caught 1 WARNING condition
; caught 8 STYLE-WARNING conditions

The problem is that the SEQ and DO-SEQ macros aren't yet loaded. There are some order dependencies that make fixing this a little bit tricky but I imagine it's relatively straightforward.

@slburson
Copy link
Owner

This has been fixed, in 46459b8, a692b5c, 69a0d59, and others.

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

2 participants