[PATCH 0/2] Recipe xml validation

Jan Tluka jtluka at redhat.com
Thu May 16 13:39:10 UTC 2013


Hi, I have doubts about including this patch as I'm afraid it would
break running of recipes that have <define> tag in it. The schemes
posted in patch do not deal with this tag so any recipe that contains
the tag won't be processed.

I'm currently working on another schema for validation. I was digging into
RelaxNG a bit and I found it more flexible with respect of ordering of
tags.

I'm proposing some of the following
1. make validation optional or provide a way to disable validation
2. add <define> in the schema
3. implement validator as a standalone app

I think the schema also misses 'source' attribute. Could we postpone
the second patch until the scheme is checked by users for any possible
issues?

-Jan

Thu, May 16, 2013 at 03:22:52PM CEST, olichtne at redhat.com wrote:
>From: Ondrej Lichtner <olichtne at redhat.com>
>
>The following two patches add files required for recipe validation and
>impplement a basic form of validation with the XML Schema file provided. At the
>moment the validation relies on the schema file to be in the same directory as
>the lnst-ctl script. In the future this should be changed to a more
>standardized location.
>
>At the moment though this should be sufficient to try out the validation, it
>reflects the currently accepted recipe format so it can help with changing your
>older recipes to the new format.
>
>Ondrej Lichtner (2):
>  add recipe.xsd and recipe.dtd
>  lnst-ctl: add recipe validation
>
> lnst-ctl   |  22 +++++++
> recipe.dtd |  56 ++++++++++++++++++
> recipe.xsd | 196 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
> 3 files changed, 274 insertions(+)
> create mode 100644 recipe.dtd
> create mode 100644 recipe.xsd
>
>-- 
>1.7.11.7
>
>_______________________________________________
>LNST-developers mailing list
>LNST-developers at lists.fedorahosted.org
>https://lists.fedorahosted.org/mailman/listinfo/lnst-developers


More information about the LNST-developers mailing list