Skip to content

0.3.2-rc2: [RSDK-11469] Add Windows support to the CI (#17)

Choose a tag to compare

@vpandiarajan20 vpandiarajan20 released this 15 Aug 21:04
· 4 commits to main since this release
a74a94d
and other refactoring:
- When you run `make setup`, the script that runs is now called `setup.sh`, and it does setup work.
- There's a new `make dist/main` target that builds the executable, separate from the setup stuff
- There's a new step in `publish.yml` to build and upload on Windows. I've marked the file as one to run on `workflow_dispatch` so you can manually run it without creating a release, and have [gotten that to work](https://github.com/viam-modules/tensorflow-cpu/actions/runs/16654086812).
- The requirements.txt did not pin versions of stuff, and it rotted and could not find mutually compatible versions of everything. So, I copied the versions pinned in https://github.com/viam-modules/keras-detector/. I was going to pin all versions so this wouldn't happen again, and got surprised that `pip freeze` gives no output! This happens on both Windows and Linux. So, I gave up on that for the moment.
- Tried out: everything builds and uploads correctly, and the Windows version appears to run on a Windows machine.


* remove trailing whitespace

* put quotes around echoed things, else you can run into trouble with punctuation

* the installer script just installs things

* name the setup script correctly

* fiddling with the workflows, not done

* cribbing off of the meeting with Abe and Khari earlier today

* whoops: setup-python is on v5 these days

* make the setup script more verbose to help diagnose what goes wrong

* tar the right files on Windows

* REVERTME: get the workflow_dispatch to run

* debugging on Windows

* try to resolve dependency mismatch

* debugging

* re-enable release version validation

* pr feedback: mark setup as a phony make target

* remove duplicated code

* and refactor out the duplicated 'source' command, too

* bugfix, perhaps

* bugfix