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

Some required changes for CLSQL to work under Lispworks 7. #12

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tdrhq
Copy link

@tdrhq tdrhq commented Sep 5, 2020

I've been using this for a while now. LW7 requires an extra option to
make sure slot values are not optimized when using MOP.

The second fix is minor, I don't remember how exactly it affects
CLSQL. But the feature check +:lispworks6 was used, whereas we really
need something like lispworks6+. Since that feature doesn't exist we
follow LW's guidelines and check for whether it's not lispworks4 or
5.

I've been using this for a while now. LW7 requires an extra option to
make sure slot values are not optimized when using MOP.

The second fix is minor, I don't remember how exactly it affects
CLSQL. But the feature check +:lispworks6 was used, whereas we really
need something like lispworks6+. Since that feature doesn't exist we
follow LW's guidelines and check for whether it's *not* lispworks4 or
5.
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

Successfully merging this pull request may close these issues.

None yet

1 participant