X2Go Bug report logs - #116
Only install what is needed

Full log

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

Received: (at control) by bugs.x2go.org; 19 Jul 2013 16:38:34 +0000
From mike.gabriel@das-netzwerkteam.de  Fri Jul 19 18:38:25 2013
X-Spam-Checker-Version: SpamAssassin 3.3.2 (2011-06-06) on
	ymir.das-netzwerkteam.de
X-Spam-Level: 
X-Spam-Status: No, score=0.0 required=5.0 tests=RCVD_IN_DNSWL_BLOCKED,
	URIBL_BLOCKED autolearn=unavailable version=3.3.2
Received: from freya.das-netzwerkteam.de (freya.das-netzwerkteam.de [88.198.48.199])
	by ymir (Postfix) with ESMTPS id 7A4655DB0A;
	Fri, 19 Jul 2013 18:38:25 +0200 (CEST)
Received: from grimnir.das-netzwerkteam.de (grimnir.das-netzwerkteam.de [78.46.204.98])
	by freya.das-netzwerkteam.de (Postfix) with ESMTPS id 53575BDB;
	Fri, 19 Jul 2013 18:38:25 +0200 (CEST)
Received: from localhost (localhost [127.0.0.1])
	by grimnir.das-netzwerkteam.de (Postfix) with ESMTP id 47EF53BBCC;
	Fri, 19 Jul 2013 18:38:25 +0200 (CEST)
X-Virus-Scanned: Debian amavisd-new at grimnir.das-netzwerkteam.de
Received: from grimnir.das-netzwerkteam.de ([127.0.0.1])
	by localhost (grimnir.das-netzwerkteam.de [127.0.0.1]) (amavisd-new, port 10024)
	with ESMTP id a+8ZhV7h0MxL; Fri, 19 Jul 2013 18:38:25 +0200 (CEST)
Received: from localhost (localhost [127.0.0.1])
	by grimnir.das-netzwerkteam.de (Postfix) with ESMTP id 202353BC02;
	Fri, 19 Jul 2013 18:38:25 +0200 (CEST)
Received: from localhost (localhost [127.0.0.1])
	by grimnir.das-netzwerkteam.de (Postfix) with ESMTP id 003693BBCC;
	Fri, 19 Jul 2013 18:38:24 +0200 (CEST)
Received: by grimnir.das-netzwerkteam.de (Postfix, from userid 33)
	id 772A83BBFD; Fri, 19 Jul 2013 18:38:24 +0200 (CEST)
Received: from m-047.informatik.uni-kiel.de (m-047.informatik.uni-kiel.de
 [134.245.254.47]) by mail.das-netzwerkteam.de (Horde Framework) with HTTP;
 Fri, 19 Jul 2013 18:38:24 +0200
Message-ID: <20130719183824.161539jaxcockmm8@mail.das-netzwerkteam.de>
X-Priority: 3 (Normal)
Date: Fri, 19 Jul 2013 18:38:24 +0200
From: Mike Gabriel <mike.gabriel@das-netzwerkteam.de>
To: Orion Poplawski <orion@cora.nwra.com>
Cc: 116@bugs.x2go.org, control@bugs.x2go.org
Subject: Re: [X2Go-Dev] Bug#238: Bundled libraries
References: <51BA8380.9070000@cora.nwra.com>
 <20130614121339.211254jthtsexcr7@mail.das-netzwerkteam.de>
 <51DDC024.9020204@cora.nwra.com>
In-Reply-To: <51DDC024.9020204@cora.nwra.com>
MIME-Version: 1.0
Content-Type: multipart/signed;
 boundary="=_36svn1o5n7nk";
 protocol="application/pgp-signature";
 micalg="pgp-sha1"
Content-Transfer-Encoding: 7bit
User-Agent: Internet Messaging Program (IMP) H3 (4.3.4)

[Message part 1 (text/plain, inline)]

tag #116 patch
thanks

Hi Orion,

On Mi 10 Jul 2013 22:12:20 CEST Orion Poplawski wrote:

> On 06/14/2013 04:13 AM, Mike Gabriel wrote:
>> Hi Orion,
>>
>> On Fr 14 Jun 2013 04:44:16 CEST Orion Poplawski wrote:
>>
>>> Package: nx-libs
>>>
>>> nx-libs has copies of a number of libraries in nx-X11/extras/ (and  
>>> some glue
>>> in nx-X11/libs).  We should get rid of these and use system  
>>> provided versions.
>>>
>>> As a first stab, I'm able to remove the following without any harm:
>>>
>>> rm -r nx-X11/extras/{drm,fonts,ogl-sample,regex,ttf2pt1,x86emu,zlib}
>>> rm -r nx-X11/lib/{regex,zlib}
>>>
>>> expat appears to be essentially the 1.95.6 version.  And can be disabled by
>>> setting HasExpat YES in nx-X11/config/cf/host.def.
>>>
>>> fontconfig is an unmodified fontconfig 2.3.2 and can be disabled with
>>> HasFontconfig YES.
>>>
>>> rman is not needed and can be disabled with BuildRman NO.
>>>
>>> That's a far as I've gotten today.
>>
>> Thanks for having started working on this. You earlier reported this request
>> via BTS (#116) and getting a fix for this issue rolling is pretty  
>> valuable, I
>> guess.
>>
>> We won't be able to remove the source folders you referred to from our
>> nx-libs.git repos as the git repos also documents how NoMachine releases get
>> merged into NX (redistributed). However, we should get rid of those  
>> folders at
>> build-time (.deb packages) and get rid of those folders from the tarballs
>> (.rpm et al. packages).
>>
>> So, if you work on this (I know that you work on .rpm based systems), please
>> edit nx-libs.git/debian/roll-tarball.sh for us whenever you want to remove
>> files from the tarball. If you need some other build options, provide that
>> change as a patch for our patch system. Just send the patch, I will  
>> transform
>> that into an appropriate quilt patch.
>>
>> For the .deb build process, we have to move folders out of the way during
>> build-time via /debian/rules. I can see to that, once you have a  
>> diff for the
>> roll-tarball.sh script.
>>
>> Greets+Thanks!
>> Mike
>
> Okay, here goes my initial changes.
>
> I'd still like to unbundle Xpm if possible, but it does get linked  
> to libXcomp.  And Mesa may be just too integrated.

I am currently testing your patches and once the .deb packages build  
with the patch 0002_ applied, I will push the patch to nx-libs.git.

Thanks for this piece of digging code work!

Mike



-- 

DAS-NETZWERKTEAM
mike gabriel, herweg 7, 24357 fleckeby
fon: +49 (1520) 1976 148

GnuPG Key ID 0x25771B31
mail: mike.gabriel@das-netzwerkteam.de, http://das-netzwerkteam.de

freeBusy:
https://mail.das-netzwerkteam.de/freebusy/m.gabriel%40das-netzwerkteam.de.xfb

[Message part 2 (application/pgp-signature, inline)]

Send a report that this bug log contains spam.