Re: kdev-python not compiling cleanly on Alpine Linux Py_ssize_t due to musl?

Björn Strömberg <[email protected]> Tue, 2 Jan 2024 05:54:34 +0100
Newsgroups gmane.comp.kde.devel.kdevelop
Message-ID <CAHo5mS2e21PhVtRWYbhKKYHJkxFCq4Y5RDqcZkN4HtgcFBqFFQ@mail.gmail.com>
--0000000000002b6ad3060def4cf9
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

looks like those deprecations are atleast in some due to Qt6 changes, so
they might not be warranted for 23.08 branch when were two months from
24.02..

On Tue, Jan 2, 2024 at 5:22=E2=80=AFAM Bj=C3=B6rn Str=C3=B6mberg <bjorn.str=
[email protected]>
wrote:

> Hello Sven,
>
> yes that was the header I was expecting was the problem last night..
>
> added it on row 1 in asttransformer.cpp fixes the issue, now its compilin=
g
> again. since it was a missing symbol, it has to be before asttransformer.=
h
> which feels a bit smelly..
>
> there is also a few warnings about deprecations. is this something that i=
s
> wanted to be fixed?
> i could roll this up in a quick merge request on invent.kde.org since i
> need to create the patch files via git for the packaging anyway.
>
>
>
> On Tue, Jan 2, 2024 at 1:28=E2=80=AFAM Sven Brauch <mail-ITmcY+a7/[email protected]> w=
rote:
>
>> Hi,
>>
>> On 01.01.24 22:23, Bj=C3=B6rn Str=C3=B6mberg wrote:
>> > currently the problem is that compilation fails of asttransformer.cpp
>> on
>> > pyport.h and unknown type ssize_t in this block of the header..
>>  >
>> > this sounds like something that might be related to Alpine Linux using
>> > musl as its libc,
>>
>> yes, that sounds related, at least a quick google search shows similar
>> patches to other projects to make them build with musl. Try adding
>>
>> #include <sys/types.h>
>>
>> in asttransformer.cpp. Does that work? If yes, we can certainly fix that
>> upstream.
>>
>> Greetings,
>> Sven
>>
>

--0000000000002b6ad3060def4cf9
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"ltr">looks like those deprecations are atleast in some due to Q=
t6 changes, so they might not be warranted for 23.08 branch when were two m=
onths from 24.02..</div><br><div class=3D"gmail_quote"><div dir=3D"ltr" cla=
ss=3D"gmail_attr">On Tue, Jan 2, 2024 at 5:22=E2=80=AFAM Bj=C3=B6rn Str=C3=
=B6mberg &lt;<a href=3D"mailto:[email protected]" target=3D"_blan=
k">[email protected]</a>&gt; wrote:<br></div><blockquote class=3D=
"gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(2=
04,204,204);padding-left:1ex"><div dir=3D"ltr">Hello Sven,<div><br></div><d=
iv>yes that was the header I was expecting was the problem last night..</di=
v><div><br></div><div>added it on row 1 in asttransformer.cpp fixes the iss=
ue, now its compiling again. since it was a missing symbol, it has to be be=
fore asttransformer.h which feels a bit smelly..</div><div><br></div><div>t=
here is also a few warnings about deprecations. is this something that is w=
anted to be fixed?=C2=A0</div><div>i could roll this up in a quick merge re=
quest on <a href=3D"http://invent.kde.org" target=3D"_blank">invent.kde.org=
</a> since i need to create the patch files via git for the packaging anywa=
y.</div><div><br></div><div><br></div></div><br><div class=3D"gmail_quote">=
<div dir=3D"ltr" class=3D"gmail_attr">On Tue, Jan 2, 2024 at 1:28=E2=80=AFA=
M Sven Brauch &lt;<a href=3D"mailto:mail-ITmcY+a7/[email protected]" target=3D"_blank">m=
ail-ITmcY+a7/[email protected]</a>&gt; wrote:<br></div><blockquote class=3D"gmail_quote"=
 style=3D"margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);p=
adding-left:1ex">Hi,<br>
<br>
On 01.01.24 22:23, Bj=C3=B6rn Str=C3=B6mberg wrote:<br>
&gt; currently the problem is that compilation fails of asttransformer.cpp =
on <br>
&gt; pyport.h and unknown type ssize_t in this block of the header..<br>
=C2=A0&gt;<br>
&gt; this sounds like something that might be related to Alpine Linux using=
 <br>
&gt; musl as its libc,<br>
<br>
yes, that sounds related, at least a quick google search shows similar <br>
patches to other projects to make them build with musl. Try adding<br>
<br>
#include &lt;sys/types.h&gt;<br>
<br>
in asttransformer.cpp. Does that work? If yes, we can certainly fix that <b=
r>
upstream.<br>
<br>
Greetings,<br>
Sven<br>
</blockquote></div>
</blockquote></div>

--0000000000002b6ad3060def4cf9--