[gems.appimage] improvement: removing the TryExec field from the installed desktop entries to avoid issues with some DEs

This commit is contained in:
Vinicius Moreira
2022-10-24 08:13:14 -03:00
parent 828be31d74
commit 58aee20e21
2 changed files with 12 additions and 1 deletions

View File

@@ -4,6 +4,12 @@ All notable changes to this project will be documented in this file.
The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/).
## [STAGING]
### Fixes
- AppImage
- some desktop entries not being displayed on the desktop environment menu (requires the AppImage to be reinstalled) [#287](https://github.com/vinifmor/bauh/issues/287)
## [0.10.3] 2022-05-30
### Features

View File

@@ -2,7 +2,7 @@ import os
import re
from typing import Optional
RE_DESKTOP_EXEC = re.compile(r'(Exec\s*=(.+)(\n?))')
RE_DESKTOP_EXEC = re.compile(r'(\w*Exec\s*=(.+)(\n?))')
RE_MANY_SPACES = re.compile(r'\s+')
@@ -24,6 +24,11 @@ def replace_desktop_entry_exec_command(desktop_entry: str, appname: str, file_pa
for exec_groups in execs:
full_match = exec_groups[0]
if full_match.startswith("TryExec"): # TryExec cause issues in some DE to display the app icon
final_entry = final_entry.replace(full_match, "")
continue
cmd = RE_MANY_SPACES.sub(' ', exec_groups[1].strip())
if cmd:
words = cmd.split(' ')