Bug 1494096 - Use mariadb-connector-c-devel instead of mysql-devel or mariadb-devel
Summary: Use mariadb-connector-c-devel instead of mysql-devel or mariadb-devel
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: gdal
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Pavel Raiskup
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 1486480 1490492
TreeView+ depends on / blocked
 
Reported: 2017-09-21 12:56 UTC by Michal Schorm
Modified: 2017-12-19 07:29 UTC (History)
9 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2017-12-19 07:29:21 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)
Patch to allow the build of gdal with MariaDB Connector/C 3.x (1.03 KB, patch)
2017-10-20 15:32 UTC, Augusto Caringi
no flags Details | Diff
patch to spec file (1.80 KB, patch)
2017-10-20 15:33 UTC, Augusto Caringi
no flags Details | Diff

Description Michal Schorm 2017-09-21 12:56:17 UTC
Hello, MariaDB went through big changes and your package will need a patch in order to build and work properly.

I tried to solve the issue, but I wasn't successful yet (at the time of bug creation).
I'll look into it more and post here the patch eventualy.


This change is currently applicable to the Rawhide (f28) only.


You can check builds I tried in COPR:
https://copr.fedorainfracloud.org/coprs/g/db-sig/mariadb-10.2/packages/



For more information check the tracking bugzilla.

Comment 1 Augusto Caringi 2017-10-20 15:31:49 UTC
Hi,

    I'm attaching a patch to fix the build and also a patch with small changes that should be applied to the spec file.

    This patch is just a quick fix and I recommend to get in touch with upstream about this problem to allow them to make a generic fix.

    Link to built packages:

    https://copr.fedorainfracloud.org/coprs/g/db-sig/mariadb-10.2/build/645433/

Comment 2 Augusto Caringi 2017-10-20 15:32:57 UTC
Created attachment 1341300 [details]
Patch to allow the build of gdal with MariaDB Connector/C 3.x

Comment 3 Augusto Caringi 2017-10-20 15:33:26 UTC
Created attachment 1341301 [details]
patch to spec file

Comment 4 Pavel Raiskup 2017-12-13 13:01:03 UTC
(In reply to Augusto Caringi from comment #2)
> Created attachment 1341300 [details]
> Patch to allow the build of gdal with MariaDB Connector/C 3.x

Has the patch been proposed upstream?

Comment 5 Augusto Caringi 2017-12-13 13:32:56 UTC
(In reply to Pavel Raiskup from comment #4)
> (In reply to Augusto Caringi from comment #2)
> > Created attachment 1341300 [details]
> > Patch to allow the build of gdal with MariaDB Connector/C 3.x
> 
> Has the patch been proposed upstream?

Not by me...

My idea was just to quick fix the problem...

"This patch is just a quick fix and I recommend to get in touch with upstream about this problem to allow them to make a generic fix."

Also, there is another related ticket: #1490492

:)

Comment 6 Augusto Caringi 2017-12-13 13:54:40 UTC
(In reply to Pavel Raiskup from comment #4)
> (In reply to Augusto Caringi from comment #2)
> > Created attachment 1341300 [details]
> > Patch to allow the build of gdal with MariaDB Connector/C 3.x
> 
> Has the patch been proposed upstream?

Answering again... 

The problem was fixed upstream:

http://trac.osgeo.org/gdal/ticket/7079
http://trac.osgeo.org/gdal/wiki/Release/2.2.3-News

And this new updated version (2.2.3) is already in rawhide:

https://src.fedoraproject.org/rpms/gdal/c/552efb7294e952a01c7377fdb2fca19394c453d1?branch=master

Comment 7 Michal Schorm 2017-12-13 14:50:37 UTC
Looks fixed then.

Comment 8 Pavel Raiskup 2017-12-13 15:07:57 UTC
Not yet, buildrequires fix is needed and %bootstrap should be disabled.

Comment 9 Pavel Raiskup 2017-12-14 15:13:47 UTC
Proposed fix:
https://src.fedoraproject.org/rpms/gdal/pull-request/3

Testing build:
https://koji.fedoraproject.org/koji/taskinfo?taskID=23682707

I'll push to master and build new package on Monday, please stop
me if you feel it's not a good idea.

Comment 10 Pavel Raiskup 2017-12-19 07:29:21 UTC
(In reply to Pavel Raiskup from comment #9)
> Proposed fix:
> https://src.fedoraproject.org/rpms/gdal/pull-request/3

^^^ done.


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