fairyglade/ly
fairyglade/ly
Fork 44
Code Issues 27 Pull requests 3 Releases 15 Activity

support using different login shell #897

Open
DrSensor wants to merge 1 commit from DrSensor/ly:feature/login_shell into master
pull from: DrSensor/ly:feature/login_shell
merge into: fairyglade:master
Conversation 1 Commits 1 Files changed 4 +10 -2
DrSensor commented 2025-12-21 23:05:32 +01:00
First-time contributor
Copy link

What are the changes about?

This add a config to use different login shell when entering a session.

What existing issue does this resolve?

N/A (no issue attached)

This can slim down the memory consumption when using different login shell other than /bin/sh.
When you use different login shell it may look like this:

systemd─┬─...
        .
        ├─...
        .
        ├─ly───fish───ly───sh───systemctl

notice there is 2 different shell spawned

Pre-requisites

  • I have tested & confirmed the changes work locally
## What are the changes about? This add a config to use different login shell when entering a session. ## What existing issue does this resolve? N/A (no issue attached) This can slim down the memory consumption when using different login shell other than `/bin/sh`. When you use different login shell it may look like this: ``` systemd─┬─... . ├─... . ├─ly───fish───ly───sh───systemctl ``` notice there is 2 different shell spawned ## Pre-requisites - [x] I have tested & confirmed the changes work locally
DrSensor canceled time tracking 2025-12-21 23:06:27 +01:00
DrSensor canceled time tracking 2025-12-22 19:05:06 +01:00
AnErrupTion commented 2025-12-29 23:42:14 +01:00
Owner
Copy link

@DrSensor I'm not sure this makes sense, especially since you can change /bin/sh to be your preferred shell. What are the savings in memory usage you found?

@DrSensor I'm not sure this makes sense, especially since you can change /bin/sh to be your preferred shell. What are the savings in memory usage you found?
This pull request has changes conflicting with the target branch.
  • src/main.zig
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u feature/login_shell:DrSensor-feature/login_shell
git switch DrSensor-feature/login_shell

Merge

Merge the changes and update on Forgejo.
git switch master
git merge --no-ff DrSensor-feature/login_shell
git switch DrSensor-feature/login_shell
git rebase master
git switch master
git merge --ff-only DrSensor-feature/login_shell
git switch DrSensor-feature/login_shell
git rebase master
git switch master
git merge --no-ff DrSensor-feature/login_shell
git switch master
git merge --squash DrSensor-feature/login_shell
git switch master
git merge --ff-only DrSensor-feature/login_shell
git switch master
git merge DrSensor-feature/login_shell
git push origin master
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
bug

This issue or pull request discusses a bug

  
clue

This issue or pull request contains valuable information

  
docs

This issue or pull request discusses documentation

  
feature

This issue or pull request discusses a feature

  
ko

This issue is not an issue

  
needs review

This issue or pull request must be confirmed/reviewed

  
ok

This issue or pull request was confirmed/reviewed

  
wip

This issue or pull request is a draft

  
wont

This issue or pull request won't be fixed/merged

No labels
bug
clue
docs
feature
ko
needs review
ok
wip
wont
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
fairyglade/ly!897
No description provided.