From: Akihiko Odaki <akihiko.odaki@gmail.com>
To: Peter Maydell <peter.maydell@linaro.org>
Cc: Programmingkid <programmingkidx@gmail.com>,
QEMU devel list <qemu-devel@nongnu.org>
Subject: Re: Picture missing in About dialog on cocoa ui
Date: Fri, 9 Jul 2021 09:15:05 +0900 [thread overview]
Message-ID: <CAMVc7JULWjm_ME0bO0VAmqw=8Jr-LvqJWxFk1KxSMk2orMDgvQ@mail.gmail.com> (raw)
In-Reply-To: <CAFEAcA-vGe5BQg6HVtub5mDD6CtQN1OKGPE3Q8eJsjqyDCROnw@mail.gmail.com>
I tried [NSApp applicationIconImage] but had no luck. I suspect it is
because Rez is for Carbon development and deprecated while NSApp is
from AppKit. Loading from the filesystem is inevitable.
By the way, I knew Rez is from Carbon but didn't know it is
deprecated, which I have figured out just now by reading man. Reading
the icon file and assigning it to [NSApp applicationIconImage] is a
possible solution, but it requires modifications for all graphical
backends (cocoa, gtk, and sdl) and of course an external icon file. Do
you think removing the dependency on Rez is worth it?
See "[PATCH 1/4] cutils: Introduce bundle mechanism" for a general fix
of the problem when loading a "bundled" file.
Regards,
Akihiko Odaki
On Fri, Jul 9, 2021 at 2:51 AM Peter Maydell <peter.maydell@linaro.org> wrote:
>
> On Thu, 8 Jul 2021 at 17:28, Programmingkid <programmingkidx@gmail.com> wrote:
> > The problem with e31746ecf8dd2f25f687c94ac14016a3ba5debfc is it requires a
> > picture file to be found in a certain path. My original code used QEMU's
> > icon to obtain a picture. The reason why the picture in the About dialog
> > stopped appearing was because of the move to the meson build system.
> > A new patch has just been committed that fixes the missing icon issue.
> > Using 'git revert e31746ecf8dd2f25f687c94ac14016a3ba5debfc' fixes the
> > missing picture issue in the About dialog.
>
> If the icon is the same (same resolution, etc) then just using it
> does seem better than loading it off the filesystem. But we should
> also sort out why get_relocated_path() isn't working, because if QEMU.
> can't load files that way then other things will also be broken I think.
>
> -- PMM
next prev parent reply other threads:[~2021-07-09 0:16 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-07-07 17:00 Picture missing in About dialog on cocoa ui Programmingkid
2021-07-08 5:37 ` Akihiko Odaki
2021-07-08 8:58 ` Peter Maydell
2021-07-08 16:23 ` Programmingkid
2021-07-08 16:27 ` Programmingkid
2021-07-08 17:50 ` Peter Maydell
2021-07-08 18:38 ` Programmingkid
2021-08-16 20:55 ` Programmingkid
2021-08-17 8:59 ` Peter Maydell
2021-07-09 0:15 ` Akihiko Odaki [this message]
2021-07-10 2:43 ` Programmingkid
2021-07-10 3:30 ` Akihiko Odaki
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='CAMVc7JULWjm_ME0bO0VAmqw=8Jr-LvqJWxFk1KxSMk2orMDgvQ@mail.gmail.com' \
--to=akihiko.odaki@gmail.com \
--cc=peter.maydell@linaro.org \
--cc=programmingkidx@gmail.com \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).