Package Details: python-syncedlyrics 1.0.1-1

Git Clone URL: https://aur.archlinux.org/python-syncedlyrics.git (read-only, click to copy)
Package Base: python-syncedlyrics
Description: Get an LRC format (synchronized) lyrics for your music
Upstream URL: https://github.com/moehmeni/syncedlyrics
Keywords: lrc lyrics lyrics-fetcher lyrics-finder lyrics-search music music-bot music-player synced-lyrics synchronization
Licenses: MIT
Submitter: txtsd
Maintainer: txtsd (pekkarr)
Last Packager: txtsd
Votes: 9
Popularity: 0.90
First Submitted: 2023-03-15 19:06 (UTC)
Last Updated: 2024-11-10 07:12 (UTC)

Pinned Comments

txtsd commented on 2024-07-20 09:16 (UTC) (edited on 2024-07-20 09:17 (UTC) by txtsd)

Tests often fail when a search provider is unavailable.

Build with makepkg --nocheck to skip tests.

If using an AUR helper like yay, it's yay -S --mflags --nocheck.

Latest Comments

1 2 3 Next › Last »

gesh commented on 2024-12-23 19:12 (UTC)

I don't see where it says that in the wiki page, but I'll respect your position. (Especially since with https://github.com/maximbaz/rebuild-detector, it is easy to find which packages need to be rebuilt)

txtsd commented on 2024-12-23 17:38 (UTC)

@gesh AUR packages don't do bumps for rebuilds. It's the user's responsibility to rebuild.

See: https://wiki.archlinux.org/title/Arch_User_Repository#Updating_packages

gesh commented on 2024-12-23 15:53 (UTC)

Note: Package needs to be rebuilt for Python 3.13 (a bump to pkgrel suffices)

gesh commented on 2024-09-30 15:57 (UTC)

Note -- given that we're not building with the python-setuptools backend, the python-wheel makedep is unnecessary.

txtsd commented on 2024-07-20 09:16 (UTC) (edited on 2024-07-20 09:17 (UTC) by txtsd)

Tests often fail when a search provider is unavailable.

Build with makepkg --nocheck to skip tests.

If using an AUR helper like yay, it's yay -S --mflags --nocheck.

ahbanavi commented on 2024-07-19 20:10 (UTC) (edited on 2024-07-19 20:11 (UTC) by ahbanavi)

I can not install it

tests.py .....F..                                                                                                [100%]

======================================================= FAILURES =======================================================
_____________________________________________________ test_genius ______________________________________________________

    def test_genius():
>       _test_provider("Genius")

tests.py:42:
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _

provider = 'Genius', kwargs = {}, lrc = None, @py_assert3 = False
@py_format5 = 'assert False\n{False = isinstance(None, str)\n}'

    def _test_provider(provider: str, **kwargs):
        lrc = syncedlyrics.search(search_term=q, providers=[provider], **kwargs)
        logging.debug(lrc)
>       assert isinstance(lrc, str)
E       assert False
E        +  where False = isinstance(None, str)

tests.py:15: AssertionError
=============================================== short test summary info ================================================
FAILED tests.py::test_genius - assert False

txtsd commented on 2024-03-07 12:29 (UTC)

@gesh That's correct. The pypi version does not ship with the tests.py file.

@pekkarr I was not aware of this RFC. Thanks for showing it to me. I will undo the changes shortly.

gesh commented on 2024-03-07 12:27 (UTC)

I mean, --nocheck is there in case the test failures are bogus. (as is the case rn, raised upstream: https://github.com/moehmeni/syncedlyrics/issues/25)

pekkarr commented on 2024-03-07 12:26 (UTC) (edited on 2024-03-07 12:28 (UTC) by pekkarr)

@txtsd PyPI shouldn't be used as a source for python packages, see Arch Linux RFC 20. It's better to get the source archive from the github repo, and this makes it possible to run the tests too. Also using the GitHub repo in the url variable would make more sense than the PyPI url.