Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove Ion.egg-info and build_sources.sh #1669

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

krishnans2006
Copy link
Member

Proposed changes

  • Remove Ion.egg-info
  • Make build_sources.sh obsolete, and remove any references to it in other scripts and CI

Brief description of rationale

Python eggs are no longer supported: they have been entirely replaced by wheels (see here, here, and here). Therefore, carrying around an Ion.egg-info and having to build sources is highly unnecessary.

Below is a (truncated) successful installation of intranet as a pip package, without Ion.egg-info:

Processing ./intranet
  Installing build dependencies ... done
  Getting requirements to build wheel ... done
  Preparing metadata (pyproject.toml) ... done
...
Building wheels for collected packages: Ion
  Building wheel for Ion (pyproject.toml) ... done
  Created wheel for Ion: filename=Ion-1.0-py3-none-any.whl size=18169480 sha256=699a7c9abe22951aa166a4e5daf474e089d7be38dee907de9433a31cae94eecb
  Stored in directory: /home/krishnan/.cache/pip/wheels/7e/80/92/06ba346fc4361cb005b13fc798756c7e861000bc5219cbecc5
Successfully built Ion
Installing collected packages: ...
Successfully installed ...

This change is made possible by #1648.

@krishnans2006 krishnans2006 self-assigned this May 2, 2024
@krishnans2006 krishnans2006 requested a review from a team as a code owner May 2, 2024 21:35
@krishnans2006 krishnans2006 force-pushed the pyproject-migration branch 2 times, most recently from 0483844 to 012d190 Compare May 2, 2024 21:53
Python eggs are no longer supported: they have been entirely replaced by
wheels. Therefore, carrying around an `Ion.egg-info` and having to build
sources is highly unnecessary.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant