NCAR / ccpp-framework

Common Community Physics Package (CCPP)
http://www.dtcenter.org/community-code/common-community-physics-package-ccpp/
Other
26 stars 64 forks source link

xmllint opt-out flag #606

Open mwaxmonsky opened 4 weeks ago

mwaxmonsky commented 4 weeks ago

Description

As discovered in #601, xmllint is always being checked and run if found. This creates an implicit dependency on xmllint that shouldn't be required for all runs of capgen.

Solution

Add an explicit flag to disable xmllint-ing and associated handling for checking if xmllint is available in the environment if enabled. This way if the flag is not turned on and xmllint is not found in the environment, an error can be generated instead of silently failing and just printing to stdout.

Alternatives (optional)

Discussed an opt-in flag but most uses cases are going to require linting.

Related to (optional)

601