March 29, 2022 Re: arsd-minigui - couple of questions | ||||
|---|---|---|---|---|
| ||||
 Posted in reply to sai | Interestingly I get the same error if I change the pragma test to "BLAH" > >lld-link: error: BLAH is not allowed in .drectve I suspect something is going wrong when passing the linker options from pragma to the linker itself. However, if I uncomment the pragma and explicitly place the manifest file it works (as you mentioned before). I will use it this way for now. But fixing the pragma will be awesome if we can get rid of the manifest file. | |||
March 29, 2022 Re: arsd-minigui - couple of questions | ||||
|---|---|---|---|---|
| ||||
Posted in reply to sai | Found this: https://bugs.llvm.org/show_bug.cgi?id=38797 Seems like a linker bug? So we have to wait for the fix to show up in dmd package. | |||
March 29, 2022 Re: arsd-minigui - couple of questions | ||||
|---|---|---|---|---|
| ||||
 Posted in reply to sai | On Tuesday, 29 March 2022 at 13:39:25 UTC, sai wrote:
> I searched for "manifestdependency" and "drectve" (w/o quotes) in arsd lib and couldn't find them so not sure how to debug this further.
Ah sorry, you caught me in the middle of something. Yes, I remember seeing this before, lld-link is buggy and doesn't support this :( I tried this before and it failed, but this time I tested again after seeing it in the Windows API docs on my laptop computer... which has the Microsoft linker installed, which does support it. So it worked for me and I committed without checking the other linker.
This is annoying. You can edit the source to comment that I guess and just use the .manifest file instead. Ugh.
| |||
March 29, 2022 Re: arsd-minigui - couple of questions | ||||
|---|---|---|---|---|
| ||||
Posted in reply to sai | On Tuesday, 29 March 2022 at 14:24:13 UTC, sai wrote:
> Seems like a linker bug? So we have to wait for the fix to show up in dmd package.
I just tried copying the lld-link from llvm version 14 upstream into the dmd directory and it does fix the problem, so will try to get the dmd release manager to do that for the future. Then I can enable this convenient thing again when that updates based on a dmd version check.
| |||
March 29, 2022 Re: arsd-minigui - couple of questions | ||||
|---|---|---|---|---|
| ||||
Posted in reply to Adam D Ruppe | Thanks Adam for all the help :) | |||
April 11, 2022 Re: arsd-minigui - couple of questions | ||||
|---|---|---|---|---|
| ||||
Posted in reply to sai | One more request, is it possible for you to do to ImageBox what you did to Button to show the transparent images (png)? Because Imagebox is showing black color for transparent pixels. I tried to see the git history to see if I can make that change to Imagebox and submit a pull request, but I am lost and don't think I understand the code well enough. | |||
April 11, 2022 Re: arsd-minigui - couple of questions | ||||
|---|---|---|---|---|
| ||||
Posted in reply to sai | On Monday, 11 April 2022 at 12:40:29 UTC, sai wrote: > One more request, is it possible for you to do to ImageBox what you did to Button to show the transparent images (png)? Because Imagebox is showing black color for transparent pixels. oh yeah this one is different because the imagebox custom draws it, so it is really as simple as passing `true` to the opacity there too when it converts. Try this diff: https://github.com/adamdruppe/arsd/commit/912047ccdda7e7775f64431a631519e6024d2494 i think it fixes it, let me know. | |||
April 11, 2022 Re: arsd-minigui - couple of questions | ||||
|---|---|---|---|---|
| ||||
Posted in reply to Adam D Ruppe | On Monday, 11 April 2022 at 13:14:16 UTC, Adam D Ruppe wrote:
> On Monday, 11 April 2022 at 12:40:29 UTC, sai wrote:
>> One more request, is it possible for you to do to ImageBox what you did to Button to show the transparent images (png)? Because Imagebox is showing black color for transparent pixels.
>
> oh yeah this one is different because the imagebox custom draws it, so it is really as simple as passing `true` to the opacity there too when it converts.
>
> Try this diff:
>
> https://github.com/adamdruppe/arsd/commit/912047ccdda7e7775f64431a631519e6024d2494
>
> i think it fixes it, let me know.
Done, it works. Awesome thanks.
| |||
Permalink
Reply