[Buildroot] [PATCH] package/rpi-userland: update to fix no-longer applying patch

[Maxime Hadjinlian]

On Sun, Feb 10, 2013 at 4:10 PM, Samuel Martin <s.martin49 at gmail.com> wrote:
> Hi all,
>
> 2013/2/10 Floris Bos <bos at je-eigen-domein.nl>:
>> On 02/10/2013 03:14 PM, Maxime Hadjinlian wrote:
>>>
>>> On Sun, Feb 10, 2013 at 2:24 PM, Floris Bos <bos at je-eigen-domein.nl>
>>> wrote:
>>>>
>>>> On 02/10/2013 12:58 PM, Yann E. MORIN wrote:
>>>>>
>>>>> Our patches do not apply as-is against the version of rpi-userland we
>>>>> package (weird...).
>>>>
>>>>
>>>> RPI_USERLAND_SITE = http://github.com/raspberrypi/userland/tarball/master
>>>>
>>>> Shouldn't that be something like this?
>>>>
>>>> RPI_USERLAND_SITE =
>>>> http://github.com/raspberrypi/userland/tarball/$(RPI_USERLAND_VERSION)
>>>>
>>> You are using the version variable only if the projects has tag, which
>>> this one hasn't.
>>> Then, you are forced to use sha1 version, then you give the branch
>>> where this sha1 can be found.
>>> As specified in the doc :
>>>
>>> http://buildroot.uclibc.org/downloads/manual/manual.html#github-download-url
>>
>>
>> Documentation is incorrect/outdated.
>>
>> Github only looks at the folder name, ignores the filename, and will
>> redirect you to
>> https://nodeload.github.com/raspberrypi/userland/legacy.tar.gz/master
>>
> [...]
>>
>> With
>> "RPI_USERLAND_SITE=http://github.com/raspberrypi/userland/tarball/$(RPI_USERLAND_VERSION)"
>> it does work:
> [...]
>
Indeed, thanks for finding this.

> Fair enough.
> Care to fix/update the doc?
>
> Also, rpi-userland is not the only package fetched from github...
> So patches for all other potentially broken packages will be welcome as well.
Yep, we should do a little grep magic here to find all the possible
incorrect packages.
>
> Regards,
>
> --
> Samuel
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot