-
Notifications
You must be signed in to change notification settings - Fork 54
The action installs the wrong version of cabal of MacOS #15
Comments
Sorry for not replying earlier. Thanks for pointing this out! So, there's an interesting chain of events happening here.
The problem is we really don't want to stick a bunch of install logic in the function that checks whether or not something is installed. I encountered this before when ghcup didn't have a way to check which versions of cabal were installed in a file path manner. Now it does, so that leads to the natural solution of async function isInstalled(
const installedPath = await fs
- .access(`${ghcupPath}/cabal`)
+ .access(`${ghcupPath}/cabal-${version}`)
.then(() => ghcupPath)
.catch(() => undefined); So that solves "systems that rely on ghcup to install cabal and then try to install a new different of cabal later won't have that version correctly setup" (for real, this time). The second issue is that As it stands, pre-releases of GHC and Cabal look like this:
There is absolutely no way to make this consistent,currently. Any pre-release logic is going to have to happen entirely on the end-user's side of things, and I don't feel comfortable setting up CI with insecure flags and hammering a personal server to test that something not designed to be used in CI can indeed be hacked into a CI pipeline. |
Please see: "I switched all urls to http. Your workaround can be removed." - Julian Ospald |
That'll work for now; I'm happy to remove the testing of 3.4.0.0-rc4 if it's needed in the future (eg @phadej requests that) |
@jared-w I would prefer that GitHub doesn't "ddos" my (to some approximation) server. Testing release candidates is however welcome. Clarification: I don't have access to |
@phadej that's very reasonable and understandable. Thanks for the update! I'm going to close this as the related PR was merged. And I'll address the windows one today as well. Feel free to open a new issue if things stop working :) |
@phadej could we upload cabal pre-releases into |
See https://github.com/input-output-hk/cardano-db-sync/pull/441/checks?check_run_id=1586145305
Note, on Linux, the correct version gets installed:
The text was updated successfully, but these errors were encountered: