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

New package: matrix-mirage #22834

Closed
wants to merge 8 commits into from

Conversation

Francesco149
Copy link
Contributor

this is a client for the matrix protocol and the various dependencies it wants.

@Francesco149
Copy link
Contributor Author

Francesco149 commented Jun 12, 2020

oops looks like i forgot to include the version in the commit msg, ill fix later

@Francesco149
Copy link
Contributor Author

okay reworded everything. other than that they were building fine on the CI

@travankor
Copy link
Contributor

travankor commented Jun 13, 2020

I built this package and ran into this error when I tried to run it. Seems similar to mirukana/mirage#15

Got library name:  "/usr/lib/qt5/qml/io/thp/pyotherside/libpyothersideplugin.so"                          
 "PyOtherSide error: Traceback (most recent call last):\n\n  File \"qrc:/src/backend/qml_bridge.py\", line 164, in <module>\n    BRIDGE = QMLBridge()\n\n  File \"qrc:/src/backend/qml_bridge.py\", line 50, in __init__\n    from .backend import Backend\n\n  File \"qrc:/src/backend/backend.py\", line 16, in <module>\n    from .matrix_client import MatrixClient\n\n  File \"qrc:/src/backend/matrix_client.py\", line 27, in <module>\n    from pymediainfo import MediaInfo\n\n  File \"/usr/lib/python3.8/site-packages/pymediainfo/__init__.py\", line 9, in <module>\n    from pkg_resources import get_distribution, DistributionNotFound\n\n  File \"/usr/lib/python3.8/site-packages/pkg_resources/__init__.py\", line 3253, in <module>\n    def _initialize_master_working_set():\n\n  File \"/usr/lib/python3.8/site-packages/pkg_resources/__init__.py\", line 3236, in _call_aside\n    f(*args, **kwargs)\n\n  File \"/usr/lib/python3.8/site-packages/pkg_resources/__init__.py\", line 3265, in _initialize_master_working_set\n    working_set = WorkingSet._build_master()\n\n  File \"/usr/lib/python3.8/site-packages/pkg_resources/__init__.py\", line 575, in _build_master\n    ws = cls()\n\n  File \"/usr/lib/python3.8/site-packages/pkg_resources/__init__.py\", line 568, in __init__\n    self.add_entry(entry)\n\n  File \"/usr/lib/python3.8/site-packages/pkg_resources/__init__.py\", line 624, in add_entry\n    for dist in find_distributions(entry, True):\n\n  File \"/usr/lib/python3.8/site-packages/pkg_resources/__init__.py\", line 2067, in find_on_path\n    for dist in factory(fullpath):\n\n  File \"/usr/lib/python3.8/site-packages/pkg_resources/__init__.py\", line 2139, in distributions_from_metadata\n    yield Distribution.from_location(\n\n  File \"/usr/lib/python3.8/site-packages/pkg_resources/__init__.py\", line 2591, in from_location\n    return cls(\n\n  File \"/usr/lib/python3.8/site-packages/pkg_resources/__init__.py\", line 2995, in _reload_version\n    md_version = self._get_version()\n\n  File \"/usr/lib/python3.8/site-packages/pkg_resources/__init__.py\", line 2773, in _get_version\n    version = _version_from_file(lines)\n\n  File \"/usr/lib/python3.8/site-packages/pkg_resources/__init__.py\", line 2557, in _version_from_file\n    line = next(iter(version_lines), '')\n\n  File \"/usr/lib/python3.8/site-packages/pkg_resources/__init__.py\", line 2768, in _get_metadata\n    for line in self.get_metadata_lines(name):\n\n  File \"/usr/lib/python3.8/site-packages/pkg_resources/__init__.py\", line 1434, in get_metadata_lines\n    return yield_lines(self.get_metadata(name))\n\n  File \"/usr/lib/python3.8/site-packages/pkg_resources/__init__.py\", line 1422, in get_metadata\n    value = self._get(path)\n\n  File \"/usr/lib/python3.8/site-packages/pkg_resources/__init__.py\", line 1618, in _get\n    with open(path, 'rb') as stream:\n\nPermissionError: [Errno 13] Permission denied: '/usr/lib/python3.8/site-packages/blist-1.3.6-py3.8.egg-info/PKG-INFO'\n"
Unhandled PyOtherSide error: Cannot import module: backend.qml_bridge (Traceback (most recent call last):

  File "qrc:/src/backend/qml_bridge.py", line 164, in <module>
    BRIDGE = QMLBridge()

  File "qrc:/src/backend/qml_bridge.py", line 50, in __init__
    from .backend import Backend

  File "qrc:/src/backend/backend.py", line 16, in <module>
    from .matrix_client import MatrixClient

  File "qrc:/src/backend/matrix_client.py", line 27, in <module>
    from pymediainfo import MediaInfo

  File "/usr/lib/python3.8/site-packages/pymediainfo/__init__.py", line 9, in <module>
    from pkg_resources import get_distribution, DistributionNotFound

  File "/usr/lib/python3.8/site-packages/pkg_resources/__init__.py", line 3253, in <module>
    def _initialize_master_working_set():

  File "/usr/lib/python3.8/site-packages/pkg_resources/__init__.py", line 3236, in _call_aside
    f(*args, **kwargs)

  File "/usr/lib/python3.8/site-packages/pkg_resources/__init__.py", line 3265, in _initialize_master_working_set
    working_set = WorkingSet._build_master()

  File "/usr/lib/python3.8/site-packages/pkg_resources/__init__.py", line 575, in _build_master
    ws = cls()

  File "/usr/lib/python3.8/site-packages/pkg_resources/__init__.py", line 568, in __init__
    self.add_entry(entry)

  File "/usr/lib/python3.8/site-packages/pkg_resources/__init__.py", line 624, in add_entry
    for dist in find_distributions(entry, True):

  File "/usr/lib/python3.8/site-packages/pkg_resources/__init__.py", line 2067, in find_on_path
    for dist in factory(fullpath):

  File "/usr/lib/python3.8/site-packages/pkg_resources/__init__.py", line 2139, in distributions_from_metadata
    yield Distribution.from_location(

  File "/usr/lib/python3.8/site-packages/pkg_resources/__init__.py", line 2591, in from_location
    return cls(

  File "/usr/lib/python3.8/site-packages/pkg_resources/__init__.py", line 2995, in _reload_version
    md_version = self._get_version()

  File "/usr/lib/python3.8/site-packages/pkg_resources/__init__.py", line 2773, in _get_version
    version = _version_from_file(lines)

  File "/usr/lib/python3.8/site-packages/pkg_resources/__init__.py", line 2557, in _version_from_file
    line = next(iter(version_lines), '')

  File "/usr/lib/python3.8/site-packages/pkg_resources/__init__.py", line 2768, in _get_metadata
    for line in self.get_metadata_lines(name):

  File "/usr/lib/python3.8/site-packages/pkg_resources/__init__.py", line 1434, in get_metadata_lines
    return yield_lines(self.get_metadata(name))

  File "/usr/lib/python3.8/site-packages/pkg_resources/__init__.py", line 1422, in get_metadata
    value = self._get(path)

  File "/usr/lib/python3.8/site-packages/pkg_resources/__init__.py", line 1618, in _get
    with open(path, 'rb') as stream:

PermissionError: [Errno 13] Permission denied: '/usr/lib/python3.8/site-packages/blist-1.3.6-py3.8.egg-info/PKG-INFO'
)
"PyOtherSide error: Traceback (most recent call last):\n\n  File \"<string>\", line 1, in <module>\n\nNameError: name 'BRIDGE' is not defined\n"
Unhandled PyOtherSide error: Function not found: 'BRIDGE.call_backend_coro' (Traceback (most recent call last):

  File "<string>", line 1, in <module>

NameError: name 'BRIDGE' is not defined
)

Secondly, the binary conflicts with the package mirage and will overwrite files if you have mirage and matrix-mirage installed.

@Francesco149
Copy link
Contributor Author

right, i forgot about that. i need to rename the binary.

hmm strange - that error did not happen to me, I'll look into it

@Francesco149
Copy link
Contributor Author

PermissionError: [Errno 13] Permission denied: '/usr/lib/python3.8/site-packages/blist-1.3.6-py3.8.egg-info/PKG-INFO'

ah yes, i remember having to chmod a python package that for some reason wasn't world readable and i have no idea why. I'm gonna try to reinstall all the dependencies and see if it breaks

@Francesco149
Copy link
Contributor Author

yeah blist is definitely being installed with wrong permissions... but why?

$ ls -latr /usr/lib/python3.8/site-packages/blist-1.3.6-py3.8.egg-info/
total 48
-rw-r-----   1 root root     6 Jun 12 17:13 top_level.txt
-rw-r-----   1 root root   644 Jun 12 17:13 SOURCES.txt
-rw-r-----   1 root root  8790 Jun 12 17:13 PKG-INFO
-rw-r-----   1 root root     1 Jun 12 17:13 not-zip-safe
-rw-r-----   1 root root     1 Jun 12 17:13 dependency_links.txt

@Francesco149
Copy link
Contributor Author

apparently this is just how the tarball is. guess we'll chmod it manually... DanielStutzbach/blist#72

@Francesco149
Copy link
Contributor Author

fixed by just grabbing the tarball off github instead

@Chocimier
Copy link
Member

Please make new packages python3-only.

@Francesco149
Copy link
Contributor Author

are you sure about that? what's the disadvantage of keeping python2 templates around? we could just not build them when we drop python2

@Chocimier
Copy link
Member

Chocimier commented Jun 13, 2020

It's not as easy as stop to build them in future. All py2 packages will need to be checked if they aren't required by other packages. Removing packages from repo require manual work too. Py2 modules are not accepted since few months.

@travankor
Copy link
Contributor

It works now but qt5-graphicaleffects should be a dependency somewhere (matrix-mirage pyotherside?).

It's not as easy as stop to build them in future. All py2 packages will need to be checked if they aren't required by other packages. Removing packages from repo require manual work too. Py2 modules are not accepted since few months.

@Chocimier Not related to this PR, but olm-python3 depends on olm-python, which seems wrong.

@Francesco149
Copy link
Contributor Author

alright added the dep

@Chocimier
Copy link
Member

@travankor, thanks for report, fixed.

@mirukana
Copy link

Note the only submodule still used as of v0.5.1 is hsluv-c, no need to fetch the others.

@Francesco149
Copy link
Contributor Author

ohh thanks for the info

@Francesco149
Copy link
Contributor Author

rebased and removed unnecessary submodules

@Francesco149
Copy link
Contributor Author

rebased (2 deps were added by other people in recent commits) and updated to 0.5.2

@Francesco149
Copy link
Contributor Author

updated to 0.6.0

@travankor
Copy link
Contributor

travankor commented Aug 25, 2020

0.6.2 is out

@Chocimier
Copy link
Member

blist need patch for py3.8: https://patch-diff.githubusercontent.com/raw/DanielStutzbach/blist/pull/78.patch
For tests: tinycss2 needs pre_check() { vsed -i setup.cfg -e s/--flake8// }
cssselect2 needs checkdepends="python3-pytest" and pre_check() { vsed -i setup.cfg -e s/--isort// }
CairoSVG needs checkdepends="${depends}", but tests will still fail without modifications of cairocffi: will push these later.

@Chocimier
Copy link
Member

Fixed cairocffi.

@ericonr
Copy link
Member

ericonr commented Dec 18, 2020

@Francesco149 ping

@ericonr ericonr added the new-package This PR adds a new package label Dec 20, 2020
@abenson
Copy link
Contributor

abenson commented Nov 6, 2021

Seemingly abandoned.

@abenson abenson closed this Nov 6, 2021
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 5, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
new-package This PR adds a new package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants