Bug 1494096

Summary: Use mariadb-connector-c-devel instead of mysql-devel or mariadb-devel
Product: [Fedora] Fedora Reporter: Michal Schorm <mschorm>
Component: gdalAssignee: Pavel Raiskup <praiskup>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: acaringi, alex, devrim, jmlich83, mmahut, orion, pertusus, praiskup, volker27
Target Milestone: ---Keywords: Reopened
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2017-12-19 07:29:21 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1486480, 1490492    
Attachments:
Description Flags
Patch to allow the build of gdal with MariaDB Connector/C 3.x
none
patch to spec file none

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.