Automatically add the emscripten-forge channel if it is missing?

[jtpio]

Problem

Currently the default channels are only added if there is no environment file:

xeus/jupyterlite_xeus/add_on.py

Lines 130 to 147 in eda12f8

	# open the env yaml file if it's provided
	if env_file.exists():
	with open(env_file, "r") as file:
	yaml_content = yaml.safe_load(file)
	env_name = yaml_content.get("name", "xeus-env")
	env_prefix = root_prefix / "envs" / env_name
	self.prefix = str(env_prefix)
	create_conda_env_from_env_file(root_prefix, yaml_content, env_file.parent)
	else:
	create_conda_env_from_specs(
	env_name=env_name,
	root_prefix=root_prefix,
	specs=["xeus-python"],
	channels=["https://repo.mamba.pm/emscripten-forge", "conda-forge"],
	)

Proposed Solution

Maybe the add-on should check if the user-provided environment.yml includes https://repo.mamba.pm/emscripten-forge in the list of channels, and add it automatically otherwise?

Additional context

This could help improving compatibility with existing environment.yml files without having to create a copy of the file with https://repo.mamba.pm/emscripten-forge in it.

And also for use cases such as repo2jupyterlite: jupyterlite/repo2jupyterlite#19

[martinRenou]

I don't think we should add https://repo.mamba.pm/emscripten-forge automagically if it's not there.

People should be free to use any channel they want (if they mirror emscripten-forge or provide their own channel with emscripten packages).

[martinRenou]

If no channel is specified, it's used by default though: https://github.com/jupyterlite/xeus/blob/main/jupyterlite_xeus/create_conda_env.py#L47

[jtpio]

Ah ok, I thought we used to have something like this in jupyterlite-xeus-python. But maybe because it was always installing xeus-python by default.

[jtpio]

If no channel is specified, it's used by default though: https://github.com/jupyterlite/xeus/blob/main/jupyterlite_xeus/create_conda_env.py#L47

Could it make sense to also do this if only the conda-forge channel is specified? I guess this could address most use cases.

[martinRenou]

Could it make sense to also do this if only the conda-forge channel is specified? I guess this could address most use cases

I see, yeah that could make sense!

[martinRenou]

Could it make sense to also do this if only the conda-forge channel is specified? I guess this could address most use cases.

Let's put this behind an optional flag though, as we probably should not change the environment file without the user knowing.

[jtpio]

Let's put this behind an optional flag though, as we probably should not change the environment file without the user knowing.

That sounds good, and would at least help experiment with that without too much disruption.

Projects like https://github.com/jupyterlite/repo2jupyterlite could then enable that option by default.