Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cannot compile with XLSX #3686

Closed
gzotti opened this issue Mar 24, 2024 · 6 comments
Closed

Cannot compile with XLSX #3686

gzotti opened this issue Mar 24, 2024 · 6 comments
Labels
dependencies dependencies (such like ShowMySky, QXlsx, etc.) related issues importance: critical Real showstopper, program fails here
Milestone

Comments

@gzotti
Copy link
Member

gzotti commented Mar 24, 2024

Expected Behaviour

Activating ENABLE_XLSX in CMake configuration leads to successful compilation

Actual Behaviour

Compilation fails with 126 errors.
grafik

Maybe just some headers missing? I cannot say when I had disabled this, so unsure when significant changes were made. Sorry, no time currently to go hunting after this.
There are also warnings (C4821) about missing Unicode signatures (BOM).

Steps to reproduce

Setup QtCreator, MSVC2019, Qt6.5.3, configure as above.

System

  • Stellarium version: master (6c869cc)
  • Operating system: Windows 11
  • Qt version 6.5.3/MSVC2019
  • Graphics Card: irrelevant

Note

I hope CI works for 24.1, so I target 24.2 here.

@gzotti gzotti added importance: critical Real showstopper, program fails here dependencies dependencies (such like ShowMySky, QXlsx, etc.) related issues labels Mar 24, 2024
@gzotti gzotti added this to the 24.2 milestone Mar 24, 2024
@10110111
Copy link
Contributor

What does xlsxcellformula.h contain? In which encoding? Normally it contains the definition of class CellFormula and should be in ASCII with UNIX line endings. It builds perfectly for me on Linux/GCC.

@alex-w
Copy link
Member

alex-w commented Mar 24, 2024

I cannot reproduce the issue in Windows 10/MSVC2019/Qt 6.5.3

P.S. CI has same configuration, but without a problem: https://ci.appveyor.com/project/alex-w/stellarium/builds/49460702

@alex-w
Copy link
Member

alex-w commented Mar 24, 2024

@gzotti could you cleanup the building tree and check it again?

@gzotti
Copy link
Member Author

gzotti commented Mar 24, 2024

OK, after full cleanout (my attempts before only ran cmake, sorry I am short in time these weeks.) it seems we are back at the "usual" INDI warnings which should go away iff anybody steps up to INDI2....

[120/1068 7.5/sec] Building CXX object src\external\CMakeFiles\indiclient.dir\__\__\_deps\indiclient-src\libs\indibase\indiproperty.cpp.obj
[121/1068 7.5/sec] Building C object src\external\CMakeFiles\indiclient.dir\__\__\_deps\indiclient-src\libs\indicom.c.obj
D:\StelDev\GIT\build-stellarium-Desktop_Qt_6_5_3_MSVC2019_64bit-Release\_deps\indiclient-src\libs\indicom.h(45): warning C4005: "_USE_MATH_DEFINES": Makro-Neudefinition
Befehlszeile(1): note: Siehe vorherige Definition von "_USE_MATH_DEFINES"
[122/1068 7.5/sec] Building CXX object _deps\qxlsxqt6-build\CMakeFiles\QXlsx.dir\source\xlsxdocument.cpp.obj
[123/1068 7.5/sec] Building CXX object src\external\CMakeFiles\indiclient.dir\__\__\_deps\indiclient-src\libs\indibase\basedevice.cpp.obj
D:\StelDev\GIT\build-stellarium-Desktop_Qt_6_5_3_MSVC2019_64bit-Release\_deps\indiclient-src\libs\indicom.h(45): warning C4005: "_USE_MATH_DEFINES": Makro-Neudefinition
D:\StelDev\GIT\build-stellarium-Desktop_Qt_6_5_3_MSVC2019_64bit-Release\_deps\indiclient-src\libs\indibase\basedevice.cpp: note: Siehe vorherige Definition von "_USE_MATH_DEFINES"
[124/1068 7.5/sec] Linking CXX static library _deps\qxlsxqt6-build\QXlsxQt6.lib
[125/1068 7.1/sec] Building CXX object src\external\CMakeFiles\indiclient.dir\__\__\_deps\indiclient-src\libs\indibase\baseclient.cpp.obj


@gzotti gzotti closed this as completed Mar 24, 2024
@alex-w alex-w added the state: published The fix has been published for testing in weekly binary package label Apr 21, 2024
Copy link

Hello @gzotti!

Please check the fresh version (development snapshot) of Stellarium:
https://github.com/Stellarium/stellarium-data/releases/tag/weekly-snapshot

@alex-w alex-w removed the state: published The fix has been published for testing in weekly binary package label Jun 23, 2024
Copy link

Hello @gzotti!

Please check the latest stable version of Stellarium:
https://github.com/Stellarium/stellarium/releases/latest

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies dependencies (such like ShowMySky, QXlsx, etc.) related issues importance: critical Real showstopper, program fails here
Development

No branches or pull requests

3 participants