[Buildroot] [PATCH] autobuild-run: remove only tarballs from download dir
Yann E. MORIN
yann.morin.1998 at free.fr
Sun Apr 15 19:49:15 UTC 2018
Thomas, Ricardo, All,
On 2018-04-15 21:42 +0200, Thomas Petazzoni spake thusly:
> On Fri, 13 Apr 2018 10:34:31 -0300, Ricardo Martincoski wrote:
>
> > diff --git a/scripts/autobuild-run b/scripts/autobuild-run
> > index 33d0ae9..6a01151 100755
> > --- a/scripts/autobuild-run
> > +++ b/scripts/autobuild-run
> > @@ -291,6 +291,9 @@ def prepare_build(**kwargs):
> > # recursively find files under root
> > def find_files(root):
> > for r, d, f in os.walk(root):
> > + if '.git' in d:
> > + d[:] = list()
> > + continue
>
> Just to check how this is supposed to work. We have this:
>
> dl/<package>/git/.git
>
> Correct ?
>
> So you're idea is that when we are inside dl/<package>/git/, one of the
> sub-directories is .git, and therefore we shouldn't remove anything in
> dl/<package>/git/ ?
>
> My concern is that I'm not sure if what you've done prevents from
> removing files inside dl/<package>/git or only inside
> dl/<package>/git/.git. I would find it more to do something like:
>
> if "git" in d:
> d.remove("git")
>
> but perhaps you haven't done this for some good reason ?
>
> Another concern is how to fix those autobuilders that have already
> removed some random files from their cached Git repositories? Should we
> ask the people who run those autobuilders to entirely wipe the download
> folders of their autobuilder instances ? Or do we have a smart (but
> simple) thing to do to avoid this ?
For this last part, we've already discused this with Ricardo in another
thread: we run git-fsck, and if there is an error, we ditch the git tree
and clone from scratch.
See the current work I have to further robustify (does that even exist?)
the git backend:
https://git.buildroot.org/~ymorin/git/buildroot/log/?h=yem/git-robust
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. |
'------------------------------^-------^------------------^--------------------'
More information about the buildroot
mailing list