Fix FTBFS due to redefine the 'round' symbol in korganizer. #48

Merged
SlavekB merged 1 commits from fix/korganizer-round-redefinition into master 4 years ago
Owner

Unfortunately, this emerged as a new issue during the R14.0.9 build test on the newly released FreeBSD 12.2. Because R14.0.9 is already tagged for the final release, it is not possible to incorporate this fix at this time.

Unfortunately, this emerged as a new issue during the R14.0.9 build test on the newly released FreeBSD 12.2. Because R14.0.9 is already tagged for the final release, it is not possible to incorporate this fix at this time.
SlavekB added this to the R14.0.10 release milestone 4 years ago
Owner

Same as for the "log" PR, I find this problem a bit weird. Why it does not show up in other distros? Perhaps in BSD we are using different building option?

Same as for the "log" PR, I find this problem a bit weird. Why it does not show up in other distros? Perhaps in BSD we are using different building option?
Poster
Owner

Here round conflicts with the function defined in math.h:

FAILED: korganizer/printing/CMakeFiles/korg_stdprinting-shared.dir/calprintpluginbase.cpp.o
/usr/bin/c++ -DHAVE_CONFIG_H -D_FILE_OFFSET_BITS=64 -Dkorg_stdprinting_shared_EXPORTS -Ikorganizer/printing -I../ -I../korganizer/interfaces -I../libtdepim -I/opt/trinity/include -I/usr/local/include/tqt3 -I/usr/local/include/tqt -I/usr/local/include -g  -DQT_NO_ASCII_CAST -DQT_CLEAN_NAMESPACE -DQT_NO_STL -DQT_NO_COMPAT -DQT_NO_TRANSLATION -DQT_THREAD_SUPPORT -D_REENTRANT -I/usr/local/include/tqt3 -I/usr/local/include/tqt -include tqt.h -DQT_NO_ASCII_CAST -DQT_CLEAN_NAMESPACE -DQT_NO_STL -DQT_NO_COMPAT -DQT_NO_TRANSLATION -DQT_THREAD_SUPPORT -D_REENTRANT -includetqt.h -g -fPIC -MD -MT korganizer/printing/CMakeFiles/korg_stdprinting-shared.dir/calprintpluginbase.cpp.o -MF korganizer/printing/CMakeFiles/korg_stdprinting-shared.dir/calprintpluginbase.cpp.o.d -o korganizer/printing/CMakeFiles/korg_stdprinting-shared.dir/calprintpluginbase.cpp.o -c ../korganizer/printing/calprintpluginbase.cpp
../korganizer/printing/calprintpluginbase.cpp:41:12: error: functions that differ only in their return type cannot be overloaded
inline int round(const double x)
       ~~~ ^
/usr/include/math.h:318:8: note: previous declaration is here
double  round(double);
~~~~~~  ^
Here `round` conflicts with the function defined in `math.h`: ``` FAILED: korganizer/printing/CMakeFiles/korg_stdprinting-shared.dir/calprintpluginbase.cpp.o /usr/bin/c++ -DHAVE_CONFIG_H -D_FILE_OFFSET_BITS=64 -Dkorg_stdprinting_shared_EXPORTS -Ikorganizer/printing -I../ -I../korganizer/interfaces -I../libtdepim -I/opt/trinity/include -I/usr/local/include/tqt3 -I/usr/local/include/tqt -I/usr/local/include -g -DQT_NO_ASCII_CAST -DQT_CLEAN_NAMESPACE -DQT_NO_STL -DQT_NO_COMPAT -DQT_NO_TRANSLATION -DQT_THREAD_SUPPORT -D_REENTRANT -I/usr/local/include/tqt3 -I/usr/local/include/tqt -include tqt.h -DQT_NO_ASCII_CAST -DQT_CLEAN_NAMESPACE -DQT_NO_STL -DQT_NO_COMPAT -DQT_NO_TRANSLATION -DQT_THREAD_SUPPORT -D_REENTRANT -includetqt.h -g -fPIC -MD -MT korganizer/printing/CMakeFiles/korg_stdprinting-shared.dir/calprintpluginbase.cpp.o -MF korganizer/printing/CMakeFiles/korg_stdprinting-shared.dir/calprintpluginbase.cpp.o.d -o korganizer/printing/CMakeFiles/korg_stdprinting-shared.dir/calprintpluginbase.cpp.o -c ../korganizer/printing/calprintpluginbase.cpp ../korganizer/printing/calprintpluginbase.cpp:41:12: error: functions that differ only in their return type cannot be overloaded inline int round(const double x) ~~~ ^ /usr/include/math.h:318:8: note: previous declaration is here double round(double); ~~~~~~ ^ ```
MicheleC approved these changes 4 years ago
SlavekB merged commit 1762272863 into master 4 years ago
SlavekB deleted branch fix/korganizer-round-redefinition 4 years ago

Reviewers

MicheleC approved these changes 4 years ago
The pull request has been merged as 1762272863.
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: TDE/tdepim#48
Loading…
There is no content yet.