Not a fan of using ? in a file name but that's a minor nitpick (I understand why you're doing it though)
Optional but since we're not using _REV anywhere except in MASTER_SITES perhaps consider use it directly and remove the variable?

LGTM