Re: svn commit: r1919799 - /subversion/branches/cmake/CMakeLists.txt

Daniel Sahlberg <[email protected]>
Newsgroups gmane.comp.version-control.subversion.svn,gmane.comp.clustering.spark.livy.devel
Message-ID <CAMHy98OxfECTisW8QdwtcpH+WRRc2xYBiBqmKv+HYAZnDLEjww@mail.gmail.com>
Den lör 10 aug. 2024 kl 21:25 skrev <[email protected]>:

> Author: rinrab
> Date: Sat Aug 10 19:25:38 2024
> New Revision: 1919799
>
> URL: http://svn.apache.org/viewvc?rev=1919799&view=rev
> Log:
> On the 'cmake' branch: Support compilation of Perl SWIG bindings.
>
> For finding the Perl dev-kit, the FindPerlLibs.cmake built-in CMake module
> will be utilized. This module doesn't actually create a target, so we will
> do it manually and will just setup the external-perl target properties with
> values from FindPerlLibs. The targets for the Perl SWIG bindings are
> already
> created by gen-cmake while other Perl-specific options will be handled by
> the
> UseSWIG module.
>
> * CMakeLists.txt
>   (swig.perl): Create the external-perl target for the Perl bindings.
>

Great work!

Hate to crash the party but it didn't work for me.

Had to do the obvious header:
$ python ./build/generator/swig/external_runtime.py ./build.conf swig perl

But still got
[[[
[ 68%] Building C object
CMakeFiles/libsvn_swig_perl.dir/subversion/bindings/swig/perl/libsvn_swig_perl/swigutil_pl.c.o
In file included from
/home/dsg/svn_cmake/subversion/bindings/swig/perl/libsvn_swig_perl/swigutil_pl.c:32:
/usr/lib/x86_64-linux-gnu/perl/5.38/CORE/perlio.h:252:15: error: unknown
type name ‘off64_t’; did you mean ‘off_t’?
  252 | PERL_CALLCONV Off_t PerlIO_tell(PerlIO *);
      |               ^~~~~
/usr/lib/x86_64-linux-gnu/perl/5.38/CORE/perlio.h:255:41: error: unknown
type name ‘off64_t’; did you mean ‘off_t’?
  255 | PERL_CALLCONV int PerlIO_seek(PerlIO *, Off_t, int);
      |                                         ^~~~~
]]]

It seems off64_t require #define _LARGEFILE64_SOURCE

Again, sorry, out of time for tonight but I couldn't help checking this
exciting commit! =)

Cheers,
Daniel

>
>
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.