Skip to content

Fixed missing include statement

Nathan Misner requested to merge ndiddy/discover:fix_missing_include into master

This commit fixes a compiler error when building Discover on the development version of KDE Neon by adding an include statement for the QDir header. Here's what the compiler error looks like if anyone's curious:

/home/nathan/kde/src/discover/discover/main.cpp: In function ‘void processArgs(QCommandLineParser*, DiscoverObject*)’:
/home/nathan/kde/src/discover/discover/main.cpp:67:111: error: ‘QDir’ has not been declared
   67 |         discoverObject->openLocalPackage(QUrl::fromUserInput(parser->value(QStringLiteral("local-filename")), QDir::currentPath(), QUrl::AssumeLocalFile));
      |                                                                                                               ^~~~
/home/nathan/kde/src/discover/discover/main.cpp:72:51: error: ‘QDir’ has not been declared
   72 |         const QUrl url = QUrl::fromUserInput(arg, QDir::currentPath(), QUrl::AssumeLocalFile);
      |

I'm not sure why you're not seeing an error in your CI, maybe a Qt update changed how some of the headers are organized?

Merge request reports