<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div class="">Hi Frediano, thanks for review.</div><div class="">See below.</div><div class=""><br class=""></div><br class=""><div><blockquote type="cite" class=""><div class="">On 8 Jun 2017, at 13:42 PM, Frediano Ziglio <<a href="mailto:fziglio@redhat.com" class="">fziglio@redhat.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div class=""><div style="font-family: 'times new roman', 'new york', times, serif; font-size: 12pt;" class=""><div class=""><br class=""></div><blockquote style="border-left-width: 2px; border-left-style: solid; border-left-color: rgb(16, 16, 255); margin-left: 5px; padding-left: 5px; font-weight: normal; font-style: normal; text-decoration: none; font-family: Helvetica, Arial, sans-serif; font-size: 12pt;" class=""><div class=""><br class=""></div>Hello All,<div class=""><br class=""></div><div class="">This patch introduces Appveyor CI configuration file.</div><div class=""><br class=""></div><div class="">Having this patch pushed and qxl-wddm-dod repository registered at Appveyor,</div><div class="">every push or merge request will be automatically built on their servers.</div></blockquote><div class=""><br class=""></div><div class="">If you need to comment a single patch to add explanation maybe this additional<br class=""></div><div class="">explanation should be added to the commit message.<br class=""></div></div></div></div></blockquote><div><br class=""></div><div>Yes, good idea. I will add.</div><blockquote type="cite" class=""><div class=""><div class=""><div style="font-family: 'times new roman', 'new york', times, serif; font-size: 12pt;" class=""><blockquote style="border-left-width: 2px; border-left-style: solid; border-left-color: rgb(16, 16, 255); margin-left: 5px; padding-left: 5px; font-weight: normal; font-style: normal; text-decoration: none; font-family: Helvetica, Arial, sans-serif; font-size: 12pt;" class=""><div class="">Build status notifications will be sent to this list (<a href="mailto:spice-devel@lists.freedesktop.org" class="" target="_blank">spice-devel@lists.freedesktop.org</a>)</div></blockquote><div class="">It seems that the message arrive as html with no corresponding test, see</div><div class=""><a href="https://lists.freedesktop.org/archives/spice-devel/2017-June/037963.html" class="">https://lists.freedesktop.org/archives/spice-devel/2017-June/037963.html.</a></div><div class="">Is possible to have pure text messages instead?<br class=""></div></div></div></div></blockquote><div><br class=""></div><div>Probably yes, I will check if Appveyor supports this.</div><blockquote type="cite" class=""><div class=""><div class=""><div style="font-family: 'times new roman', 'new york', times, serif; font-size: 12pt;" class=""><blockquote style="border-left-width: 2px; border-left-style: solid; border-left-color: rgb(16, 16, 255); margin-left: 5px; padding-left: 5px; font-weight: normal; font-style: normal; text-decoration: none; font-family: Helvetica, Arial, sans-serif; font-size: 12pt;" class=""><div class="">Best Regards,</div><div class="">Dmitry</div><div class=""><br class=""><div class=""><blockquote class=""><div class="">On 8 Jun 2017, at 10:10 AM, Dmitry Fleytman <<a href="mailto:dmitry@daynix.com" class="" target="_blank">dmitry@daynix.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div class="">Signed-off-by: Dmitry Fleytman <<a href="mailto:dmitry@daynix.com" class="" target="_blank">dmitry@daynix.com</a>><br class="">---<br class=""> .appveyor.yml | 18 ++++++++++++++++++<br class=""> 1 file changed, 18 insertions(+)<br class=""> create mode 100644 .appveyor.yml<br class=""><br class="">diff --git a/.appveyor.yml b/.appveyor.yml<br class="">new file mode 100644<br class="">index 0000000..f3f3e5c<br class="">--- /dev/null<br class="">+++ b/.appveyor.yml<br class="">@@ -0,0 +1,18 @@<br class="">+image: Visual Studio 2015<br class="">+</div></div></blockquote></div></div></blockquote><div class="">This cause to stick to a version of images and if image are going to be removed in the<br class=""></div><div class="">future this won't work. I use AppVeyor for another project and I know images are</div><div class="">updated and obsolete ones are removed after a while. From my experience is better to<br class=""></div><div class="">fix the issues with the last machine version.<br class=""></div></div></div></div></blockquote><div><br class=""></div><div>Not sure I understand. What do you mean by “last machine version”?</div><blockquote type="cite" class=""><div class=""><div class=""><div style="font-family: 'times new roman', 'new york', times, serif; font-size: 12pt;" class=""><blockquote style="border-left-width: 2px; border-left-style: solid; border-left-color: rgb(16, 16, 255); margin-left: 5px; padding-left: 5px; font-weight: normal; font-style: normal; text-decoration: none; font-family: Helvetica, Arial, sans-serif; font-size: 12pt;" class=""><div class=""><div class=""><blockquote class=""><div class=""><div class=""><br class="">+version: build-{build}-{branch}<br class="">+<br class="">+before_build:<br class="">+ - ren "C:\Program Files (x86)\Windows Kits\10\include\00wdf" "wdf"<br class="">+<br class="">+build_script: buildAll.bat<br class="">+<br class="">+skip_commits:<br class="">+ message: /\[ci skip\]/<br class="">+</div></div></blockquote></div></div></blockquote><div class="">what does this?<br class=""></div></div></div></div></blockquote><div><br class=""></div><div>A nice feature borrowed from Travis-CI - If commit message includes “[ci skip]” - CI is not run for this commit.</div><div><br class=""></div><blockquote type="cite" class=""><div class=""><div class=""><div style="font-family: 'times new roman', 'new york', times, serif; font-size: 12pt;" class=""><blockquote style="border-left-width: 2px; border-left-style: solid; border-left-color: rgb(16, 16, 255); margin-left: 5px; padding-left: 5px; font-weight: normal; font-style: normal; text-decoration: none; font-family: Helvetica, Arial, sans-serif; font-size: 12pt;" class=""><div class=""><div class=""><blockquote class=""><div class=""><div class=""><br class="">+notifications:<br class="">+ - provider: Email<br class="">+ to: <a href="mailto:spice-devel@lists.freedesktop.org" class="" target="_blank"></a><a class="">spice-devel@lists.freedesktop.org</a><br data-mce-bogus="1" class=""></div></div></blockquote></div></div></blockquote><div class="">Not strictly related to this patch, is usually a good idea to add sender e-mail<br class=""></div><div class="">to the ML. Did you do it?<br class=""></div></div></div></div></blockquote><div><br class=""></div><div>Not yet. I did not register upstream repository with Appveyor also.</div><div>Will do it after the patch is inside.</div><blockquote type="cite" class=""><div class=""><div class=""><div style="font-family: 'times new roman', 'new york', times, serif; font-size: 12pt;" class=""><blockquote style="border-left-width: 2px; border-left-style: solid; border-left-color: rgb(16, 16, 255); margin-left: 5px; padding-left: 5px; font-weight: normal; font-style: normal; text-decoration: none; font-family: Helvetica, Arial, sans-serif; font-size: 12pt;" class=""><div class=""><div class=""><blockquote class=""><div class=""><div class=""><br class="">+ on_build_success: false<br class="">+ on_build_failure: true<br class="">+ on_build_status_changed: true</div></div></blockquote></div></div></blockquote><div class="">Frediano</div><div class=""><br class=""></div></div></div></div></blockquote></div><br class=""></body></html>