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 /usr/local/share/applications to the list of desktop files paths. #5

Merged
merged 1 commit into from Aug 3, 2015
Merged

Add /usr/local/share/applications to the list of desktop files paths. #5

merged 1 commit into from Aug 3, 2015

Conversation

elboulangero
Copy link
Contributor

Hi,

this path is quite standard, at least on Debian, for everything that was not installed using the package manager.

Just out of curiosity, I see that there is a config file for obmenu-generator in ~/.config, but it's generated and modifications are lost. So, the only way to change configuration is to modify obmenu-generator itself, isn't it ? Or is there another way ?

trizen pushed a commit that referenced this pull request Aug 3, 2015
Add /usr/local/share/applications to the list of desktop files paths.
@trizen trizen merged commit e4f7be1 into trizen:master Aug 3, 2015
@trizen
Copy link
Owner

trizen commented Aug 3, 2015

Hi,

Thanks for the patch.

About the config file, once generated, it will not be generated again, just updated periodically. This means, that any new value added later to the config file will not actually be lost (except for comments and indentations).

@elboulangero
Copy link
Contributor Author

OK, thanks for the explanation.

I just spotted another little problem (completely different, but no need to create a ticket for such a small thing). In schema.pl, obmenu-generator is refered as $0. The problem I ran into is that I just regenerated a menu using ./obmenu-generator. Now, if I try again to re-generate a menu, but this time by right-clicking on the menu, it fails, because it can't find ./obmenu-generator (of course it can't).

So I guess maybe the basename should be used, instead of $0. Except for absolute paths maybe ?

@trizen
Copy link
Owner

trizen commented Aug 3, 2015

Fixed. Thanks

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.

None yet

2 participants