Re: [PATCH 2/2] deb-pkg: add source package
On Fri, Apr 10, 2015 at 04:15:14PM +0300, riku.voipio@linaro.org wrote:
> From: Riku Voipio <riku.voipio@linaro.org>
>
> By passing BUILD_SOURCE=y variable, make deb-pkg builds a debian source
> package. It will generate a minimal debian/rules file that calls back
> to make deb-pkg. Generated source package will build the same kernel
> .config than what was available for make deb-pkg.
>
> The source package is useful for gpl compliance, or for feeding to a
> automated debian package builder.
>
> Patch depends on the "deb-pkg: move setting debarch for a separate function"
> for correct changelog filenames.
>
> Signed-off-by: Riku Voipio <riku.voipio@linaro.org>
> ---
great this is a much requested feature for wider adoption of make
deb-pkg. In general acked-by me, just minor comment below.
I do not like the BUILD_SOURCE=y variable,
I think it should just be like the other scripts and do it by default.
What we do need is a target that *only* compiles the linux image.
> scripts/package/builddeb | 42 ++++++++++++++++++++++++++++++++++++++++++
> 1 file changed, 42 insertions(+)
>
> diff --git a/scripts/package/builddeb b/scripts/package/builddeb
> index e397815..3d77fd3 100755
> --- a/scripts/package/builddeb
> +++ b/scripts/package/builddeb
> @@ -272,12 +272,23 @@ On Debian GNU/Linux systems, the complete text of the GNU General Public
> License version 2 can be found in \`/usr/share/common-licenses/GPL-2'.
> EOF
>
> +
> +build_depends="bc, "
> +if [ -n "$BUILD_TOOLS" ]
why this dual stage?
> +then
> + build_depends="$build_depends python-dev, libperl-dev, bison, flex, \
> +libaudit-dev, libdw-dev, libelf-dev, libiberty-dev, libnewt-dev, autoconf, \
> +automake, libtool, libglib2.0-dev, libudev-dev, libwrap0-dev, libiberty-dev, \
> +libunwind8-dev [amd64 arm64 i386], libnuma-dev [amd64 arm64 i386 powerpc ppc64 ppc64el] "
how did you generate this list, this seems bogus to me?!
python-dev should probably be python
why would you need automake?
plus I do seem to miss cpio, kmod.
> +fi
> +
> # Generate a control file
> cat <<EOF > debian/control
> Source: linux-upstream
> Section: kernel
> Priority: optional
> Maintainer: $maintainer
> +Build-Depends: $build_depends
> Standards-Version: 3.8.4
> Homepage: http://www.kernel.org/
> EOF
> @@ -425,4 +436,35 @@ EOF
> create_package "$tools_packagename" "$tools_dir"
> fi
>
> +if [ -n "$BUILD_SOURCE" ]
> +then
> + cat <<EOF > debian/rules
> +#!/usr/bin/make -f
> +
> +build:
> + cp debian/config .config
> + \$(MAKE) oldconfig
> +
> +binary-arch:
> + \$(MAKE) KDEB_PKGVERSION=${packageversion} BUILD_TOOLS=$BUILD_TOOLS deb-pkg
> +
> +clean:
> + \$(MAKE) clean
> +
> +binary: binary-arch
> +EOF
> +
> + (cd $KBUILD_SRC; git archive --prefix=linux-upstream-${version}/ HEAD)|gzip -9 > ../linux-upstream_${version}.orig.tar.gz
> + cp $KCONFIG_CONFIG debian/config
> + tar caf ../linux-upstream_${packageversion}.debian.tar.gz debian/{config,copyright,rules,changelog,control}
> + dpkg-source -cdebian/control -ldebian/changelog --format="3.0 (custom)" --target-format="3.0 (quilt)" \
> + -b / ../linux-upstream_${version}.orig.tar.gz ../linux-upstream_${packageversion}.debian.tar.gz
> + mv linux-upstream_${packageversion}*dsc ..
> + dpkg-genchanges > ../linux-upstream_${packageversion}_${debarch}.changes
> +else
> + dpkg-genchanges -b > ../linux-upstream_${packageversion}_${debarch}.changes
> +fi
> +
> +
> +
> exit 0
> --
> 2.1.4
>
>
> --
> To UNSUBSCRIBE, email to debian-kernel-REQUEST@lists.debian.org
> with a subject of "unsubscribe". Trouble? Contact listmaster@lists.debian.org
> Archive: [🔎] 34017611f8a0056a09d2c38412efd7828efe00fe.1428671643.git.riku.voipio@linaro.org">https://lists.debian.org/[🔎] 34017611f8a0056a09d2c38412efd7828efe00fe.1428671643.git.riku.voipio@linaro.org
>
Reply to: