Skip to content

dzen2: new package#1386

Closed
drvirgilio wants to merge 1 commit intoNixOS:masterfrom
drvirgilio:dzen2
Closed

dzen2: new package#1386
drvirgilio wants to merge 1 commit intoNixOS:masterfrom
drvirgilio:dzen2

Conversation

@drvirgilio
Copy link
Contributor

"Dzen is a general purpose messaging, notification and menuing program for X11.
It was designed to be fast, tiny and scriptable in any language."
-- https://github.com/robm/dzen

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you use the substituteInPlace function instead of sed? It's provided by Nix and works the same on all platforms.

"Dzen is a general purpose messaging, notification and menuing program for X11.
It was designed to be fast, tiny and scriptable in any language."
-- https://github.com/robm/dzen
@drvirgilio
Copy link
Contributor Author

I replaced sed with substituteInPlace. Let me know if there is anything else to fix. Thanks.

@the-kenny
Copy link
Contributor

Merged as 84b09e2

(I took the liberty to fix the commit message & remove a whitespace error)

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