All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH] Add documentation about docker registry
@ 2022-09-04 13:46 Thierry GUIBERT via buildroot
  2023-04-16 12:36 ` Yann E. MORIN
  0 siblings, 1 reply; 2+ messages in thread
From: Thierry GUIBERT via buildroot @ 2022-09-04 13:46 UTC (permalink / raw)
  To: yann.morin.1998, thomas.petazzoni, buildroot; +Cc: Thierry GUIBERT

Signed-off-by: Thierry GUIBERT <thierry.guibert@croix-rouge.fr>
---
 docs/manual/adding-board-support.txt | 19 ++++++++++++++++---
 1 file changed, 16 insertions(+), 3 deletions(-)

diff --git a/docs/manual/adding-board-support.txt b/docs/manual/adding-board-support.txt
index cd90a5871f..f68e192468 100644
--- a/docs/manual/adding-board-support.txt
+++ b/docs/manual/adding-board-support.txt
@@ -48,9 +48,22 @@ and configurations in these directories, and reference them from the main
 Buildroot configuration. Refer to xref:customize[] for more details.
 
 Before submitting patches for new boards it is recommended to test it by
-building it using latest gitlab-CI docker container. To do this use
-utils/docker-run script and inside it issue these commands:
+building it using latest gitlab-CI docker container. 
+To do this use utils/docker-run script and inside it issue these commands:
 --------------------
- $ make +<boardname>_defconfig+
+ $ make <boardname>_defconfig
  $ make
 --------------------
+
+By defaut, buildroot developpers use our gitlab.com registry and it should
+be convenient for most usages. 
+If you still want to build your own docker images :
+--------------------
+ $ DOCKER_REGISTRY=registry.gitlab.com
+ $ IMAGE_NAME=buildroot.org/buildroot/base
+ $ IMAGE_TAG=YYYYMMDD.HHMM
+ $ docker build  --pull --no-cache \
+	-t ${DOCKER_REGISTRY}/${IMAGE_NAME}:${IMAGE_TAG} support/docker
+ $ docker push ${DOCKER_REGISTRY}/${IMAGE_NAME}:${IMAGE_TAG}
+--------------------
+
-- 
2.37.2

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

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

* Re: [Buildroot] [PATCH] Add documentation about docker registry
  2022-09-04 13:46 [Buildroot] [PATCH] Add documentation about docker registry Thierry GUIBERT via buildroot
@ 2023-04-16 12:36 ` Yann E. MORIN
  0 siblings, 0 replies; 2+ messages in thread
From: Yann E. MORIN @ 2023-04-16 12:36 UTC (permalink / raw)
  To: Thierry GUIBERT; +Cc: thomas.petazzoni, buildroot

Thierry, All,

On 2022-09-04 15:46 +0200, Thierry GUIBERT via buildroot spake thusly:
> Signed-off-by: Thierry GUIBERT <thierry.guibert@croix-rouge.fr>
> ---
>  docs/manual/adding-board-support.txt | 19 ++++++++++++++++---
>  1 file changed, 16 insertions(+), 3 deletions(-)
> 
> diff --git a/docs/manual/adding-board-support.txt b/docs/manual/adding-board-support.txt
> index cd90a5871f..f68e192468 100644
> --- a/docs/manual/adding-board-support.txt
> +++ b/docs/manual/adding-board-support.txt
> @@ -48,9 +48,22 @@ and configurations in these directories, and reference them from the main
>  Buildroot configuration. Refer to xref:customize[] for more details.
>  
>  Before submitting patches for new boards it is recommended to test it by
> -building it using latest gitlab-CI docker container. To do this use
> -utils/docker-run script and inside it issue these commands:
> +building it using latest gitlab-CI docker container. 
> +To do this use utils/docker-run script and inside it issue these commands:
>  --------------------
> - $ make +<boardname>_defconfig+
> + $ make <boardname>_defconfig

Both spurious changes in this commit.

Still, there was clearly incorrect formatting, so I've fix that in a
followup commit, thanks.

>   $ make
>  --------------------
> +
> +By defaut, buildroot developpers use our gitlab.com registry and it should

s/defaut/default/
s/developpers/developers/

> +be convenient for most usages. 
> +If you still want to build your own docker images :
> +--------------------
> + $ DOCKER_REGISTRY=registry.gitlab.com
> + $ IMAGE_NAME=buildroot.org/buildroot/base
> + $ IMAGE_TAG=YYYYMMDD.HHMM
> + $ docker build  --pull --no-cache \
> +	-t ${DOCKER_REGISTRY}/${IMAGE_NAME}:${IMAGE_TAG} support/docker
> + $ docker push ${DOCKER_REGISTRY}/${IMAGE_NAME}:${IMAGE_TAG}
> +--------------------

I don't think it is very interesting to document how to build a local
image and push it to a registry; that's the basics of docker.

Instead, I've reworded to instruct people where to look for the offical
images, and that they can use it as the FROM directive in their
Dockerfile.

Applied to master with the above changes, thanks.

Regards,
Yann E. MORIN.

> +
> -- 
> 2.37.2
> 
> _______________________________________________
> buildroot mailing list
> buildroot@buildroot.org
> https://lists.buildroot.org/mailman/listinfo/buildroot

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 561 099 427 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

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

end of thread, other threads:[~2023-04-16 12:36 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-09-04 13:46 [Buildroot] [PATCH] Add documentation about docker registry Thierry GUIBERT via buildroot
2023-04-16 12:36 ` Yann E. MORIN

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.