mailarchive of the ptxdist mailing list
 help / color / mirror / Atom feed
* [ptxdist] [PATCH 1/2] barebox: bump default barebox version
@ 2016-09-13  8:04 Uwe Kleine-König
  2016-09-13  8:04 ` [ptxdist] [PATCH 2/2] barebox: update the default barebox config name analogous to KERNEL_CONFIG Uwe Kleine-König
  0 siblings, 1 reply; 5+ messages in thread
From: Uwe Kleine-König @ 2016-09-13  8:04 UTC (permalink / raw)
  To: ptxdist

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
---
 platforms/barebox.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/platforms/barebox.in b/platforms/barebox.in
index a9270bb5465e..01dee92b5526 100644
--- a/platforms/barebox.in
+++ b/platforms/barebox.in
@@ -23,7 +23,7 @@ if BAREBOX
 config BAREBOX_VERSION
 	prompt "barebox version"
 	string
-	default "2014.01.0"
+	default "2016.08.0"
 
 config BAREBOX_MD5
 	prompt "barebox source md5sum"
-- 
2.8.1


_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

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

* [ptxdist] [PATCH 2/2] barebox: update the default barebox config name analogous to KERNEL_CONFIG
  2016-09-13  8:04 [ptxdist] [PATCH 1/2] barebox: bump default barebox version Uwe Kleine-König
@ 2016-09-13  8:04 ` Uwe Kleine-König
  2016-09-13  9:21   ` Michael Olbrich
  0 siblings, 1 reply; 5+ messages in thread
From: Uwe Kleine-König @ 2016-09-13  8:04 UTC (permalink / raw)
  To: ptxdist

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
---
 platforms/barebox.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/platforms/barebox.in b/platforms/barebox.in
index 01dee92b5526..6cae458e7085 100644
--- a/platforms/barebox.in
+++ b/platforms/barebox.in
@@ -32,7 +32,7 @@ config BAREBOX_MD5
 config BAREBOX_CONFIG
 	prompt "config file"
 	string
-	default "barebox.config"
+	default "bareboxconfig-${PTXCONF_BAREBOX_VERSION}"
 	help
 	  This entry specifies the .config file used to compile
 	  barebox.
-- 
2.8.1


_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

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

* Re: [ptxdist] [PATCH 2/2] barebox: update the default barebox config name analogous to KERNEL_CONFIG
  2016-09-13  8:04 ` [ptxdist] [PATCH 2/2] barebox: update the default barebox config name analogous to KERNEL_CONFIG Uwe Kleine-König
@ 2016-09-13  9:21   ` Michael Olbrich
  2016-09-13  9:54     ` Uwe Kleine-König
  0 siblings, 1 reply; 5+ messages in thread
From: Michael Olbrich @ 2016-09-13  9:21 UTC (permalink / raw)
  To: ptxdist

I prefer a filename without the version. That way I can update the version
and just do a 'ptxdist oldconfig barebox'.
I think we should change the default for KERNEL_CONFIG to match barebox
instead.

Michael

On Tue, Sep 13, 2016 at 10:04:22AM +0200, Uwe Kleine-König wrote:
> Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
> ---
>  platforms/barebox.in | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/platforms/barebox.in b/platforms/barebox.in
> index 01dee92b5526..6cae458e7085 100644
> --- a/platforms/barebox.in
> +++ b/platforms/barebox.in
> @@ -32,7 +32,7 @@ config BAREBOX_MD5
>  config BAREBOX_CONFIG
>  	prompt "config file"
>  	string
> -	default "barebox.config"
> +	default "bareboxconfig-${PTXCONF_BAREBOX_VERSION}"
>  	help
>  	  This entry specifies the .config file used to compile
>  	  barebox.
> -- 
> 2.8.1
> 
> 
> _______________________________________________
> ptxdist mailing list
> ptxdist@pengutronix.de

-- 
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 |

_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

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

* Re: [ptxdist] [PATCH 2/2] barebox: update the default barebox config name analogous to KERNEL_CONFIG
  2016-09-13  9:21   ` Michael Olbrich
@ 2016-09-13  9:54     ` Uwe Kleine-König
  2016-09-13 10:54       ` Michael Olbrich
  0 siblings, 1 reply; 5+ messages in thread
From: Uwe Kleine-König @ 2016-09-13  9:54 UTC (permalink / raw)
  To: ptxdist

Hello,

On Tue, Sep 13, 2016 at 11:21:27AM +0200, Michael Olbrich wrote:
> I prefer a filename without the version.

I like it with version because this allows to keep the old config
(similar to the old patch stack, that is versioned, too) for a while
before/after updating kernel or barebox.

Best regards
Uwe

-- 
Pengutronix e.K.                           | Uwe Kleine-König            |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |

_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

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

* Re: [ptxdist] [PATCH 2/2] barebox: update the default barebox config name analogous to KERNEL_CONFIG
  2016-09-13  9:54     ` Uwe Kleine-König
@ 2016-09-13 10:54       ` Michael Olbrich
  0 siblings, 0 replies; 5+ messages in thread
From: Michael Olbrich @ 2016-09-13 10:54 UTC (permalink / raw)
  To: ptxdist

On Tue, Sep 13, 2016 at 11:54:36AM +0200, Uwe Kleine-König wrote:
> On Tue, Sep 13, 2016 at 11:21:27AM +0200, Michael Olbrich wrote:
> > I prefer a filename without the version.
> 
> I like it with version because this allows to keep the old config
> (similar to the old patch stack, that is versioned, too) for a while
> before/after updating kernel or barebox.

I think keeping the old stuff is really annoying because 'for a while'
means 'forever' in most cases. The end result is, that every kernel and
barebox version ever used in a BSP has a patch stack and config nobody
knows which of those are still needed.

And it's still an option anyways. We're talking about the default value
here and most people don't switch back and forth between versions.

Michael

-- 
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 |

_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

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

end of thread, other threads:[~2016-09-13 10:54 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-09-13  8:04 [ptxdist] [PATCH 1/2] barebox: bump default barebox version Uwe Kleine-König
2016-09-13  8:04 ` [ptxdist] [PATCH 2/2] barebox: update the default barebox config name analogous to KERNEL_CONFIG Uwe Kleine-König
2016-09-13  9:21   ` Michael Olbrich
2016-09-13  9:54     ` Uwe Kleine-König
2016-09-13 10:54       ` Michael Olbrich

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