Bug 1129632 - FTBFS without ModemManager
Summary: FTBFS without ModemManager
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: NetworkManager
Version: 21
Hardware: s390x
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Dan Williams
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: ZedoraTracker
TreeView+ depends on / blocked
 
Reported: 2014-08-13 11:56 UTC by Dan Horák
Modified: 2014-08-18 15:11 UTC (History)
2 users (show)

Fixed In Version: NetworkManager-0.9.10.0-4.git20140704.fc22
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2014-08-18 15:11:06 UTC


Attachments (Terms of Use)

Description Dan Horák 2014-08-13 11:56:00 UTC
starting with NetworkManager-0.9.9.95-1.git20140609.fc21 NetworkManager fails to build on s390(x) because it made ModemManager-glib-devel a hard requirement.

from build.log
...
checking for NSS... yes
checking pppd/pppd.h usability... yes
checking pppd/pppd.h presence... yes
checking for pppd/pppd.h... yes
checking for pppd... /usr/sbin/pppd
checking for pppoe... no
checking for MM_GLIB... no
configure: error: Couldn't find libmm-glib
error: Bad exit status from /var/tmp/rpm-tmp.mqROss (%build)
RPM build errors:
    Bad exit status from /var/tmp/rpm-tmp.mqROss (%build)
Child return code was: 1
EXCEPTION: Command failed. See logs for output.
...

for full logs please see http://s390.koji.fedoraproject.org/koji/taskinfo?taskID=1488265


Either MM-glib must be an optional dependency in configure or we need

eg.
diff --git a/NetworkManager.spec b/NetworkManager.spec
index 969cede..301a52b 100644
--- a/NetworkManager.spec
+++ b/NetworkManager.spec
@@ -145,7 +145,7 @@ BuildRequires: wimax-devel
 BuildRequires: systemd >= 200-3 systemd-devel
 BuildRequires: libsoup-devel
 BuildRequires: libndp-devel >= 1.0
-%if 0%{?with_wwan} && (0%{?rhel} || (0%{?fedora} && 0%{?fedora} > 19))
+%if 0%{?rhel} || (0%{?fedora} && 0%{?fedora} > 19)
 BuildRequires: ModemManager-glib-devel >= 1.0
 %endif
 %if 0%{?with_nmtui}


Note You need to log in before you can comment on or make changes to this bug.