Comment 2 for bug 1307554

Revision history for this message
jazzynico (jazzynico) wrote :

The attached patch adds a new dependency type:

<dependency type="os">[win|notwin]</dependency>

I'm still not sure if the type name is well chosen (I hesitate between "os" and "system").

Note that it's a very basic implementation that solves the bug, with very limited modifications to the INX schema. We could also easily add other operating systems if needed.

Also note that including or excluding an OS from the element's value is not very clean, but using a different attribute would be a bit more difficult (and add confusion due to the existence of other dependency types...). IMHO for such a feature we should keep the schema changes as limited as possible.