mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH] kbuild: deb-pkg: specify targets in debian/rules as .PHONY
@ 2023-04-25 11:08 Masahiro Yamada
  2023-04-25 18:08 ` Nathan Chancellor
  0 siblings, 1 reply; 3+ messages in thread
From: Masahiro Yamada @ 2023-04-25 11:08 UTC (permalink / raw)
  To: linux-kbuild
  Cc: linux-kernel, Masahiro Yamada, Nathan Chancellor,
	Nick Desaulniers, Nicolas Schier

If a file with the same name exists, the target is not run.

For example, the following command fails.

  $ make O=build-arch bindeb-pkg
    [ snip ]
  sed: can't read modules.order: No such file or directory
  make[6]: *** [../Makefile:1577: __modinst_pre] Error 2
  make[5]: *** [../scripts/Makefile.package:150: intdeb-pkg] Error 2
  make[4]: *** [../Makefile:1657: intdeb-pkg] Error 2
  make[3]: *** [debian/rules:14: binary-arch] Error 2
  dpkg-buildpackage: error: debian/rules binary subprocess returned exit status 2
  make[2]: *** [../scripts/Makefile.package:139: bindeb-pkg] Error 2

Signed-off-by: Masahiro Yamada <masahiroy@kernel.org>
---

 scripts/package/mkdebian | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/scripts/package/mkdebian b/scripts/package/mkdebian
index a4c2c2276223..b6cb95473548 100755
--- a/scripts/package/mkdebian
+++ b/scripts/package/mkdebian
@@ -269,6 +269,8 @@ cat <<EOF > debian/rules
 srctree ?= .
 KERNELRELEASE = ${KERNELRELEASE}
 
+.PHONY: clean build build-arch build-indep binary binary-arch binary-indep
+
 build-indep:
 build-arch:
 	\$(MAKE) -f \$(srctree)/Makefile ARCH=${ARCH} \
-- 
2.37.2


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] kbuild: deb-pkg: specify targets in debian/rules as .PHONY
  2023-04-25 11:08 [PATCH] kbuild: deb-pkg: specify targets in debian/rules as .PHONY Masahiro Yamada
@ 2023-04-25 18:08 ` Nathan Chancellor
  2023-04-26 14:12   ` Masahiro Yamada
  0 siblings, 1 reply; 3+ messages in thread
From: Nathan Chancellor @ 2023-04-25 18:08 UTC (permalink / raw)
  To: Masahiro Yamada
  Cc: linux-kbuild, linux-kernel, Nick Desaulniers, Nicolas Schier

On Tue, Apr 25, 2023 at 08:08:59PM +0900, Masahiro Yamada wrote:
> If a file with the same name exists, the target is not run.
> 
> For example, the following command fails.
> 
>   $ make O=build-arch bindeb-pkg
>     [ snip ]
>   sed: can't read modules.order: No such file or directory
>   make[6]: *** [../Makefile:1577: __modinst_pre] Error 2
>   make[5]: *** [../scripts/Makefile.package:150: intdeb-pkg] Error 2
>   make[4]: *** [../Makefile:1657: intdeb-pkg] Error 2
>   make[3]: *** [debian/rules:14: binary-arch] Error 2
>   dpkg-buildpackage: error: debian/rules binary subprocess returned exit status 2
>   make[2]: *** [../scripts/Makefile.package:139: bindeb-pkg] Error 2
> 
> Signed-off-by: Masahiro Yamada <masahiroy@kernel.org>

Reviewed-by: Nathan Chancellor <nathan@kernel.org>

> ---
> 
>  scripts/package/mkdebian | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/scripts/package/mkdebian b/scripts/package/mkdebian
> index a4c2c2276223..b6cb95473548 100755
> --- a/scripts/package/mkdebian
> +++ b/scripts/package/mkdebian
> @@ -269,6 +269,8 @@ cat <<EOF > debian/rules
>  srctree ?= .
>  KERNELRELEASE = ${KERNELRELEASE}
>  
> +.PHONY: clean build build-arch build-indep binary binary-arch binary-indep
> +
>  build-indep:
>  build-arch:
>  	\$(MAKE) -f \$(srctree)/Makefile ARCH=${ARCH} \
> -- 
> 2.37.2
> 

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] kbuild: deb-pkg: specify targets in debian/rules as .PHONY
  2023-04-25 18:08 ` Nathan Chancellor
@ 2023-04-26 14:12   ` Masahiro Yamada
  0 siblings, 0 replies; 3+ messages in thread
From: Masahiro Yamada @ 2023-04-26 14:12 UTC (permalink / raw)
  To: Nathan Chancellor
  Cc: linux-kbuild, linux-kernel, Nick Desaulniers, Nicolas Schier

On Wed, Apr 26, 2023 at 3:08 AM Nathan Chancellor <nathan@kernel.org> wrote:
>
> On Tue, Apr 25, 2023 at 08:08:59PM +0900, Masahiro Yamada wrote:
> > If a file with the same name exists, the target is not run.
> >
> > For example, the following command fails.
> >
> >   $ make O=build-arch bindeb-pkg
> >     [ snip ]
> >   sed: can't read modules.order: No such file or directory
> >   make[6]: *** [../Makefile:1577: __modinst_pre] Error 2
> >   make[5]: *** [../scripts/Makefile.package:150: intdeb-pkg] Error 2
> >   make[4]: *** [../Makefile:1657: intdeb-pkg] Error 2
> >   make[3]: *** [debian/rules:14: binary-arch] Error 2
> >   dpkg-buildpackage: error: debian/rules binary subprocess returned exit status 2
> >   make[2]: *** [../scripts/Makefile.package:139: bindeb-pkg] Error 2
> >
> > Signed-off-by: Masahiro Yamada <masahiroy@kernel.org>
>
> Reviewed-by: Nathan Chancellor <nathan@kernel.org>

Applied to linux-kbuild.



>
> > ---
> >
> >  scripts/package/mkdebian | 2 ++
> >  1 file changed, 2 insertions(+)
> >
> > diff --git a/scripts/package/mkdebian b/scripts/package/mkdebian
> > index a4c2c2276223..b6cb95473548 100755
> > --- a/scripts/package/mkdebian
> > +++ b/scripts/package/mkdebian
> > @@ -269,6 +269,8 @@ cat <<EOF > debian/rules
> >  srctree ?= .
> >  KERNELRELEASE = ${KERNELRELEASE}
> >
> > +.PHONY: clean build build-arch build-indep binary binary-arch binary-indep
> > +
> >  build-indep:
> >  build-arch:
> >       \$(MAKE) -f \$(srctree)/Makefile ARCH=${ARCH} \
> > --
> > 2.37.2
> >



-- 
Best Regards
Masahiro Yamada

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2023-04-26 14:13 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-04-25 11:08 [PATCH] kbuild: deb-pkg: specify targets in debian/rules as .PHONY Masahiro Yamada
2023-04-25 18:08 ` Nathan Chancellor
2023-04-26 14:12   ` Masahiro Yamada

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®