Review Request 127638: build system option to build "kdev-clang" and the rest of the IDE separately

Aleix Pol Gonzalez aleixpol at kde.org
Tue Apr 12 22:54:04 UTC 2016



> On April 12, 2016, 11:31 p.m., Milian Wolff wrote:
> > -2
> > 
> > the ide without the plugins is unusable, this patch makes zero sense to me.
> > 
> > also note that you can already disable sub folders if you really want to do that (I doubt it). Just look at the cache via ccmake e.g.:
> > 
> > ```
> >  BUILD_TESTING                    ON
> >  BUILD_clang                      ON
> >  BUILD_cmake                      ON
> >  BUILD_cmakebuilder               ON
> >  BUILD_custom-buildsystem         ON
> >  BUILD_custom-definesandinclude   ON
> >  BUILD_custommake                 ON
> >  BUILD_executeplasmoid            ON
> >  BUILD_ghprovider                 ON
> >  BUILD_kdeprovider                ON
> >  BUILD_kdev_includepathsconvert   ON
> >  BUILD_kdev_makefileresolver      OFF
> >  BUILD_kdev_msvcdefinehelper      OFF
> >  BUILD_manpage                    ON
> >  BUILD_plugins                    ON
> >  BUILD_qmake                      ON
> >  BUILD_qmake_parser               OFF
> >  BUILD_qmakebuilder               ON
> >  BUILD_qmljs                      ON
> >  BUILD_qthelp                     ON
> > ```

Agreed, I don't see the value in the patch either.


- Aleix


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/127638/#review94558
-----------------------------------------------------------


On April 12, 2016, 6:34 p.m., René J.V. Bertin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/127638/
> -----------------------------------------------------------
> 
> (Updated April 12, 2016, 6:34 p.m.)
> 
> 
> Review request for KDE Software on Mac OS X and KDevelop.
> 
> 
> Repository: kdevelop
> 
> 
> Description
> -------
> 
> This patch is in reaction to a discussion on kdevelop-devel about weakening KDevelop's dependency on a specific Clang (libclang) version.
> 
> The patch introduces 2 CMake options:
> - `KDEVELOP_BUILD_IDE` builds just the IDE, without the clang-based parser (formerly kdev-clang)
> - `KDEVELOP_BUILD_CLANG_PARSER` builds just the clang-based parser
> 
> Nothing changes when both options are on (the default): all of KDevelop is built. When both options are off, an error is raised.
> 
> The idea is that this should make it a little bit easier for distributions/packaging systems and users to change the clang toolchain.
> 
> I presume that the languages subdirectory could be built first or last (that would the patch a bit simpler)?
> 
> 
> Diffs
> -----
> 
>   CMakeLists.txt 2d3faa7 
>   languages/CMakeLists.txt 4e2fde2 
>   languages/clang/CMakeLists.txt c19a951 
> 
> Diff: https://git.reviewboard.kde.org/r/127638/diff/
> 
> 
> Testing
> -------
> 
> On OS X and Linux, against Qt 5.6.0, FWs 5.20.0, prefix=/opt/local
> 
> 
> Thanks,
> 
> René J.V. Bertin
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20160412/19e9badf/attachment.html>


More information about the KDevelop-devel mailing list