Fix logic in 32-bit architecture check

[ktbarrett]

Closes #166.

[goanpeca]

Thanks for working on this @ktbarrett

Is there a way we can test this by updating some of the examples that run on CI ?

[ktbarrett]

@goanpeca I'll look into adding some CI checks for other architectures. I see you have the checks oriented around the README examples, would you like some additional documentation added to the README?

[goanpeca]

Absolutely :)

We will move that to a docs site eventually

[goanpeca]

Also could you point this PR against the develop branch? And not the master one?

Thanks!

[chrisjbillington]

Any chance of getting this merged soon?

[ktbarrett]

Apparently I don't know anything.

[goanpeca]

@ktbarrett thanks for working on this!

Apparently I don't know anything.

You need to compile the TS file into JS to be able to see any actual changes run on CI. See the the steps here:

https://github.com/conda-incubator/setup-miniconda/blob/master/CONTRIBUTING.md

[goanpeca]

https://github.com/conda-incubator/setup-miniconda/pull/168/checks?check_run_id=2319868097#step:3:19

there is. $latest instead of latest

[goanpeca]

@ktbarrett since the 32bit installers are old, they might be using and old conda by default that does not provide the conda executable inside a condabin folder, but on a bin folder.

/usr/share/miniconda3/condabin/conda you will need to update the code accordingly where we define where to look for conda executables.

Thanks!

[goanpeca]

@ktbarrett its turtles all the way down, now you see why adding support for this was on hold :)

So conda init is a command we rely on for correctly initializing conda, so in the 32 bit installer the conda that comes bundled does not provide that option, which means we wont be able to properly activate environments :-|

[goanpeca]

Ideas @bollwyvl ?

[jaimergp]

I think the first version to offer conda init and $CONDA_ROOT/condabin was 4.6.0. At least it looks like that from the release notes.

We can check which version is trying to get installed and error out if <4.6. Maybe there are other rough edges down there, but we can start with this.

cc @chrisjbillington @ktbarrett

[ktbarrett]

closing in favor of #189.