Message ID | 1429583936-13335-2-git-send-email-fabio.porcedda@gmail.com |
---|---|
State | Accepted |
Commit | ccbf35e1d43502641913b07be290c6dc752b5614 |
Headers | show |
>>>>> "Fabio" == Fabio Porcedda <fabio.porcedda@gmail.com> writes: > Signed-off-by: Fabio Porcedda <fabio.porcedda@gmail.com> > --- > package/cache-calibrator/cache-calibrator.hash | 2 ++ > 1 file changed, 2 insertions(+) > create mode 100644 package/cache-calibrator/cache-calibrator.hash > diff --git a/package/cache-calibrator/cache-calibrator.hash b/package/cache-calibrator/cache-calibrator.hash > new file mode 100644 > index 0000000..df61d17 > --- /dev/null > +++ b/package/cache-calibrator/cache-calibrator.hash > @@ -0,0 +1,2 @@ > +# Locally calculated > +sha256 2018ed8fa733155d44ceb1c0066c5cf8df7771cdf7cfca0a07b8dd9bebd9c221 calibrator.c This will cause issues when/if that file ever gets updated - But then atleast we know and can update the hash. Committed, thanks.
Peter, All, On 2015-04-21 09:07 +0200, Peter Korsgaard spake thusly: > >>>>> "Fabio" == Fabio Porcedda <fabio.porcedda@gmail.com> writes: > > > Signed-off-by: Fabio Porcedda <fabio.porcedda@gmail.com> > > --- > > package/cache-calibrator/cache-calibrator.hash | 2 ++ > > 1 file changed, 2 insertions(+) > > create mode 100644 package/cache-calibrator/cache-calibrator.hash > > > diff --git a/package/cache-calibrator/cache-calibrator.hash b/package/cache-calibrator/cache-calibrator.hash > > new file mode 100644 > > index 0000000..df61d17 > > --- /dev/null > > +++ b/package/cache-calibrator/cache-calibrator.hash > > @@ -0,0 +1,2 @@ > > +# Locally calculated > > +sha256 2018ed8fa733155d44ceb1c0066c5cf8df7771cdf7cfca0a07b8dd9bebd9c221 calibrator.c > > This will cause issues when/if that file ever gets updated - But then > atleast we know and can update the hash. It also means that Buildroot 2015.05 will stop working at some point when this file is updated. The problem is not much about Buildroot master , which is easily fixable, but about our releases, that get broken as soon as upstream changes that file. I am not really happy with either solutions: either add a hash and break releases in the future, or not add a hash file, and miss updates, especially in the autobuilders (even though they do remove downloads from time to time. One case for the adding the hash, too, is that it will tell people that their local copy in their local cache (e.g. my ~/src/ ) is out-of-date. Still, I'm not convinced we want that hash... Do we have other such packages (I think I read there was another version-less package for now)? Regards, Yann E. MORIN. > Committed, thanks. > > -- > Bye, Peter Korsgaard > _______________________________________________ > buildroot mailing list > buildroot@busybox.net > http://lists.busybox.net/mailman/listinfo/buildroot
Dear Yann E. MORIN, On Tue, 21 Apr 2015 19:22:46 +0200, Yann E. MORIN wrote: > Do we have other such packages (I think I read there was another > version-less package for now)? For makedevs, we simply bundle the source code in Buildroot. But it's more a core tool, that we have improved for our own purposes. calibrator.c is only 38KB, so we could integrate it in package/cache-calibrator/ directly, but it's not very nice either. Thoas
>>>>> "Yann" == Yann E MORIN <yann.morin.1998@free.fr> writes: Hi, >> This will cause issues when/if that file ever gets updated - But then >> atleast we know and can update the hash. > It also means that Buildroot 2015.05 will stop working at some point > when this file is updated. The problem is not much about Buildroot > master , which is easily fixable, but about our releases, that get > broken as soon as upstream changes that file. > I am not really happy with either solutions: either add a hash and break > releases in the future, or not add a hash file, and miss updates, > especially in the autobuilders (even though they do remove downloads > from time to time. True - I don't really think there's any good solution to this (besides maybe convincing upstream to version their releases). > One case for the adding the hash, too, is that it will tell people that > their local copy in their local cache (e.g. my ~/src/ ) is out-of-date. > Still, I'm not convinced we want that hash... Maybe not. Perhaps the breakage from wrong hash is worse than the potential breakage from newer versions. Also notice that the download support falls back to our local copy on sources.buildroot.net if the hash doesn't match. > Do we have other such packages (I think I read there was another > version-less package for now)? Yes, we do: git grep 'SOURCE =.*\.c$' package/cache-calibrator/cache-calibrator.mk:CACHE_CALIBRATOR_SOURCE = calibrator.c package/devmem2/devmem2.mk:DEVMEM2_SOURCE = devmem2.c package/whetstone/whetstone.mk:WHETSTONE_SOURCE = whetstone.c
On Tue, Apr 21, 2015 at 10:43 PM, Peter Korsgaard <peter@korsgaard.com> wrote: >>>>>> "Yann" == Yann E MORIN <yann.morin.1998@free.fr> writes: > > Hi, > >>> This will cause issues when/if that file ever gets updated - But then > >> atleast we know and can update the hash. > > > It also means that Buildroot 2015.05 will stop working at some point > > when this file is updated. The problem is not much about Buildroot > > master , which is easily fixable, but about our releases, that get > > broken as soon as upstream changes that file. > > > I am not really happy with either solutions: either add a hash and break > > releases in the future, or not add a hash file, and miss updates, > > especially in the autobuilders (even though they do remove downloads > > from time to time. > > True - I don't really think there's any good solution to this (besides > maybe convincing upstream to version their releases). > > > One case for the adding the hash, too, is that it will tell people that > > their local copy in their local cache (e.g. my ~/src/ ) is out-of-date. > > > Still, I'm not convinced we want that hash... > > Maybe not. Perhaps the breakage from wrong hash is worse than the > potential breakage from newer versions. Also notice that the download > support falls back to our local copy on sources.buildroot.net if the > hash doesn't match. > > > Do we have other such packages (I think I read there was another > > version-less package for now)? > > Yes, we do: > > git grep 'SOURCE =.*\.c$' > package/cache-calibrator/cache-calibrator.mk:CACHE_CALIBRATOR_SOURCE = calibrator.c > package/devmem2/devmem2.mk:DEVMEM2_SOURCE = devmem2.c > package/whetstone/whetstone.mk:WHETSTONE_SOURCE = whetstone.c > If the upstream is not willing to version those files what about creating a github repositories for them? Also to be able to check the hash on github files we just need to attach the binary for each release. BR
diff --git a/package/cache-calibrator/cache-calibrator.hash b/package/cache-calibrator/cache-calibrator.hash new file mode 100644 index 0000000..df61d17 --- /dev/null +++ b/package/cache-calibrator/cache-calibrator.hash @@ -0,0 +1,2 @@ +# Locally calculated +sha256 2018ed8fa733155d44ceb1c0066c5cf8df7771cdf7cfca0a07b8dd9bebd9c221 calibrator.c
Signed-off-by: Fabio Porcedda <fabio.porcedda@gmail.com> --- package/cache-calibrator/cache-calibrator.hash | 2 ++ 1 file changed, 2 insertions(+) create mode 100644 package/cache-calibrator/cache-calibrator.hash