Bug 460168

Summary: FedoraDS' adminutil requires non-existent "icu.pc" on non-RH/Fedora OS
Product: [Retired] 389 Reporter: bwailea+11
Component: Directory ServerAssignee: Rich Megginson <rmeggins>
Status: CLOSED CURRENTRELEASE QA Contact: Viktor Ashirov <vashirov>
Severity: medium Docs Contact:
Priority: medium    
Version: 1.1.1CC: jgalipea, srl
Target Milestone: ---Keywords: VerifiedUpstream
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2015-12-07 16:50:49 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 543590    
Attachments:
Description Flags
patch nkinder: review+

Description bwailea+11 2008-08-26 15:57:54 UTC
FedoraDS "Building" docs @,

	http://directory.fedoraproject.org/wiki/Building#External_Requirements

state,

	" ... The directory server configure script will first attempt to find these components using pkg-config (e.g. pkg-config --libs nspr) or the package specific config script (e.g. icu-config or net-snmp-config) ..."

Build/install of 'adminutil' however installs creates "adminutil.pc",

	cat /usr/lib/pkgconfig/adminutil.pc
		prefix=/usr
		exec_prefix=/usr
		libdir=${exec_prefix}/lib
		includedir=${prefix}/include
		
		Name: adminutil
		Description: Utility library for directory server administration
		Version: 1.1.6
		Requires: nspr, nss, svrcore, mozldap, icu
		Libs: -ladmsslutil -ladminutil
		Cflags: -I${includedir}/admsslutil -I${includedir}/adminutil

which instructs pkg-config to "require", and fails for lack of, icu.pc,

	pkg-config --libs adminutil
		Package adminutil was not found in the pkg-config search path.
		Perhaps you should add the directory containing `adminutil.pc'
		to the PKG_CONFIG_PATH environment variable
		No package 'adminutil' found
		containing a "Require" for icu -- 

installation of ICU from raw source creates/installs *no* icu.pc, rather icu-config.

rpm-install of icu on OpenSuse does the same -- no icu.pc.

afaict, only redhat/fedora seem to install/require the "non-standard" icu.pc, despite the docs.

modding adminutil.pc,

	-	Requires: nspr, nss, svrcore, mozldap, icu
	+	Requires: nspr, nss, svrcore, mozldap

removes the 'fail'.  as for downstream,

richm @ #fedora-ds
	"the only packages that use adminutil are fedora-ds-admin and fedora-ds-dsgw - and both of those require icu separately, so removing that Requires should cause no problems."

nonetheless, DS sources should depend on standard/documented icu-config, rather than on mfg'd icu.pc.

Comment 1 Rich Megginson 2010-02-26 19:21:34 UTC
Created attachment 396645 [details]
patch

Comment 2 Rich Megginson 2010-02-26 23:10:59 UTC
commit 2720ccb5a5637ec9224ef180437f68505d07135a
Author: Rich Megginson <rmeggins>
Date:   Fri Feb 26 12:19:15 2010 -0700

    Reviewed by: nkinder (Thanks!)
    Branch: HEAD

Comment 3 Fedora Update System 2010-02-27 02:04:25 UTC
389-adminutil-1.1.10-1.fc12 has been submitted as an update for Fedora 12.
http://admin.fedoraproject.org/updates/389-adminutil-1.1.10-1.fc12

Comment 4 Fedora Update System 2010-02-27 02:04:30 UTC
389-adminutil-1.1.10-1.fc13 has been submitted as an update for Fedora 13.
http://admin.fedoraproject.org/updates/389-adminutil-1.1.10-1.fc13

Comment 5 Fedora Update System 2010-02-27 02:04:34 UTC
389-adminutil-1.1.10-1.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/389-adminutil-1.1.10-1.fc11

Comment 8 Steven R. Loomis 2012-08-15 23:38:32 UTC
(In reply to comment #0)
> installation of ICU from raw source creates/installs *no* icu.pc, rather
> icu-config.

That's because the downstream (RedHat) went ahead and created icu.pc without contributing the patch back up. 

When I implemented http://bugs.icu-project.org/trac/ticket/6981 we created separate .pc's, because there are multiple patches.

I filed a ticket in ICU, http://bugs.icu-project.org/trac/ticket/9478 to create a deprecated icu.pc for compatibility.