Skip to content

Comments

update d/rules (workaround issue in desktop file)#563

Merged
corrados merged 1 commit intojamulussoftware:masterfrom
trebmuh:patch-1
Sep 1, 2020
Merged

update d/rules (workaround issue in desktop file)#563
corrados merged 1 commit intojamulussoftware:masterfrom
trebmuh:patch-1

Conversation

@trebmuh
Copy link
Member

@trebmuh trebmuh commented Sep 1, 2020

The exec field is replaced with jamulus-headless which isn't what's expected by the user to be executed.

Pinging @tormodvolden since you've been working on the debian/ files recently.

@corrados : maybe it could be a good idea to generate a desktop file for jamulus-GUI ("jamulus.desktop") and another desktop file for jamulus-headless ("jamulus-headless.desktop").

@corrados
Copy link
Contributor

corrados commented Sep 1, 2020

@corrados : maybe it could be a good idea to generate a desktop file for jamulus-GUI ("jamulus.desktop") and another desktop file for jamulus-headless ("jamulus-headless.desktop").

Yes, good idea.

@corrados corrados merged commit f1c2b4b into jamulussoftware:master Sep 1, 2020
@trebmuh trebmuh deleted the patch-1 branch September 1, 2020 16:49
@tormodvolden
Copy link
Contributor

Thanks! I hadn't noticed that the desktop file was autogenerated and didn't check it. This probably explains some Lintian warnings, I thought it was only confused...

A desktop file for jamulus-headless is IMO pointless.

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.

3 participants