[Bug 43595] New: Rainmeter doesn't run, demands windows 7 or later

https://bugs.winehq.org/show_bug.cgi?id=43595

Bug ID: 43595
           Summary: Rainmeter doesn't run, demands windows 7 or later
           Product: Wine
           Version: 2.15
          Hardware: x86
               URL: https://www.rainmeter.net/
                OS: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: -unknown
          Assignee: [e-mail]
          Reporter: [e-mail]
      Distribution: ---

Tested with rainmeter 4.0 final release and 4.1 beta. Tested with wine 2.15 and
wine-staging 2.15.

Install works after setting the windows verison t owindows 10, but when running
the program it quits with a messagebox:
> Rainmeter requires Windows 7 SP1 (with Platform Update) or later.
> For Windows XP or Vista, you can download Rainmeter 3.3 from ww.rainmeter.net

Log is empty except for
> fixme:module:load_library unsupported flag(s) used (flags: 0x00000800)

[Bug 43595] Rainmeter doesn't run, demands windows 7 or later

https://bugs.winehq.org/show_bug.cgi?id=43595

Fabian Maurer  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |download

[Bug 43595] Rainmeter doesn't run, demands windows 7 or later

https://bugs.winehq.org/show_bug.cgi?id=43595

Louis Lenders  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
          Component|-unknown                    |d2d
     Ever confirmed|0                           |1
             Status|UNCONFIRMED                 |NEW
                 CC|                            |[e-mail]

--- Comment #1 from Louis Lenders  ---
Confirming, I guess it`s a d2d issue:

trace:d2d:D2D1CreateFactory factory_type 0, iid
{bb12d362-daee-4b9a-aa1d-14ba401cfa1f}, factory_options 0x33f9ec, factory
0x10176a50.
trace:d2d:D2D1CreateFactory Created factory 0x149f38.
trace:d2d:d2d_factory_QueryInterface iface 0x149f38, iid
{bb12d362-daee-4b9a-aa1d-14ba401cfa1f}, out 0x10176a50.
warn:d2d:d2d_factory_QueryInterface {bb12d362-daee-4b9a-aa1d-14ba401cfa1f} not
implemented, returning E_NOINTERFACE.
trace:d2d:d2d_factory_Release 0x149f38 decreasing refcount to 0.
trace:msgbox:MSGBOX_OnInit L"Rainmeter requires Windows 7 SP1 (with Platform
Update) or later.\n\nFor Windows XP or Vista, you can download Rainmeter 3.3
from www.rainmeter.net"

Seems that is IID_ID2D1Factory1 {bb12d362-daee-4b9a-aa1d-14ba401cfa1f},
seehttps://github.com/rainmeter/rainmeter/blob/master/Build/Installer/Installer...

[Bug 43595] Rainmeter doesn't run, demands windows 7 or later

https://bugs.winehq.org/show_bug.cgi?id=43595

--- Comment #2 from Louis Lenders  ---
Created attachment 59745
  --> https://bugs.winehq.org/attachment.cgi?id=59745
stub for Shell_NotifyIconGetRect

I saw reporter of bug 44052 submitted stub for ID2D1Factory1, so I thought lets
try this for this bug.

The application now starts after doing:

* Apply patches 
https://www.winehq.org/pipermail/wine-devel/2017-November/119764.html
https://www.winehq.org/pipermail/wine-devel/2017-November/119767.html

It then runs into unimplemented Shell_NotifyIconGetRect:

* Apply attached patch with stub for that Shell_NotifyIconGetRect

The application starts now but unfortunately there`s no text, and the windows
are almost black. Might be due to unimplemented d2d things, or my outdated
GPU??
I`ll attach a screenshot

[Bug 43595] Rainmeter doesn't run, demands windows 7 or later

https://bugs.winehq.org/show_bug.cgi?id=43595

--- Comment #3 from Louis Lenders  ---
Created attachment 59747
  --> https://bugs.winehq.org/attachment.cgi?id=59747
no text/black windows

no text/black windows, see screenshot

[Bug 43595] Rainmeter doesn't run, demands windows 7 or later

https://bugs.winehq.org/show_bug.cgi?id=43595

--- Comment #4 from Nikolay Sivov  ---
I'd expect it needs more d2d now that you advertise ID2D1Factory1. Could you
attach plain output too?

[Bug 43595] Rainmeter doesn't run, demands windows 7 or later

https://bugs.winehq.org/show_bug.cgi?id=43595

--- Comment #5 from Louis Lenders  ---
Created attachment 59752
  --> https://bugs.winehq.org/attachment.cgi?id=59752
log with LIBGL_ALWAYS_SOFTWARE=1

Hi Nicolai, here`s a log while running the application.

In this log I ran with LIBGL_ALWAYS_SOFTWARE=1 en MaxVersionGL key set
(0x300002); my GPU is really outdated, when I run without this I get lots of
errors in the console.

I`ll attach another log without LIBGL_ALWAYS_SOFTWARE=1 hereafter, though I
doubt that one is usefull.

[Bug 43595] Rainmeter doesn't run, demands windows 7 or later

https://bugs.winehq.org/show_bug.cgi?id=43595

--- Comment #6 from Nikolay Sivov  ---
That one could mean missing font:

---
fixme:d2d:d2d_d3d_render_target_DrawTextLayout Failed to draw text layout, hr
0x80004005.
---

+dwrite trace will make it clear.

[Bug 43595] Rainmeter doesn't run, demands windows 7 or later

https://bugs.winehq.org/show_bug.cgi?id=43595

--- Comment #7 from Louis Lenders  ---
Created attachment 59753
  --> https://bugs.winehq.org/attachment.cgi?id=59753
log without LIBGL_ALAYS_SOFTWARE set

log without LIBGL_ALAYS_SOFTWARE set (so plain output)

[Bug 43595] Rainmeter doesn't run, demands windows 7 or later

https://bugs.winehq.org/show_bug.cgi?id=43595

--- Comment #8 from Louis Lenders  ---
(In reply to Nikolay Sivov from comment #6)
> That one could mean missing font:
>
> fixme:d2d:d2d_d3d_render_target_DrawTextLayout Failed to draw text layout,
> hr 0x80004005.
>
> +dwrite trace will make it clear.

Ok, thanks for the hint. It says 
warn:dwrite:layout_resolve_fonts [0,5): failed to map family L"Trebuchet MS",
collection 0x13fd98, hr 0x80004005.

I did winetricks allfonts and the issue is resolved, it shows text now.

Hopefully the d2d patches from bug 44052 get committed in near future, then we
can close this bug too. I`ll send the shell32 stub later this week. Thanks for
the hints.

[Bug 43595] Rainmeter doesn't run, demands windows 7 or later (needs (stub ) ID2D1Factory1)

https://bugs.winehq.org/show_bug.cgi?id=43595

Louis Lenders  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|Rainmeter doesn't run,      |Rainmeter doesn't run,
                   |demands windows 7 or later  |demands windows 7 or later
                   |                            |(needs (stub)
                   |                            |ID2D1Factory1)
           Keywords|                            |patch

--- Comment #9 from Louis Lenders  ---
Patch for stub Shell_NotifyIconGetRect inshell32 is now in git; 
now only needs ID2D1Factory1 (from Lucian Post from bug 44052). Added info.

[Bug 43595] Rainmeter doesn't run, demands windows 7 or later (needs (stub ) ID2D1Factory1)

https://bugs.winehq.org/show_bug.cgi?id=43595

--- Comment #10 from Nikolay Sivov  ---
A stub for ID2D1Factory1 was committed, see
https://source.winehq.org/git/wine.git/?a=commit;h=9d9fbc0e4adbaadf92d740cf5....
Please retest.

[Bug 43595] Rainmeter doesn't run, demands windows 7 or later (needs (stub ) ID2D1Factory1)

https://bugs.winehq.org/show_bug.cgi?id=43595

--- Comment #11 from Louis Lenders  ---
Created attachment 60324
  --> https://bugs.winehq.org/attachment.cgi?id=60324
background not refreshed

Yes, the application starts fine now.

There`s still an issue here that the background is not refreshed when I
minimize the virtual desktop and maximize it again (see screenshot) but that
could be due to the fact I had to run with LIBGL_ALWAYS_SOFTWARE=1 to see text
anyway. 
With a decent recent GPU it might work fine.

Anyway, the initial issue is fixed.

[Bug 43595] Rainmeter doesn't run, demands windows 7 or later (needs (stub ) ID2D1Factory1)

https://bugs.winehq.org/show_bug.cgi?id=43595

Louis Lenders  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
      Fixed by SHA1|                            |9d9fbc0e4adbaadf92d740cf588
                   |                            |db412ea2f633d
         Resolution|---                         |FIXED

--- Comment #12 from Louis Lenders  ---
fixed

[Bug 43595] Rainmeter doesn't run, demands windows 7 or later (needs (stub ) ID2D1Factory1)

https://bugs.winehq.org/show_bug.cgi?id=43595

--- Comment #13 from Nikolay Sivov  ---
Could you attach terminal output please? In case some new d2d fixmes showed up.

[Bug 43595] Rainmeter doesn't run, demands windows 7 or later (needs (stub ) ID2D1Factory1)

https://bugs.winehq.org/show_bug.cgi?id=43595

--- Comment #14 from Louis Lenders  ---
Created attachment 60327
  --> https://bugs.winehq.org/attachment.cgi?id=60327
terminal output

attached terminal output

Apparently only one fixme from d2d shows up. see attachment

[Bug 43595] Rainmeter doesn't run, demands windows 7 or later (needs (stub ) ID2D1Factory1)

https://bugs.winehq.org/show_bug.cgi?id=43595

Alexandre Julliard  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |CLOSED

--- Comment #15 from Alexandre Julliard  ---
Closing bugs fixed in 3.1.

[Bug 43595] Rainmeter doesn't run, demands windows 7 or later ( needs ID2D1Factory1 stub)

https://bugs.winehq.org/show_bug.cgi?id=43595

Nikolay Sivov  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|Rainmeter doesn't run,      |Rainmeter doesn't run,
                   |demands windows 7 or later  |demands windows 7 or later
                   |(needs (stub)               |(needs ID2D1Factory1 stub)
                   |ID2D1Factory1)              |