conda / menuinst

Cross platform menu item installation
https://conda.github.io/menuinst/
BSD 3-Clause "New" or "Revised" License
38 stars 44 forks source link

Windows-only shortcut is created by Linux and MacOS shell installers #234

Open dokempf opened 3 months ago

dokempf commented 3 months ago

Checklist

What happened?

I have a Windows only shortcut defined here: https://github.com/conda-forge/helios-menuinst-feedstock/blob/main/recipe/src/menu.json#L32

However, the shell installers for Linux and MacOS also add this shortcut (which is of course not functional). You can download the installers for reproduction here: https://heibox.uni-heidelberg.de/d/5672545fba1b4185a3be/

Is this a bug or did I misread the documentation about how to specify platforms?

Conda Info

No response

Conda Config

No response

Conda list

No response

Additional Context

No response

dokempf commented 3 months ago

After reading through #240 and debugging with the generated paths, I can confirm the following:

"platforms": {
  "win": {}
}

will trigger generation on all platforms, which I consider a bug.

The following works:

"platforms": {
  "win": {},
  "linux": null,
  "osx": null
}

I will use this for now in my package.