• Bug#1057573: linpac: FTBFS: error: invalid use of incomplete type =?UTF

    From David Ranch@21:1/5 to Dave Hibberd on Fri Jan 5 18:40:01 2024
    This is a multi-part message in MIME format.
    Hello Dave,

    I received the following email yesterday which gives me the impression
    that your patch wasn't accepted. What needs to be done here by Jan 16th?
    --

    linpac 0.28-2 is marked for autoremoval from testing on 2024-01-16

    It is affected by these RC bugs:
    1057573: linpac: FTBFS: error: invalid use of incomplete type ‘WINDOW’ {aka ‘struct _win_st’}
    https://bugs.debian.org/1057573


    This mail is generated by: https://salsa.debian.org/release-team/release-tools/-/blob/master/mailer/mail_autoremovals.pl

    Autoremoval data is generated by: https://salsa.debian.org/qa/udd/-/blob/master/udd/testing_autoremovals_gatherer.pl
    --


    On 01/01/2024 08:55 AM, Dave Hibberd wrote:
    Hi there!

    Yeah I’ve pushed his patch already to our git, so if anyone else on
    the team wants to upload it they can - I am still waiting on our
    account managers to finalise my Debian developer application before I
    have free reign to upload things, and I’ll switch this package to
    fall under my ownership like a collection of other packet things to
    make life a little easier going forward.

    Cheers
    DH

    On 1 Jan 2024, at 16:46, David Ranch <dranch@trinnet.net> wrote:

     Hello DaveH,

    I have pushed the required fixes to the develop branch of Linpac. I
    do plan on adding some other enhancements to Linpac before publishing
    an new official release but I don't know if getting out this fix is
    important to the Debian release process. Maybe the similar third
    party fix offered by Svenis good enough for now to keep Linpac in the
    Debian Unstable/Testing repos?

    --David
    KI6ZHD



    On 12/18/2023 02:40 AM, Dave Hibberd wrote:
    Hi both,

    I'll prepare a team upload for this in advance of the new upstream
    release (thanks David), and upload it independently or with support
    depending on the outcome of the DAM stage of my NM process
    (https://nm.debian.org/process/1224/)

    Cheers
    DH

    --
    Hibby
    MM0RFN

    On Sat, 16 Dec 2023, at 6:32 PM, David Ranch wrote:
    Hello Sven, Debian team,

    I was about to apply a very similar fix to the "develop" branch of
    Linpac at https://sourceforge.net/p/linpac/linpac/ci/develop/tree
    though my changes didn't include the "-1" at the end of the
    changes. Not sure if that's needed / important. Regardless, I am
    planning to eventually merge the develop branch into the Master
    branch and releae 0.29 in the near future which will include this
    and other fixes.

    --David
    KI6ZHD

    On 12/16/2023 10:01 AM, Sven Joachim wrote:
    Control: tags -1 + patch

    On 2023-12-05 23:07 +0100, Santiago Vila wrote:


    Package: src:linpac
    Version: 0.28-2
    Severity: serious
    Tags: ftbfs

    Dear maintainer:

    During a rebuild of all packages in unstable, your package failed to build:

    --------------------------------------------------------------------------------
    [...]
    g++ -DHAVE_CONFIG_H -I. -I../../.. -Wdate-time -D_FORTIFY_SOURCE=2 -g -O2 -ffile-prefix-map=/<<PKGBUILDDIR>>=. -fstack-protector-strong -fstack-clash-protection -Wformat -Werror=format-security -fcf-protection -c -o mail_screen.o mail_screen.cc
    mail_screen.cc: In function ‘void init_main_screen()’:
    mail_screen.cc:39:16: error: invalid use of incomplete type ‘WINDOW’ {aka ‘struct _win_st’}
    39 | maxx = stdscr->_maxx;
    | ^~
    In file included from mail_screen.cc:13:
    /usr/include/curses.h:442:16: note: forward declaration of ‘WINDOW’ {aka ‘struct _win_st’}
    442 | typedef struct _win_st WINDOW;
    | ^~~~~~~
    mail_screen.cc:40:16: error: invalid use of incomplete type ‘WINDOW’ {aka ‘struct _win_st’}
    40 | maxy = stdscr->_maxy;
    | ^~
    /usr/include/curses.h:442:16: note: forward declaration of ‘WINDOW’ {aka ‘struct _win_st’}
    442 | typedef struct _win_st WINDOW;
    | ^~~~~~~
    mail_screen.cc: In function ‘void redraw()’:
    mail_screen.cc:70:15: error: invalid use of incomplete type ‘WINDOW’ {aka ‘struct _win_st’}
    70 | main_window->_clear = TRUE;
    | ^~
    /usr/include/curses.h:442:16: note: forward declaration of ‘WINDOW’ {aka ‘struct _win_st’}
    442 | typedef struct _win_st WINDOW;

    The attached patch, which can be added to the series file fixes, these >>>>> errors and two additional ones in src/linpac.cc, but I have only tested >>>>> that the package builds, not if it works. Note that getmaxx(win)
    returns win->_maxx + 1, and similar for getmaxy.

    Cheers,
    Sven






    <html>
    <head>
    <meta content="text/html; charset=utf-8" http-equiv="Content-Type">
    </head>
    <body bgcolor="#FFFFFF" text="#000000">
    <font size="-1">Hello Dave,<br>
    <br>
    I received the following email yesterday which gives me the
    impression that your patch wasn't accepted.  What needs to be done
    here by Jan 16th?<br>
    --</font><br>
    <pre wrap="">linpac 0.28-2 is marked for autoremoval from testing on 2024-01-16

    It is affected by these RC bugs:
    1057573: linpac: FTBFS: error: invalid use of incomplete type ‘WINDOW’ {aka ‘struct _win_st’}
    <a class="moz-txt-link-freetext" href="https://bugs.debian.org/1057573">https://bugs.debian.org/1057573</a>


    This mail is generated by:
    <a class="moz-txt-link-freetext" href="https://salsa.debian.org/release-team/release-tools/-/blob/master/mailer/mail_autoremovals.pl">https://salsa.debian.org/release-team/release-tools/-/blob/master/mailer/mail_autoremovals.pl</a>

    Autoremoval data is generated by:
    <a class="moz-txt-link-freetext" href="https://salsa.debian.org/qa/udd/-/blob/master/udd/testing_autoremovals_gatherer.pl">https://salsa.debian.org/qa/udd/-/blob/master/udd/testing_autoremovals_gatherer.pl</a>
    --
    </pre>
    <br>
    <div class="moz-cite-prefix">On 01/01/2024 08:55 AM, Dave Hibberd
    wrote:<br>
    </div>
    <blockquote
    cite="mid:EABFBBF1-B248-48B3-ABF9-DDC760C34B3F@vehibberd.com"
    type="cite">
    <meta http-equiv="content-type" content="text/html; charset=utf-8">
    Hi there! 
    <div><br>
    </div>
    <div>Yeah I’ve pushed his patch already to our git, so if anyone
    else on the team wants to upload it they can - I am still
    waiting on our account managers to finalise my Debian developer
    application before I have free reign to upload things,  and I’ll
    switch this package to fall under my ownership like a collection
    of other packet things to make life a little easier going
    forward.<br id="lineBreakAtBeginningOfSignature">
    <div dir="ltr"><br>
    </div>
    <div dir="ltr">Cheers</div>
    <div dir="ltr">DH</div>
    <div dir="ltr"><br>
    <blockquote type="cite">On 1 Jan 2024, at 16:46, David Ranch
    <a class="moz-txt-link-rfc2396E" href="mailto:dranch@trinnet.net">&lt;dranch@trinnet.net&gt;</a> wrote:<br>
    <br>
    </blockquote>
    </div>
    <blockquote type="cite">
    <div dir="ltr">
    <meta content="text/html; charset=utf-8"
    http-equiv="Content-Type">
    <font size="-1">Hello DaveH,<br>
    <br>
    I have pushed the required fixes to the develop branch of
    Linpac.  I do plan on adding some other enhancements to
    Linpac before publishing an new official release but I
    don't know if getting out this fix is important to the
    Debian release process.  Maybe the similar third party fix
    offered by </font>Sven<font size="-1"> is good enough for
    now to keep Linpac in the Debian Unstable/Testing repos?<br>
    <br>
    --David<br>
    KI6ZHD<br>
    <br>
    <br>
    </font><br>
    <div class="moz-cite-prefix">On 12/18/2023 02:40 AM, Dave
    Hibberd wrote:<br>
    </div>
    <blockquote
    cite="mid:a7890291-1f17-41d6-b94d-86fe74e71125@app.fastmail.com"
    type="cite">
    <title></title>
    <div style="font-family:menlo, consolas, monospace;">Hi
    both,<br>
    </div>
    <div style="font-family:menlo, consolas, monospace;"><br>
    </div>
    <div style="font-family:menlo, consolas, monospace;">I'll
    prepare a team upload for this in advance of the new
    upstream release (thanks David), and upload it
    independently or with support depending on the outcome
    of the DAM stage of my NM process (<a
    moz-do-not-send="true"
    href="https://nm.debian.org/process/1224/">https://nm.debian.org/process/1224/</a>)</div>
    <div style="font-family:menlo, consolas, monospace;"><br>
    </div>
    <div style="font-family:menlo, consolas, monospace;">Cheers<br>
    </div>
    <div style="font-family:menlo, consolas, monospace;">DH<br>
    </div>
    <div style="font-family:menlo, consolas, monospace;"><br>
    </div>
    <div id="sig57366953">
    <div class="signature">-- <br>
    </div>
    <div class="signature">  Hibby<br>
    </div>
    <div class="signature">  MM0RFN<br>
    </div>
    </div>
    <div style="font-family:menlo, consolas, monospace;"><br>
    </div>
    <div>On Sat, 16 Dec 2023, at 6:32 PM, David Ranch wrote:<br>
    </div>
    <blockquote type="cite" id="qt" style="">
    <div><span class="size" style="">Hello Sven, Debian
    team,<br>
    <br>
    I was about to apply a very similar fix to the
    "develop" branch of Linpac at <a
    moz-do-not-send="true"
    class="qt-moz-txt-link-freetext"
    href="https://sourceforge.net/p/linpac/linpac/ci/develop/tree">https://sourceforge.net/p/linpac/linpac/ci/develop/tree</a>
    though my changes didn't include the "-1" at the end
    of the changes.  Not sure if that's needed /
    important.  Regardless, I am planning to eventually
    merge the develop branch into the Master branch and
    releae 0.29 in the near future which will include
    this and other fixes.<br>
    <br>
    --David<br>
    KI6ZHD<br>
    <br>
    </span></div>
    <div class="qt-moz-cite-prefix">On 12/16/2023 10:01 AM,
    Sven Joachim wrote:<br>
    </div>
    <blockquote cite="mid:87h6kiowky.fsf@turtle.gmx.de"
    type="cite">
    <pre>Control: tags -1 + patch

    On 2023-12-05 23:07 +0100, Santiago Vila wrote:


    </pre>
    <blockquote type="cite">
    <pre>Package: src:linpac
    Version: 0.28-2
    Severity: serious
    Tags: ftbfs

    Dear maintainer:

    During a rebuild of all packages in unstable, your package failed to build:

    --------------------------------------------------------------------------------
    [...]
    g++ -DHAVE_CONFIG_H -I. -I../../.. -Wdate-time -D_FORTIFY_SOURCE=2 -g -O2 -ffile-prefix-map=/&lt;&lt;PKGBUILDDIR&gt;&gt;=. -fstack-protector-strong -fstack-clash-protection -Wformat -Werror=format-security -fcf-protection -c -o mail_screen.o mail_
    screen.cc
    mail_screen.cc: In function ‘void init_main_screen()’: mail_screen.cc:39:16: error: invalid use of incomplete type ‘WINDOW’ {aka ‘struct _win_st’}
    39 | maxx = stdscr-&gt;_maxx;
    | ^~
    In file included from mail_screen.cc:13:
    /usr/include/curses.h:442:16: note: forward declaration of ‘WINDOW’ {aka ‘struct _win_st’}
    442 | typedef struct _win_st WINDOW;
    | ^~~~~~~
    mail_screen.cc:40:16: error: invalid use of incomplete type ‘WINDOW’ {aka ‘struct _win_st’}
    40 | maxy = stdscr-&gt;_maxy;
    | ^~
    /usr/include/curses.h:442:16: note: forward declaration of ‘WINDOW’ {aka ‘struct _win_st’}
    442 | typedef struct _win_st WINDOW;
    | ^~~~~~~
    mail_screen.cc: In function ‘void redraw()’:
    mail_screen.cc:70:15: error: invalid use of incomplete type ‘WINDOW’ {aka ‘struct _win_st’}
    70 | main_window-&gt;_clear = TRUE;
    | ^~
    /usr/include/curses.h:442:16: note: forward declaration of ‘WINDOW’ {aka ‘struct _win_st’}
    442 | typedef struct _win_st WINDOW;

    </pre>
    </blockquote>
    <pre>The attached patch, which can be added to the series file fixes, these
    errors and two additional ones in src/linpac.cc, but I have only tested
    that the package builds, not if it works. Note that getmaxx(win)
    returns win-&gt;_maxx + 1, and similar for getmaxy.

    Cheers,
    Sven


    </pre>
    </blockquote>
    </blockquote>
    <div style="font-family:menlo, consolas, monospace;"><br>
    </div>
    </blockquote>
    <br>
    </div>
    </blockquote>
    </div>
    </blockquote>
    <br>
    </body>
    </html>

    --- SoupGate-Win32 v1.05
    * Origin: fsxNet Usenet Gateway (21:1/5)