mail archive of the barebox mailing list
 help / color / mirror / Atom feed
* [PATCH] scripts: Delete non-barebox content from scripts/.
@ 2009-12-21 10:19 Robert P. J. Day
  2009-12-21 10:51 ` Sascha Hauer
  0 siblings, 1 reply; 3+ messages in thread
From: Robert P. J. Day @ 2009-12-21 10:19 UTC (permalink / raw)
  To: U-Boot Version 2 (barebox)


Remove scripts/ content having nothing to do with barebox:

  - pnmtologo
  - conmakehash
  - unifdef
  - genksyms

---

  if there's anything else that can be added to this patch (that is,
scripts/ content that can be tossed), i can resubmit a more complete
patch.  otherwise, might as well just apply as is.


diff --git a/scripts/Makefile b/scripts/Makefile
index a6829c5..c22a8b6 100644
--- a/scripts/Makefile
+++ b/scripts/Makefile
@@ -3,14 +3,8 @@
 # the kernel for the build process.
 # ---------------------------------------------------------------------------
 # kallsyms:      Find all symbols in barebox
-# pnmttologo:    Convert pnm files to logo files
-# conmakehash:   Create chartable
-# conmakehash:	 Create arrays for initializing the kernel console tables

 hostprogs-$(CONFIG_KALLSYMS)     += kallsyms
-hostprogs-$(CONFIG_LOGO)         += pnmtologo
-hostprogs-$(CONFIG_VT)           += conmakehash
-hostprogs-$(CONFIG_PROM_CONSOLE) += conmakehash
 hostprogs-y                      += bin2c
 hostprogs-y                      += mkimage
 hostprogs-y                      += bareboxenv
@@ -18,10 +12,6 @@ hostprogs-$(CONFIG_ARCH_NETX)    += gen_netx_image

 always		:= $(hostprogs-y) $(hostprogs-m)

-# The following hostprogs-y programs are only build on demand
-hostprogs-y += unifdef
-
-#subdir-$(CONFIG_MODVERSIONS) += genksyms
 subdir-y                     += mod

 # Let clean descend into subdirs



========================================================================
Robert P. J. Day                               Waterloo, Ontario, CANADA

            Linux Consulting, Training and Kernel Pedantry.

Web page:                                          http://crashcourse.ca
Twitter:                                       http://twitter.com/rpjday
========================================================================

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

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

* Re: [PATCH] scripts: Delete non-barebox content from scripts/.
  2009-12-21 10:19 [PATCH] scripts: Delete non-barebox content from scripts/ Robert P. J. Day
@ 2009-12-21 10:51 ` Sascha Hauer
  2009-12-21 11:11   ` Robert P. J. Day
  0 siblings, 1 reply; 3+ messages in thread
From: Sascha Hauer @ 2009-12-21 10:51 UTC (permalink / raw)
  To: Robert P. J. Day; +Cc: U-Boot Version 2 (barebox)

On Mon, Dec 21, 2009 at 05:19:06AM -0500, Robert P. J. Day wrote:
> 
> Remove scripts/ content having nothing to do with barebox:
> 
>   - pnmtologo
>   - conmakehash
>   - unifdef
>   - genksyms
> 
> ---
> 
>   if there's anything else that can be added to this patch (that is,
> scripts/ content that can be tossed), i can resubmit a more complete
> patch.  otherwise, might as well just apply as is.

Applied as is. If there's more to cleanup, a second patch doesn't hurt.

Sascha


> 
> 
> diff --git a/scripts/Makefile b/scripts/Makefile
> index a6829c5..c22a8b6 100644
> --- a/scripts/Makefile
> +++ b/scripts/Makefile
> @@ -3,14 +3,8 @@
>  # the kernel for the build process.
>  # ---------------------------------------------------------------------------
>  # kallsyms:      Find all symbols in barebox
> -# pnmttologo:    Convert pnm files to logo files
> -# conmakehash:   Create chartable
> -# conmakehash:	 Create arrays for initializing the kernel console tables
> 
>  hostprogs-$(CONFIG_KALLSYMS)     += kallsyms
> -hostprogs-$(CONFIG_LOGO)         += pnmtologo
> -hostprogs-$(CONFIG_VT)           += conmakehash
> -hostprogs-$(CONFIG_PROM_CONSOLE) += conmakehash
>  hostprogs-y                      += bin2c
>  hostprogs-y                      += mkimage
>  hostprogs-y                      += bareboxenv
> @@ -18,10 +12,6 @@ hostprogs-$(CONFIG_ARCH_NETX)    += gen_netx_image
> 
>  always		:= $(hostprogs-y) $(hostprogs-m)
> 
> -# The following hostprogs-y programs are only build on demand
> -hostprogs-y += unifdef
> -
> -#subdir-$(CONFIG_MODVERSIONS) += genksyms
>  subdir-y                     += mod
> 
>  # Let clean descend into subdirs
> 
> 
> 
> ========================================================================
> Robert P. J. Day                               Waterloo, Ontario, CANADA
> 
>             Linux Consulting, Training and Kernel Pedantry.
> 
> Web page:                                          http://crashcourse.ca
> Twitter:                                       http://twitter.com/rpjday
> ========================================================================
> 
> _______________________________________________
> barebox mailing list
> barebox@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/barebox
> 

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

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

* Re: [PATCH] scripts: Delete non-barebox content from scripts/.
  2009-12-21 10:51 ` Sascha Hauer
@ 2009-12-21 11:11   ` Robert P. J. Day
  0 siblings, 0 replies; 3+ messages in thread
From: Robert P. J. Day @ 2009-12-21 11:11 UTC (permalink / raw)
  To: Sascha Hauer; +Cc: U-Boot Version 2 (barebox)

On Mon, 21 Dec 2009, Sascha Hauer wrote:

> On Mon, Dec 21, 2009 at 05:19:06AM -0500, Robert P. J. Day wrote:
> >
> > Remove scripts/ content having nothing to do with barebox:
> >
> >   - pnmtologo
> >   - conmakehash
> >   - unifdef
> >   - genksyms
> >
> > ---
> >
> >   if there's anything else that can be added to this patch (that
> > is, scripts/ content that can be tossed), i can resubmit a more
> > complete patch.  otherwise, might as well just apply as is.
>
> Applied as is. If there's more to cleanup, a second patch doesn't
> hurt.
>
> Sascha

  generally, i'll submit separate patches if, say, some cleanup is
obvious while additional cleanup is not quite so straightforward.  the
above struck me as what was immediately trivial so it makes sense that
it can be the initial patch for that directory.  thanks.

rday
--



========================================================================
Robert P. J. Day                               Waterloo, Ontario, CANADA

            Linux Consulting, Training and Kernel Pedantry.

Web page:                                          http://crashcourse.ca
Twitter:                                       http://twitter.com/rpjday
========================================================================

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

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

end of thread, other threads:[~2009-12-21 11:11 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-12-21 10:19 [PATCH] scripts: Delete non-barebox content from scripts/ Robert P. J. Day
2009-12-21 10:51 ` Sascha Hauer
2009-12-21 11:11   ` Robert P. J. Day

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