From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wg0-f43.google.com ([74.125.82.43]:64633 "EHLO mail-wg0-f43.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752881AbbAFLj3 (ORCPT ); Tue, 6 Jan 2015 06:39:29 -0500 Received: by mail-wg0-f43.google.com with SMTP id k14so11448522wgh.16 for ; Tue, 06 Jan 2015 03:39:27 -0800 (PST) MIME-Version: 1.0 Reply-To: sedat.dilek@gmail.com In-Reply-To: <1420278619-6637-1-git-send-email-sedat.dilek@gmail.com> References: <1420278619-6637-1-git-send-email-sedat.dilek@gmail.com> Date: Tue, 6 Jan 2015 12:39:27 +0100 Message-ID: Subject: Re: [PATCH v6] builddeb: Try to determine distribution From: Sedat Dilek Content-Type: text/plain; charset=UTF-8 Sender: linux-kbuild-owner@vger.kernel.org List-ID: To: Sedat Dilek Cc: Ben Hutchings , maximilian attems , Thorsten Glaser , Alexander Wirt , Michal Marek , linux-kbuild@vger.kernel.org On Sat, Jan 3, 2015 at 10:50 AM, Sedat Dilek wrote: > lsb_release command is a good choice to determine the distribution > name for the changelog file in the generated Debian packages [1]. > Its installation is no precondition. > In Debian it is still not essential or build-essential. > > Ben gave some helpful informations and detailed explanations in [2]. > There he also suggested to have an option to explicitly set the > distribution name (see $KDEB_CHANGELOG_DIST variable). > > Embedded the improvement as suggested by Thorsten (see [3]): > > "This is suboptimal: if KDEB_CHANGELOG_DIST is defined, > lsb_release is not necessary. The following snippet > also omits using its output if it fails but still > produces any:" > > Dealing with this issue I learned about "The Colon in the Shell." and > possible pitfalls in this area (see [4,5]). Furthermore, refreshed my > knowledge about redirecting outputs with the echo command (see [5]). > Special thanks to Thorsten, I enjoyed the IRC session with you. > > Cooked together the snippets of Ben and Thorsten (see [2,3]). > Tested against Linux v3.19-rc2. > > Thanks goes to Alexander, Ben, maximilian and Thorsten for the very > vital help. > > [1] https://lkml.org/lkml/2012/4/23/516 > [2] http://marc.info/?l=linux-kbuild&m=142022188322321&w=2 > [3] http://marc.info/?l=linux-kbuild&m=142023476825460&w=2 > [4] http://blog.brlink.eu/index.html#i70 > [5] https://www.mirbsd.org/permalinks/wlog-10_e20141209-tg.htm > [6] http://stackoverflow.com/questions/23489934/echo-2-some-text-what-does-it-mean-in-shell-scripting > > CC: Ben Hutchings > CC: maximilian attems > CC: Thorsten Glaser > CC: Alexander Wirt > CC: Michal Marek > CC: linux-kbuild@vger.kernel.org > Suggested-by: Ben Hutchings > Suggested-by: Thorsten Glaser > Reviewed-by: Ben Hutchings > Acked-by: maximilian attems > [ dileks: Reviewed his suggested diff in RFC v4 ] > Reviewed-by: Thorsten Glaser > Signed-off-by: Sedat Dilek > --- > v6: Based on RFC v5 with added reviews and acks > v5: Improve commit-message and fix Thorsten's email-address > v4: Embed improvements suggested by Thorsten > v3: Massaged the commit-message > v2: Suppress error messages when lsb_release is not installed. > (Thanks maximilian and Ben.) > > scripts/package/builddeb | 13 ++++++++++++- > 1 file changed, 12 insertions(+), 1 deletion(-) > > diff --git a/scripts/package/builddeb b/scripts/package/builddeb > index 5972624..f631c77 100755 > --- a/scripts/package/builddeb > +++ b/scripts/package/builddeb > @@ -217,9 +217,20 @@ else > fi > maintainer="$name <$email>" > > +# Try to determine distribution > +if [ -n "$KDEB_CHANGELOG_DIST" ]; then > + distribution=$KDEB_CHANGELOG_DIST > +elif distribution=$(lsb_release -cs 2>/dev/null) && [ -n "$distribution" ]; then > + : # nothing to do in this case > +else > + distribution="unstable" > + echo >&2 "Using default distribution of 'unstable' in the changelog" > + echo >&2 "Install lsb-release or set \$KDEB_CHANGELOG_DIST explicitly" > +fi > + > # Generate a simple changelog template > cat < debian/changelog > -linux-upstream ($packageversion) unstable; urgency=low > +linux-upstream ($packageversion) $distribution; urgency=low > > * Custom built Linux kernel. > > -- > 2.2.1 > According to your explanations about your development workflow and Git trees this can go now to kbuild.git#misc. Or is something wrong with that one (commit message, missing credits, etc.)? Thanks. - Sedat -