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

Support for shell buffers (needs workaround or, preferably, upstream support) #22

Open
jacquesdurden opened this issue Nov 27, 2020 · 9 comments
Assignees
Labels
enhancement New feature or request help wanted Extra attention is needed
Milestone

Comments

@jacquesdurden
Copy link

Burly is great! Thank you for this super handy tool.

I have only a little problem, or maybe it's actually a new feature request.

I'd like to save a layout of say 3 windows in my frame with one of the window open on a shell (using just shell). My wish would be that by invoking the Burly bookmark of this layout, the 3 windows would reappear with my shell reopened exactly as before.

For now, while Burly is perfectly fine for standard windows open on files, I got this error message if one of them contains a shell Wrong type argument: stringp, nil

Thank you.

@alphapapa alphapapa self-assigned this Nov 28, 2020
@alphapapa alphapapa added bug Something isn't working enhancement New feature or request labels Nov 28, 2020
@alphapapa alphapapa added this to the 0.1.1 milestone Nov 28, 2020
@alphapapa
Copy link
Owner

Hi Jacques,

Thanks, I'm glad you like it.

Supporting the various kinds of shell/terminal buffers would best be handled by giving each of those buffers' major modes a bookmark-make-record-function so their buffers could be bookmarked by Emacs. Ideally those functions would be merged into Emacs upstream so that everyone would benefit from them.

Other than that, we could add some mode-specific workarounds in this package, but we'd have to write essentially the same code, so it might as well be upstreamed.

I rarely keep shells open in Emacs, so I probably wouldn't use this feature very much right now. Would you be able/willing to work on that?

For now, while Burly is perfectly fine for standard windows open on files, I got this error message if one of them contains a shell Wrong type argument: stringp, nil

That part is a bug, so thanks for reporting that.

@alphapapa alphapapa added the help wanted Extra attention is needed label Nov 28, 2020
@captain-yoshi
Copy link

This module is great (elegant, simple and effective). Like @jacquesdurden I would like to store eshell with Burly.

I have come across this post that led me to this eshell bookmark github repo. I didn't have the chance to test this, but this could be a good starting point and/or solution.

@jacquesdurden
Copy link
Author

Thank you for this solution @captain-yoshi! I tried it and it works! (just be sure though to well install eshell-bookmark with its activation command in your init.el, see its github page) This could be a good info for the Burly team too. I just have really minor positioning issues in the window containing eshell at startup but I can live with this. Thank you again.

@captain-yoshi

This comment has been minimized.

@jacquesdurden

This comment has been minimized.

@captain-yoshi

This comment has been minimized.

@alphapapa
Copy link
Owner

@captain-yoshi Thanks for sharing the links. I added @Fuco1's article and package to the notes file. Ideally the eshell-bookmark package would be upstreamed into Emacs. Since I don't much use eshell (yet?), I'll leave that to you to pursue.

Regarding that error: it means what it says: the function is void. That means you haven't loaded it into your Emacs.

Since Doom and generic Emacs issues are off-topic here, please use an appropriate forum to seek help with them.

Thanks.

@Fuco1
Copy link

Fuco1 commented Dec 19, 2020

I agree it would be nice to push upstream, I think someone even tried to do it already. Unfortunately I'm quite unable to tend to my packages these days, but it's certainly open for anyone (I have signed the fsf papers).

@alphapapa
Copy link
Owner

AFAIK there is no longer a bug here with regard to Burly.

The enhancement to shell in Emacs itself, to add bookmark support, is still needed. Maybe someone will get to that someday...

@alphapapa alphapapa changed the title Burly with window(s) containing shell Support for shell buffers (needs workaround or, preferably, upstream support) Aug 31, 2023
@alphapapa alphapapa removed the bug Something isn't working label Aug 31, 2023
@alphapapa alphapapa modified the milestones: 0.1.1, Future Aug 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

4 participants