← Back to team overview

ubuntu-touch-coreapps-reviewers team mailing list archive

Re: [Merge] lp:~laney/ubuntu-terminal-app/notshowin-unity into lp:ubuntu-terminal-app

 

On Tue, Sep 27, 2016 at 11:08:32AM -0000, Alan Pope 🍺🐧🐱 wrote:
> Interestingly this fails click-review, if built as a click (which we do for the phone, although armhf, not amd64 as in the example below)...
> 
> Now executing: click-review ./com.ubuntu.terminal_0.7.217_amd64.click
> Errors
> ------
>  - desktop:validates:terminal
> 	did not validate: (ValidationError in file '/tmp/review-pp96fg9l/com.ubuntu.terminal.desktop': 
> - 'Unity7' is not a registered OnlyShowIn value )
> 	http://askubuntu.com/questions/417377/what-does-desktop-validates-mean/417378

Sigh, desktop-file-validate has a whitelist of allowed names.

-- 
Iain Lane                                  [ iain@xxxxxxxxxxxxxxxxxxx ]
Debian Developer                                   [ laney@xxxxxxxxxx ]
Ubuntu Developer                                   [ laney@xxxxxxxxxx ]

https://code.launchpad.net/~laney/ubuntu-terminal-app/notshowin-unity/+merge/306855
Your team Ubuntu Terminal Developers is requested to review the proposed merge of lp:~laney/ubuntu-terminal-app/notshowin-unity into lp:ubuntu-terminal-app.


Follow ups

References