jotego / jtbin

Binary files for MiSTerFPGA, Pocket and other platforms
https://patreon.com/jotego
246 stars 69 forks source link

Change the tool flow to prevent bad uploads #303

Closed jotego closed 1 year ago

jotego commented 1 year ago

jtupdate adds file to JTBIN by default, which has caused inadverted uploads of broken cores a few times. The issue arises as:

  1. while working on a release for system x (say mist)
  2. I check that other system (say mister) is still healthy by running a batch compilation using jtupdate
  3. when I commit the changes to jtbin, the mister files get uploaded too, despite there was no intention to do so

Some solutions:

jotego commented 1 year ago

Keeping JTBIN clean with git clean -f before working on a new update seems to prevent this problem. jtupdate should check that JTBIN is clean before running

jotego commented 1 year ago

Changes made in JTUTIL v1.0.0 and JTFRAME v1.4.3