I'm torn on this. I hate how we do that across all our packages - it leads to crazy duplicated xml that has to be kept in sync across multiple packages, even for minor changes.

The idea of loading it dynamically is that only local xml ever has to be edited.