Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Yann E. MORIN <yann.morin.1998@free.fr>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH] Update github helper URL.
Date: Thu, 13 Nov 2014 21:22:34 +0100	[thread overview]
Message-ID: <20141113202234.GD3641@free.fr> (raw)
In-Reply-To: <CAAXf6LWNoKjSPRrjpDROBHK6xvqC1cP6BsybRuMX64veGOTc7A@mail.gmail.com>

Thomas, Arnout, Patrick, All,

On 2014-11-13 21:06 +0100, Thomas De Schampheleire spake thusly:
> On Thu, Nov 13, 2014 at 8:58 PM, Arnout Vandecappelle <arnout@mind.be> wrote:
> > On 13/11/14 20:48, Patrick Williams wrote:
> >> Github changed the URL syntax from 'archive' back to 'tarball'.
> >>
> >> Signed-off-by: Patrick Williams <iawillia@us.ibm.com>
> >> ---
> >>  package/pkg-download.mk | 2 +-
> >>  1 file changed, 1 insertion(+), 1 deletion(-)
> >>
> >> diff --git a/package/pkg-download.mk b/package/pkg-download.mk
> >> index 4cfb913..03907f1 100644
> >> --- a/package/pkg-download.mk
> >> +++ b/package/pkg-download.mk
> >> @@ -56,7 +56,7 @@ notdomain=$(patsubst $(call domain,$(1),$(2))$(call domainseparator,$(2))%,%,$(c
> >>  domainseparator=$(if $(1),$(1),/)
> >>
> >>  # github(user,package,version): returns site of GitHub repository
> >> -github = https://github.com/$(1)/$(2)/archive/$(3)
> >> +github = https://github.com/$(1)/$(2)/tarball/$(3)
> >>
> >>  ################################################################################
> >>  # The DOWNLOAD_* helpers are in charge of getting a working copy
> >>
> >
> >  Yep, that seems to work! So Yann, stop working :-)

Meh...

> >  I haven't tested if it's exactly the same tarball. It redirects to a URL
> > containing legacy.tar.gz instead of tar.gz, so who knows...
> 
> I can't follow all this madness.
> 
> >>> a10disp a899cc6dcf909360aac00482ede5369f3656f226 Downloading
> --2014-11-13 20:57:54--
> https://github.com/hglm/a10disp/archive/a899cc6dcf909360aac00482ede5369f3656f226/a10disp-a899cc6dcf909360aac00482ede5369f3656f226.tar.gz
> Resolving github.com (github.com)... 192.30.252.129
> Connecting to github.com (github.com)|192.30.252.129|:443... connected.
> HTTP request sent, awaiting response... 302 Found
> Location: https://codeload.github.com/hglm/a10disp/tar.gz/a899cc6dcf909360aac00482ede5369f3656f226
> [following]
> --2014-11-13 20:57:54--
> https://codeload.github.com/hglm/a10disp/tar.gz/a899cc6dcf909360aac00482ede5369f3656f226
> Resolving codeload.github.com (codeload.github.com)... 192.30.252.146
> Connecting to codeload.github.com
> (codeload.github.com)|192.30.252.146|:443... connected.
> HTTP request sent, awaiting response... 200 OK
> Length: unspecified [application/x-gzip]
> 
> Here the original URL with 'archive' still works. So what's the fuzz about?

DAng... It works again. It surely was broken yesterday: all downlaod
would fallback to using http://sources.buildroot.net

See also the mail from Peter stating it did not work for him, too:
    http://lists.busybox.net/pipermail/buildroot/2014-November/111953.html

So, they "fixed" it over-night. Sigh, that's not reliable.

OK, so we can probably post-pone the resolution for post-release.

Regards,
Yann E. MORIN.

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'

  parent reply	other threads:[~2014-11-13 20:22 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-11-13 19:48 [Buildroot] [PATCH] Update github helper URL Patrick Williams
2014-11-13 19:58 ` Arnout Vandecappelle
2014-11-13 20:05   ` Arnout Vandecappelle
2014-11-13 20:06   ` Thomas De Schampheleire
2014-11-13 20:15     ` Arnout Vandecappelle
2014-11-13 20:20     ` Samuel Martin
2014-11-13 20:22     ` Yann E. MORIN [this message]
2014-11-14 15:13       ` Peter Korsgaard
2014-11-14 15:25         ` Samuel Martin
2014-11-14 17:51           ` Peter Korsgaard

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20141113202234.GD3641@free.fr \
    --to=yann.morin.1998@free.fr \
    --cc=buildroot@busybox.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox