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

Read config file from $XDG_CONFIG_HOME/spotify-adblock/config.toml #175

Merged
merged 3 commits into from
Sep 15, 2024

Conversation

guihkx
Copy link
Contributor

@guihkx guihkx commented Apr 7, 2024

@guihkx
Copy link
Contributor Author

guihkx commented Apr 7, 2024

P.S.: This will greatly simplify the setting up needed for Spotify for Flatpak.

I've sent another patch on the Flatpak side to make LD_PRELOAD be inherited easier, too.

If this PR gets accepted, I'll update the Flatpak documentation next.

@SamDc73

This comment was marked as spam.

src/lib.rs Outdated Show resolved Hide resolved
@guihkx guihkx force-pushed the xdg-config branch 2 times, most recently from c5a5ae6 to ea34c4f Compare September 14, 2024 23:01
@abba23
Copy link
Owner

abba23 commented Sep 15, 2024

The new path should also be added to the Configuration section in the README.

@guihkx
Copy link
Contributor Author

guihkx commented Sep 15, 2024

The new path should also be added to the Configuration section in the README.

Done in the last commit. I've removed the reference to Flatpak because it's not really Flatpak-specific.

Once this gets merged, I'll send a new PR updating instructions for Flatpak, which will greatly simplify things.

@abba23 abba23 merged commit 7391666 into abba23:main Sep 15, 2024
1 check passed
@abba23
Copy link
Owner

abba23 commented Sep 15, 2024

Thank you for the PR!

@guihkx guihkx deleted the xdg-config branch September 15, 2024 11:35
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.

3 participants