Skip to content

Install dunstify#701

Merged
tsipinakis merged 2 commits into
dunst-project:masterfrom
braunbearded:install-dunstify
Apr 3, 2020
Merged

Install dunstify#701
tsipinakis merged 2 commits into
dunst-project:masterfrom
braunbearded:install-dunstify

Conversation

@braunbearded
Copy link
Copy Markdown

Put dunstify in $PATH, so you can run it easily without manually moving files, like dunst itself

@bebehei
Copy link
Copy Markdown
Member

bebehei commented Apr 3, 2020

I guess you have to fix the tests 😉 You have to make changes here: https://github.com/dunst-project/dunst/blob/master/test/test-install.sh

@braunbearded
Copy link
Copy Markdown
Author

Fixed: Dont know how I missed that

@tsipinakis
Copy link
Copy Markdown
Member

I have rejected similar PRs before on the notion that dunstctl aka #651 would be feature-equivalent with dunstify and one could use that instead.
This doesn't look to be the case looking at the current implementation in that PR, so I agree here, let's add dunstify officially in.

@tsipinakis tsipinakis merged commit 12873cc into dunst-project:master Apr 3, 2020
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