[Bug 494986] Review Request: mysqludf_xql - MySQL UDF library for XML output

bugzilla at redhat.com bugzilla at redhat.com
Fri May 8 12:06:38 UTC 2009


Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=494986





--- Comment #8 from Yanko Kaneti <yaneti at declera.com>  2009-05-08 08:06:37 EDT ---
(In reply to comment #7)
> Yanko: you don't need to call ldconfig; you're not installing a library, just a
> shared object.

Empirical evdence doesn't agree, (test rpm without ldconfig):
# service mysqld start
Starting MySQL:                                            [  OK  ]
[root at f10test ~]# rpm -Uhv
/home/rpmbuild/rpmbuild/RPMS/i386/mysqludf_xql-0.9.7-8.fc10.i386.rpm
Preparing...                ########################################### [100%]
   1:mysqludf_xql           ########################################### [100%]
[root at f10test ~]# mysql mysql -e 'CREATE FUNCTION lib_mysqludf_xql_info RETURNS
STRING SONAME "lib_mysqludf_xql.so";'
ERROR 1126 (HY000) at line 1: Can't open shared library 'lib_mysqludf_xql.so'
(errno: 22 lib_mysqludf_xql.so: cannot open shared object file: No such file or
directory)
[root at f10test ~]# ldconfig 
[root at f10test ~]# mysql mysql -e 'CREATE FUNCTION lib_mysqludf_xql_info RETURNS
STRING SONAME "lib_mysqludf_xql.so";'
[root at f10test ~]# 


> rm -rf $RPM_BUILD_ROOT/%{myplugdir}/*.{a,la}
> Why do you call a recursive rm here?

Just a habit. Doubt it will hurt anything, but the -r  can be removed.

> Isn't it possible to avoid build of the static library by passing
> --disable-static to configure?

not completely sufficient, thus just additional baggage.
error: Installed (but unpackaged) file(s) found:
   /usr/lib/mysql/lib_mysqludf_xql.la

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.




More information about the Fedora-package-review mailing list