[PATCH] babeltrace: switch source URI

[PATCH] babeltrace: switch source URI

[Trevor Woerner]

It looks like http://www.efficios.com/files/babeltrace is down, therefore
switch to https://github.com/efficios/babeltrace/archive.

The md5sum/sha256sum updates are (I'm guessing) a result of switching from
.tar.bz2 to .tar.gz.

Signed-off-by: Trevor Woerner <twoerner@gmail.com>
---
 meta/recipes-kernel/lttng/babeltrace_1.5.5.bb | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/meta/recipes-kernel/lttng/babeltrace_1.5.5.bb b/meta/recipes-kernel/lttng/babeltrace_1.5.5.bb
index 92d86b92e8..2098264c67 100644
--- a/meta/recipes-kernel/lttng/babeltrace_1.5.5.bb
+++ b/meta/recipes-kernel/lttng/babeltrace_1.5.5.bb
@@ -10,10 +10,9 @@ DEPENDS = "glib-2.0 util-linux popt bison-native flex-native"
 
 inherit autotools pkgconfig
 
-SRC_URI = "http://www.efficios.com/files/babeltrace/babeltrace-${PV}.tar.bz2 \
-"
+SRC_URI = "https://github.com/efficios/babeltrace/archive/v${PV}.tar.gz"
 
 EXTRA_OECONF = "--disable-debug-info"
 
-SRC_URI[md5sum] = "7decafd46093e4e47c14e7307d498986"
-SRC_URI[sha256sum] = "409146789b4a6b81f6a275fcad932a030743d444f6de5bd4a34aaf17aa72e8ac"
+SRC_URI[md5sum] = "aa70da8947b2104b976302d8f65ddee5"
+SRC_URI[sha256sum] = "ce11a5725e7f4838b4a1cc20966eca77bf77a80c43a072062708d7eac7ebf272"
-- 
2.17.0.rc2.3.gc2a499e6c

Re: [PATCH] babeltrace: switch source URI

[Martin Jansa]

[-- Attachment #1: Type: text/plain, Size: 2067 bytes --]

There is already a fix for this, see:
http://lists.openembedded.org/pipermail/openembedded-core/2018-May/150423.html

Using github archives is even worse as they are regenerated from time to
time. Also when you really want to use v${PV}.tar.gz archive, then
use downloadfilename parameter to fetch it with some more meaningful
filename and to prevent possible conflicts with other badly named tarballs.

On Mon, May 7, 2018 at 5:27 PM, Trevor Woerner <twoerner@gmail.com> wrote:

> It looks like http://www.efficios.com/files/babeltrace is down, therefore
> switch to https://github.com/efficios/babeltrace/archive.
>
> The md5sum/sha256sum updates are (I'm guessing) a result of switching from
> .tar.bz2 to .tar.gz.
>
> Signed-off-by: Trevor Woerner <twoerner@gmail.com>
> ---
>  meta/recipes-kernel/lttng/babeltrace_1.5.5.bb | 7 +++----
>  1 file changed, 3 insertions(+), 4 deletions(-)
>
> diff --git a/meta/recipes-kernel/lttng/babeltrace_1.5.5.bb
> b/meta/recipes-kernel/lttng/babeltrace_1.5.5.bb
> index 92d86b92e8..2098264c67 100644
> --- a/meta/recipes-kernel/lttng/babeltrace_1.5.5.bb
> +++ b/meta/recipes-kernel/lttng/babeltrace_1.5.5.bb
> @@ -10,10 +10,9 @@ DEPENDS = "glib-2.0 util-linux popt bison-native
> flex-native"
>
>  inherit autotools pkgconfig
>
> -SRC_URI = "http://www.efficios.com/files/babeltrace/babeltrace-${
> PV}.tar.bz2 \
> -"
> +SRC_URI = "https://github.com/efficios/babeltrace/archive/v${PV}.tar.gz"
>
>  EXTRA_OECONF = "--disable-debug-info"
>
> -SRC_URI[md5sum] = "7decafd46093e4e47c14e7307d498986"
> -SRC_URI[sha256sum] = "409146789b4a6b81f6a275fcad932a
> 030743d444f6de5bd4a34aaf17aa72e8ac"
> +SRC_URI[md5sum] = "aa70da8947b2104b976302d8f65ddee5"
> +SRC_URI[sha256sum] = "ce11a5725e7f4838b4a1cc20966eca
> 77bf77a80c43a072062708d7eac7ebf272"
> --
> 2.17.0.rc2.3.gc2a499e6c
>
> --
> _______________________________________________
> Openembedded-core mailing list
> Openembedded-core@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-core
>

[-- Attachment #2: Type: text/html, Size: 3808 bytes --]

Re: [PATCH] babeltrace: switch source URI

[Trevor Woerner]

Hi Martin,

On Mon 2018-05-07 @ 05:33:26 PM, Martin Jansa wrote:
> There is already a fix for this, see:
> http://lists.openembedded.org/pipermail/openembedded-core/2018-May/150423.html

Oops, I should have checked, sorry!

> Using github archives is even worse as they are regenerated from time to
> time.

Crazy! But good to know.

> Also when you really want to use v${PV}.tar.gz archive, then
> use downloadfilename parameter to fetch it with some more meaningful
> filename and to prevent possible conflicts with other badly named tarballs.

Oh, I see, in one's own DL_DIR. So (not that this patch is relevant anymore)
instead of:

	-SRC_URI = "http://www.efficios.com/files/babeltrace/babeltrace-${PV}.tar.bz2 \
	-"
	+SRC_URI = "https://github.com/efficios/babeltrace/archive/v${PV}.tar.gz"

better:

	-SRC_URI = "http://www.efficios.com/files/babeltrace/babeltrace-${PV}.tar.bz2 \
	-"
	+SRC_URI = "https://github.com/efficios/babeltrace/archive/v${PV}.tar.gz;downloadfilename=babeltrace-v${PV}.tar.gz"

In this case, would the checksums need a tweak too, or do they know to look
for the downloadfilename file name?

Re: [PATCH] babeltrace: switch source URI

[Martin Jansa]

[-- Attachment #1: Type: text/plain, Size: 1440 bytes --]

Checksums don't need any special treatment with downloadfilename parameter.

On Mon, May 7, 2018 at 5:48 PM, Trevor Woerner <twoerner@gmail.com> wrote:

> Hi Martin,
>
> On Mon 2018-05-07 @ 05:33:26 PM, Martin Jansa wrote:
> > There is already a fix for this, see:
> > http://lists.openembedded.org/pipermail/openembedded-core/
> 2018-May/150423.html
>
> Oops, I should have checked, sorry!
>
> > Using github archives is even worse as they are regenerated from time to
> > time.
>
> Crazy! But good to know.
>
> > Also when you really want to use v${PV}.tar.gz archive, then
> > use downloadfilename parameter to fetch it with some more meaningful
> > filename and to prevent possible conflicts with other badly named
> tarballs.
>
> Oh, I see, in one's own DL_DIR. So (not that this patch is relevant
> anymore)
> instead of:
>
>         -SRC_URI = "http://www.efficios.com/files/babeltrace/babeltrace-${
> PV}.tar.bz2 \
>         -"
>         +SRC_URI = "https://github.com/efficios/
> babeltrace/archive/v${PV}.tar.gz"
>
> better:
>
>         -SRC_URI = "http://www.efficios.com/files/babeltrace/babeltrace-${
> PV}.tar.bz2 \
>         -"
>         +SRC_URI = "https://github.com/efficios/
> babeltrace/archive/v${PV}.tar.gz;downloadfilename=babeltrace-v${PV}.tar.gz
> "
>
> In this case, would the checksums need a tweak too, or do they know to look
> for the downloadfilename file name?
>

[-- Attachment #2: Type: text/html, Size: 2638 bytes --]

Re: [PATCH] babeltrace: switch source URI

[Burton, Ross]

[-- Attachment #1: Type: text/plain, Size: 648 bytes --]

On 7 May 2018 at 16:33, Martin Jansa <martin.jansa@gmail.com> wrote:

> There is already a fix for this, see:
> http://lists.openembedded.org/pipermail/openembedded-core/
> 2018-May/150423.html
>
> Using github archives is even worse as they are regenerated from time to
> time. Also when you really want to use v${PV}.tar.gz archive, then
> use downloadfilename parameter to fetch it with some more meaningful
> filename and to prevent possible conflicts with other badly named tarballs.
>

http://lists.openembedded.org/pipermail/openembedded-devel/2017-September/114916.html
has a bit more background to this problem fwiw.

Ross

[-- Attachment #2: Type: text/html, Size: 1296 bytes --]