X2Go Bug report logs - #1051
failing .deb builds due to flaw in code-to-version conversion scripts

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

Reported by: Mike Gabriel <mike.gabriel@das-netzwerkteam.de>

Date: Mon, 4 Jul 2016 12:45:02 UTC

Severity: important

Reply or subscribe to this bug.

Toggle useless messages

View this report as an mbox folder, status mbox, maintainer mbox


Report forwarded to x2go-dev@lists.x2go.org, X2Go Developers <x2go-dev@lists.x2go.org>:
Bug#1051; Package buildscripts. (Mon, 04 Jul 2016 12:45:02 GMT) Full text and rfc822 format available.

Acknowledgement sent to Mike Gabriel <mike.gabriel@das-netzwerkteam.de>:
New Bug report received and forwarded. Copy sent to X2Go Developers <x2go-dev@lists.x2go.org>. (Mon, 04 Jul 2016 12:45:02 GMT) Full text and rfc822 format available.

Message #5 received at submit@bugs.x2go.org (full text, mbox):

From: Mike Gabriel <mike.gabriel@das-netzwerkteam.de>
To: submit@bugs.x2go.org
Subject: failing .deb builds due to flaw in code-to-version conversion scripts
Date: Mon, 04 Jul 2016 12:40:16 +0000
[Message part 1 (text/plain, inline)]
Package: buildscripts
Severity: important

Hi Mihai,

I have played with X2Go Server a little today, trying to get the new  
execution logic for nxagent/x2goagent into X2Go Server 4.1.0.0. While  
doing that, I stumbled over build failures due to a flaw in the  
buildscripts.

This fixes the occurring issues:

```
jenkins@japsand:~/buildscripts$ git diff
diff --git a/bin/debian-codename-to-version.sh  
b/bin/debian-codename-to-version.sh
index 8bd0384..315fcbd 100755
--- a/bin/debian-codename-to-version.sh
+++ b/bin/debian-codename-to-version.sh
@@ -76,7 +76,5 @@ case "${codename}" in
        ("rex") echo "1.2";;
        ("buzz") echo "1.1";;

-       (*) ret="1";;
+       (*) echo "1";;
 esac
-
-return "${ret}"
diff --git a/bin/ubuntu-codename-to-version.sh  
b/bin/ubuntu-codename-to-version.sh
index 943e130..f0c1418 100755
--- a/bin/ubuntu-codename-to-version.sh
+++ b/bin/ubuntu-codename-to-version.sh
@@ -66,7 +66,5 @@ case "${codename}" in
        ("hoary") echo "5.04";;
        ("warty") echo "4.10";;

-       (*) ret="1";;
+       (*) echo "1";;
 esac
-
-return "${ret}"
```

light+love
Mike
-- 

DAS-NETZWERKTEAM
mike gabriel, herweg 7, 24357 fleckeby
mobile: +49 (1520) 1976 148
landline: +49 (4354) 8390 139

GnuPG Fingerprint: 9BFB AEE8 6C0A A5FF BF22  0782 9AF4 6B30 2577 1B31
mail: mike.gabriel@das-netzwerkteam.de, http://das-netzwerkteam.de

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

Information forwarded to x2go-dev@lists.x2go.org, X2Go Developers <x2go-dev@lists.x2go.org>:
Bug#1051; Package buildscripts. (Mon, 04 Jul 2016 20:20:02 GMT) Full text and rfc822 format available.

Acknowledgement sent to Mihai Moldovan <ionic@ionic.de>:
Extra info received and forwarded to list. Copy sent to X2Go Developers <x2go-dev@lists.x2go.org>. (Mon, 04 Jul 2016 20:20:02 GMT) Full text and rfc822 format available.

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

From: Mihai Moldovan <ionic@ionic.de>
To: Mike Gabriel <mike.gabriel@das-netzwerkteam.de>, 1051@bugs.x2go.org
Subject: Re: [X2Go-Dev] failing .deb builds due to flaw in code-to-version conversion scripts
Date: Mon, 4 Jul 2016 22:18:40 +0200
[Message part 1 (text/plain, inline)]
On 04.07.2016 02:40 PM, Mike Gabriel wrote:
> Package: buildscripts
> Severity: important
> 
> Hi Mihai,
> 
> I have played with X2Go Server a little today, trying to get the new  
> execution logic for nxagent/x2goagent into X2Go Server 4.1.0.0. While  
> doing that, I stumbled over build failures due to a flaw in the  
> buildscripts.
> 
> This fixes the occurring issues:
> 
> ```
> jenkins@japsand:~/buildscripts$ git diff
> diff --git a/bin/debian-codename-to-version.sh  
> b/bin/debian-codename-to-version.sh
> index 8bd0384..315fcbd 100755
> --- a/bin/debian-codename-to-version.sh
> +++ b/bin/debian-codename-to-version.sh
> @@ -76,7 +76,5 @@ case "${codename}" in
>          ("rex") echo "1.2";;
>          ("buzz") echo "1.1";;
> 
> -       (*) ret="1";;
> +       (*) echo "1";;
>   esac
> -
> -return "${ret}"
> diff --git a/bin/ubuntu-codename-to-version.sh  
> b/bin/ubuntu-codename-to-version.sh
> index 943e130..f0c1418 100755
> --- a/bin/ubuntu-codename-to-version.sh
> +++ b/bin/ubuntu-codename-to-version.sh
> @@ -66,7 +66,5 @@ case "${codename}" in
>          ("hoary") echo "5.04";;
>          ("warty") echo "4.10";;
> 
> -       (*) ret="1";;
> +       (*) echo "1";;
>   esac
> -
> -return "${ret}"
> ```

NAK, that would go against the script's contract as detailed in the comments.
There shouldn't be any output in case the code name provided is invalid, the
status is supposed to be reflected by the return value. There were a few
problems I fixed, though.

How about

http://code.x2go.org/gitweb?p=buildscripts.git;a=commitdiff;h=3153ec59c323cdc2efb3d394f0e4e2f98ca28c93

and

http://code.x2go.org/gitweb?p=buildscripts.git;a=commitdiff;h=8bb92fd510b40fe80759460fb96c1fafa1423fba

instead?



Mihai

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

Send a report that this bug log contains spam.


X2Go Developers <owner@bugs.x2go.org>. Last modified: Tue Dec 18 10:48:31 2018; Machine Name: ymir.das-netzwerkteam.de

X2Go Bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.