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

macOS support improvements #63

Merged
merged 10 commits into from
Apr 13, 2024

Conversation

Stealthii
Copy link
Contributor

@Stealthii Stealthii commented Apr 12, 2024

This PR restores macOS compatibility, and fixes a few remaining issues with current widgets.

Major changes

  • Support macOS in netspeed widget

Minor changes

  • Use calculated data from nowplaying-cli, fixes Unicode support
  • Use GNU date to format duration/position in music-tmux-statusbar

Bug & security fixes

  • Appropriately handle now-playing durations above 1 hour
  • Use GNU coreutils on macOS if available

Additional information

A dependency on GNU coreutils/bc, and Bash 4+ is implied with this update. The bash requirement only affects macOS at present, however Homebrew instructions are being provided for dependencies and use of this package manager is assumed to provide tmux and other utilities that are already used by this theme.

@Stealthii Stealthii changed the title macOS support macOS support improvements Apr 12, 2024
@Stealthii Stealthii force-pushed the feature/macos-support branch from 5d8f5d7 to a5abadd Compare April 12, 2024 18:20
@janoamaral janoamaral self-requested a review April 12, 2024 20:56
@janoamaral janoamaral added the enhancement New feature or request label Apr 12, 2024
Copy link
Owner

@janoamaral janoamaral left a comment

Choose a reason for hiding this comment

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

As I'm a Linux user only, having this improvings on macOS helps the project greatly. Just fix the conflicts and we are good to go.

In order to navigate rendering issues from raw data in nowplaying-cli,
the output has been switched to use named properties in an associative
array.

Included is a fix to playback state based on rate, handling playback at
different speeds.

This will require bash v4 or higher.
This change includes a fix for durations over 1 hour, where the time is
updated to the dashed variant rather than rolling over each hour.
Flattened requirements into one clear list, with sections provided for
OS flavors.
@Stealthii Stealthii force-pushed the feature/macos-support branch from d563fad to bcd2992 Compare April 13, 2024 22:59
@janoamaral janoamaral merged commit d6fce99 into janoamaral:master Apr 13, 2024
1 check passed
@Stealthii Stealthii deleted the feature/macos-support branch April 13, 2024 23:28
uNyanda pushed a commit to uNyanda/tokyo-night-tmux that referenced this pull request Nov 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants