From: Yann E. MORIN <yann.morin.1998@free.fr>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 1/1] linuxptp: bump to the latest version
Date: Sun, 10 Sep 2017 11:24:50 +0200 [thread overview]
Message-ID: <20170910092450.GC3536@scaer> (raw)
In-Reply-To: <20170910080432.0a6bc7b6@windsurf.lan>
On 2017-09-10 08:04 +0200, Thomas Petazzoni spake thusly:
> On Sat, 9 Sep 2017 22:53:06 +0200, Petr Kulhavy wrote:
> > Is there a command to just clone and compress the repo via BR?
> > The <package>-extract make target fails if the hash doesn't exist and
> > consequently deletes the temporary files.
> Yeah, it's a bit annoying. If you put a none hash temporarily, then you
> can have the tarball downloaded, calculate its hash, and add it. We
> also had proposals like https://patchwork.ozlabs.org/patch/791357/ to
> help with this.
IIRC, I was opposed to that change, because we want the user to go and
get the hash as provided by upstream (e.g. in a release email).
Having the infra pre-calculate the hash locally defeats the very purpose
of the hashes: check that what we got is what upstream provides.
We accept local calculation of hashes only in the case that upstream
does not provide it.
As an aside, the patch does two things, so should be split.
I'll go and reply that to that original patch.
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. |
'------------------------------^-------^------------------^--------------------'
next prev parent reply other threads:[~2017-09-10 9:24 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-09-09 17:17 [Buildroot] [PATCH 1/1] linuxptp: bump to the latest version Petr Kulhavy
2017-09-09 20:08 ` Thomas Petazzoni
2017-09-09 20:53 ` Petr Kulhavy
2017-09-10 6:04 ` Thomas Petazzoni
2017-09-10 9:24 ` Yann E. MORIN [this message]
2017-09-10 10:31 ` Petr Kulhavy
2017-09-10 18:18 ` Yann E. MORIN
2017-09-10 18:40 ` Thomas Petazzoni
2017-09-10 23:30 ` Petr Kulhavy
2017-09-11 20:04 ` Yann E. MORIN
2017-09-10 9:57 ` Petr Kulhavy
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=20170910092450.GC3536@scaer \
--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