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

user/ignis: new package #3245

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

supermikea
Copy link

No description provided.

user/ignis/template.py Outdated Show resolved Hide resolved
user/python-loguru/template.py Outdated Show resolved Hide resolved
@q66
Copy link
Member

q66 commented Dec 5, 2024

the python modules should use PYPI_SITE correctly

the typing modules should not be necessary, just patch them out

@q66
Copy link
Member

q66 commented Dec 5, 2024

i added mypy separately, you can drop whatever is related to that

user/ignis/template.py Outdated Show resolved Hide resolved
user/ignis/template.py Outdated Show resolved Hide resolved
user/ignis/template.py Outdated Show resolved Hide resolved
user/ignis/template.py Outdated Show resolved Hide resolved
user/python-loguru/template.py Outdated Show resolved Hide resolved
user/ignis/template.py Outdated Show resolved Hide resolved
@supermikea
Copy link
Author

This should be it (assuming "autotraced" means that it's automatically pulled in)

@q66
Copy link
Member

q66 commented Dec 5, 2024

this is not fully fixed, there is still libgvc.so in /usr/lib even though there shouldn't (it should be in a designated private directory) - this is an upstream issue, and changing the layer shell dependency to gtk4-layer-shell is the correct thing to do, but it also breaks it because the code tries to open the .so symlink that is in -devel, this is also an upstream issue

you'll have to patch around these somehow, or get upstream to fix it

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