I'm not sure this is correct. There's no place else where we use or set USING_OIIO_PUGI. I think that instead the correct solution is that later on, instead of 

    #ifdef USING_OIIO_PUGI

it should say 

    #ifndef USE_EXTERNAL_PUGIXML