<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="">I’m sending the next version with updated commit message.<div class="">See more answers below…</div><div class=""><br class=""><div><blockquote type="cite" class=""><div class="">On 8 Jun 2017, at 15:00 PM, Dmitry Fleytman <<a href="mailto:dmitry@daynix.com" class="">dmitry@daynix.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div class="" style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;">Hi Frediano, thanks for review.</div><div class="" style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;">See below.</div><div class="" style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;"><br class=""></div><br class="" style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;"><div style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><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 class="" style="font-family: 'times new roman', 'new york', times, serif; font-size: 12pt;"><div class=""><br class=""></div><blockquote class="" 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;"><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 class=""><br class=""></div><div class="">Yes, good idea. I will add.</div></div></div></blockquote><div><br class=""></div>Added.<br class=""><blockquote type="cite" class=""><div class=""><div style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><blockquote type="cite" class=""><div class=""><div class=""><div class="" style="font-family: 'times new roman', 'new york', times, serif; font-size: 12pt;"><blockquote class="" 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;"><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 class=""><br class=""></div><div class="">Probably yes, I will check if Appveyor supports this.</div></div></div></blockquote><div><br class=""></div><div>Unfortunately Appveyor cannot send plain text e-mails.</div><blockquote type="cite" class=""><div class=""><div style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><blockquote type="cite" class=""><div class=""><div class=""><div class="" style="font-family: 'times new roman', 'new york', times, serif; font-size: 12pt;"><blockquote class="" 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;"><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 class=""><br class=""></div><div class="">Not sure I understand. What do you mean by “last machine version”?</div></div></div></blockquote><div><br class=""></div><div><br class=""></div><div>Frediano, please elaborate.</div><br class=""><blockquote type="cite" class=""><div class=""><div style="font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px;" class=""><blockquote type="cite" class=""><div class=""><div class=""><div class="" style="font-family: 'times new roman', 'new york', times, serif; font-size: 12pt;"><blockquote class="" 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;"><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 class=""><br class=""></div><div class="">A nice feature borrowed from Travis-CI - If commit message includes “[ci skip]” - CI is not run for this commit.</div><div class=""><br class=""></div><blockquote type="cite" class=""><div class=""><div class=""><div class="" style="font-family: 'times new roman', 'new york', times, serif; font-size: 12pt;"><blockquote class="" 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;"><div class=""><div class=""><blockquote class=""><div class=""><div class=""><br class="">+notifications:<br class="">+ - provider: Email<br class="">+ to:<span class="Apple-converted-space"> </span><a href="mailto:spice-devel@lists.freedesktop.org" class="" target="_blank"></a><a class=""></a><a href="mailto:spice-devel@lists.freedesktop.org" 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 class=""><br class=""></div><div class="">Not yet. I did not register upstream repository with Appveyor also.</div><div class="">Will do it after the patch is inside.</div><blockquote type="cite" class=""><div class=""><div class=""><div class="" style="font-family: 'times new roman', 'new york', times, serif; font-size: 12pt;"><blockquote class="" 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;"><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></div></div></blockquote></div></div></blockquote></div><br class=""></div></body></html>