factoriotools / factorio-docker

Factorio headless server in a Docker container
https://hub.docker.com/r/factoriotools/factorio/
MIT License
1.08k stars 241 forks source link

Updated update.sh and clear data from broken github action! #505

Closed MrBasicX closed 1 month ago

jonojr commented 1 month ago

I'm just going to check this out and make sure it works

LsHallo commented 1 month ago

Same thing for me: yq: -i/--in-place can only be used with -y/-Y

miragecentury commented 1 month ago

Result for buildinfo.json from a quick test.


  "2.0.7": {
    "sha256": "e6a64a9ee5b1d3773cd1f40aa5cf78e91960eade6c508fd365c34752afdf2762",
    "tags": [
      "latest",
      "stable",
      "stable-2.0.7",
      "2",
      "2.0",
      "2.0.7"
    ]
  }```
miragecentury commented 1 month ago

yq: -i/--in-place can only be used with -y/-Y

For me, it's a yq version mismatch. ( Not a behavior to be expected in the action run ).

jonojr commented 1 month ago

Same thing for me: yq: -i/--in-place can only be used with -y/-Y

I get the same, but the lines using it have not been changed, and we get past the issue with getting the checksum. I am betting this is a version mismatch for yq, so I think we are fine

LsHallo commented 1 month ago

Same thing for me: yq: -i/--in-place can only be used with -y/-Y

I get the same, but the lines using it have not been changed, and we get past the issue with getting the checksum. I am betting this is a version mismatch for yq, so I think we are fine

yq: -i/--in-place can only be used with -y/-Y

For me, it's a yq version mismatch. ( Not a behavior to be expected in the action run ).

Yeah thinking the same. LGTM

jonojr commented 1 month ago

@Fank I can get the checksums now with this version, so I think we are gtg

MelaineGerard commented 1 month ago

I can confirm the result of the buildinfo.json