Bug 2325989 - Please branch and build gpsbabel for EPEL 10
Summary: Please branch and build gpsbabel for EPEL 10
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora EPEL
Classification: Fedora
Component: gpsbabel
Version: epel10
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Tom.Rix
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On: 2327523
Blocks: EPEL10Tracker 2318296 2327519
TreeView+ depends on / blocked
 
Reported: 2024-11-13 18:04 UTC by Xavier Bachelot
Modified: 2025-04-07 04:25 UTC (History)
8 users (show)

Fixed In Version: gpsbabel-1.9.0-7.el10_0
Clone Of:
Environment:
Last Closed: 2024-12-03 01:40:42 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Xavier Bachelot 2024-11-13 18:04:54 UTC
Hi,

Could you please branch and build gpsbabel for EPEL 10 ?
It is needed to build another package.

Regards,
Xavier

Comment 1 Carl George 🤠 2024-11-19 07:53:42 UTC
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

Comment 2 Sergio Basto 2024-11-19 22:18:03 UTC
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

Comment 3 Sandro Mani 2024-11-19 23:02:42 UTC
I've submitted gdal-3.10.0-2.el10_0: https://koji.fedoraproject.org/koji/taskinfo?taskID=126032444

Comment 4 Carl George 🤠 2024-11-19 23:09:51 UTC
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

Comment 5 Sergio Basto 2024-11-20 02:17:40 UTC
(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 .

Comment 6 Carl George 🤠 2024-11-20 05:50:52 UTC
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.

Comment 7 Troy Dawson 2024-11-20 14:29:26 UTC
shapelib is now built for epel10.
It has been tagged into override and is available in epel10-build.
Let's get this done.

Comment 8 Sergio Basto 2024-11-21 14:36:39 UTC
we need gpsbabel , not shapelib

Comment 9 Troy Dawson 2024-11-21 14:46:57 UTC
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

Comment 10 Carl George 🤠 2024-11-21 19:24:02 UTC
> 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.

Comment 11 Fedora Update System 2024-12-02 14:13:12 UTC
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

Comment 12 Fedora Update System 2024-12-03 01:40:42 UTC
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.

Comment 13 Sergio Basto 2024-12-07 05:29:34 UTC
(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 .

Comment 14 Red Hat Bugzilla 2025-04-07 04:25:03 UTC
The needinfo request[s] on this closed bug have been removed as they have been unresolved for 120 days


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