gogglesmm / gogglesmm

Goggles Music Manager
https://gogglesmm.dev
GNU General Public License v3.0
52 stars 13 forks source link

Update to FOX 1.7.67 APIs #90

Closed zx2c4 closed 4 years ago

gogglesguy commented 4 years ago

Thanks, I think some of these changes were already introduced before 1.7.67. Either way, first need to update the internal fox library (1.7.62) in order for these changes to be merged.

gogglesguy commented 4 years ago

I've updated the internal fox library to 1.7.67 and included the necessary changes to make everything compile. Won't be needing this specific PR. Thanks!

nerijus commented 3 years ago

I tried to build develop branch with fox-1.7.73, but it failed at:

[ 74%] Building CXX object src/CMakeFiles/gogglesmm.dir/GMPodcastSource.cpp.o
/home/nerijus/rpmbuild/BUILD/gogglesmm-1.2.1/src/GMPodcastSource.cpp: In function 'FX::FXString gm_rfc1123(FX::FXTime)':
/home/nerijus/rpmbuild/BUILD/gogglesmm-1.2.1/src/GMPodcastSource.cpp:62:43: error: invalid conversion from 'FX::FXTime' {aka 'long int'} to 'const FXchar*' {aka 'const char*'} [-fpermissive]
   62 |   str += FXSystem::universalTime("%T GMT",time);
      |                                           ^~~~
      |                                           |
      |                                           FX::FXTime {aka long int}
compilation terminated due to -Wfatal-errors.

Could you please update source to build with the latest FOX?