|author||Sedat Dilek <firstname.lastname@example.org>||2015-01-03 10:50:19 +0100|
|committer||Michal Marek <email@example.com>||2015-01-06 20:42:46 +0100|
builddeb: Try to determine distribution
lsb_release command is a good choice to determine the distribution name for the changelog file in the generated Debian packages . Its installation is no precondition. In Debian it is still not essential or build-essential. Ben gave some helpful informations and detailed explanations in . 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 ): "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 ). 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.  https://lkml.org/lkml/2012/4/23/516  http://marc.info/?l=linux-kbuild&m=142022188322321&w=2  http://marc.info/?l=linux-kbuild&m=142023476825460&w=2  http://blog.brlink.eu/index.html#i70  https://www.mirbsd.org/permalinks/wlog-10_e20141209-tg.htm  http://stackoverflow.com/questions/23489934/echo-2-some-text-what-does-it-mean-in-shell-scripting CC: Alexander Wirt <firstname.lastname@example.org> Suggested-by: Ben Hutchings <email@example.com> Suggested-by: Thorsten Glaser <firstname.lastname@example.org> Reviewed-by: Ben Hutchings <email@example.com> Acked-by: maximilian attems <firstname.lastname@example.org> [ dileks: Reviewed his suggested diff in RFC v4 ] Reviewed-by: Thorsten Glaser <email@example.com> Signed-off-by: Sedat Dilek <firstname.lastname@example.org> Signed-off-by: Michal Marek <email@example.com>
Diffstat (limited to 'scripts/package')
1 files changed, 12 insertions, 1 deletions
diff --git a/scripts/package/builddeb b/scripts/package/builddeb
index 2c68c8bf6c76..88dbf23b6970 100755
@@ -217,9 +217,20 @@ else
+# Try to determine distribution
+if [ -n "$KDEB_CHANGELOG_DIST" ]; then
+elif distribution=$(lsb_release -cs 2>/dev/null) && [ -n "$distribution" ]; then
+ : # nothing to do in this case
+ echo >&2 "Using default distribution of 'unstable' in the changelog"
+ echo >&2 "Install lsb-release or set \$KDEB_CHANGELOG_DIST explicitly"
# Generate a simple changelog template
cat <<EOF > debian/changelog
-linux-upstream ($packageversion) unstable; urgency=low
+linux-upstream ($packageversion) $distribution; urgency=low
* Custom built Linux kernel.