On Fri, Jan 2, 2015 at 9:21 PM, maximilian attems <max@xxxxxxx> wrote: > > > On Fri, Jan 02, 2015 at 09:14:29PM +0100, Sedat Dilek wrote: >> This is based on ideas of me and Ben. >> Thanks goes to Thorsten and Alexander for the vital help in 2012. >> >> Like in my initial patch [1] lsb_release is used to determine >> the short codename as distribution name for debian/changelog. > > In contrast to the code snippet from Ben, you assume lsb_release > to be installed, which is a wrong assumption. > So I should change that line? +codename=$(lsb_release --codename --short) -codename=$(lsb_release --codename --short 2>/dev/null) - Sedat - >> Ben gave some helpful and detailed information in [2]. >> There he also suggested to have an option to explicitly set the >> distribution (see $KDEB_CHANGELOG_DIST variable). >> >> I have hardcoded the script-name when the default distribution >> is used. This is a bit ugly. >> >> This patch is based on the snippet of Ben in [2] and tested >> against Linux v3.19-rc2. >> >> [1] https://lkml.org/lkml/2012/4/23/516 >> [2] http://marc.info/?l=linux-kbuild&m=142022188322321&w=2 >> -- To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
