Ruby gem overmind by prog-supdex · Pull Request #176 · DarthSim/overmind · GitHub
Skip to content

Ruby gem overmind#176

Merged
Envek merged 4 commits into
DarthSim:masterfrom
prog-supdex:packagin_ruby
Mar 25, 2024
Merged

Ruby gem overmind#176
Envek merged 4 commits into
DarthSim:masterfrom
prog-supdex:packagin_ruby

Conversation

@prog-supdex

Copy link
Copy Markdown
Collaborator

This PR adds the ability to generate gem files that pack the overmind library with dependencies tmux into a gem file for various architectures of the Linux and macos platforms by running a rake task overmind:build:all.

It also prepares the gem file for FreeBSD, but without tmux in the box

The rake task will download the overmind and tmux binaries and package them into a platform-specific gem.

To build the gem files, you need to have Ruby >= 3.0.

This PR also adds a NOTICE file and additional information to the README.md.

@prog-supdex prog-supdex requested review from DarthSim and Envek March 18, 2024 18:18
Comment thread NOTICE Outdated

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

What is the purpose of this notice file? Why we need it?

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Let's keep it in sync with overmind itself!

Suggested change
VERSION = "0.1.2"
VERSION = "2.4.0"

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

done!

# Class for downloading Overmind binaries.
class OvermindDownloader < BaseDownloader
NAME = "overmind"
DEFAULT_VERSION = "2.4.0"

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Let's use here gem version also (if we keep it in sync with overmind one)

Suggested change
DEFAULT_VERSION = "2.4.0"
DEFAULT_VERSION = ::Overmind::VERSION

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

done!

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Missing steps to build tmux and what are pre-requisites (having Overmind itself already released on GitHub, right?)

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

thanks, I added to RELEASING.md

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Do we need a separate changelog?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I removed it

Comment thread packaging/rubygems/lib/overmind/cli.rb Outdated

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

If we want to always use bundled tmux, its path should be prepended, not appended

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

thanks, fixed!

Comment thread NOTICE Outdated

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

I believe this should be located in packaging/rubygems and should be included in the gem. Also, it should contain licenses of tmux deps

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Done!

@prog-supdex prog-supdex requested review from DarthSim and Envek March 20, 2024 14:34
@Envek Envek merged commit 6b30a8e into DarthSim:master Mar 25, 2024
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