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

app-launcher-list-apps: Symbol’s value as variable is void: counsel-linux-apps-faulty #6

Open
elizagamedev opened this issue Mar 15, 2022 · 2 comments

Comments

@elizagamedev
Copy link

I had some faulty desktop files (i.e. apps with an empty Name field) and this error occurred. This seems to be a leftover from the counsel package.

@Vivien-lelouette
Copy link

I have the exact same error. I am not yet an expert on Desktop files debuging.
Do you have a way to find out which Desktop file causes the issue ?

I am going to debug this on the Emacs side in the near future, but if there is a more efficient way I'll be happy to know it :)

Note: I do think this should be excluded from the result instead of throwing an error. If I have time in the following weeks (highly unlikely), I might try to fork and code this behaviour.

@Vivien-lelouette
Copy link

Vivien-lelouette commented Apr 9, 2022

Turns out you just need to remove the push to that variable.
The entry is already skipped, only this push causes the crash.

I made a fork available here: https://github.com/Vivien-lelouette/app-launcher.
There is only this line removal in the HEAD.
@SebastienWae, should I make a merge request ?

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

No branches or pull requests

2 participants