GNOME Bugzilla – Bug 624386
[PATCH] for new feature: xmllint --noxref
Last modified: 2011-02-22 00:42:36 UTC
We regularly need to run xmllint on xml-files out of context. For that usecase, we know that IDREFs won't work, but we'd like to validate the rest of the xml files anyway. Therefore I created patch to add a commandline option --noxref to xmllint.
Created attachment 165917 [details] [review] --noxref : turn IDREF linkend errors into warnings
the patch is in use at suse. https://build.opensuse.org/package/files?package=libxml2&project=devel%3Alibraries%3Ac_c%2B%2B
Patch makes little sense to me. what do you mean by "out of context" ? If you mean the DTD can't be accessed, well that's what catalogs are for ! http://xmlsoft.org/catalog.html Now breaking validation rules, but just the ID/IDREF check makes absolutely no sense to me, either you are validating and then it's a validation error and reported as such but it's not a fatal error and parsing and processing should continue, or you're not validating and then I don't see how you're getting just ID errors if the DTD is not found. Patch hardly justified, introducing a unclear semantic change, you should really have asked the mailing-list before pushing that in SuSE ! in any case NAK to such a patch without a better clarification of the problem ... Daniel
Comment on attachment 165917 [details] [review] --noxref : turn IDREF linkend errors into warnings Rejected as per last comment.
Closing this bug report as no further information has been provided. Please feel free to reopen this bug if you can provide the information asked for. Thanks!