Bug 460168 - FedoraDS' adminutil requires non-existent "icu.pc" on non-RH/Fedora OS
Summary: FedoraDS' adminutil requires non-existent "icu.pc" on non-RH/Fedora OS
Status: CLOSED CURRENTRELEASE
Alias: None
Product: 389
Classification: Retired
Component: Directory Server
Version: 1.1.1
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Rich Megginson
QA Contact: Viktor Ashirov
URL:
Whiteboard:
Keywords: VerifiedUpstream
Depends On:
Blocks: 389_1.2.6
TreeView+ depends on / blocked
 
Reported: 2008-08-26 15:57 UTC by bwailea+11
Modified: 2015-12-07 16:50 UTC (History)
2 users (show)

(edit)
Clone Of:
(edit)
Last Closed: 2015-12-07 16:50:49 UTC


Attachments (Terms of Use)
patch (1.11 KB, patch)
2010-02-26 19:21 UTC, Rich Megginson
nkinder: review+
Details | Diff

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@redhat.com>
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.


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