X2Go Bug report logs - #812
x2gogetapps should ignore NoDisplay=true desktop application entries

version graph

Package: x2goserver; Maintainer for x2goserver is X2Go Developers <x2go-dev@lists.x2go.org>; Source for x2goserver is src:x2goserver.

Reported by: Jason Alavaliant <alavaliant@ra09.com>

Date: Tue, 10 Mar 2015 08:35:01 UTC

Severity: normal

Tags: patch, pending

Found in version

Fixed in versions,

Done: X2Go Release Manager <git-admin@x2go.org>

Bug is archived. No further changes may be made.

Full log

Message #10 received at 812@bugs.x2go.org (full text, mbox, reply):

Received: (at 812) by bugs.x2go.org; 11 Mar 2015 20:14:36 +0000
From ionic@ionic.de  Wed Mar 11 21:14:34 2015
X-Spam-Checker-Version: SpamAssassin 3.3.2 (2011-06-06) on
X-Spam-Status: No, score=0.8 required=5.0 tests=BAYES_50,T_DKIM_INVALID,
	URIBL_BLOCKED autolearn=ham version=3.3.2
Received: from Root24.de (powered.by.root24.eu [])
	by ymir.das-netzwerkteam.de (Postfix) with ESMTP id CC4BD3BE7C
	for <812@bugs.x2go.org>; Wed, 11 Mar 2015 21:14:34 +0100 (CET)
Received: from nopileos.local (home.ionic.de [])
	by mail.ionic.de (Postfix) with ESMTPSA id AF1DF4F088B5;
	Wed, 11 Mar 2015 21:14:32 +0100 (CET)
DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=ionic.de; s=default;
	t=1426104874; bh=13m/v+mehFIik2hrnh40brYfEydpoVqJcjn5rLvVSCI=;
Message-ID: <5500A221.7090301@ionic.de>
Date: Wed, 11 Mar 2015 21:14:25 +0100
From: Mihai Moldovan <ionic@ionic.de>
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.9; rv:31.0) Gecko/20100101 Thunderbird/31.5.0
MIME-Version: 1.0
To: Jason Alavaliant <alavaliant@ra09.com>, 812@bugs.x2go.org
Subject: Re: [X2Go-Dev] Bug#812: x2gogetapps should ignore NoDisplay=true
 desktop application entries
References: <54FEABB7.5090705@ra09.com>
In-Reply-To: <54FEABB7.5090705@ra09.com>
Content-Type: multipart/signed; micalg=pgp-sha512;
[Message part 1 (text/plain, inline)]
Hi Jason,

On 10.03.2015 09:30 AM, Jason Alavaliant wrote:
> Currently the x2gogetapps command prints out all .desktop application
> files irrespective of if they have the value NoDisplay=true set in the
> file or not.
> [...]
> The attached patch updates the x2gogetapps command to check the file
> to see if NoDisplay=true is set and if so doesn't print out the
> contents of that file removing all the duplicates and other entries
> not intended to be listed in a normal application launch menu.

Thank you for the patch!

I've reviewed your patch and re-implemented it.

First, there's also a "Hidden" key which indicates that the application
shall be regarded as deleted (and not show up either.) We didn't respect
that either, previously.

Further, there's no need to read the whole file once we stumble upon
Hidden or NoDisplay. We can stop processing there. The same goes when
finding a non-Desktop Entry group after a Desktop Entry group.

The new patch can be seen here:


or downloaded here:


Can you please test if my patch fixes your problem, too? I'd rather
commit this more complete patch to the main branches.


[signature.asc (application/pgp-signature, attachment)]

Send a report that this bug log contains spam.

X2Go Developers <owner@bugs.x2go.org>. Last modified: Tue Jan 19 19:18:33 2021; Machine Name: ymir.das-netzwerkteam.de

X2Go Bug tracking system

Debbugs is free software and licensed under the terms of the GNU Public License version 2. The current version can be obtained from https://bugs.debian.org/debbugs-source/.

Copyright © 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson, 2005-2017 Don Armstrong, and many other contributors.