Note: This is a public test instance of Red Hat Bugzilla. The data contained within is a snapshot of the live data so any changes you make will not be reflected in the production Bugzilla. Email is disabled so feel free to test any aspect of the site that you want. File any problems you find or give feedback at
Bug 1501933 - scriptlet issue: error: file /usr/lib64/mysql: No such file or directory
Summary: scriptlet issue: error: file /usr/lib64/mysql: No such file or directory
Alias: None
Product: Fedora
Classification: Fedora
Component: mariadb-connector-c
Version: rawhide
Hardware: Unspecified
OS: Unspecified
Target Milestone: ---
Assignee: Michal Schorm
QA Contact: Fedora Extras Quality Assurance
Depends On:
Blocks: 1486480
TreeView+ depends on / blocked
Reported: 2017-10-13 13:55 UTC by Pavel Raiskup
Modified: 2017-10-30 12:42 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Last Closed: 2017-10-30 12:42:29 UTC
Type: Bug

Attachments (Terms of Use)

Comment 1 Pavel Raiskup 2017-10-13 15:03:02 UTC
> %pre devel
> rpm -qf /usr/lib64/mysql       
Calling rpm from rpm is a bit scary :).

> if [ $? -eq 1 ]; then
>   if [ "$(ls -A /usr/lib64/mysql)" ]; then
>     mv /usr/lib64/mysql /usr/lib64/.mysql_update_tmp
>   fi
>   rm -rf /usr/lib64/{mysql*,mariadb*}
> fi

I'm not really sure what's the purpose of this stuff, could we remove that?

Comment 2 Michal Schorm 2017-10-16 12:58:41 UTC
Hi, I already know about this issue and I know it will need a chnage in some way.

* Several versions of mariadb package in several Fedora version had incorrectly
  set ownership over the "/usr/lib/mysql" directory.
* Several other versions of connector-c tried to install "/usr/lib/mysql" as a
  symlink to a directory, which broke even more stuff.

* Add scriptlet, which check, if user upgrades from somehow-broken mariadb or 
  mariadb-connector-c version and fix the issues, that block the clean upgrade.
  (Please note, that without such a scriplet, users would stick to broken verison 
  and wouldn't be able to update to new verison)

* Calling RPM from RPM: Well, should I rather 'rm -rf' in libdir without any 
  check? :)
* There should be more checks and work added, Like checking the "mariadb" 
  directory too
* Calling "rm -rf" inside libdir is unpleasant thing to do at all. However I must 
  find a way to fix broken mariadbs and connectors idstributed amongst users.

Comment 3 Michal Schorm 2017-10-28 00:46:55 UTC

Move mariadb library from "/usr/lib64/mysql" or "/usr/lib64/mariadb" directly to "/usr/lib64".

Scriptlet now only checks, if those locations are symlinks. In such a case, it removes them.

Comment 4 Pavel Raiskup 2017-10-29 08:27:26 UTC
A bit more comments (some unrelated), re-opening to not fill another bug:

 11 %if 0%{fedora} >= 28
 12 # RPM does not expand "less" correctly
 13 %else
 14 # mariadb-connector-c is a part of mariadb. *Every* file is meant to conflict.
 15 # From F28 the client library part was removed from mariadb and tis package is required instead.
 16 # On F<28 is the package present from both historical and compatibility reasons.
 17 Conflicts: mariadb-libs mariadb-devel
 18 %endif

What's the point in having this conditional?  Not only that it Conflicts,
I think you should also unconditionally obsolete them (to actually enforce
that those packages are uninstalled automatically for any upgrade path).

Shouldn't actually devel obsolete devel, and libs obsolete mariadb-libs? 

 28 %if 0%{?__isa_bits} == 64
 29 Provides:
 30 Provides:
 31 Provides:
 32 %else
 33 Provides:
 34 Provides:
 35 Provides:
 36 %endif # isa_bits

This seems weird.  What's the point to provide sonames, if we do not even
provide compatibility symlinks?  Dependency resolvers are just "cheated".

 99 # Remove static linked libraries and symlinks to them
100 rm %{buildroot}%{_libdir}/libmariadbclient.a
101 rm %{buildroot}%{_libdir}/libmysqlclient.a
102 rm %{buildroot}%{_libdir}/libmysqlclient_r.a

As a good security measure, do `find %buildroot -name '*.a' -delete.

Comment 5 Pavel Raiskup 2017-10-29 08:31:39 UTC
107 %pre devel
108 if [ -h %{_libdir}/mariadb ]; then
109   unlink %{_libdir}/mariadb
110 fi
111 if [ -h %{_libdir}/mysql ]; then
112   unlink %{_libdir}/mysql
113 fi

Per [1], it seems like we should use %pretrans instead.  I would very much
appreciate if there was citation where the idea comes from (proving that
we don't do ugly hacks, but just implementing known work-around).


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