Code review comment for lp:~elopio/ubuntu-ui-toolkit/fake_app_fixture

Revision history for this message
Leo Arias (elopio) wrote :

> Would it make sense to verify the fields in the .desktop file? The shell isn't
> typically very talkative in case something's wrong.

kalikiana, I had reported a bug about that. I see now that it's a duplicate of:
https://bugs.launchpad.net/ubuntu/+source/qtubuntu/+bug/1204595
And it seems to have some work. I'll ask tomorrow.

It wouldn't hurt to add an extra check here. What are the fields that we should require?

If that's ok to you, I'll do it in a separate branch.

« Back to merge proposal