Try harder to find email and maintainer name. Debian's own devscripts all use DEBEMAIL or DEBFULLNAME prior to an eventual EMAIL or NAME environment variable. Match their logic. "Anonymous Maintainer" sounds nicer then Kernel compiler if no name is found. As bonus add more descriptive changelog entry. Cc: Andres Salomon <dilinger@xxxxxxxxxx> Signed-off-by: maximilian attems <max@xxxxxxx> --- scripts/package/builddeb | 25 ++++++++++++++++++++----- 1 files changed, 20 insertions(+), 5 deletions(-) diff --git a/scripts/package/builddeb b/scripts/package/builddeb index 1264b8e..dd61591 100644 --- a/scripts/package/builddeb +++ b/scripts/package/builddeb @@ -66,14 +66,29 @@ EOF chmod 755 "$tmpdir/DEBIAN/$script" done -name="Kernel Compiler <$(id -nu)@$(hostname -f)>" +# Try to determine maintainer and email values +if [ -n "${DEBEMAIL}" ]; then + email=${DEBEMAIL} +elif [ -n "${EMAIL}" ]; then + email=${EMAIL} +else + email=$(id -nu)@$(hostname -f) +fi +if [ -n "${DEBFULLNAME}" ]; then + name=${DEBFULLNAME} +elif [ -n "${NAME}" ]; then + name=${NAME} +else + name="Anonymous Maintainer" +fi +maintainer="${name} <${email}>" # Generate a simple changelog template cat <<EOF > debian/changelog linux ($version-$revision) unstable; urgency=low - * A standard release + * New upstream release - -- $name $(date -R) + -- ${maintainer} $(date -R) EOF # Generate a control file @@ -83,7 +98,7 @@ cat <<EOF > debian/control Source: linux Section: base Priority: optional -Maintainer: $name +Maintainer: $maintainer Standards-Version: 3.6.1 Package: $packagename @@ -105,7 +120,7 @@ cat <<EOF > debian/control Source: linux Section: base Priority: optional -Maintainer: $name +Maintainer: $maintainer Standards-Version: 3.6.1 Package: $packagename -- 1.6.2.1 -- 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