coverallsapp / coverage-reporter

Self-contained, universal coverage uploader binary. Under development.
https://coveralls.io
MIT License
45 stars 14 forks source link

fix: link sqlite3 statically on windows #107

Closed nebolsin closed 8 months ago

nebolsin commented 8 months ago

Alternative to #104 and #105.

Use statically linked sqlite3 lib on windows (which we compile from SQLite3 "amalgamation" source distribution) to avoid conflicts with other sqlite3.dll available in the system.

Here's an output of dumpbin /dependents coveralls.exe of the produced executable, which confirms no dependency on sqlite3.dll:

Microsoft (R) COFF/PE Dumper Version 14.37.32824.0
Copyright (C) Microsoft Corporation.  All rights reserved.

Dump of file coveralls.exe

File Type: EXECUTABLE IMAGE

  Image has the following dependencies:

    CRYPT32.dll
    ADVAPI32.dll
    WS2_32.dll
    KERNEL32.dll
    dbghelp.dll
    bcrypt.dll
    USER32.dll

Also adds Vagrantfile and provision script to simplify setting up local Windows development environment.

coveralls-official[bot] commented 8 months ago

Pull Request Test Coverage Report for Build 6721845286


Totals Coverage Status
Change from base Build 6498958900: 0.0%
Covered Lines: 940
Relevant Lines: 1003

💛 - Coveralls
afinetooth commented 8 months ago

🔥