user/lidm: New package - #6112
Conversation
What about tuigreet? |
| make_build_args = ["PREFIX=/usr"] | ||
| make_install_args = ["PREFIX=/usr"] | ||
| make_use_env = True |
There was a problem hiding this comment.
why is this needed? seems like it isn't
| hostmakedepends = ["linux-pam-devel", "linux-headers"] | ||
| makedepends = ["linux-pam-devel", "linux-headers"] | ||
|
|
||
| options = ["etcfiles", "!check"] |
There was a problem hiding this comment.
Should go after sha256, and you should provide a reason
|
|
||
| def post_install(self): | ||
| self.install_license("LICENSE") | ||
| self.do( |
There was a problem hiding this comment.
Use self.install_service. and don't seperate self.do invocations with a semicolon, thats a gross hack and is difficult to read
There was a problem hiding this comment.
it just doesn't work period, this just creates a bunch of weirdly named directories and doesn't install any service file (which, if it did, it would get installed in a wrong path and the lint would fail, and also the upstream service file doesn't conform to requirements)
3 participants
Description
new package: LiDM
LiDM is a really light unix login manager made in C, and highly customizable sadly in /etc, figuring out /usr/share/examples/ might take some time. Whats more important for why i want this in the repos is that the only login managers are sddm and gdm, with no option for a lighter TUI login manager, and compared to other TUI login managers (ly, lemurs, etc.) LiDM has a short dependency list, is written in a pretty standard language, and from my testing just works.
I understand if this gets turned down from the repos
Checklist
I have read CONTRIBUTING.md
I acknowledge that overtly not following the above or the below will result in my pull request getting closed
I acknowledge https://gts.chimera-linux.org/@chimera/statuses/01KWARHE1BXBMXB8WPXK0BBM1B
I have read Packaging.md
I have built and tested my changes on my machine
I will take responsibility for my template and keep it up to date