diff mbox

[1,of,2,v3] manual: github commits should be specified in full

Message ID 63f3504eafe27efb9269.1383404247@argentina
State Accepted
Commit 1ae1f276066baf24ab85a4b05571c776b52cf16a
Headers show

Commit Message

Thomas De Schampheleire Nov. 2, 2013, 2:57 p.m. UTC
Signed-off-by: Thomas De Schampheleire <thomas.de.schampheleire@gmail.com>

---
v3: change tarball name in example (missed comment Samuel)
v2: remove parentheses around 'full' (comment Samuel)

 docs/manual/adding-packages-tips.txt |  7 +++----
 1 files changed, 3 insertions(+), 4 deletions(-)

Comments

Samuel Martin Nov. 2, 2013, 3:01 p.m. UTC | #1
Thomas,

2013/11/2 Thomas De Schampheleire <patrickdepinguin@gmail.com>

> Signed-off-by: Thomas De Schampheleire <thomas.de.schampheleire@gmail.com>
>
Acked-by: Samuel Martin <s.martin49@gmail.com>


> ---
> v3: change tarball name in example (missed comment Samuel)
> v2: remove parentheses around 'full' (comment Samuel)
>
>  docs/manual/adding-packages-tips.txt |  7 +++----
>  1 files changed, 3 insertions(+), 4 deletions(-)
>
> diff --git a/docs/manual/adding-packages-tips.txt
> b/docs/manual/adding-packages-tips.txt
> --- a/docs/manual/adding-packages-tips.txt
> +++ b/docs/manual/adding-packages-tips.txt
> @@ -44,14 +44,13 @@ However, it is possible to download tarb
>  on github.
>
>  ------------------------
> -FOO_VERSION = v1.0 # tag or (abbreviated) commit ID
> +FOO_VERSION = v1.0 # tag or full commit ID
>  FOO_SITE = http://github.com/<user>/<package>/tarball/$(FOO_VERSION)
>  ------------------------
>
>  .Notes
>  - The FOO_VERSION can either be a tag or a commit ID.
>  - The tarball name generated by github matches the default one from
> -  Buildroot (e.g.: +foo-1234567.tar.gz+),
> +  Buildroot (e.g.: +foo-f6fb6654af62045239caed5950bc6c7971965e60.tar.gz+),
>    so it is not necessary to specify it in the +.mk+ file.
> -- When using a commit ID as version, usually the first 7 characters of
> -  the SHA1 are enough.
> +- When using a commit ID as version, you should use the full 40 hex
> characters.
> _______________________________________________
> buildroot mailing list
> buildroot@busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
>

Regards,
diff mbox

Patch

diff --git a/docs/manual/adding-packages-tips.txt b/docs/manual/adding-packages-tips.txt
--- a/docs/manual/adding-packages-tips.txt
+++ b/docs/manual/adding-packages-tips.txt
@@ -44,14 +44,13 @@  However, it is possible to download tarb
 on github.
 
 ------------------------
-FOO_VERSION = v1.0 # tag or (abbreviated) commit ID
+FOO_VERSION = v1.0 # tag or full commit ID
 FOO_SITE = http://github.com/<user>/<package>/tarball/$(FOO_VERSION)
 ------------------------
 
 .Notes
 - The FOO_VERSION can either be a tag or a commit ID.
 - The tarball name generated by github matches the default one from
-  Buildroot (e.g.: +foo-1234567.tar.gz+),
+  Buildroot (e.g.: +foo-f6fb6654af62045239caed5950bc6c7971965e60.tar.gz+),
   so it is not necessary to specify it in the +.mk+ file.
-- When using a commit ID as version, usually the first 7 characters of
-  the SHA1 are enough.
+- When using a commit ID as version, you should use the full 40 hex characters.