hellerve / sbcli

A REPL for my SBCL needs
GNU General Public License v3.0
94 stars 13 forks source link

Expose quicklisp location to user #33

Closed cab-1729 closed 4 months ago

cab-1729 commented 5 months ago

Allow for different quicklisp location.

vindarel commented 5 months ago

There might be an issue if we wanted to quickload some libraries in our init file right?

cab-1729 commented 5 months ago

There might be an issue if we wanted to quickload some libraries in our init file right?

init file? I don't get it. Please forgive me, I'm a little new to Lisp.

vindarel commented 5 months ago

I'm a little new to Lisp.

it's great you send PRs already :)

I should have said sbcli's config file.

we load one:

(if (probe-file *config-file*)
  (load *config-file*))

with your PR we load quicklisp afterwards.

What if we want to quickload a library in our config file, before quicklisp is loaded?

(there might be an issue, I don't have the answer)

cab-1729 commented 5 months ago

If the user intends to to use quickload in the sbclirc shouldn't he have to load quicklisp in the same file separately? If the user does so then there would be no issue. It makes sense for a user to load dependencies in the same file they are being used, rather than relying on it being already loaded in another dependency. I think this is a general programming practice. For example, this is the reason why we use #ifndef header guards in C. To stop a dependency from being loaded more than once, because the programmer is expected to load each dependency in each file it is used even if those files are compiled together. I hope I'm making sense.