Summary: | Build failled with mlpipelinemacros.h:53:68: error | ||
---|---|---|---|
Product: | [Applications] digikam | Reporter: | pascal lacroix <kikoucalou> |
Component: | Portability-Compilation | Assignee: | Digikam Developers <digikam-bugs-null> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | caulier.gilles, metzpinguin, michael_miller |
Priority: | NOR | ||
Version: | 8.6.0 | ||
Target Milestone: | --- | ||
Platform: | Mint (Ubuntu based) | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: | 8.6.0 | |
Sentry Crash Report: |
Description
pascal lacroix
2025-01-11 08:04:13 UTC
Hi Pascal, I completed a significant merge yesterday. The AI models, classifiers, and pipeline for Autotagging has been completely rewritten. I'll look at the pipeline macros to see why they won't compile for you. Cheers, Mike Pascal compiles with Qt5. I think qMax in the macros should be replaced with std:max for testing purposes. Maik (In reply to Maik Qualmann from comment #2) > Pascal compiles with Qt5. I think qMax in the macros should be replaced with > std:max for testing purposes. > > Maik Hi Maik, Good idea. I think the problem is the different types (int vs qint64). I was to commit a change that casts all calls to qMax to (qint64). I'll also change it to std::max. Hopefully the two changes together fix the issue. Cheers, Mike Hi Michael, Hmm, are you sure about qint64? I actually checked it before, both are actually int. Maik (In reply to Maik Qualmann from comment #4) > Hi Michael, > > Hmm, are you sure about qint64? I actually checked it before, both are > actually int. > > Maik One is an int, the other is a long long int > deduced conflicting types for parameter ‘const T’ (‘int’ and ‘qint64’ {aka ‘long long int’}) Cheers, Mike Révision sous Git : f80...6a69 https://invent.kde.org/graphics/digikam/commit/f8060cae7c9252bc56087264e738f19510816a69 build and run nicely on linux mint 22 Thanks ! (In reply to pascal lacroix from comment #6) > Révision sous Git : > f80...6a69 > https://invent.kde.org/graphics/digikam/commit/ > f8060cae7c9252bc56087264e738f19510816a69 > > build and run nicely on linux mint 22 > > Thanks ! Wonderful! I'm happy that fixed it for you. Cheers, Mike Hi Michael, I have several compile warnings here, unfortunately in German. In Datei, eingebunden von /home/maik/Devel/digikam/core/libs/mlfoundation/mlpipelinefoundation.h:36, von /home/maik/Devel/digikam/core/libs/mlfoundation/mlpipelinefoundation.cpp:16: /home/maik/Devel/digikam/core/libs/mlfoundation/mlpipelinemacros.h:82:1: Warnung: mehrzeiliger Kommentar [-Wcomment] 82 | // #define MLPIPELINE_CATCH(pipelineStageName) \ | ^ Maik (In reply to Maik Qualmann from comment #8) > Hi Michael, > > I have several compile warnings here, unfortunately in German. > > In Datei, eingebunden von > /home/maik/Devel/digikam/core/libs/mlfoundation/mlpipelinefoundation.h:36, > von > /home/maik/Devel/digikam/core/libs/mlfoundation/mlpipelinefoundation.cpp:16: > /home/maik/Devel/digikam/core/libs/mlfoundation/mlpipelinemacros.h:82:1: > Warnung: mehrzeiliger Kommentar [-Wcomment] > 82 | // #define MLPIPELINE_CATCH(pipelineStageName) \ > | ^ > > Maik Hi Maik, No worries. It's been a long time since I learned German, but I still understand enough to see the problem. It should be fixed in my latest commit. Cheers, Mike |