diff mbox

fbdev: make scripts/pnmtologo dependency portable

Message ID 1350352380-5324-1-git-send-email-yselkowitz@users.sourceforge.net (mailing list archive)
State New, archived
Headers show

Commit Message

Yaakov (Cygwin/X) Oct. 16, 2012, 1:53 a.m. UTC
From: Yaakov Selkowitz <yselkowitz@users.sourceforge.net>

Commit a53c9d5b7115173fba9f82ff8120b624ef206f48 added a dependency on
scripts/pnmtologo to all autogenerated .c files.  An explicit rule
is required on platforms where the .exe suffix is used for hostprogs.

Signed-off-by: Yaakov Selkowitz <yselkowitz@users.sourceforge.net>
---
Michal,

This was originally sent to linux-fbdev, but was forwarded to linux-kbuild
(but not you) on 12 June.  It has yet to be applied to Linus' tree yet, though.

Also applies to all 3.x stable branches

 drivers/video/logo/Makefile |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

Comments

Michal Marek Oct. 30, 2012, 1:24 p.m. UTC | #1
On 16.10.2012 03:53, Yaakov (Cygwin/X) wrote:
> From: Yaakov Selkowitz <yselkowitz@users.sourceforge.net>
> 
> Commit a53c9d5b7115173fba9f82ff8120b624ef206f48 added a dependency on
> scripts/pnmtologo to all autogenerated .c files.  An explicit rule
> is required on platforms where the .exe suffix is used for hostprogs.
> 
> Signed-off-by: Yaakov Selkowitz <yselkowitz@users.sourceforge.net>
> ---
> Michal,
> 
> This was originally sent to linux-fbdev, but was forwarded to linux-kbuild
> (but not you) on 12 June.  It has yet to be applied to Linus' tree yet, though.
> 
> Also applies to all 3.x stable branches
> 
>  drivers/video/logo/Makefile |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
> 
> diff --git a/drivers/video/logo/Makefile b/drivers/video/logo/Makefile
> index 3b43781..3854dbb 100644
> --- a/drivers/video/logo/Makefile
> +++ b/drivers/video/logo/Makefile
> @@ -38,6 +38,7 @@ extra-y += $(call logo-cfiles,_clut224,ppm)
>  extra-y += $(call logo-cfiles,_gray256,pgm)
>  
>  pnmtologo := scripts/pnmtologo
> +$(pnmtologo): $(objtree)/scripts/pnmtologo

Please add an explanation to the changelog how it works.
scripts/pnmtologo is dependent on $(objtree)/scripts/pnmtologo (same
file??), how does it fix the problem that the file is called
scripts/pnmtologo.exe on Cygwin? I do not see any rule for hostprog.exe
-> hostprog in our makefiles.

Michal
--
To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/drivers/video/logo/Makefile b/drivers/video/logo/Makefile
index 3b43781..3854dbb 100644
--- a/drivers/video/logo/Makefile
+++ b/drivers/video/logo/Makefile
@@ -38,6 +38,7 @@  extra-y += $(call logo-cfiles,_clut224,ppm)
 extra-y += $(call logo-cfiles,_gray256,pgm)
 
 pnmtologo := scripts/pnmtologo
+$(pnmtologo): $(objtree)/scripts/pnmtologo
 
 # Create commands like "pnmtologo -t mono -n logo_mac_mono -o ..."
 quiet_cmd_logo = LOGO    $@