[Bug 59368] New: [feature request] Allow app developers to set dlloverrides

[WineHQ Bugzilla]

http://bugs.winehq.org/show_bug.cgi?id=59368

            Bug ID: 59368
           Summary: [feature request] Allow app developers to set
                    dlloverrides
           Product: Wine
           Version: unspecified
          Hardware: x86-64
                OS: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: -unknown
          Assignee: [e-mail]
          Reporter: [e-mail]
      Distribution: ---

Whilst most projects specify dll overrides in their documentation this'd allow
the developer of the app to configure the default behaviour.
> GameFolder
> > game.exe
> > game2.exe
> > `wine_dll_overrides.toml`

wine_dll_overrides.toml would look something like

```
[game.exe]
dinput8=n,b
winmm=n,b

[game2.exe]
rtworkq=n,b
```

Might make a PR for this depending on what upstream folks think

-- 
Do not reply to this email, post in Bugzilla using the
above URL to reply.
You are receiving this mail because:
You are watching all bug changes.

[Bug 59368] [feature request] Allow app developers to set dlloverrides

[WineHQ Bugzilla]

http://bugs.winehq.org/show_bug.cgi?id=59368

mata <[e-mail]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

--- Comment #1 from mata <[e-mail]> ---
That more or less already exists, and it even supports more than just dll
overrides:
https://gitlab.winehq.org/wine/wine/-/wikis/Useful-Registry-Keys

So, to provide some defaults, a reg file yould be used, for example:



Windows Registry Editor Version 5.00

[HKEY_CURRENT_USER\Software\Wine\AppDefaults\AppName.exe\DllOverrides]
"dllName1"="builtin,native"
"dllName2"="native"
"dllName3"=""

...


and imported using:

    wine reg import fileName.reg


Not sure if it really would mare sense to add an additional location to
populate the overrides, besides the already existing options via
WINEDLLOVERRIDES env variable, the global default registry and the
per-application registry.

-- 
Do not reply to this email, post in Bugzilla using the
above URL to reply.
You are receiving this mail because:
You are watching all bug changes.

[Bug 59368] [feature request] Allow app developers to set dlloverrides

[WineHQ Bugzilla]

http://bugs.winehq.org/show_bug.cgi?id=59368

--- Comment #2 from [e-mail] ---
(In reply to mata from comment #1)
> That more or less already exists, and it even supports more than just dll
> overrides:
> https://gitlab.winehq.org/wine/wine/-/wikis/Useful-Registry-Keys
>
> So, to provide some defaults, a reg file yould be used, for example:
>
> Windows Registry Editor Version 5.00
>
> [HKEY_CURRENT_USER\Software\Wine\AppDefaults\AppName.exe\DllOverrides]
> "dllName1"="builtin,native"
> "dllName2"="native"
> "dllName3"=""
>
> ...
>
> and imported using:
>
> wine reg import fileName.reg

Yea I already knew about this approach but it requires user to run a .reg file
or the developer to write code that does the regedit editing which won't work
with dll mods [like
this](https://www.nexusmods.com/plantsvszombiesgardenwarfare/mods/25) because
how can the dll override itself

I believe the config approach is better.

-- 
Do not reply to this email, post in Bugzilla using the
above URL to reply.
You are receiving this mail because:
You are watching all bug changes.

[Bug 59368] [feature request] Allow app developers to set dlloverrides

[WineHQ Bugzilla]

http://bugs.winehq.org/show_bug.cgi?id=59368

--- Comment #3 from Alexandre Julliard <[e-mail]> ---
That's not a good idea. The correct native/builtin choice depends both on the
version of the native dll and on the Wine version being used. As Wine makes
progress, the goal is for builtin to always be the right choice. We don't want
the app to be hardcoding a configuration that will most likely become incorrect
in the future.

-- 
Do not reply to this email, post in Bugzilla using the
above URL to reply.
You are receiving this mail because:
You are watching all bug changes.