[PR #139] [MERGED] Swap use of subprocess.call for webbrowser.open in util.py #788

New issue

Closed

opened 2026-02-28 00:01:43 +03:00 by kerem · 0 comments

kerem commented

2026-02-28 00:01:43 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/spotipy-dev/spotipy/pull/139
Author: @delucks
Created: 12/21/2016
Status: ✅ Merged
Merged: 12/31/2016
Merged by: @plamere

Base: master ← Head: master

📝 Commits (1)

3587e10 Swap use of subprocess.call for webbrowser.open in util.py

📊 Changes

1 file changed (+3 additions, -3 deletions)

View changed files

📝 spotipy/util.py (+3 -3)

📄 Description

This swaps out the subprocess call to "open {uri}" to use the webbrowser module, which is more portable. I wrote this after noticing that my client (Linux on amd64) didn't properly open my browser when it printed the message about doing so.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/spotipy-dev/spotipy/pull/139 **Author:** [@delucks](https://github.com/delucks) **Created:** 12/21/2016 **Status:** ✅ Merged **Merged:** 12/31/2016 **Merged by:** [@plamere](https://github.com/plamere) **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (1) - [`3587e10`](https://github.com/spotipy-dev/spotipy/commit/3587e107c0b576167fe2b2401630a215853e25f9) Swap use of subprocess.call for webbrowser.open in util.py ### 📊 Changes **1 file changed** (+3 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `spotipy/util.py` (+3 -3) </details> ### 📄 Description This swaps out the subprocess call to "open {uri}" to use the webbrowser module, which is more portable. I wrote this after noticing that my client (Linux on amd64) didn't properly open my browser when it printed the message about doing so. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>