libera/#slime - IRC Chatlog

7:20:19 jackdaniel luis: do you think it is OK to merge that?

7:20:34 jackdaniel or more broadly - is anyone opposed to merging the pr that makes swank.asd an "ordinary" system?

8:18:22 luis jackdaniel: yes, I think so. Let's do it.

8:25:15 luis jackdaniel: your pr differs from previous attempts in that, for instance, it does not split each contrib into its own system, but I like this simpler start.

8:26:10 luis jackdaniel: also, it'd be nice if there was only one place to define the loading order (for both the ASDF and non-ASDF mechanism) but that might be more work than it's worth, not sure.

8:59:14 jackdaniel luis: I think that having a "common root" would only complicate the code (i.e it will be less readable)

8:59:41 jackdaniel i.e you'd need to load the actual code before the asd definition, or you'd need to depend on asdf in the standalone loader

9:00:41 jackdaniel and it is only a few lines, so it does not duplicate "much" when it comes to the operational logic. especially that I can imagine that people may expect different behavior from asd and from the loader (i.e slime having some extra hooks to enable/disable contribs)

9:01:43 jackdaniel regarding contribs, I don't think I've ever seen loaded them separately; if you like I can make each of them a separate system though

9:02:14 jackdaniel but I'd rather have it the way it is and improve upon it when a real use case arrives

9:09:24 luis jackdaniel: I was thinking maybe something like a .sexp file with a load order list that can be READ. But it's just a random thought, no need to complicate this PR.

9:10:27 jackdaniel great :) so should I merge it or do you want to have the honor of merging it? ;)

9:11:17 jackdaniel hum, I should add an entry to NEWS before that

9:13:00 luis jackdaniel: https://github.com/joaotavora/sly/blob/master/slynk/slynk.asd is what I was thinking about, probably.

9:13:30 luis jackdaniel: got for it.

9:13:34 luis *go

9:14:04 jackdaniel thanks!

9:14:08 jackdaniel ACTION goes for it