[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: Switching SAGA to cmake



On 4/14/22 11:21, Sebastiaan Couwenberg wrote:
On 4/14/22 10:36, Johan Van de Wauw wrote:
As SAGA is switching to cmake[1], I'm now adjusting the debian package
build to use cmake as well.

For now I managed to build a working package, but I've disabled two tools
[2].

If I enable them, I get these errors:
--
:CMake Error at src/tools/io/io_shapes_dxf/CMakeLists.txt:10
(target_include_directories):
   Cannot specify include directories for target "io_shapes_dxf" which is not
   built by this project.


CMake Error at src/tools/io/io_shapes_dxf/CMakeLists.txt:11
(target_link_libraries):
   Cannot specify link libraries for target "io_shapes_dxf" which is not
built
   by this project.
--
if anyone has an idea why this happens, feel free to help fixing the issue.

"
  The named <target> must have been created by a command such as
  add_executable() or add_library() and must not be an ALIAS target.
"
https://cmake.org/cmake/help/latest/command/target_include_directories.html

saga-gis/src/tools/CMakePluginTemplate.cmake call add_library and should likely be included after the project() call instead of at the end of the file.

Please act on the issues raised on the pkg-grass-devel lists:

https://alioth-lists.debian.net/pipermail/pkg-grass-devel/2022-April/094097.html https://alioth-lists.debian.net/pipermail/pkg-grass-devel/2022-April/094087.html https://alioth-lists.debian.net/pipermail/pkg-grass-devel/2022-April/094086.html

I've pushed changes to fix several issues, please address the remaining ones.

You really need to push the upstream branch as it needs to be updated with another repacked upstream release from which the CC BY-NC-SA documentation is also removed. I have already deleted the 8.2.0+dfsg tag, you should delete that in your local copy too.

Did you see the related lintian issue?

 missing-field-in-dep5-copyright License [debian/copyright:252]

Kind Regards,

Bas

--
 GPG Key ID: 4096R/6750F10AE88D4AF1
Fingerprint: 8182 DE41 7056 408D 6146  50D1 6750 F10A E88D 4AF1


Reply to: