All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCHv3] tar: move package to 'system tools'
@ 2015-09-01 16:05 Francesco Nwokeka
  2015-09-01 16:08 ` Vicente Olivert Riera
  2015-09-01 20:29 ` Peter Korsgaard
  0 siblings, 2 replies; 4+ messages in thread
From: Francesco Nwokeka @ 2015-09-01 16:05 UTC (permalink / raw)
  To: buildroot

Package was previously in the 'Development' section
---
 package/Config.in | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/package/Config.in b/package/Config.in
index 6ba21a1..58b7380 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -140,9 +140,6 @@ if BR2_PACKAGE_BUSYBOX_SHOW_OTHERS
 endif
 	source "package/sstrip/Config.in"
 	source "package/subversion/Config.in"
-if BR2_PACKAGE_BUSYBOX_SHOW_OTHERS
-	source "package/tar/Config.in"
-endif
 	source "package/tree/Config.in"
 	source "package/yasm/Config.in"
 endmenu
@@ -1508,6 +1505,7 @@ endif
 	source "package/systemd/Config.in"
 if BR2_PACKAGE_BUSYBOX_SHOW_OTHERS
 	source "package/sysvinit/Config.in"
+	source "package/tar/Config.in"
 endif
 	source "package/util-linux/Config.in"
 endmenu
-- 
2.5.0

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

* [Buildroot] [PATCHv3] tar: move package to 'system tools'
  2015-09-01 16:05 [Buildroot] [PATCHv3] tar: move package to 'system tools' Francesco Nwokeka
@ 2015-09-01 16:08 ` Vicente Olivert Riera
  2015-09-01 16:15   ` Francesco Nwokeka
  2015-09-01 20:29 ` Peter Korsgaard
  1 sibling, 1 reply; 4+ messages in thread
From: Vicente Olivert Riera @ 2015-09-01 16:08 UTC (permalink / raw)
  To: buildroot

Dear Francesco Nwokeka,

On 09/01/2015 05:05 PM, Francesco Nwokeka wrote:
> Package was previously in the 'Development' section
> ---
>  package/Config.in | 4 +---
>  1 file changed, 1 insertion(+), 3 deletions(-)
> 
> diff --git a/package/Config.in b/package/Config.in
> index 6ba21a1..58b7380 100644
> --- a/package/Config.in
> +++ b/package/Config.in
> @@ -140,9 +140,6 @@ if BR2_PACKAGE_BUSYBOX_SHOW_OTHERS
>  endif
>  	source "package/sstrip/Config.in"
>  	source "package/subversion/Config.in"
> -if BR2_PACKAGE_BUSYBOX_SHOW_OTHERS
> -	source "package/tar/Config.in"
> -endif
>  	source "package/tree/Config.in"
>  	source "package/yasm/Config.in"
>  endmenu
> @@ -1508,6 +1505,7 @@ endif
>  	source "package/systemd/Config.in"
>  if BR2_PACKAGE_BUSYBOX_SHOW_OTHERS
>  	source "package/sysvinit/Config.in"
> +	source "package/tar/Config.in"
>  endif
>  	source "package/util-linux/Config.in"
>  endmenu
> 

Reviewed-by: Vicente Olivert Riera <Vincent.Riera@imgtec.com>
Tested-by: Vicente Olivert Riera <Vincent.Riera@imgtec.com>

Could you please mark your two previous patches as superseded in
Patchwork? You will need to register if you don't have a Patchwork account.

Regards,

Vincent.

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

* [Buildroot] [PATCHv3] tar: move package to 'system tools'
  2015-09-01 16:08 ` Vicente Olivert Riera
@ 2015-09-01 16:15   ` Francesco Nwokeka
  0 siblings, 0 replies; 4+ messages in thread
From: Francesco Nwokeka @ 2015-09-01 16:15 UTC (permalink / raw)
  To: buildroot

On Tuesday 01 September 2015 17:08:35 Vicente Olivert Riera wrote:
> Dear Francesco Nwokeka,
> 
> On 09/01/2015 05:05 PM, Francesco Nwokeka wrote:
> > Package was previously in the 'Development' section
> > ---
> > 
> >  package/Config.in | 4 +---
> >  1 file changed, 1 insertion(+), 3 deletions(-)
> > 
> > diff --git a/package/Config.in b/package/Config.in
> > index 6ba21a1..58b7380 100644
> > --- a/package/Config.in
> > +++ b/package/Config.in
> > @@ -140,9 +140,6 @@ if BR2_PACKAGE_BUSYBOX_SHOW_OTHERS
> > 
> >  endif
> >  
> >  	source "package/sstrip/Config.in"
> >  	source "package/subversion/Config.in"
> > 
> > -if BR2_PACKAGE_BUSYBOX_SHOW_OTHERS
> > -	source "package/tar/Config.in"
> > -endif
> > 
> >  	source "package/tree/Config.in"
> >  	source "package/yasm/Config.in"
> >  
> >  endmenu
> > 
> > @@ -1508,6 +1505,7 @@ endif
> > 
> >  	source "package/systemd/Config.in"
> >  
> >  if BR2_PACKAGE_BUSYBOX_SHOW_OTHERS
> >  
> >  	source "package/sysvinit/Config.in"
> > 
> > +	source "package/tar/Config.in"
> > 
> >  endif
> >  
> >  	source "package/util-linux/Config.in"
> >  
> >  endmenu
> 
> Reviewed-by: Vicente Olivert Riera <Vincent.Riera@imgtec.com>
> Tested-by: Vicente Olivert Riera <Vincent.Riera@imgtec.com>
> 
> Could you please mark your two previous patches as superseded in
> Patchwork? You will need to register if you don't have a Patchwork account.

Done, previous patches have been marked as "superseded".


> Regards,
> 
> Vincent.

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

* [Buildroot] [PATCHv3] tar: move package to 'system tools'
  2015-09-01 16:05 [Buildroot] [PATCHv3] tar: move package to 'system tools' Francesco Nwokeka
  2015-09-01 16:08 ` Vicente Olivert Riera
@ 2015-09-01 20:29 ` Peter Korsgaard
  1 sibling, 0 replies; 4+ messages in thread
From: Peter Korsgaard @ 2015-09-01 20:29 UTC (permalink / raw)
  To: buildroot

>>>>> "Francesco" == Francesco Nwokeka <francesco.nwokeka@gmail.com> writes:

 > Package was previously in the 'Development' section

You forgot to add your signed-off-by, but as you already added it in the
earlier version I've added it locally and committed, thanks.

-- 
Bye, Peter Korsgaard

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

end of thread, other threads:[~2015-09-01 20:29 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-09-01 16:05 [Buildroot] [PATCHv3] tar: move package to 'system tools' Francesco Nwokeka
2015-09-01 16:08 ` Vicente Olivert Riera
2015-09-01 16:15   ` Francesco Nwokeka
2015-09-01 20:29 ` Peter Korsgaard

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.