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

Fix pypi installation #745

Closed
senier opened this issue Aug 18, 2021 · 0 comments · Fixed by #794
Closed

Fix pypi installation #745

senier opened this issue Aug 18, 2021 · 0 comments · Fixed by #794
Assignees
Labels

Comments

@senier
Copy link
Member

senier commented Aug 18, 2021

When installing RecordFlux from the command line, we currently get:

$ pip install RecordFlux==0.5.0
Collecting RecordFlux==0.5.0
  Downloading RecordFlux-0.5.0.tar.gz (157 kB)
     |████████████████████████████████| 157 kB 5.5 MB/s 
ERROR: Packages installed from PyPI cannot depend on packages which are not also hosted on PyPI.
RecordFlux depends on RecordFlux-language@ git+https://github.com/Componolit/RecordFlux-language.git@v0.6.0#egg=RecordFlux-language 

Consequently, we need to change the way RecordFlux-language is incorporated.

Options:

  1. Host RecordFlux-language on PyPI
  2. Incorporate RecordFlux-language into RecordFlux repository
  3. Make RecordFlux-language an external prerequisite

In AdaCore/RecordFlux-parser#37 we'll go for option 1 and try to make it a regular PyPI package.

@senier senier created this issue from a note in RecordFlux 0.6 (To do) Aug 18, 2021
@senier senier added the bug label Aug 18, 2021
@senier senier self-assigned this Aug 18, 2021
@senier senier moved this from To do to In progress in RecordFlux 0.6 Aug 18, 2021
@senier senier moved this from In progress to To do in RecordFlux 0.6 Aug 26, 2021
@senier senier moved this from To do to In progress in RecordFlux 0.6 Sep 14, 2021
senier pushed a commit that referenced this issue Oct 5, 2021
@senier senier moved this from In progress to Under review in RecordFlux 0.6 Oct 5, 2021
senier pushed a commit that referenced this issue Oct 5, 2021
RecordFlux 0.6 automation moved this from Under review to Merged Oct 5, 2021
senier pushed a commit that referenced this issue Oct 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

1 participant