Bug 253677

Summary: compilation fails due to non existant /incorrect gps data
Product: [Applications] marble Reporter: Geoff Madden <mgmadden>
Component: generalAssignee: marble-bugs
Status: RESOLVED FIXED    
Severity: normal CC: nienhueser, rahn
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Gentoo Packages   
OS: Linux   
Latest Commit: Version Fixed In:
Attachments: emerge info file
machine info
build log for marble-4.5.2
environment file

Description Geoff Madden 2010-10-09 17:16:29 UTC
Created attachment 52368 [details]
emerge info file

Version:           4.5 (using Devel) 
OS:                Linux

I have retried compiling 4 times,added gpsd geos libgeotiff all to no avail. The compile falls over in the same place each time

Reproducible: Always

Steps to Reproduce:
emerge -u world,configuration completes ok and around 86% of the compile succeeds

Actual Results:  
stalled with error msg

Expected Results:  
complete compile and installation
Comment 1 Geoff Madden 2010-10-09 17:20:00 UTC
Created attachment 52369 [details]
machine info
Comment 2 Geoff Madden 2010-10-09 17:20:59 UTC
Created attachment 52370 [details]
build log for marble-4.5.2
Comment 3 Geoff Madden 2010-10-09 17:21:52 UTC
Created attachment 52371 [details]
environment file
Comment 4 Geoff Madden 2010-10-09 17:22:52 UTC
it'd be nice to finish the update
Comment 5 Dennis Nienhüser 2010-10-11 10:42:21 UTC
I need the version number of sci-geosciences/gpsd [1] installed on your system, please.

[1] http://packages.gentoo.org/package/sci-geosciences/gpsd
Comment 6 Geoff Madden 2010-10-12 16:26:06 UTC
(In reply to comment #5)
> I need the version number of sci-geosciences/gpsd [1] installed on your system,
> please.
> 
> [1] http://packages.gentoo.org/package/sci-geosciences/gpsd

2.39-r1
Comment 7 Geoff Madden 2010-10-12 17:50:43 UTC
(In reply to comment #6)
> (In reply to comment #5)
> > I need the version number of sci-geosciences/gpsd [1] installed on your system,
> > please.
> > 
> > [1] http://packages.gentoo.org/package/sci-geosciences/gpsd
> 
> 2.39-r1

I looked at the ebuild file again tonite ,noticed that gdal was also a dependency so installed that,but the compile fell over at the 97% mark GpsdPositionProvider.
Which I gather was around the same area.
Comment 8 Dennis Nienhüser 2010-10-12 17:57:33 UTC
2.38 and 2.39 both have eph
2.39 introduces the MAJOR_VERSION, sets it to 3.
2.90 introduces epx, epy and removes eph, but does not change MAJOR_VERSION even though that change was incompatible. Therefore we can't rely on MAJOR_VERSION only, but need another check. Checking if WATCH_ENABLED is #defined should be fine as that was introduced with 2.90.

@Torsten: You're still running a 2.3x version of gpsd, right? Can you test if it still works fine with the additional check for WATCH_ENABLE?

#if defined( GPSD_API_MAJOR_VERSION ) && ( GPSD_API_MAJOR_VERSION >= 3 ) 
should become 
#if defined ( WATCH_ENABLE ) && defined( GPSD_API_MAJOR_VERSION ) && ( GPSD_API_MAJOR_VERSION >= 3 ) 

@Geoff: Up- or downgrading to any other gpsd version should fix it for you if you don't want to wait for KDE 4.5.3 or a Gentoo patch.
Comment 9 Geoff Madden 2010-10-13 07:13:03 UTC
(In reply to comment #8)
> 2.38 and 2.39 both have eph
> 2.39 introduces the MAJOR_VERSION, sets it to 3.
> 2.90 introduces epx, epy and removes eph, but does not change MAJOR_VERSION
> even though that change was incompatible. Therefore we can't rely on
> MAJOR_VERSION only, but need another check. Checking if WATCH_ENABLED is
> #defined should be fine as that was introduced with 2.90.
> 
> @Torsten: You're still running a 2.3x version of gpsd, right? Can you test if
> it still works fine with the additional check for WATCH_ENABLE?
> 
> #if defined( GPSD_API_MAJOR_VERSION ) && ( GPSD_API_MAJOR_VERSION >= 3 ) 
> should become 
> #if defined ( WATCH_ENABLE ) && defined( GPSD_API_MAJOR_VERSION ) && (
> GPSD_API_MAJOR_VERSION >= 3 ) 
> 
> @Geoff: Up- or downgrading to any other gpsd version should fix it for you if
> you don't want to wait for KDE 4.5.3 or a Gentoo patch.

The update to 2.95 fixed the problem,I'm curious as to why this wasn't included into the update ebuild ,as this would have pulled in the later version ,and the problem wouldn't have arisen,still things are now progressing,,, Thank's