This fixes a regression that was introduced in
commit d5f5d399.

Traceback (most recent call last):
  ...
  File "slapos/recipe/build/__init__.py", line 191, in pipeCommand
    raise zc.buildout.UserError('Failed while running command:'
  File "slapos/recipe/build/__init__.py", line 192, in <genexpr>
    + ''.join('\n  ' + q for q in failed))
TypeError: cannot concatenate 'str' and 'tuple' objects

With only 384M, debian-11.0.0-amd64-netinst.iso fails to initialize
the network interface.

Since we switched to running tests with python -m unittest discover,
doctests were not longer ran.

When using init or install option of slapos.recipe.build to
write python code, the traceback were hard to understand, because
the current line was not included in the traceback.

This put the script in linecache before compiling it, so that in
case of error the problematic line appears in traceback.

There is still the "problem" that the traceback line numbers starts
from the beginning of the script and not the actual line from
buildout profile, but this does not seem feasible to adjust the line
numbers, so let's ignore this.

It is even the recommended recipe for simple download.

Update use to tolerate errors in develop mode, to prevent buildout from
deleting develop working copies if anything happens, then this behavior was
extended to non-develop mode, apparently by mistake.

In non-develop mode, we want error to propagate, if something went wrong with
updating this repository, buildout should delete and retry.

The purpose is to replace colletive.recipe.shelloutput

/reviewed-on nexedi/slapos.recipe.build!11

These options were used by a subclass that does not exist anymore.

This reverts the following commits partially:
  d157d8a1
  e92d31cf

With buildout 2, indentation is either fully removed (if anything
starts on the same line as '<option-name> =') or preverved (the same
number of leading spaces if stripped). Any use that required these
commits was a misuse. They could even break valid Python.

This fixes a problem visible when installing openssl 1.1.1d from
component/openssl. At the end of install step, some symlinks are created
inside openssl install directory for all certificates from
ca-certificates part, then the "make read only" step occurs, which
recursively chmod on files and folder in openssl installed parts, but
since this was following symlinks, it was also trying to make readonly
in ca-certificates parts, which in my case was belonging to another
unix user, because it was a shared part folder from outer level slapos.

/reviewed-on nexedi/slapos.recipe.build!10

It's a nightmare to convert from DOS to GPT (required when moving to a
disk bigger than 2TB), because GRUB can only be installed reliably
from the system to boot.

ext4 was chosen for the 'discard' option, but formatting takes too
much time and space. I was surprised to see my VM being paused
(no left space on host) because of ext4lazyinit.

An unrelated change is about develop mode: the behaviour was simplified to do nothing if a working copy already exists.

TODO: handle removal of submodules

/reviewed-on nexedi/slapos.recipe.build!7

Note that we don't change the URL for BAD_GIT_REPOSITORY as we need them
to return 500 while trying to clone. If we use Nexedi Gitlab repo for
non-existing repo, we get redirected to login page. Hence, we keep using
git.erp5.org for `repo/nowhere`.

/reviewed-on nexedi/slapos.recipe.build!6