After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 786404 - [PATCH] application: update application ID to match .desktop file
[PATCH] application: update application ID to match .desktop file
Status: RESOLVED FIXED
Product: gnome-packagekit
Classification: Core
Component: general
git master
Other Linux
: Normal normal
: ---
Assigned To: gnome-packagekit-maint
gnome-packagekit-maint
Depends on:
Blocks:
 
 
Reported: 2017-08-17 08:58 UTC by Xiang Fan
Modified: 2017-08-27 02:47 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
0001-application-update-application-ID-to-match-.desktop-.patch (1.52 KB, patch)
2017-08-17 08:58 UTC, Xiang Fan
committed Details | Review

Description Xiang Fan 2017-08-17 08:58:06 UTC
Created attachment 357774 [details] [review]
0001-application-update-application-ID-to-match-.desktop-.patch

Fixes missing window name and icon in Wayland.
Comment 1 Richard Hughes 2017-08-17 15:31:01 UTC
Comment on attachment 357774 [details] [review]
0001-application-update-application-ID-to-match-.desktop-.patch

LGTM, thanks.
Comment 2 Jeremy Bicha 2017-08-26 20:36:38 UTC
Xiang Fan, do you have commit rights to git.gnome.org or do you need someone to push this for you?
Comment 3 Xiang Fan 2017-08-27 02:30:46 UTC
(In reply to Jeremy Bicha from comment #2)
> Xiang Fan, do you have commit rights to git.gnome.org or do you need someone
> to push this for you?

Thank you for commenting.

Yes, I need someone to push this. I thought it was because of the freezes or something, but I don't really understand how this works.

Can you please push this patch (and another one with accepted-commit_now https://bugzilla.gnome.org/show_bug.cgi?id=786189 )?
Comment 4 Jeremy Bicha 2017-08-27 02:37:47 UTC
Sure, no problem. Pushing for you now.

The way this works is the maintainer marked your patch as accepted-commit_now.

But if you're new to GNOME and don't have git commit rights, you'll need to ping someone that does to commit the patch for you.

I cam across this bug since it also affected me and didn't recognize your name so I thought I'd ask if you needed someone to commit it. :)

When you're ready, this page describes applying for git commit rights:
https://wiki.gnome.org/AccountsTeam/NewAccounts
Comment 5 Xiang Fan 2017-08-27 02:47:55 UTC
(In reply to Jeremy Bicha from comment #4)
> Sure, no problem. Pushing for you now.
> 
> The way this works is the maintainer marked your patch as
> accepted-commit_now.
> 
> But if you're new to GNOME and don't have git commit rights, you'll need to
> ping someone that does to commit the patch for you.
> 
> I cam across this bug since it also affected me and didn't recognize your
> name so I thought I'd ask if you needed someone to commit it. :)
> 
> When you're ready, this page describes applying for git commit rights:
> https://wiki.gnome.org/AccountsTeam/NewAccounts

Thank you :)