From mike.gabriel@das-netzwerkteam.de  Mon Feb  9 14:07:54 2015
Received: (at 783) by bugs.x2go.org; 9 Feb 2015 13:07:56 +0000
X-Spam-Checker-Version: SpamAssassin 3.3.2 (2011-06-06) on
	ymir.das-netzwerkteam.de
X-Spam-Level: 
X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,URIBL_BLOCKED
	autolearn=ham version=3.3.2
Received: from freya.das-netzwerkteam.de (freya.das-netzwerkteam.de [88.198.48.199])
	by ymir.das-netzwerkteam.de (Postfix) with ESMTPS id 8CEC35DBCA
	for <783@bugs.x2go.org>; Mon,  9 Feb 2015 14:07:54 +0100 (CET)
Received: from grimnir.das-netzwerkteam.de (grimnir.das-netzwerkteam.de [78.46.204.98])
	by freya.das-netzwerkteam.de (Postfix) with ESMTPS id 44F9BA3E;
	Mon,  9 Feb 2015 14:07:54 +0100 (CET)
Received: from localhost (localhost [127.0.0.1])
	by grimnir.das-netzwerkteam.de (Postfix) with ESMTP id 35C6C3C278;
	Mon,  9 Feb 2015 14:07:54 +0100 (CET)
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 9pQpJtamVJOt; Mon,  9 Feb 2015 14:07:54 +0100 (CET)
Received: from grimnir.das-netzwerkteam.de (localhost [127.0.0.1])
	by grimnir.das-netzwerkteam.de (Postfix) with ESMTPS id 0765B3C071;
	Mon,  9 Feb 2015 14:07:54 +0100 (CET)
Received: from bifrost.das-netzwerkteam.de (bifrost.das-netzwerkteam.de
 [178.62.101.154]) by mail.das-netzwerkteam.de (Horde Framework) with HTTP;
 Mon, 09 Feb 2015 13:07:53 +0000
Date: Mon, 09 Feb 2015 13:07:53 +0000
Message-ID: <20150209130753.Horde.L19M7fVjVKN39pdqOKW9-w1@mail.das-netzwerkteam.de>
From: Mike Gabriel <mike.gabriel@das-netzwerkteam.de>
To: Schlomo Schapiro <schlomo.schapiro@immobilienscout24.de>,
 783@bugs.x2go.org
Subject: Re: [X2Go-Dev] Bug#783: RPM Pre Script fail on Upgrade (Nightly)
In-Reply-To: <54D89EBC.9050602@immobilienscout24.de>
User-Agent: Internet Messaging Program (IMP) H5 (6.2.2)
Accept-Language: de,en
Organization: DAS-NETZWERKTEAM
X-Originating-IP: 178.62.101.154
X-Remote-Browser: Mozilla/5.0 (X11; Linux x86_64; rv:32.0) Gecko/20100101
 Firefox/32.0 Iceweasel/32.0
Content-Type: multipart/signed; boundary="=_P9VEr0CFoaheyPZ2puFxUw2";
 protocol="application/pgp-signature"; micalg=pgp-sha1
MIME-Version: 1.0

This message is in MIME format and has been PGP signed.

--=_P9VEr0CFoaheyPZ2puFxUw2
Content-Type: text/plain; charset=utf-8; format=flowed; DelSp=Yes
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

Hi Schlomo.

On  Mo 09 Feb 2015 12:49:16 CET, Schlomo Schapiro wrote:

> Package: x2goserver
> Version: 4.1.0.0-0.0x2go1.0.git20150201.1236.heuler.el6
>
> RPM Upgrades of existing installations fail because of a broken %pre
> script in the x2goserver-common RPM:
>
> Resolving Dependencies
> --> Running transaction check
> ---> Package x2goserver-common.x86_64
> 0:4.1.0.0-0.0x2go1.0.git20150113.1214.heuler.el6 will be updated
> ---> Package x2goserver-common.x86_64
> 0:4.1.0.0-0.0x2go1.0.git20150201.1236.heuler.el6 will be an update
> --> Finished Dependency Resolution
>
> Dependencies Resolved
>
> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
>  Package                      Arch
> Version
> Repository                    Size
> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
> Updating:
>  x2goserver-common            x86_64
> 4.1.0.0-0.0x2go1.0.git20150201.1236.heuler.el6
> is24-x2go-nightly            7.2 k
>
> Transaction Summary
> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
> Upgrade       1 Package(s)
>
> Total download size: 7.2 k
> Is this ok [y/N]: y
> Downloading Packages:
> x2goserver-common-4.1.0.0-0.0x2go1.0.git20150201.1236.heuler.el6.x86_64.r=
pm
> | 7.2 kB     00:00
> Running rpm_check_debug
> Running Transaction Test
> Transaction Test Succeeded
> Running Transaction
> Error in PREIN scriptlet in rpm package
> x2goserver-common-4.1.0.0-0.0x2go1.0.git20150201.1236.heuler.el6.x86_64
> groupadd: group 'x2gouser' already exists
> useradd: user 'x2gouser' already exists
> error:
> %pre(x2goserver-common-4.1.0.0-0.0x2go1.0.git20150201.1236.heuler.el6.x86=
_64)
> scriptlet failed, exit status 9
> error:   install: %pre scriptlet failed (2), skipping
> x2goserver-common-4.1.0.0-0.0x2go1.0.git20150201.1236.heuler.el6
> Running post transaction command: report-yum-to-graphite.sh
> x2goserver-common-4.1.0.0-0.0x2go1.0.git20150113.1214.heuler.el6.x86_64
> was supposed to be removed but is not!
>   Verifying  :
> x2goserver-common-4.1.0.0-0.0x2go1.0.git20150113.1214.heuler.el6.x86_64
> 1/2
>   Verifying  :
> x2goserver-common-4.1.0.0-0.0x2go1.0.git20150201.1236.heuler.el6.x86_64
> 2/2
>
> Failed:
>   x2goserver-common.x86_64
> 0:4.1.0.0-0.0x2go1.0.git20150113.1214.heuler.el6
>
>   x2goserver-common.x86_64
> 0:4.1.0.0-0.0x2go1.0.git20150201.1236.heuler.el6
>
>
> Complete!
>
> The bad script is this:
>
> if getent group x2gouser 1>/dev/null; then
>     groupadd -r x2gouser
> fi
> if getent passwd x2gouser >/dev/null; then
>     useradd -r -g x2gouser -d /var/lib/x2go -s /sbin/nologin \
>             -c "x2go" x2gouser
> fi
>
> It should have a ! in the if clause to only call groupadd / useradd=20=20
>=20if the group or user DO NOT EXIST!!

Yep, should already have been fixed for several days by [1].

> Don't you test upgrades???

Nope, not always.

/me wonders, if your nightly builds are not obtained from=20=20
packages.x2go.org=20but from a non-up-to-date repo. Possibly we may have=20=
=20
a=20problem in the EPEL-6 nightly builds archive.

Mike


[1]=20=20
http://code.x2go.org/gitweb?p=3Dx2goserver.git;a=3Dcommitdiff;h=3D0838f4686=
fc0dd15c66840fecd665e38a7ea1bca
--=20

DAS-NETZWERKTEAM
mike=20gabriel, 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.x=
fb

--=_P9VEr0CFoaheyPZ2puFxUw2
Content-Type: application/pgp-signature
Content-Description: Digitale PGP-Signatur
Content-Disposition: inline

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1

iQIbBAABAgAGBQJU2LEpAAoJEJr0azAldxsxcA0P91KXjFHIbGwAKVIdQyKx3FSE
FVoP7e9Y2jZNAMJtOwM/rYKdM4K2zfUVLfR54qOu5mep19OIic8GCKugGdP9KT3R
hlsJ7dmI03UulfsY/Bk6QgAWKGraQYva8OkD6Fl9GXMqexlhtvAkJ/72xQS+mm3G
AB46DXPAIkHQwuqG1aFO4jru1DmncQYh7PqLY3esU3KWtUsS9DELV6gvDgatk2it
mnGcidQ1gW1HF3clj0aK0cyXAIW2XQQkBCD9GchMYSAkM5rBKWf278cZf4Dg5aJ0
ZZdinu6YmTiCyX2tXC9lqpVpaMQCJW8fh6/rmSmgnESykl2AOU9kGA2/kG44h20c
sdrKMNP/llc+dDQWOuEtYQYYjngFcl23I20/MBzchLDo0A6gKi8iBmLaZ9gUFchb
JByV2c0a0w+gNZPwxYGlKYfyA3TsvagbXVibMe5ZyheGUdD+MLIsaGjfTiYmxjkS
YQNEbhic2cAbSGcw1CFGr0/9keJZZAC0p0UHILl4lJJqMCzxxGKZ61+O3Vvasgzn
ct/cVJ20KF/tirvrVl7rMAM2hs9kpwFG0fVwz62n6JBMXCb4yerVFqXP/VX4qafb
+FMZru+sD8nLcTNSRJUi8m4qCFTYWVfovPMRTjvcJ7JlxFCNeGWtVDA86vxkKes/
GrHGx3WcGwHoVb+vYOM=
=bCW3
-----END PGP SIGNATURE-----

--=_P9VEr0CFoaheyPZ2puFxUw2--

