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.

Sign up for GitHub

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

dbus: add manpages and missing build dependencies #6835

Closed
wants to merge 5 commits into from
Closed

dbus: add manpages and missing build dependencies #6835

Changes from 4 commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
ddfa122
dbus: add manpages and missing build dependencies
Nov 11, 2016
5306718
dbus: restore skip-bootstrapping-unless-on-head behavior
Nov 12, 2016
199d47c
dbus: fix comment typo
Nov 12, 2016
aee0e5f
dbus: fix audit violations and move forward in version
Nov 12, 2016
4102da9
dbus: remove comment for docbook dir
Nov 12, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 13 additions & 1 deletion Formula/dbus.rb
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,10 +22,18 @@ class Dbus < Formula
url "https://anongit.freedesktop.org/git/dbus/dbus.git"

depends_on "autoconf" => :build
depends_on "autoconf-archive" => :build
depends_on "automake" => :build
depends_on "libtool" => :build
end

depends_on "xmlto" => :build

# Docbook is a dependency of xmlto, but the XML_CATALOG_FILES env-set in
# install() uses files created by docbook, so best to be explicit in case
# the install process for docbook ever stops creating those files.
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's drop this entire comment and the explicit docbook dependency. ascii, deheader, pgbouncer, reposurgeon, and shadowsocks-libev all have the same situation and none declares a docbook dependency.

depends_on "docbook" => :build

# Patch applies the config templating fixed in https://bugs.freedesktop.org/show_bug.cgi?id=94494
# Homebrew pr/issue: 50219
patch do
Expand All @@ -37,12 +45,16 @@ def install
# Fix the TMPDIR to one D-Bus doesn't reject due to odd symbols
ENV["TMPDIR"] = "/tmp"

# Manpages won't build without a current docbook catalog. This should exist
Copy link
Contributor

@ilovezfs ilovezfs Nov 12, 2016 •
edited
Loading

Choose a reason for hiding this comment

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

This comment is not needed. There are 29 cases just like this in core.

# if xmlto and docbook (build dependencies of this package) are installed.
Copy link
Contributor

Choose a reason for hiding this comment

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

This comment is not needed. There are 29 cases just like this in core.

ENV["XML_CATALOG_FILES"] = "#{etc}/xml/catalog"

system "./autogen.sh", "--no-configure" if build.head?
system "./configure", "--disable-dependency-tracking",
"--prefix=#{prefix}",
"--localstatedir=#{var}",
"--sysconfdir=#{etc}",
"--disable-xml-docs",
"--enable-xml-docs",
"--disable-doxygen-docs",
"--enable-launchd",
"--with-launchd-agent-dir=#{prefix}",
Expand Down