[Libguestfs] [PATCH supermin NOT TO BE APPLIED] Supermin 5 rewrite.

2014-02-24 Thread Richard W.M. Jones
I am rewriting supermin to make it more robust. In particular, instead of storing the list of host files to be copied into the appliance (which is not robust if the host package is updated and new files are added), it now stores the list of packages, and works out the dependencies and full list

[Libguestfs] [PATCH] builder: add a mandatory 'arch' key in index files

2014-02-24 Thread Pino Toscano
Introduce a mandatory arch= key in all the entries of index files, to identify which architecture is each. Adapt the long and JSON list outputs to print also this new field. This introduces an incompatibility with index files created with virt-builder 1.26, as they will be rejected until entries

[Libguestfs] [PATCH] sysprep: use Mkdtemp to create the temporary directory

2014-02-24 Thread Pino Toscano
Use the safer mkdtemp instead of manually creating a path. --- sysprep/Makefile.am | 2 ++ sysprep/sysprep_operation_script.ml | 4 +--- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/sysprep/Makefile.am b/sysprep/Makefile.am index 2600477..9c9ab14 100644 ---

[Libguestfs] [PATCH] builder: split INI C - OCaml glue code in own module

2014-02-24 Thread Pino Toscano
Move in an own module the code which calls the C virt_builder_parse_index and does the array - list conversion of the result. This way this code can be easily called also in places different than Index_parser without the need to copy the types mapping, etc. Just code motion, no actual behaviour

Re: [Libguestfs] [PATCH] sysprep: use Mkdtemp to create the temporary directory

2014-02-24 Thread Richard W.M. Jones
On Mon, Feb 24, 2014 at 03:42:03PM +0100, Pino Toscano wrote: Use the safer mkdtemp instead of manually creating a path. --- sysprep/Makefile.am | 2 ++ sysprep/sysprep_operation_script.ml | 4 +--- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git

Re: [Libguestfs] [PATCH] builder: add a mandatory 'arch' key in index files

2014-02-24 Thread Richard W.M. Jones
On Mon, Feb 24, 2014 at 11:36:29AM +0100, Pino Toscano wrote: Introduce a mandatory arch= key in all the entries of index files, to identify which architecture is each. Adapt the long and JSON list outputs to print also this new field. This introduces an incompatibility with index files

Re: [Libguestfs] [PATCH] builder: split INI C - OCaml glue code in own module

2014-02-24 Thread Richard W.M. Jones
On Mon, Feb 24, 2014 at 03:12:13PM +0100, Pino Toscano wrote: Move in an own module the code which calls the C virt_builder_parse_index and does the array - list conversion of the result. This way this code can be easily called also in places different than Index_parser without the need to

Re: [Libguestfs] [PATCH] builder: add a mandatory 'arch' key in index files

2014-02-24 Thread Pino Toscano
On Monday 24 February 2014 15:15:11 Richard W.M. Jones wrote: On Mon, Feb 24, 2014 at 11:36:29AM +0100, Pino Toscano wrote: Introduce a mandatory arch= key in all the entries of index files, to identify which architecture is each. Adapt the long and JSON list outputs to print also this