<p></p>
<p>When running the unittests on Windows, all Ctags tests fail because TagManager writes the tag files with CRLF line endings but the tests compare against the test samples which have LF line endings (most of them).<br>
There is a probably easy solution: as <a class="user-mention" data-hovercard-type="user" data-hovercard-url="/users/techee/hovercard" data-octo-click="hovercard-link-click" data-octo-dimensions="link_type:self" href="https://github.com/techee">@techee</a> pointed out in <a class="issue-link js-issue-link" data-error-text="Failed to load title" data-id="746131049" data-permission-text="Title is private" data-url="https://github.com/geany/geany/issues/2666" data-hovercard-type="pull_request" data-hovercard-url="/geany/geany/pull/2666/hovercard?comment_id=734731377&comment_type=issue_comment" href="https://github.com/geany/geany/pull/2666#issuecomment-734731377">#2666 (comment)</a>, writing the tag file in binary mode will use LF line endings when writing the tags file (and <em>should</em> have no effect on non-Windows systems <g-emoji class="g-emoji" alias="crossed_fingers" fallback-src="https://github.githubassets.com/images/icons/emoji/unicode/1f91e.png">🤞</g-emoji>).<br>
There are six test samples which have CRLF line endings which still causes the tests to fail. Those could be converted to LF line endings:</p>
<ul>
<li>3184782.sql.tags</li>
<li>3526726.tex.tags</li>
<li>bug1570779.sql.tags</li>
<li>ingres_procedures.sql.tags</li>
<li>matlab_backtracking.m.tags</li>
<li>matlab_test.m.tags</li>
</ul>
<p>Apart from the Ctags unittests, we have some tests for various utility functions in <a href="https://github.com/geany/geany/blob/master/tests/test_utils.c">https://github.com/geany/geany/blob/master/tests/test_utils.c</a> where few filesystem related tests fail on Windows. Those could be probably <code>ifdef</code>'d for Windows and/or separate tests for Windows be added.</p>
<p>Once these issues are fixed, we could enable <code>make check</code> in CI!?</p>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/geany/geany/issues/2677">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AAIOWJ52CFYQGSKSETOZU63SSJD5VANCNFSM4UGQMLEQ">unsubscribe</a>.<img src="https://github.com/notifications/beacon/AAIOWJYF2D4SCBCYAJDZHN3SSJD5VA5CNFSM4UGQMLE2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4LHATKEQ.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/geany/geany/issues/2677",
"url": "https://github.com/geany/geany/issues/2677",
"name": "View Issue"
},
"description": "View this Issue on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>