<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Wed, Jul 15, 2015 at 11:07 AM, Brad King <span dir="ltr"><<a href="mailto:brad.king@kitware.com" target="_blank" onclick="window.open('https://mail.google.com/mail/?view=cm&tf=1&to=brad.king@kitware.com&cc=&bcc=&su=&body=','_blank');return false;">brad.king@kitware.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 07/15/2015 10:57 AM, Daniel Pfeifer wrote:<br>
> The new element `ChangeId` is added to Build.xml and Test.xml.<br>
> Did you consider adding it as an attribute in `cmCTest::StartXML` instead?<br>
> That would make this (very useful) information available in all xml files.<br>
<br>
</span>I think that would make sense since all submitted .xml files are<br>
associated with the change identified by the ChangeId.<br>
<br>
Zack?<br></blockquote><div><br></div><div><br></div><div>I did consider that.  The reason I went with the current approach is that StartXML() only defines attributes on the <Site> tag, which doesn't logically seem like the right location for this information.  Change ID should be associated with the build/test/update/etc step being performed.  Instead, I followed the example set by <StartDateTime>, which is independently set by each of the CTest handlers.</div><div><br></div><div>That being said, I don't have very strong feelings about this, so if you're all okay with Change ID being a Site attribute I could move it to StartXML().</div></div></div></div>