<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Sun, Aug 25, 2013 at 7:06 PM, Dave Reisner <span dir="ltr"><<a href="mailto:d@falconindy.com" target="_blank">d@falconindy.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Sun, Aug 25, 2013 at 06:35:03PM +0200, Maxime Gauduin wrote:<br>
> Fixes FS#36654: <a href="https://bugs.archlinux.org/task/36654" target="_blank">https://bugs.archlinux.org/task/36654</a>.<br>
><br>
> Signed-off-by: Maxime Gauduin <<a href="mailto:alucryd@gmail.com">alucryd@gmail.com</a>><br>
> ---<br>
>  <a href="http://makechrootpkg.in" target="_blank">makechrootpkg.in</a> | 10 ++++++++++<br>
>  1 file changed, 10 insertions(+)<br>
><br>
> diff --git a/<a href="http://makechrootpkg.in" target="_blank">makechrootpkg.in</a> b/<a href="http://makechrootpkg.in" target="_blank">makechrootpkg.in</a><br>
> index 8c64ae1..b842d2f 100644<br>
> --- a/<a href="http://makechrootpkg.in" target="_blank">makechrootpkg.in</a><br>
> +++ b/<a href="http://makechrootpkg.in" target="_blank">makechrootpkg.in</a><br>
> @@ -296,6 +296,16 @@ _chrootbuild() {<br>
>               done<br>
>       done<br>
><br>
> +     # Same with bzr sources<br>
> +     for dir in /srcdest /startdir; do<br>
> +             cd $dir<br>
> +             for bzrdir in */.bzr; do<br>
> +                     rm ${bzrdir%/.bzr}<br>
<br>
</div>Isn't this going to be a directory? How does this rm call succeed?<br>
<div class="im"><br>
> +                     cp -a ${dir}_host/${bzrdir%/.bzr} .<br>
> +                     chown -R nobody ${bzrdir%/.bzr}<br>
> +             done<br>
> +     done<br>
<br>
</div>General mention: please quote liberally rather than assuming that the<br>
contents will never be quoteworthy.<br>
<div class="HOEnZb"><div class="h5"><br>
> +<br>
>       cd /startdir<br>
><br>
>       # XXX: Keep PKGBUILD writable for pkgver()<br>
> --<br>
> 1.8.3.4<br>
><br>
</div></div></blockquote></div><br></div><div class="gmail_extra">Right, all I did was copy paste the equivalent for svn in the script, replacing svn with bzr. I'll modify both svn and bzr with quotes and add -rf to the rm command.<br clear="all">

</div><div class="gmail_extra"><br>-- <br><div dir="ltr">Maxime</div>
</div></div>