Skip to content
This repository has been archived by the owner on Jul 4, 2023. It is now read-only.

Added gupnp-tools 0.8.10 (new formula) #40483

Closed
wants to merge 1 commit into from
Closed

Conversation

ened
Copy link
Contributor

@ened ened commented Jun 8, 2015

Provides a collection of developer tools utilising GUPnP and GTK+.

"--prefix=#{prefix}"
system "make", "install"
end
end
Copy link
Member

Choose a reason for hiding this comment

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

Can a test be added to do something more substantial than e.g. --version or --help? See cmake.rb for an example of an application formula with a good test and tinyxml2.rb for an example of a library formula with a good test. Thanks!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Difficult - as the formula is for a GUI tool. Suggestion?

@MikeMcQuaid
Copy link
Member

@MikeMcQuaid
Copy link
Member

Thanks for your contribution to Homebrew! Without people like you submitting PRs we couldn't run this project. You rock!

For future reference the preferred commit message format for new formulae is gupnp-tools 0.8.10 (new formula).

@bfontaine
Copy link
Contributor

Just saw this, isn’t it the same formula as the one in homebrew-x11?

@DomT4
Copy link
Member

DomT4 commented Jun 11, 2015

^ Yup, looks like it. We can remove it from x11 and remove any reference to migrating it in tap_migrations.

It was probably only moved to X11 in the first place because GTK+3 originally had an X11 backend (It no longer does). Either that or the X11 dependency is actually optional, in which case it's cool to stay in the core as well.

@DomT4
Copy link
Member

DomT4 commented Jun 16, 2015

DomT4 added a commit that referenced this pull request Jun 16, 2015
This formula was moved back to the core in #40483
@Homebrew Homebrew locked and limited conversation to collaborators Jul 10, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants