Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH] Add local directory for non-public packages
@ 2007-01-23 22:58 Ulf Samuelsson
  2007-01-24  0:11 ` Erik Andersen
  0 siblings, 1 reply; 12+ messages in thread
From: Ulf Samuelsson @ 2007-01-23 22:58 UTC (permalink / raw)
  To: buildroot

This adds a "buildroot/local" directory.
The structure of "local" is the same as that of "packages"
but the contents of "local" is never
intended to be merged upstream.

It allows simple maintenance of those application
since you only need to move the local directory from
one version of buildroot to a newer.

Best Regards
Ulf Samuelsson



-------------- next part --------------
A non-text attachment was scrubbed...
Name: Makefile_local.patch
Type: text/x-patch
Size: 1458 bytes
Desc: not available
Url : http://busybox.net/lists/buildroot/attachments/20070123/74be3a88/attachment-0001.bin 

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

* [Buildroot] [PATCH] Add local directory for non-public packages
  2007-01-23 22:58 [Buildroot] [PATCH] Add local directory for non-public packages Ulf Samuelsson
@ 2007-01-24  0:11 ` Erik Andersen
  2007-01-24  1:13   ` Ulf Samuelsson
  2007-01-24 11:37   ` Daniel Laird
  0 siblings, 2 replies; 12+ messages in thread
From: Erik Andersen @ 2007-01-24  0:11 UTC (permalink / raw)
  To: buildroot

On Tue Jan 23, 2007 at 11:58:40PM +0100, Ulf Samuelsson wrote:
> This adds a "buildroot/local" directory.
> The structure of "local" is the same as that of "packages"
> but the contents of "local" is never
> intended to be merged upstream.
> 
> It allows simple maintenance of those application
> since you only need to move the local directory from
> one version of buildroot to a newer.
> 
> Best Regards
> Ulf Samuelsson

See package/customize/ which tries to do the same thing.

If you think perhaps package/customize/ should be renamed
or better documented I'm open to that.  Adding a second
directory doing the same thing however seems unwise.


 -Erik

--
Erik B. Andersen             http://codepoet-consulting.com/
--This message was written using 73% post-consumer electrons--

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

* [Buildroot] [PATCH] Add local directory for non-public packages
  2007-01-24  0:11 ` Erik Andersen
@ 2007-01-24  1:13   ` Ulf Samuelsson
  2007-01-24  9:17     ` Thomas Lundquist
  2007-01-24 11:58     ` Daniel Laird
  2007-01-24 11:37   ` Daniel Laird
  1 sibling, 2 replies; 12+ messages in thread
From: Ulf Samuelsson @ 2007-01-24  1:13 UTC (permalink / raw)
  To: buildroot

Erik Andersen wrote:
> On Tue Jan 23, 2007 at 11:58:40PM +0100, Ulf Samuelsson wrote:
>> This adds a "buildroot/local" directory.
>> The structure of "local" is the same as that of "packages"
>> but the contents of "local" is never
>> intended to be merged upstream.
>>
>> It allows simple maintenance of those application
>> since you only need to move the local directory from
>> one version of buildroot to a newer.
>>
>> Best Regards
>> Ulf Samuelsson
>
> See package/customize/ which tries to do the same thing.
>

Package "customize" *copies* things into the file system
I think this is useful if you want to have things like
web pages into the root system.

"local" is for "building" packages which should be in the file system,
but may be of a prorprietary nature, or for testing purpose or whatever.

The purpose is mainly easy maintenance.
Anything in local could be in "packages", but since
it is never going into the mainstream,
so you have to update those packages every time
you update the buildroot.

By definition, this is the only patch for "local" there will ever be.
Anything going into mainstream buildroot should go into "packages"


> If you think perhaps package/customize/ should be renamed
> or better documented I'm open to that.  Adding a second
> directory doing the same thing however seems unwise.
>
>
> -Erik

Best Regards,
Ulf Samuelsson
ulf at atmel.com
GSM:  +46 (706) 22 44 57
Tel:     +46  (8) 441 54 22
Fax:     +46 (8) 441 54 29
Mail: Box 2033  174 02 Sundbyberg
Visit: Kavalleriv?gen 24
          174 58 Sundbyberg'
Sweden

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

* [Buildroot] [PATCH] Add local directory for non-public packages
  2007-01-24  1:13   ` Ulf Samuelsson
@ 2007-01-24  9:17     ` Thomas Lundquist
  2007-01-24 11:58     ` Daniel Laird
  1 sibling, 0 replies; 12+ messages in thread
From: Thomas Lundquist @ 2007-01-24  9:17 UTC (permalink / raw)
  To: buildroot

On Wed, Jan 24, 2007 at 02:13:17AM +0100, Ulf Samuelsson wrote:
> 
> "local" is for "building" packages which should be in the file system,
> but may be of a prorprietary nature, or for testing purpose or whatever.

I'd be very happy if this patch was included.




Thomas.

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

* [Buildroot] [PATCH] Add local directory for non-public packages
  2007-01-24  0:11 ` Erik Andersen
  2007-01-24  1:13   ` Ulf Samuelsson
@ 2007-01-24 11:37   ` Daniel Laird
  2007-01-24 15:02     ` Ulf Samuelsson
  2007-01-26 16:18     ` MikeW
  1 sibling, 2 replies; 12+ messages in thread
From: Daniel Laird @ 2007-01-24 11:37 UTC (permalink / raw)
  To: buildroot




Erik Andersen wrote:
> 
> On Tue Jan 23, 2007 at 11:58:40PM +0100, Ulf Samuelsson wrote:
>> This adds a "buildroot/local" directory.
>> The structure of "local" is the same as that of "packages"
>> but the contents of "local" is never
>> intended to be merged upstream.
>> 
>> It allows simple maintenance of those application
>> since you only need to move the local directory from
>> one version of buildroot to a newer.
>> 
>> Best Regards
>> Ulf Samuelsson
> 
> See package/customize/ which tries to do the same thing.
> 
> If you think perhaps package/customize/ should be renamed
> or better documented I'm open to that.  Adding a second
> directory doing the same thing however seems unwise.
> 
> 
>  -Erik
> 
> --
> Erik B. Andersen             http://codepoet-consulting.com/
> --This message was written using 73% post-consumer electrons--
> _______________________________________________
> buildroot mailing list
> buildroot at uclibc.org
> http://busybox.net/mailman/listinfo/buildroot
> 
> 
I too would be very happy to see this patch go in.  It means buildroot can
be a more generic build system allowing people to easily move from version
to version.  But also allows people to use it to build their own packages
that are not going to be ever part of the open-source world, or are so
specialised as to be of limited use to the main buildroot community.
Dan
-- 
View this message in context: http://www.nabble.com/-PATCH--Add-local-directory-for-non-public-packages-tf3078310.html#a8558857
Sent from the BuildRoot mailing list archive at Nabble.com.

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

* [Buildroot] [PATCH] Add local directory for non-public packages
  2007-01-24  1:13   ` Ulf Samuelsson
  2007-01-24  9:17     ` Thomas Lundquist
@ 2007-01-24 11:58     ` Daniel Laird
  2007-01-24 12:11       ` Ulf Samuelsson
  1 sibling, 1 reply; 12+ messages in thread
From: Daniel Laird @ 2007-01-24 11:58 UTC (permalink / raw)
  To: buildroot


Do you need to patch the top level Config.in to include local/Config.in? I
think this is needed for the menuconfig stuff.

Cheers
Dan
-- 
View this message in context: http://www.nabble.com/-PATCH--Add-local-directory-for-non-public-packages-tf3078310.html#a8559127
Sent from the BuildRoot mailing list archive at Nabble.com.

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

* [Buildroot] [PATCH] Add local directory for non-public packages
  2007-01-24 11:58     ` Daniel Laird
@ 2007-01-24 12:11       ` Ulf Samuelsson
  0 siblings, 0 replies; 12+ messages in thread
From: Ulf Samuelsson @ 2007-01-24 12:11 UTC (permalink / raw)
  To: buildroot

Daniel Laird wrote:
> Do you need to patch the top level Config.in to include
> local/Config.in? I think this is needed for the menuconfig stuff.

True.

>
> Cheers
> Dan

Best Regards,
Ulf Samuelsson
ulf at atmel.com
GSM:  +46 (706) 22 44 57
Tel:     +46  (8) 441 54 22
Fax:     +46 (8) 441 54 29
Mail: Box 2033  174 02 Sundbyberg
Visit: Kavalleriv?gen 24
          174 58 Sundbyberg'
Sweden

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

* [Buildroot] [PATCH] Add local directory for non-public packages
  2007-01-24 11:37   ` Daniel Laird
@ 2007-01-24 15:02     ` Ulf Samuelsson
  2007-01-24 20:29       ` Lars Munch
  2007-01-26 16:18     ` MikeW
  1 sibling, 1 reply; 12+ messages in thread
From: Ulf Samuelsson @ 2007-01-24 15:02 UTC (permalink / raw)
  To: buildroot

ons 2007-01-24 klockan 03:37 -0800 skrev Daniel Laird:
> 
> 
> Erik Andersen wrote:
> > 
> > On Tue Jan 23, 2007 at 11:58:40PM +0100, Ulf Samuelsson wrote:
> >> This adds a "buildroot/local" directory.
> >> The structure of "local" is the same as that of "packages"
> >> but the contents of "local" is never
> >> intended to be merged upstream.
> >> 
> >> It allows simple maintenance of those application
> >> since you only need to move the local directory from
> >> one version of buildroot to a newer.
> >> 
> >> Best Regards
> >> Ulf Samuelsson
> > 
> > See package/customize/ which tries to do the same thing.
> > 
> > If you think perhaps package/customize/ should be renamed
> > or better documented I'm open to that.  Adding a second
> > directory doing the same thing however seems unwise.
> > 
> > 
> >  -Erik
> > 
> > --
> > Erik B. Andersen             http://codepoet-consulting.com/
> > --This message was written using 73% post-consumer electrons--
> > _______________________________________________
> > buildroot mailing list
> > buildroot at uclibc.org
> > http://busybox.net/mailman/listinfo/buildroot
> > 
> > 
> I too would be very happy to see this patch go in.  It means buildroot can
> be a more generic build system allowing people to easily move from version
> to version.  But also allows people to use it to build their own packages
> that are not going to be ever part of the open-source world, or are so
> specialised as to be of limited use to the main buildroot community.
> Dan


Here is an updated patch which includes the local/Config.in.
As I explained in the previous Email, the "package/customize"
and "local" have two distinct functions.
If you move up "buildroot/package/customize" to "buildroot/customize"
and make the "Config.in" and "customize.mk" empty,
then that would be ok as well.



-------------- next part --------------
A non-text attachment was scrubbed...
Name: Makefile_local.patch
Type: text/x-patch
Size: 1771 bytes
Desc: not available
Url : http://busybox.net/lists/buildroot/attachments/20070124/25349706/attachment.bin 

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

* [Buildroot] [PATCH] Add local directory for non-public packages
  2007-01-24 15:02     ` Ulf Samuelsson
@ 2007-01-24 20:29       ` Lars Munch
  2007-01-24 21:35         ` Ulf Samuelsson
  0 siblings, 1 reply; 12+ messages in thread
From: Lars Munch @ 2007-01-24 20:29 UTC (permalink / raw)
  To: buildroot

Hi Ulf

On Wed, Jan 24, 2007 at 04:02:02PM +0100, Ulf Samuelsson wrote:
> Here is an updated patch which includes the local/Config.in.
> As I explained in the previous Email, the "package/customize"
> and "local" have two distinct functions.
> If you move up "buildroot/package/customize" to "buildroot/customize"
> and make the "Config.in" and "customize.mk" empty,
> then that would be ok as well.
[snip]

I really like this idea of having a directory for "local" packages, but I
think it would be nice to take it a step further and make it possible to
set the directory where the "local" packages are located. Then it will be
possible to have your "local" packages out of the buildroot tree.

Just my 2 cent.

-- Lars Munch

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

* [Buildroot] [PATCH] Add local directory for non-public packages
  2007-01-24 20:29       ` Lars Munch
@ 2007-01-24 21:35         ` Ulf Samuelsson
  2007-01-25 12:59           ` iks hefem
  0 siblings, 1 reply; 12+ messages in thread
From: Ulf Samuelsson @ 2007-01-24 21:35 UTC (permalink / raw)
  To: buildroot

Lars Munch wrote:
> Hi Ulf
> 
> On Wed, Jan 24, 2007 at 04:02:02PM +0100, Ulf Samuelsson wrote:
>> Here is an updated patch which includes the local/Config.in.
>> As I explained in the previous Email, the "package/customize"
>> and "local" have two distinct functions.
>> If you move up "buildroot/package/customize" to "buildroot/customize"
>> and make the "Config.in" and "customize.mk" empty,
>> then that would be ok as well.
> [snip]
> 
> I really like this idea of having a directory for "local" packages,
> but I think it would be nice to take it a step further and make it
> possible to set the directory where the "local" packages are located.
> Then it will be possible to have your "local" packages out of the
> buildroot tree. 
> 

Not a bad idea!


> Just my 2 cent.
> 
> -- Lars Munch

Best Regards,
Ulf Samuelsson

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

* [Buildroot] [PATCH] Add local directory for non-public packages
  2007-01-24 21:35         ` Ulf Samuelsson
@ 2007-01-25 12:59           ` iks hefem
  0 siblings, 0 replies; 12+ messages in thread
From: iks hefem @ 2007-01-25 12:59 UTC (permalink / raw)
  To: buildroot

+1 for me

> > Then it will be possible to have your "local" packages out of the
> > buildroot tree.
> >
>
> Not a bad idea!
>

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

* [Buildroot] [PATCH] Add local directory for non-public packages
  2007-01-24 11:37   ` Daniel Laird
  2007-01-24 15:02     ` Ulf Samuelsson
@ 2007-01-26 16:18     ` MikeW
  1 sibling, 0 replies; 12+ messages in thread
From: MikeW @ 2007-01-26 16:18 UTC (permalink / raw)
  To: buildroot

Daniel Laird <danieljlaird@...> writes:

> 
> I too would be very happy to see this patch go in.  It means buildroot can
> be a more generic build system allowing people to easily move from version
> to version.  But also allows people to use it to build their own packages
> that are not going to be ever part of the open-source world, or are so
> specialised as to be of limited use to the main buildroot community.
> Dan

All that's missing now is a menuconfig option to allow use
of an external toolchain ...

Cheers,
MikeW

PS hi to 225/810 guys Dan !

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

end of thread, other threads:[~2007-01-26 16:18 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-01-23 22:58 [Buildroot] [PATCH] Add local directory for non-public packages Ulf Samuelsson
2007-01-24  0:11 ` Erik Andersen
2007-01-24  1:13   ` Ulf Samuelsson
2007-01-24  9:17     ` Thomas Lundquist
2007-01-24 11:58     ` Daniel Laird
2007-01-24 12:11       ` Ulf Samuelsson
2007-01-24 11:37   ` Daniel Laird
2007-01-24 15:02     ` Ulf Samuelsson
2007-01-24 20:29       ` Lars Munch
2007-01-24 21:35         ` Ulf Samuelsson
2007-01-25 12:59           ` iks hefem
2007-01-26 16:18     ` MikeW

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