[SRM] xorg 1:7.3+20 for lenny
Hi SRMs,
I broke the xserver-xorg package's postinst script in the lenny r2
update. Maximiliano was kind enough to fix my mistake and test the
updated package, which is sitting in p-u-new. Please accept for 5.0.3
(sooner is probably better, so people with b0rked installs can get the
fix from p-u in the meantime). The diff from 1:7.3+19 follows.
TIA,
Julien
On Thu, Jul 30, 2009 at 20:20:32 +0000, Julien Cristau wrote:
> debian/changelog | 7 +++++++
> debian/xserver-xorg.postinst.in | 5 ++++-
> 2 files changed, 11 insertions(+), 1 deletion(-)
>
> New commits:
> commit 89b8ea3587eafe48c80740a8912d390702e6b02d
> Author: Maximiliano Curia <maxy@debian.org>
> Date: Wed Jul 22 16:26:29 2009 -0300
>
> Fix stupid bug in xserver-xorg.postinst
>
> Shell scripting fail.
> ``||'' and ``&&'' have the same precedence in shell, so we ended up with
> an empty xorg.conf after installing 1:7.3+19.
> Debian bug#535624.
>
> Signed-off-by: Julien Cristau <jcristau@debian.org>
>
> diff --git a/debian/changelog b/debian/changelog
> index 18477f8..c44450e 100644
> --- a/debian/changelog
> +++ b/debian/changelog
> @@ -1,3 +1,10 @@
> +xorg (1:7.3+20) stable; urgency=low
> +
> + * Non-maintainer upload, supervised by Julien Cristau.
> + * xserver-xorg.postinst: fix for the previous patch. Closes: #535624
> +
> + -- Maximiliano Curia <maxy@debian.org> Wed, 22 Jul 2009 16:26:29 -0300
> +
> xorg (1:7.3+19) stable; urgency=low
>
> * xserver-xorg.postinst: default to the fbdev driver on sparc, even when we
> diff --git a/debian/xserver-xorg.postinst.in b/debian/xserver-xorg.postinst.in
> index 5beb891..8aef64a 100644
> --- a/debian/xserver-xorg.postinst.in
> +++ b/debian/xserver-xorg.postinst.in
> @@ -932,7 +932,10 @@ fi
> # Don't touch the config on upgrades except to deal with known issues with old
> # configs.
> if [ -z "$UPGRADE" ] || dpkg --compare-versions "$2" le "1:7.0.14" || \
> - [ "$ARCHITECTURE" = sparc ] && dpkg --compare-versions "$2" lt-nl 1:7.3+19; then
> + { [ "$ARCHITECTURE" = sparc ] && \
> + dpkg --compare-versions "$2" lt-nl 1:7.3+19; } || \
> + { [ -n "$(find "$XORGCONFIG" -maxdepth 0 -size 0 2>/dev/null)" ] && \
> + [ "$2" = 1:7.3+19 ]; }; then
> # compare the current and stored checksums; if they do not match, assume
> # that's the way the user wants it. if we're reconfiguring, overwrite
> # it regardless and back it up.
Reply to: