Bug 2325989

There are some builds in epel10-testing that rely on this getting resolved.

https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2024-24d55505b5

https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2024-048936c915

maybe we can remove the require temporarily in https://src.fedoraproject.org/rpms/gdal/blob/rawhide/f/gdal.spec#_235 
because we use gdal as lib , not as runtime I guess

I've submitted gdal-3.10.0-2.el10_0: https://koji.fedoraproject.org/koji/taskinfo?taskID=126032444

From what I can tell skimming the upstream code, the gpsbabel command is a dependency of the GPSSBabel driver [0], which is enabled by default [1].  Therefore merely skipping the dependency is not valid.  Please revert the change in gdal-3.10.0-2 and lets wait for gpsbabel to be available.


[0] https://github.com/OSGeo/gdal/blob/v3.10.0/ogr/ogrsf_frmts/gpsbabel/ogrgpsbabeldriver.cpp#L96
[1] https://github.com/OSGeo/gdal/blob/v3.10.0/frmts/drivers.ini#L235

(In reply to Carl George 🤠 from comment #4)
> From what I can tell skimming the upstream code, the gpsbabel command is a
> dependency of the GPSSBabel driver [0], which is enabled by default [1]. 
> Therefore merely skipping the dependency is not valid.  Please revert the
> change in gdal-3.10.0-2 and lets wait for gpsbabel to be available.
> 
> 
> [0]
> https://github.com/OSGeo/gdal/blob/v3.10.0/ogr/ogrsf_frmts/gpsbabel/
> ogrgpsbabeldriver.cpp#L96
> [1] https://github.com/OSGeo/gdal/blob/v3.10.0/frmts/drivers.ini#L235


bGPSBabelFound = VSIStatL("/usr/bin/gpsbabel", &sStat) == 0;
if (!bGPSBabelFound)

Carl George , I understand your concern , but we are bootstrapping epel10,  we need land opencv etc and BTW to me it seems that the code deals well without gpsbabel .

The bootstrapping period for EPEL 10 is over, that's why the testing repo is enabled now instead of having updates auto-push to stable.  We're preparing to do a launch announcement very soon.  Now is the time to be fixing uninstallable packages, not pushing more of them from testing to stable, or introducing hacks to let a package install with broken functionality.

shapelib is now built for epel10.
It has been tagged into override and is available in epel10-build.
Let's get this done.

we need gpsbabel , not shapelib

Ping - Can gpsbabel be built on epel10.
I have verified that the Fedora rawhide version builds with no modifications on epel10.

If you do not wish to maintain gpsbabel in epel10,
or do not think you will be able to do this in a timely manner,
I would be happy to be a co-maintainer of the package (FAS tdawson);
please add me through https://src.fedoraproject.org/rpms/gpsbabel/adduser

> we need gpsbabel , not shapelib

BuildRequires: shapelib-devel

https://src.fedoraproject.org/rpms/gpsbabel/blob/rawhide/f/gpsbabel.spec#_19

That's why this bug depends on bug 2327523.

FEDORA-EPEL-2024-1c372b1e82 (gpsbabel-1.9.0-7.el10_0) has been submitted as an update to Fedora EPEL 10.0.
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2024-1c372b1e82

FEDORA-EPEL-2024-1c372b1e82 (gpsbabel-1.9.0-7.el10_0) has been pushed to the Fedora EPEL 10.0 stable repository.
If problem still persists, please make note of it in this bug report.

(In reply to Sergio Basto from comment #5)
> (In reply to Carl George 🤠 from comment #4)
> > From what I can tell skimming the upstream code, the gpsbabel command is a
> > dependency of the GPSSBabel driver [0], which is enabled by default [1]. 
> > Therefore merely skipping the dependency is not valid.  Please revert the
> > change in gdal-3.10.0-2 and lets wait for gpsbabel to be available.
> > 
> > 
> > [0]
> > https://github.com/OSGeo/gdal/blob/v3.10.0/ogr/ogrsf_frmts/gpsbabel/
> > ogrgpsbabeldriver.cpp#L96
> > [1] https://github.com/OSGeo/gdal/blob/v3.10.0/frmts/drivers.ini#L235
> 
> 
> bGPSBabelFound = VSIStatL("/usr/bin/gpsbabel", &sStat) == 0;
> if (!bGPSBabelFound)
> 
> Carl George , I understand your concern , but we are bootstrapping epel10, 
> we need land opencv etc and BTW to me it seems that the code deals well
> without gpsbabel .

I'd like to suggest instead, Requires: gpsbabel, change to, Recommends: gpsbabel
I believe is more appropriated, in general, packages not need gpsbabel to build .

The needinfo request[s] on this closed bug have been removed as they have been unresolved for 120 days