D17278: Meson plugin: First working version

Francis Herne noreply at phabricator.kde.org
Sat Dec 1 22:00:11 GMT 2018

flherne added inline comments.


> mesonbuilder.cpp:150
> +            i18n("The directory '%1' is not empty and does not seem to be an already configured build directory",
> +                 buildDir.buildDir.toLocalFile()));
> +    case INVALID_BUILD_DIR:

This condition is reached in case a previous configure attempt failed, e.g. because of a missing dependency.

KDevelop refuses to re-try the configure step (after installing the dependency) until the build dir is removed, which is quite inconvenient.

> mesonbuilder.cpp:46
> +        , m_error(error)
> +    {
> +    }

This needs `setStandardToolView(KDevelop::IOutputView::BuildView);` or similar, otherwise the output appears as a separate view with no name or icon.

  R32 KDevelop


To: dmensinger, apol
Cc: flherne, rizzitello, yurchor, kde-doc-english, kdevelop-devel, glebaccon, hase, antismap, iodelay, vbspam, skadinna, geetamc, Pilzschaf, akshaydeo, surgenight, arrowd
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20181201/7790c1e8/attachment.html>

More information about the KDevelop-devel mailing list