Skip to content

Adjust project structure and add poetry#3

Open
SimonBrandner wants to merge 2 commits intomichibo:masterfrom
SimonBrandner:org
Open

Adjust project structure and add poetry#3
SimonBrandner wants to merge 2 commits intomichibo:masterfrom
SimonBrandner:org

Conversation

@SimonBrandner
Copy link

This PR adjusts the project structure for better readability. It also adds poetry for better dependency management

Signed-off-by: Šimon Brandner <simon.bra.ag@gmail.com>
@michibo
Copy link
Owner

michibo commented Sep 16, 2023

Hi Simon, thanks for the pull request!

I have no experience with poetry, but it looks like a good idea. However, I have the following problem:

I try to run

> poetry install

and get the error

Package operations: 1 install, 0 updates, 0 removals

  • Installing pycairo (1.24.0): Failed

  TypeError

  Can't instantiate abstract class IsolatedEnv with abstract methods make_extra_environ, python_executable

  at /usr/lib/python3.11/site-packages/poetry/installation/chef.py:109 in _prepare
      105│     ) -> Path:
      106│         from subprocess import CalledProcessError
      107│
      108│         with ephemeral_environment(self._env.python) as venv:
    → 109│             env = IsolatedEnv(venv, self._pool)
      110│             builder = ProjectBuilder(
      111│                 directory,
      112│                 python_executable=env.executable,
      113│                 scripts_dir=env.scripts_dir,

I actually have pycairo installed on my system, so the program runs flawlessly without the poetry sandbox. Do you have an idea on how to fix that?

Best, M

@SimonBrandner
Copy link
Author

Argh, I haven't seen that before, no, not really sure how to help you there...

Signed-off-by: Šimon Brandner <simon.bra.ag@gmail.com>
@SimonBrandner
Copy link
Author

@michibo, we you able to figure out what's going on? I'd like to move this forward, though I am not sure how to help on your side to be frank... We could do this without poetry but I think it helps with dep management quite a bit...

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.

2 participants