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

Fix numpy-installed-p to work with case sensitive lisps #14

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

Conversation

LEHunter
Copy link

T never has to be quoted (it evaluates to itself), so get rid of the (quote ... ) in the numpy-installed-p file. Has the advantage or producing t in case sensitive lisps as well.

T never has to be quoted (it evaluates to itself), so get rid of the  (quote ... ) in the  numpy-installed-p file.  Has the advantage or producing t in case sensitive lisps as well.
@digikar99
Copy link
Owner

The final goal is to be able to handle (asdf:load-system "py4cl2-cffi") with minimal warnings. Not quoting yields a warning about undefined variable t when the readtable case used to load the system is different from compile time. I had assumed quoting it would get rid of it, but quoting creates other issues. Perhaps, the best is to handle this outside of lisp.

The intention of making it a lisp file is to make use of asdf's autocompile mechanism. By changing numpy-installed-p, it would automatically trigger the recompilation of the lisp file shared-objects.lisp. But perhaps, it might be better to perform the recompilation manually.

Let me see if I can try something to see to solve both the issues.

@LEHunter
Copy link
Author

LEHunter commented Apr 30, 2024 via email

@digikar99 digikar99 force-pushed the macos branch 6 times, most recently from e01462e to 9dbf4cb Compare May 8, 2024 02:27
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.

2 participants