GNU bug report logs - #28644
Pulseview with modular qt

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 28644 in the body.
You can then email your comments to 28644 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Theodoros Foradis <theodoros <at> foradis.org>
To: bug-guix <at> gnu.org
Cc: Marius Bakke <mbakke <at> fastmail.com>
Subject: Pulseview with modular qt
Date: Fri, 29 Sep 2017 14:27:08 +0300

Hello,

commit b1e4c0fa514d94df9735685a037693e5c6efa5b2 changed the pulseview
package to build with modular qt.

Now, some icons do not appear at all in the gui. I had tried building
with modular qt before submitting the qt package, and this is why I
opted for "qt" as an input.

The two inputs from modular qt, namely qtbase and qtsvg, seem to be the
correct ones, seeing that other distibutions (e.g. Parabola/Arch) use
the same dependencies.

My guess (not being familiar with qt), is that somehow, having the two
inputs in different prefixes, causes the problem with the package's
build system. I gave a look at the cmake files, but couldn't locate the
issue/fix.

Any ideas?
-- 
Theodoros Foradis

Message #8 received at 28644 <at> debbugs.gnu.org (full text, mbox):

From: Ricardo Wurmus <rekado <at> elephly.net>
To: Theodoros Foradis <theodoros <at> foradis.org>
Cc: 28644 <at> debbugs.gnu.org
Subject: Re: bug#28644: Pulseview with modular qt
Date: Fri, 29 Sep 2017 18:31:12 +0200

Theodoros Foradis <theodoros <at> foradis.org> writes:

> My guess (not being familiar with qt), is that somehow, having the two
> inputs in different prefixes, causes the problem with the package's
> build system. I gave a look at the cmake files, but couldn't locate the
> issue/fix.

You could try to validate your hypothesis by using a union package such
as it was done for python-pyqt+qscintilla.  That package would have only
one prefix.  If that really does make a difference we might be able to
investigate the issue further.

--
Ricardo

GPG: BCA6 89B6 3655 3801 C3C6  2150 197A 5888 235F ACAC
https://elephly.net

Message #11 received at 28644 <at> debbugs.gnu.org (full text, mbox):

From: Theodoros Foradis <theodoros <at> foradis.org>
To: Ricardo Wurmus <rekado <at> elephly.net>
Cc: Marius Bakke <mbakke <at> fastmail.com>, 28644 <at> debbugs.gnu.org
Subject: Re: bug#28644: Pulseview with modular qt
Date: Sat, 30 Sep 2017 16:23:42 +0300

Ricardo Wurmus writes:

> Theodoros Foradis <theodoros <at> foradis.org> writes:
>
>> My guess (not being familiar with qt), is that somehow, having the two
>> inputs in different prefixes, causes the problem with the package's
>> build system. I gave a look at the cmake files, but couldn't locate the
>> issue/fix.
>
> You could try to validate your hypothesis by using a union package such
> as it was done for python-pyqt+qscintilla.  That package would have only
> one prefix.  If that really does make a difference we might be able to
> investigate the issue further.

Thanks for the union package recommendation. I tried that, but
unfortunately it does not fix the icon issue.

I see that there is a wip-qt-paths branch in our git repo. I don't
really know its purpose, maybe it's worth to try rebasing it on
master and build pulseview with that?

Any other ideas?

-- 
Theodoros Foradis

Message #14 received at 28644 <at> debbugs.gnu.org (full text, mbox):

From: Marius Bakke <mbakke <at> fastmail.com>
To: Theodoros Foradis <theodoros <at> foradis.org>,
 Ricardo Wurmus <rekado <at> elephly.net>
Cc: 28644 <at> debbugs.gnu.org
Subject: Re: bug#28644: Pulseview with modular qt
Date: Wed, 04 Oct 2017 00:00:08 +0200

[Message part 1 (text/plain, inline)]

Theodoros Foradis <theodoros <at> foradis.org> writes:

> Ricardo Wurmus writes:
>
>> Theodoros Foradis <theodoros <at> foradis.org> writes:
>>
>>> My guess (not being familiar with qt), is that somehow, having the two
>>> inputs in different prefixes, causes the problem with the package's
>>> build system. I gave a look at the cmake files, but couldn't locate the
>>> issue/fix.
>>
>> You could try to validate your hypothesis by using a union package such
>> as it was done for python-pyqt+qscintilla.  That package would have only
>> one prefix.  If that really does make a difference we might be able to
>> investigate the issue further.
>
> Thanks for the union package recommendation. I tried that, but
> unfortunately it does not fix the icon issue.
>
> I see that there is a wip-qt-paths branch in our git repo. I don't
> really know its purpose, maybe it's worth to try rebasing it on
> master and build pulseview with that?

Hello, and sorry for breaking this package!  I don't know what the
wip-qt-paths branch is for, but it looks like a good place to fix the
problem, and I'm sure Hartmut and 宋文武 would appreciate more testers.

Can you try rebasing it and see if it makes a difference?  I will try to
investigate more towards the weekend as I really want to get rid of the
monolithic Qt :)

[signature.asc (application/pgp-signature, inline)]

Message #17 received at 28644 <at> debbugs.gnu.org (full text, mbox):

From: phodina <phodina <at> protonmail.com>
To: "28644 <at> debbugs.gnu.org" <28644 <at> debbugs.gnu.org>
Cc: Ricardo Wurmus <rekado <at> elephly.net>, Marius Bakke <marius <at> gnu.org>
Subject: Missing icons in pulseview
Date: Tue, 18 Oct 2022 10:48:30 +0000

[Message part 1 (text/plain, inline)]

Hi,

recently I used Pulseview to debug eMMC issues.

I was also surprised why they are some icons and others are missing.

Based on the code some are implemented as png and some as svg.

I tried to add the qtsvg-5 but it did not work and made the resulting derivation larger.

Therefore there are 2 possibilites:

- Add support for SVG by adding qtsvg-5 and any additional packages [1]
This would be preferable as the images are displayed in the time series and the bitmaps will look ugly.

2. Convert all into pngs
This will save size but will require a big patch (which will be mostly binary)
What do you think?

[1] https://sigrok.org/bugzilla/show_bug.cgi?id=239

----
Petr

[Message part 2 (text/html, inline)]

Message #20 received at 28644 <at> debbugs.gnu.org (full text, mbox):

From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
To: phodina <phodina <at> protonmail.com>
Cc: Ricardo Wurmus <rekado <at> elephly.net>,
 "28644 <at> debbugs.gnu.org" <28644 <at> debbugs.gnu.org>, Marius Bakke <marius <at> gnu.org>
Subject: Re: bug#28644: Pulseview with modular qt
Date: Tue, 28 Mar 2023 20:34:00 -0400

Hi,

phodina <phodina <at> protonmail.com> writes:

> Hi,
>
> recently I used Pulseview to debug eMMC issues.
>
> I was also surprised why they are some icons and others are missing.
>
> Based on the code some are implemented as png and some as svg.
>
> I tried to add the qtsvg-5 but it did not work and made the resulting derivation larger.
>
> Therefore there are 2 possibilites:
>
> - Add support for SVG by adding qtsvg-5 and any additional packages [1]
> This would be preferable as the images are displayed in the time series and the bitmaps will look ugly.
>
> 2. Convert all into pngs
> This will save size but will require a big patch (which will be mostly binary)
> What do you think?

Option 1 (simply adding qtsvg-5) sounds more reasonable to me.

-- 
Thanks,
Maxim

Message #25 received at 28644-done <at> debbugs.gnu.org (full text, mbox):

From: Nicolas Goaziou <mail <at> nicolasgoaziou.fr>
To: 28644-done <at> debbugs.gnu.org
Subject: Re: Pulseview with modular qt
Date: Sat, 22 Feb 2025 13:49:35 +0100

Hello,

> phodina <phodina <at> protonmail.com> writes:
> 
> > Hi,
> >
> > recently I used Pulseview to debug eMMC issues.
> >
> > I was also surprised why they are some icons and others are missing.
> >
> > Based on the code some are implemented as png and some as svg.
> >
> > I tried to add the qtsvg-5 but it did not work and made the resulting derivation larger.
> >
> > Therefore there are 2 possibilites:
> >
> > - Add support for SVG by adding qtsvg-5 and any additional packages [1]
> > This would be preferable as the images are displayed in the time series and the bitmaps will look ugly.
> >
> > 2. Convert all into pngs
> > This will save size but will require a big patch (which will be mostly binary)
> > What do you think?
> 
> Option 1 (simply adding qtsvg-5) sounds more reasonable to me.

Pulseview has been using QtSVG for more than 6 years now. Closing.

Regards,
-- 
Nicolas Goaziou

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.