-
Notifications
You must be signed in to change notification settings - Fork 2k
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
net-im/telegram-desktop-bin: fix Icon in .desktop file #10212
Conversation
Copyright policy changePlease note that on 2018-09-15 Trustees have approved new Gentoo copyright policy. All contributions made to Gentoo need to follow this policy. If you include the Signed-off-by line in your commit message, you indicate that you have read the policy and agree to its terms. For more detailed explanation, please see the new Gentoo copyright policy explained article. Pull Request assignmentAreas affected: ebuilds net-im/telegram-desktop-bin: @henning-schild, @NP-Hardass, @gentoo/proxy-maint Linked bugsBugs linked: 667784 In order to force reassignment and/or bug reference scan, please append Docs: Code of Conduct ● Copyright policy (expl.) ● Devmanual ● GitHub PRs ● Proxy-maint guide |
S="${WORKDIR}/Telegram" | ||
|
||
src_prepare() { | ||
sed -i -e 's/^Icon=telegram$/Icon=telegram-desktop/' "${WORKDIR}/tdesktop-${PV}"/lib/xdg/telegramdesktop.desktop |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
|| die
is missing here
9299d76
to
81a1c37
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also feel free to remove -r0 in the same commit since it's a minor change.
S="${WORKDIR}/Telegram" | ||
|
||
src_prepare() { | ||
sed -i -e 's/^Icon=telegram$/Icon=telegram-desktop/' "${WORKDIR}/tdesktop-${PV}"/lib/xdg/telegramdesktop.desktop || die |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I guess I'm being weird and all but why are you altering upstream .desktop file instead of fixing the install phase below to install the icon with correct name?
0dda3d9
to
42ca59e
Compare
Closes: https://bugs.gentoo.org/667784 Signed-off-by: Henning Schild <henning@hennsch.de>
42ca59e
to
7a72255
Compare
Pull request CI reportReport generated at: 2018-10-26 12:56 UTC No issues found |
Thanks. |
Mhh, looks like you merged the original PR and not the updated one |
Thanks for noticing. I'll fix that. |
Looks like we need a -r2 now, for people that synched inbetween. But let us hope noone did ;) |
Closes: https://bugs.gentoo.org/667784
Signed-off-by: Henning Schild henning@hennsch.de