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

pyproject.toml references non-existent directory louis #80

Closed
2 tasks
k-allagbe opened this issue Mar 12, 2024 · 1 comment · Fixed by #89
Closed
2 tasks

pyproject.toml references non-existent directory louis #80

k-allagbe opened this issue Mar 12, 2024 · 1 comment · Fixed by #89
Assignees

Comments

@k-allagbe
Copy link
Member

Description

During installation of the package (pip install git+https://github.com/ai-cfia/ailab-db@main) in other repositories, the setup process fails because it expects a directory named louis which does not exist in the repository. This causes the installation to fail with the error: error: package directory 'louis' does not exist.

Collecting git+https://github.com/ai-cfia/ailab-db@main (from -r requirements.txt (line 9))
  Cloning https://github.com/ai-cfia/ailab-db (to revision main) to /private/var/folders/43/4zssf0gs6sj_j_kzmzgdzx0m0000gn/T/pip-req-build-huxsnmbh
  Running command git clone --filter=blob:none --quiet https://github.com/ai-cfia/ailab-db /private/var/folders/43/4zssf0gs6sj_j_kzmzgdzx0m0000gn/T/pip-req-build-huxsnmbh
  Resolved https://github.com/ai-cfia/ailab-db to commit fd08308138d77768294306a9405008072101329f
  Installing build dependencies ... done
  Getting requirements to build wheel ... error
  error: subprocess-exited-with-error
  
  × Getting requirements to build wheel did not run successfully.
  │ exit code: 1
  ╰─> [8 lines of output]
      running egg_info
      creating ailab_db.egg-info
      writing ailab_db.egg-info/PKG-INFO
      writing dependency_links to ailab_db.egg-info/dependency_links.txt
      writing requirements to ailab_db.egg-info/requires.txt
      writing top-level names to ailab_db.egg-info/top_level.txt
      writing manifest file 'ailab_db.egg-info/SOURCES.txt'
      error: package directory 'louis' does not exist
      [end of output]
  
  note: This error originates from a subprocess, and is likely not a problem with pip.
error: subprocess-exited-with-error

× Getting requirements to build wheel did not run successfully.
│ exit code: 1
╰─> See above for output.

note: This error originates from a subprocess, and is likely not a problem with pip.

This issue seems to be a regression: d08f373

Tasks

  • Update the pyproject.toml file to remove or correct the reference to the louis directory if it's not needed.
  • Test the package installation.

Acceptance Criteria

  • The ailab-db package should install successfully.

Warning

We should kick this up in priority because it will cause deployed services that depend on the package to fail on next redeploy. We should also think of package versioning.

@k-allagbe k-allagbe changed the title setup.py references non-existent directory louis pyproject.toml references non-existent directory louis Mar 12, 2024
@rngadam
Copy link
Contributor

rngadam commented Mar 19, 2024

@FrancoisWerbrouck-CFIA could you possibly take this on since you are making changes to remove references to louis?

@k-allagbe k-allagbe self-assigned this Mar 26, 2024
k-allagbe added a commit that referenced this issue Mar 26, 2024
@k-allagbe k-allagbe linked a pull request Mar 26, 2024 that will close this issue
k-allagbe added a commit that referenced this issue Mar 26, 2024
k-allagbe added a commit that referenced this issue Mar 26, 2024
k-allagbe added a commit that referenced this issue Mar 27, 2024
k-allagbe added a commit that referenced this issue Mar 27, 2024
k-allagbe added a commit that referenced this issue Mar 27, 2024
k-allagbe added a commit that referenced this issue Mar 27, 2024
k-allagbe added a commit that referenced this issue Mar 27, 2024
k-allagbe added a commit that referenced this issue Apr 2, 2024
…louis

issue #80: remove louis reference from `pyproject.toml`
@github-project-automation github-project-automation bot moved this to Done in Database Apr 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants