Skip to content

Work around non-gnu basename implementations#403

Merged
tsipinakis merged 1 commit intodunst-project:masterfrom
bebehei:redo-354
Oct 6, 2017
Merged

Work around non-gnu basename implementations#403
tsipinakis merged 1 commit intodunst-project:masterfrom
bebehei:redo-354

Conversation

@bebehei
Copy link
Member

@bebehei bebehei commented Oct 6, 2017

Redo #354, as it got pushed out in #365

@tsipinakis
Copy link
Member

tsipinakis commented Oct 6, 2017

JFTR I also found an unrelated bug while reviewing this: n->icon is initialized after parsing the format so %I and %i don't work with the default icons. See c55308b

Other than that looks good to me

@tsipinakis tsipinakis merged commit 4783778 into dunst-project:master Oct 6, 2017
@bebehei
Copy link
Member Author

bebehei commented Oct 6, 2017

JFTR I also found an unrelated bug while reviewing this: n->icon is initialized after parsing the format so %I and %i don't work with the default icons. See c55308b

Yes, this has to be done in a complete refactor of notificiation_init.

@bebehei bebehei deleted the redo-354 branch October 6, 2017 21:51
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.

2 participants