jupyterlite / xeus-python-kernel

xeus-python in JupyterLite
https://xeus-python-kernel.readthedocs.io
BSD 3-Clause "New" or "Revised" License
30 stars 19 forks source link

If pip is included in the environment.yml file as a dependency, we should automatically drop it #166

Closed martinRenou closed 9 months ago

martinRenou commented 9 months ago

Description

cc. @psychemedia

Specifying an environment containing pip may result in unwanted errors:

name: xeus-python-kernel
channels:
  - https://repo.mamba.pm/emscripten-forge
  - https://repo.mamba.pm/conda-forge
dependencies:
  - pip

We should take care of this case and drop pip from the list automatically.

martinRenou commented 9 months ago

After trying in #170 this seems to not be problematic actually. pip being a noarch package it can be installed in the environment. We may want to filter out pip in the packed environment though. See https://github.com/emscripten-forge/empack/issues/83