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

Add lem-extesion-manager #1413

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

Sasanidas
Copy link
Member

So, with this PR I want to add https://github.com/lem-project/lem-extension-manager for the main Lem repository.

This would be a draft until I add the commands https://github.com/lem-project/lem-extension-manager/blob/main/main.lisp#L225 that are not used on the extension-manager to the main Lem codebase and I would also like to add some documentation.

@Sasanidas Sasanidas self-assigned this Jun 24, 2024
@Sasanidas Sasanidas marked this pull request as ready for review June 27, 2024 16:36
@Sasanidas Sasanidas requested a review from cxxxr June 27, 2024 16:36
@Sasanidas
Copy link
Member Author

I have to comment the library, but I think all the basic code is in place, and the package system can start working.

Some improvements would be the ability to update git packages and quicklisp ones (both are quite easy to add).

A lock file may also be nice, tho the idea is to keep the package manager light and easy to use

@Sasanidas
Copy link
Member Author

Doc written https://github.com/lem-project/lem-extension-manager

@Sasanidas
Copy link
Member Author

Ping @cxxxr

@cxxxr
Copy link
Member

cxxxr commented Nov 12, 2024

Sorry, I've been meaning to look at this sometime, but it's been on the back burner for a long time.

@cxxxr
Copy link
Member

cxxxr commented Nov 12, 2024

The word “package” seems to be the unit of extension, but it overlaps with the term “package” in Common Lisp, so I found it difficult to understand.
Wouldn't extension be better?

@cxxxr
Copy link
Member

cxxxr commented Nov 12, 2024

The README says to do it like this, but I'm worried that the information will be duplicated with asd.

                                       :source (:type :git
                                                :url "https://github.com/smithzvk/Versioned-Objects.git"
                                                :branch "advance-versioning")
                                       :dependencies (("ip-management"
                                                       :source
                                                       (:type :git
                                                        :url "https://github.com/Sasanidas/ip-management.git"))
                                                      ("1am" :source (:type :quicklisp))))

@cxxxr
Copy link
Member

cxxxr commented Nov 12, 2024

What is simple-package? The reason it has simple in the name is probably just to avoid a conflict with cl:package. Wouldn't it be better to just call it extension?

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