Created attachment 208293 [details] port diff Simple Spotify client written in rust for the terminal, using the Spotify API providing a way to find music by name, play tracks, etc. QA: Successfully built in poudriere.
I built and installed this and signed up, did all the setup, but on the step where it prompts me to "Enter the URL you were redirected to:", I enter it, then I get a timeout: thread 'main' panicked at 'send request failed: Error(Io(Custom { kind: TimedOut, error: "timed out" }), "https://accounts.spotify.com/api/token")', src/libcore/result.rs:1084:5 Any ideas what's going wrong here?
(In reply to Steve Wills from comment #1) During the initial submission, everything compiled successfully and worked without issues. I've reason to believe there's been an upstream change to one of it's dependencies that's broken it's behaviour, causing this timeout of sorts. What leaves me confused is if you try pulling Spotify-tui from cargo: cargo install spotify-tui, I no longer face this timeout, subsequently working as intended. I've opened this as an issue on the repo: https://github.com/Rigellute/spotify-tui/issues/72 and provided a stacktrace. There seems to be an immediate issue with rspotify: (last function call before it throws the timeout) ``` 10: 0x13ad842 - rspotify::spotify::util::get_token::h33b74d9688b2ca7d ``` Can you try pulling in and running spotify-tui directly from cargo?
As a side note: originally compiled on v0.5.0, tried again now and no longer works. I've also tried the latest release v0.6.2 and same results. Downloading from cargo seems to get around this but doesn't fix the remaining issue.
Could this potentially be a discrepancy between the ports build system and the standalone cargo package manager? What stumps me is that it builds from ports, but timesout on runtime. While on the contrary simply building from cargo, outside of the port, and there is no timeout as aforementioned. Wonder if I'm missing a cargo flag in the Makefile or something along the lines.
Created attachment 208386 [details] port diff * Updated port to version v0.6.2, distinfo metadata and CARGO_CRATES. * Added optional dependency for spotifyd that can be used in conjunction with spotify-tui for playback. This does _not_ fix the underlying runtime timeout error. I'm certain it has to do with CARGO_BUILD_ARGS, as discussed previously: there's no timeout error when compiling from `cargo install spotify-tui` but _only_ when compiling the port. I'm not as familiar with the rust toolchain nor cargo, so open to suggestions in what needs to be done and/or changed. My guess is there's a particular build flag being passed (or lack-of) to cargo that breaks functionality in the do-build stage. Cheers.
Created attachment 209556 [details] spotify-tui port diff * Updated port DISTVERSION to latest (rm 0.10.0) including distinfo and CARGO_CRATES; * Added python:3.6+,build and xorg to USES (needed to build xcb crate) and similarly USE_XORG for xcb; * Added BINARY_ALIAS for python (needed during the build script of xcb crate); * Added PORTDOCS to Makefile for documentation installation; * Added pkg-message.in to files/ for installation message. The remaining issue that was aforementioned in this thread has been fixed. You can now successfully authenticate with Spotify without error, and use the application as expected. Port builds in poudriere as normal and passed with portlint.
Created attachment 209558 [details] poudriere build log * Attached poudriere build log for extra information if needed.
Created attachment 209571 [details] spotify-tui port diff * Change Makefile in accordance to `portclippy` (thanks to 0mp@ for shedding light on this); * Shorten PORTDOC installation to a one-liner.
A commit references this bug: Author: swills Date: Mon Dec 2 19:29:50 UTC 2019 New revision: 518866 URL: https://svnweb.freebsd.org/changeset/ports/518866 Log: audio/spotify-tui: create port Spotify terminal client written in rust. Play music directly from your terminal, find new artists, search for your favourite music and more. WWW: https://github.com/Rigellute/spotify-tui PR: 241238 Submitted by: Lewis Cook <vulcan@wired.sh> Changes: head/audio/Makefile head/audio/spotify-tui/ head/audio/spotify-tui/Makefile head/audio/spotify-tui/distinfo head/audio/spotify-tui/files/ head/audio/spotify-tui/files/pkg-message.in head/audio/spotify-tui/pkg-descr
Committed, thanks!