On Mon, 5 Oct 2015 15:39:40 +0300 Riku Voipio <riku.voipio@xxxxxxxxxx> wrote: > On 5 October 2015 at 04:03, Doug Smythies <dsmythies@xxxxxxxxx> wrote: > > On 2015.09.22 04:25 Kalle Valo wrote: > >> Ben Hutchings <ben@xxxxxxxxxxxxxxx> writes: > >>> On Tue, 2015-09-15 at 21:11 +0300, riku.voipio@xxxxxxxxxx wrote: > >>>> From: Riku Voipio <riku.voipio@xxxxxxxxxx> > >>>> > >>>> As debian/files keeps accumulating entries, the changes file > >>>> will end up growing indefinelty. Remove the file in clean > >>>> rule and before build starts. > >>>> > >>>> Signed-off-by: Riku Voipio <riku.voipio@xxxxxxxxxx> > >>>> Reported-by: Doug Smythies <dsmythies@xxxxxxxxx> > >>> [...] > >>> > >>> FWIW: > >>> > >>> Acked-by: Ben Hutchings <ben@xxxxxxxxxxxxxxx> > >> > >> Marek, this is a pretty annoying regression. Is it possible to get this > >> to 4.3? > > > > Perhaps I do not understand the process here, but having reported the issue > > within a day of kernel 4.3-rc1 being released, and then what seems to be an > > acceptable patch being suggested, and tested, with plenty of time to make > > into -rc2, why hasn't it been included yet (-rc4)? > > Hi Andrew, > > Could you pass this patch in your series, as the regression doesn't > seem to be getting through via the maintainer. It would be > embarrassing to have it in 4.3 since it was reported and fixed timely. > The patch is at: > > http://permalink.gmane.org/gmane.linux.kbuild.devel/13710 > > Tests and acks at: > > http://comments.gmane.org/gmane.linux.kbuild.devel/13710 I'm not subscribed to linux-kbuild and the browser won't do save-as-text, so I typed it all in again. Please check the below. I'll put this in my mainline-later queue, which means I'll send it Linuswards next week if it doesn't pop up in linux-next before then. The changelog doesn't actually describe this patch as a regression fix. It makes the patch look like a regular old bugfix. And the changelog doesn't describe the severity of the bug either. Please do pay attention to such details to guide others in deciding which kernel version(s) need fixing. From: Riku Voipio <riku.voipio@xxxxxxxxxx> Subject: builddeb: remove debian/files before build As debian/files keeps accumulating entries, the changes file will end up growing indefinelty. Remove the file in clean rule and before build starts. Fixes: 3716001bcb7f ("deb-pkg: add source package") Signed-off-by: Riku Voipio <riku.voipio@xxxxxxxxxx> Reported-by: Doug Smythies <dsmythies@xxxxxxxxx> Tested-by: Doug Smythies <dsmythies@xxxxxxxxx> Tested-by: Kalle Valo <kvalo@xxxxxxxxxxxxxx> Acked-by: Ben Hutchings <ben@xxxxxxxxxxxxxxx> Cc: Michal Marek <mmarek@xxxxxxx> Cc: maximilian attems <maks@xxxxxxx> Cc: Chris J Arges <chris.j.arges@xxxxxxxxxxxxx> Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> --- scripts/package/builddeb | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff -puN scripts/package/builddeb~a scripts/package/builddeb --- a/scripts/package/builddeb~a +++ a/scripts/package/builddeb @@ -115,7 +115,7 @@ esac BUILD_DEBUG="$(grep -s '^CONFIG_DEBUG_INFO=y' $KCONFIG_CONFIG || true)" # Setup the directory structure -rm -rf "$tmpdir" "$fwdir" "$kernel_headers_dir" "$libc_headers_dir" "$dbg_dir" +rm -rf "$tmpdir" "$fwdir" "$kernel_headers_dir" "$libc_headers_dir" "$dbg_dir" $objtree/debian/files mkdir -m 755 -p "$tmpdir/DEBIAN" mkdir -p "$tmpdir/lib" "$tmpdir/boot" mkdir -p "$fwdir/lib/firmware/$version/" @@ -408,7 +408,7 @@ binary-arch: \$(MAKE) KDEB_SOURCENAME=${sourcename} KDEB_PKGVERSION=${packageversion} bindeb-pkg clean: - rm -rf debian/*tmp + rm -rf debian/*tmp debian/files mv debian/ debian.backup # debian/ might be cleaned away \$(MAKE) clean mv debian.backup debian _ -- 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
