• Bug#1104749: [Pkg-pascal-devel] Bug#1104749: Bug#1104749: Installing "L

    From Abou Al Montacir@21:1/5 to Paul Gevers on Fri May 9 23:00:01 2025
    --=-2v+VnVqFvXmF4aCBe8v+
    Content-Type: text/plain; charset="UTF-8"
    Content-Transfer-Encoding: quoted-printable

    Hi All,

    On Fri, 2025-05-09 at 22:05 +0200, Paul Gevers wrote:
    Hi,

    On 09-05-2025 17:19, Peter B wrote:
    Circular dependencies, but does that matter?


    Yes, they should be avoided.
    +1

    I've moved the icon in other packages of mine before (I think WinFF, now team maintained here), to accommodate for the choice in AppStream to consider the package containing the icon to be the thing to install. I
    don't like it, because the icon fits nicely in the -data package and I
    had to create a "main" package to begin with IIRC, but I agree with the submitter that we should adapt to how the world works in this case.
    I think we should add recommend: lazarus

    Normally, people installing using GUI tools do not worry about disk space and download time.
    This means, that they expect a fully functional, out of the box, tool. Thus, they need the lazarus package.

    The recommend should do the job, as by default it will be pulled I assume.
    --
    Cheers,
    Abou Al Montacir


    --=-2v+VnVqFvXmF4aCBe8v+
    Content-Type: text/html; charset="utf-8"
    Content-Transfer-Encoding: quoted-printable

    <html><head></head><body><div>Hi All,</div><div><br></div><div><span><pre>On Fri, 2025-05-09 at 22:05 +0200, Paul Gevers wrote:</pre></span></div><blockquote type="cite" style="margin:0 0 0 .8ex; border-left:2px #729fcf solid;padding-left:1ex"><div>Hi,<
    </div><div><br></div><div>On 09-05-2025 17:19, Peter B wrote:<br></div><blockquote type="cite" style="margin:0 0 0 .8ex; border-left:2px #729fcf solid;padding-left:1ex"><div>Circular dependencies, but does that matter?<br></div></blockquote><div><br></
    <div><br></div><div>Yes, they should be avoided.<br></div></blockquote><div>+1</div><blockquote type="cite" style="margin:0 0 0 .8ex; border-left:2px #729fcf solid;padding-left:1ex"><div><br></div><div>I've moved the icon in other packages of mine
    before (I think WinFF, now <br></div><div>team maintained here), to accommodate for the choice in AppStream
  • From Peter B@21:1/5 to Abou Al Montacir on Sat May 10 13:20:02 2025
    On 09/05/2025 21:42, Abou Al Montacir wrote:
    I think we should add recommend: lazarus

    @Paul

    Do you think we might get an RT unblock for this?

    ISTM Significant benefit with minimal risk.


    Cheers,
    Peter

    --- SoupGate-Win32 v1.05
    * Origin: fsxNet Usenet Gateway (21:1/5)
  • From Abou Al Montacir@21:1/5 to Peter B on Sun May 11 10:10:01 2025
    --=-uTnsMdyrI4v+OY7LAJJ9
    Content-Type: text/plain; charset="UTF-8"
    Content-Transfer-Encoding: quoted-printable

    Hi Peter,

    On Sat, 2025-05-10 at 12:00 +0100, Peter B wrote:
    On 09/05/2025 21:42, Abou Al Montacir wrote:
    I think we should add recommend: lazarus

    @Paul

    Do you think we might get an RT unblock for this?

    ISTM Significant benefit with minimal risk.

    Maybe we should fix the following issue before https://appstream.debian.org/sid/main/issues/lazarus-ide-4.0.html

    I don't know much about this, but we may discover a way to tell AppStream to look for Lazarus package to install this.

    I also think about having two desktop files (Lazarus-gtk and Lazarus-qt). This may be more relevant for GUI installer.

    Let's not do anything until we agree on what should be done.
    --
    Cheers,
    Abou Al Montacir


    --=-uTnsMdyrI4v+OY7LAJJ9
    Content-Type: text/html; charset="utf-8"
    Content-Transfer-Encoding: quoted-printable

    <html><head></head><body><div>Hi Peter,</div><div><br></div><div>On Sat, 2025-05-10 at 12:00 +0100, Peter B wrote:</div><blockquote type="cite" style="margin:0 0 0 .8ex; border-left:2px #729fcf solid;padding-left:1ex"><div>On 09/05/2025 21:42, Abou Al
    Montacir wrote:<br></div><blockquote type="cite" style="margin:0 0 0 .8ex; border-left:2px #729fcf solid;padding-left:1ex"><div>I think we should add recommend: lazarus<br></div></blockquote><div><br></div><div>@Paul<br></div><div><br></div><div>Do you
    think we might get an RT unblock for this?<br></div><div><br></div><div>ISTM Significant benefit with minimal risk.</div></blockquote><div><br></div><div>Maybe we should fix the following issue before</div><div><a href="https://appstream.debian.org/sid/
    main/issues/lazarus-ide-4.0.html">https://appstream.debian.org/sid/main/issues/lazarus-ide-4.0.html</a></div>
  • From Peter B@21:1/5 to asciiwolf@seznam.cz on Sun May 11 13:40:01 2025
    On 11/05/2025 12:14, asciiwolf@seznam.cz wrote:
    Maybe we should fix the following issue before https://appstream.debian.org/sid/main/issues/lazarus-ide-4.0.html

    I would not worry about this for now. While the AppStream metainfo
    file is recommended, it is not required. Valid metadata are
    automatically generated from the desktop file (and other package data)
    if the metainfo is not present.

    (And there is no point in adding it now in my opinion since it needs
    to be in the same package as the desktop file anyway).

    Agreed.

    Appstream metadata is a distro agnostic format, that upstream can use if
    they wish.
    Its absence is not a Debian packaging issue.

    IMHO its ridiculous for its absence to be a Lintian warming. Maybe a
    pedantic (if at all).


    Regards,
    Peter

    --- SoupGate-Win32 v1.05
    * Origin: fsxNet Usenet Gateway (21:1/5)
  • From asciiwolf@seznam.cz@21:1/5 to All on Sun May 11 13:20:02 2025
    Maybe we should fix the following issue before

    https://appstream.debian.org/sid/main/issues/lazarus-ide-4.0.html




    I would not worry about this for now. While the AppStream metainfo file is recommended, it is not required. Valid metadata are automatically generated from the desktop file (and other package data) if the metainfo is not
    present.




    (And there is no point in adding it now in my opinion since it needs to be
    in the same package as the desktop file anyway).

    --=_1daa6ecb04ec3a6b11483e1fJd059da-3a22-5a6f-a03f-b94b63079549_Content-Type: text/html;
    charset=utf-8
    Content-Transfer-Encoding: quoted-printable

    <html><body>&gt; Maybe we should fix the following issue before<br><div>&gt; https://appstream.debian.org/sid/main/issues/lazarus-ide-4.0.html</div><div><br></div><div>I would not worry about this for now. While the AppStream metainfo file is recommended,
    it is not required. Valid metadata are automatically generated from the desktop file (and other package data) if the metainfo is not present.</div><div><br></div><div>(And there is no point in adding it now in my opinion since it needs to be in the same
    package as the desktop file anyway).</div></body></html>

    --- SoupGate-Win32 v1.05
    * Origin: fsxNet Usenet Gateway (21:1/5)
  • From asciiwolf@seznam.cz@21:1/5 to All on Sun May 11 14:00:01 2025
    In other words, adding the correct package dependency is sufficient and will fix the issue.




    Adding the AppStream metainfo file on the other hand is optional and can
    bring its own problems.

    --=_60a1345a650356891d11a8cfJd059da-3a22-5a6f-a03f-b94b63079549_Content-Type: text/html;
    charset=utf-8
    Content-Transfer-Encoding: quoted-printable

    <html><body><div>In other words, adding the correct package dependency is sufficient and will fix the issue.</div><div><br></div><div>Adding the AppStream metainfo file on the other hand is optional and can bring its own problems.</div></body></html>

    --- SoupGate-Win32 v1.05
    * Origin: fsxNet Usenet Gateway (21:1/5)
  • From Peter B@21:1/5 to Abou Al Montacir on Sun May 11 14:00:02 2025
    On 11/05/2025 09:05, Abou Al Montacir wrote:
    I also think about having two desktop files (Lazarus-gtk and
    Lazarus-qt). This may be more relevant for GUI installer.

    Maybe for Forky.  And maybe four. Gtk2, Gtk3, Qt5, Qt6
    But I'd rather not go there for Trixie into the freeze.


    Let's not do anything until we agree on what should be done.

    I was happy with your previous suggestion,
    maybe like this?

    ======================================================

    diff --git a/debian/control b/debian/control
    index 7bacf63b..f3289d65 100644
    --- a/debian/control
    +++ b/debian/control
    @@ -89,6 +89,7 @@ Description: IDE for Free Pascal - LCL source code
     Package: lazarus-ide-4.0
     Architecture: any
     Depends: fp-compiler (>= 3.2.2+dfsg-45),
    +         lazarus--4.0,
              ${misc:Depends},
              ${shlibs:Depends}
     Recommends: fpc,
    diff --git a/debian/control.in b/debian/control.in
    index 6f9392a1..b9fe1ffb 100644
    --- a/debian/control.in
    +++ b/debian/control.in
    @@ -89,6 +89,7 @@ Description: IDE for Free Pascal - LCL source code
     Package: lazarus-ide${PACKAGESUFFIX}
     Architecture: any
     Depends: fp-compiler (>= 3.2.2+dfsg-45),
    +         lazarus-${PACKAGESUFFIX},
              ${misc:Depends},
              ${shlibs:Depends}
     Recommends: fpc,

    --- SoupGate-Win32 v1.05
    * Origin: fsxNet Usenet Gateway (21:1/5)
  • From Abou Al Montacir@21:1/5 to Peter B on Mon May 12 06:30:01 2025
    --=-A9vbn/VjwEc6cNNjFKnx
    Content-Type: text/plain; charset="UTF-8"
    Content-Transfer-Encoding: quoted-printable

    Hi Peter,

    On Sun, 2025-05-11 at 12:51 +0100, Peter B wrote:
    On 11/05/2025 09:05, Abou Al Montacir wrote:
    I also think about having two desktop files (Lazarus-gtk and
    Lazarus-qt). This may be more relevant for GUI installer.

    Maybe for Forky.  And maybe four. Gtk2, Gtk3, Qt5, Qt6
    For now, only Gtk2 and Qt5 are compiled, so that limits the candidates.

    But I'd rather not go there for Trixie into the freeze.
    That's fair

    Let's not do anything until we agree on what should be done.

    I was happy with your previous suggestion,
    maybe like this?
    I think Recommends is better as there is no strict requirement for this circular
    dependency that I'd rather avoid.

    Also please consider using lazarus${PACKAGESUFFIX} instead of lazarus- ${PACKAGESUFFIX}.
    --
    Cheers,
    Abou Al Montacir


    --=-A9vbn/VjwEc6cNNjFKnx
    Content-Type: text/html; charset="utf-8"
    Content-Transfer-Encoding: quoted-printable

    <html><head></head><body><div>Hi Peter,</div><div><br></div><div>On Sun, 2025-05-11 at 12:51 +0100, Peter B wrote:</div><blockquote type="cite" style="margin:0 0 0 .8ex; border-left:2px #729fcf solid;padding-left:1ex"><div>On 11/05/2025 09:05, Abou Al
    Montacir wrote:<br></div><blockquote type="cite" style="margin:0 0 0 .8ex; border-left:2px #729fcf solid;padding-left:1ex"><div>I also think about having two desktop files (Lazarus-gtk and <br></div><div>Lazarus-qt). This may be more relevant for GUI
    installer.<br></div></blockquote><div><br></div><div>Maybe for Forky.&nbsp; And maybe four. Gtk2, Gtk3, Qt5, Qt6<br></div></blockquote><div>For now, only Gtk2 and Qt5 are compiled, so that limits the candidates.</div><div><br></div><blockquote type="cite"
    style="margin:0 0 0 .8ex; border-left:2px #729fcf solid;padding-left:1ex"><div>But I'd rather not go there f
  • From Peter B@21:1/5 to All on Tue May 13 12:00:01 2025
    Hi Abou,

    I've push the fix to Salsa.

    Maybe its worth waiting the three days needed for version 4.0
    to migrate before uploading?


    Cheers,
    Peter

    --- SoupGate-Win32 v1.05
    * Origin: fsxNet Usenet Gateway (21:1/5)
  • From Abou Al Montacir@21:1/5 to Peter B on Tue May 13 15:20:01 2025
    --=-DE26JFuYLHNNKbdzYN0E
    Content-Type: text/plain; charset="UTF-8"
    Content-Transfer-Encoding: quoted-printable

    Hi @Peter,

    On Tue, 2025-05-13 at 10:52 +0100, Peter B wrote:
    Hi Abou,

    I've push the fix to Salsa.
    Looks good to me.

    Maybe its worth waiting the three days needed for version 4.0
    to migrate before uploading?
    Yes, let's let it migrate first.

    Maybe you can upload to experimental and test if this fixes really installation on GUI. This can help us gain some time.
    --
    Cheers,
    Abou Al Montacir


    --=-DE26JFuYLHNNKbdzYN0E
    Content-Type: text/html; charset="utf-8"
    Content-Transfer-Encoding: quoted-printable

    <html><head></head><body><div>Hi @Peter,</div><div><br></div><div>On Tue, 2025-05-13 at 10:52 +0100, Peter B wrote:</div><blockquote type="cite" style="margin:0 0 0 .8ex; border-left:2px #729fcf solid;padding-left:1ex"><div>Hi Abou,<br></div><div><br></
    <div>I've push the fix to Salsa.<br></div></blockquote><div><div><span style="font-size: 14.666667px;">Looks good to me.</span></div></div><blockquote type="cite" style="margin:0 0 0 .8ex; border-left:2px #729fcf solid;padding-left:1ex"><div><br></
    <div>Maybe its worth waiting the three days needed for version 4.0<br></div><div>to migrate before uploading?</div></blockquote><div>Yes, let's let it migrate first.</div><div><br></div><div>Maybe you can upload to experimental and test if this fixes
    really installation on GUI. This can help us gain some time.</div><pre style="caret-color: rgb(46, 52, 54);