search

debrouxl / tilibs

TILP (formerly GtkTiLink) can transfer data between Texas Instruments graphing calculators and a computer. It works with all link cables (parallel, serial, Black/Gray/Silver/Direct Link) and it supports the TI-Z80 series (73..86), the TI-eZ80 series (83PCE, 84+CE), the TI-68k series (89, 92, 92+, V200, 89T) and the Nspire series (Nspire Clickpad / Touchpad / CX, both CAS and non-CAS)
http://lpg.ticalc.org/prj_tilp
67 stars 22 forks source link

Make it buildable on Windows with VS, and add windows Github Actions CI with vcpkg #75

Closed adriweb closed 1 year ago

adriweb commented 1 year ago

This branch is already rebased on top of the other one with macOS/Linux CI, btw.

adriweb commented 1 year ago

I'm not sure why not everything gets cached as I would have expected it for the vcpkg stuff, but anyway that can be solved later.

adriweb commented 1 year ago

The install folder is now uploaded as artifacts (for each build) :)

adriweb commented 1 year ago

Compatibility with the MinGW cross-build should be OK now.

debrouxl commented 1 year ago

I have integrated patches 1-3 and 5, and rebased experimental2 on top of those. It creates both conflicts for this PR and build failures on experimental2...

adriweb commented 1 year ago

Okay, I'll rebase and push again

adriweb commented 1 year ago

I'm not surprised about the static build failures on the other branches, and the fact that they are fixed here. I did indeed fix quite a few CMake things recently on this branch.

debrouxl commented 1 year ago

I've integrated 3 of the 5 patches at the front of the experimental2 branch (for now), in order to resolve the modify / delete conflicts caused by the torture_ticalcs.c -> torture_ticalcs.cc rename. I should dust off my cross-MinGW build procedure to test the result before forwarding the master branch.

adriweb commented 1 year ago

I saw the same warning, but it was not promoted to an error. I guess we can fix that easily. Are there any other ones?

debrouxl commented 1 year ago

I'll merge the remainder and perform some post-merge fixes. Thanks for your work :)