Bug 2241091 - [Tracker] Packages that require the MariaDB client library should use mariadb-connector-c
Summary: [Tracker] Packages that require the MariaDB client library should use mariadb...
Keywords:
Status: ASSIGNED
Alias: None
Product: Fedora
Classification: Fedora
Component: mariadb-connector-c
Version: 40
Hardware: All
OS: Linux
unspecified
medium
Target Milestone: ---
Assignee: Michal Schorm
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On: 2242386 2242380 2242381 2242382 2242383 2242385
Blocks: 2245191
TreeView+ depends on / blocked
 
Reported: 2023-09-28 03:32 UTC by Michal Schorm
Modified: 2024-02-15 22:58 UTC (History)
7 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2023-10-12 10:43:14 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Michal Schorm 2023-09-28 03:32:51 UTC
There is still a number of packages, that builds against the 'mariadb-devel' package, instead of the 'mariadb-connector-c-devel'.

The 'mariadb-devel' package brings in the 'mariadb-connector-c-devel' as a dependency. Most packages then builds on top of the 'mariadb-connector-c-devel' actually, without knowing.

The one and only case in which the 'mariadb-devel' package should be used, is for building the MariaDB server plugins. Which nearly no package does. Everyone else, who needs to compile against the MariaDB client library, should use 'mariadb-connector-c-devel' package.

---

Historically, in 2017, when the MariaDB client library was first separated by the MariaDB upstream, creating the 'mariadb-connector-c' package, I tried to move as much packages as possible to build against 'mariadb-connector-c-devel'.

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

---

However nowadays, I want to stop building i686 MariaDB variant, as per:
  https://fedoraproject.org/wiki/Changes./EncourageI686LeafRemoval
but I can't, since there still are some packages who wrongly requires 'mariadb-devel' package, blocking the process of removing i686 arch.

Note: I have no plans to stop building i686 variants of 'mariadb-connector-c' package. It's far from being a leaf package.

The goal of this ticket is to track fixing the situation in all affected packages, and pinpointing the very few that builds the MariaDB server plugins, encouraging them to stop building i686 variants, so I could too.

---

# dnf --repo=rawhide --repo=rawhide-source repoquery --whatrequires mariadb-devel --alldeps
Macaulay2-0:1.21-5.fc39.src
addrwatch-0:1.0.2-7.fc39.src
coturn-0:4.6.2-3.fc39.src
exim-0:4.96-10.fc39.src
libgda-1:6.0.0-6.fc39.src
libnss-mysql-0:1.5-45.fc39.src
mariadb-embedded-devel-3:10.5.22-1.fc40.i686
mariadb-embedded-devel-3:10.5.22-1.fc40.x86_64
nagios-plugins-0:2.4.6-1.fc40.src
netatalk-5:3.1.17-1.fc40.src
slurm-0:22.05.9-5.fc40.src
trojan-0:1.16.0-20.fc39.src
vdr-epg-daemon-0:1.3.21-4.fc39.src

Comment 1 Michal Schorm 2023-09-28 08:16:42 UTC
Straightforward PRs filed:

addrwatch:        https://src.fedoraproject.org/rpms/addrwatch/pull-request/1#
coturn:           https://src.fedoraproject.org/rpms/coturn/pull-request/3
nagios-plugins:   https://src.fedoraproject.org/rpms/nagios-plugins/pull-request/8
netatalk:         https://src.fedoraproject.org/rpms/netatalk/pull-request/4
slurm:            https://src.fedoraproject.org/rpms/slurm/pull-request/6
trojan:           https://src.fedoraproject.org/rpms/trojan/pull-request/1
vdr-epg-daemon:   https://src.fedoraproject.org/rpms/vdr-epg-daemon/pull-request/1



More complicated cases remain:

exim              Brings in 'mariadb-devel' for pkgconfig
libgda            Fails to build /usr/lib64/libgda-6.0/providers/libgda-mysql-6.0.so
libnss-mysql      Changed from mariadb-connector-c-devel to mariadb-devel before, beacuse of user report: RHBZ#1495998, need to investigate.
Macaulay2         Builds for hours, I didn't have time to verify it yet

Comment 2 Andrew Bauer 2023-09-28 12:03:10 UTC
This affects RPMFusion as well:

$ dnf --repo=rpmfusion-free --repo=rpmfusion-free-source repoquery --whatrequires mariadb-devel --alldeps
kodi-0:20.1-1.fc38.src
zoneminder-0:1.36.33-3.fc38.src


I can update the zoneminder package and will alert the kodi package maintainer.

Does this change affect fedora rawhide (40) only, or will it also affect older fedoras and epel?
Zoneminder, for example, is still being maintained as far back as el7.

Comment 3 Michal Schorm 2023-09-28 12:23:05 UTC
Thank you for the heads-up, Andrew !

The change makes sense to be applied everywhere in Fedora >~= 30.
However, since I can't drop support of i686 out of nowhere in already released Fedora versions, I'm completely fine with making the change just to Fedora Rawhide.

So IMO there's no need to touch EPEL or other products / branches.

Comment 4 Michal Schorm 2023-09-28 12:31:13 UTC
Additional PRs created:

exim              https://src.fedoraproject.org/rpms/exim/pull-request/15
libgda            https://src.fedoraproject.org/rpms/libgda/pull-request/3
Macaulay2         https://src.fedoraproject.org/rpms/Macaulay2/pull-request/2
                  * Macaulay2 FTBFS on Rawhide and F39, need a fix before merging

Last complicated case that remain:

libnss-mysql      Changed from mariadb-connector-c-devel to mariadb-devel before, beacuse of user report: RHBZ#1495998, need to investigate.

Comment 5 Michal Schorm 2023-09-28 12:52:16 UTC
Additional PRs created:

libnss-mysql      https://src.fedoraproject.org/rpms/libnss-mysql/pull-request/1
                  * I haven't found any convincing argument to stick to the revert done based on the historical user report. IMO it can be merged without any issue today.

Comment 6 Andrew Bauer 2023-09-28 13:16:00 UTC
FYI, for the kodi package:
https://bugzilla.rpmfusion.org/show_bug.cgi?id=6771



After some testing, mariadb-connector-c-devel is available as far back as el8. I will adjust zoneminder and netatalk accordingly.

Comment 7 Andrew Bauer 2023-09-28 14:43:50 UTC
zoneminder and netatalk packages have been fixed

Comment 8 Michael Cronenworth 2023-09-28 19:23:49 UTC
The kodi package has been fixed.

Comment 9 Michal Schorm 2023-10-05 19:39:58 UTC
Packages fixed:
  Fedora:
    coturn
    netatalk
    vdr-epg-daemon
    libgda
    libnss-mysql

  RPMFusion:
    kodi
    zoneminder

I thank very much for a very swift response for all the affected maintainers
Thank you Andrew for addresing the RPMFusion.

---

Packages remaining:
    addrwatch       https://bugzilla.redhat.com/show_bug.cgi?id=2242380        https://src.fedoraproject.org/rpms/addrwatch/pull-request/1
    exim            https://bugzilla.redhat.com/show_bug.cgi?id=2242382        https://src.fedoraproject.org/rpms/exim/pull-request/15
    Macaulay2       https://bugzilla.redhat.com/show_bug.cgi?id=2242383        https://src.fedoraproject.org/rpms/Macaulay2/pull-request/2
    nagios-plugins  https://bugzilla.redhat.com/show_bug.cgi?id=2242381        https://src.fedoraproject.org/rpms/nagios-plugins/pull-request/8
    slurm           https://bugzilla.redhat.com/show_bug.cgi?id=2242385        https://src.fedoraproject.org/rpms/slurm/pull-request/6
    trojan          https://bugzilla.redhat.com/show_bug.cgi?id=2242386        https://src.fedoraproject.org/rpms/trojan/pull-request/1

I have created a Bugzilla ticket for each package as a polite heads-up, since not every package maintainer watches the Pagure pull requests on regular basis.

Comment 10 Michal Schorm 2023-10-12 02:31:55 UTC
A week passed and 3 more packages were fixed: addrwatch, nagios-plugins, trojan.

I'm adding a needinfo into the BZs of the remaining packages, in a hope the respective packagers will notice.

Packages remaining:
    exim            https://bugzilla.redhat.com/show_bug.cgi?id=2242382        https://src.fedoraproject.org/rpms/exim/pull-request/15
    Macaulay2       https://bugzilla.redhat.com/show_bug.cgi?id=2242383        https://src.fedoraproject.org/rpms/Macaulay2/pull-request/2
    slurm           https://bugzilla.redhat.com/show_bug.cgi?id=2242385        https://src.fedoraproject.org/rpms/slurm/pull-request/6

Comment 11 Fedora Update System 2023-10-12 10:39:55 UTC
FEDORA-2023-cb19fb73d3 has been submitted as an update to Fedora 40. https://bodhi.fedoraproject.org/updates/FEDORA-2023-cb19fb73d3

Comment 12 Fedora Update System 2023-10-12 10:43:14 UTC
FEDORA-2023-cb19fb73d3 has been pushed to the Fedora 40 stable repository.
If problem still persists, please make note of it in this bug report.

Comment 13 Jaroslav Škarvada 2023-10-12 10:44:29 UTC
Sorry, I used bad BZ reference in the exim fix.

Comment 14 Michal Schorm 2023-10-19 10:44:30 UTC
A week passed and 1 more package was fixed: exim

I haven't got a response withing a week from the 'needinfo' BZs, so I've sent the maintainers a direct e-mails to the addresses of the 'needinfo's.

Packages remaining:
    Macaulay2       https://bugzilla.redhat.com/show_bug.cgi?id=2242383        https://src.fedoraproject.org/rpms/Macaulay2/pull-request/2
    slurm           https://bugzilla.redhat.com/show_bug.cgi?id=2242385        https://src.fedoraproject.org/rpms/slurm/pull-request/6

Comment 15 Philip Kovacs 2023-10-21 22:09:45 UTC
Sorry for the delay -- slurm now builds against mariadb-connector-c-devel in rawhide not maridb-devel.

Comment 16 Aoife Moloney 2024-02-15 22:58:32 UTC
This bug appears to have been reported against 'rawhide' during the Fedora Linux 40 development cycle.
Changing version to 40.


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