Bug 273137

Summary: emulators/wine: Fix untracked dependency of alsa-plugins and 32-Bit pkg messages for wine7 and wine-devel
Product: Ports & Packages Reporter: Alexander Vereeken <Alexander88207>
Component: Individual Port(s)Assignee: Gerald Pfeifer <gerald>
Status: Closed FIXED    
Severity: Affects Some People Flags: gerald: maintainer-feedback+
Priority: ---    
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
Fix untracked dependency of alsa-plugins and 32-Bit pkg messages for wine7 and wine-devel
none
Fix untracked dependency of alsa-plugins and 32-Bit pkg messages for wine7 and wine-devel (V2) none

Description Alexander Vereeken freebsd_triage 2023-08-14 16:55:48 UTC
Created attachment 244098 [details]
Fix untracked dependency of alsa-plugins and 32-Bit pkg messages for wine7 and wine-devel

Hello Gerald,

sry if there are a few changes in one.

1. Fix untracked dependency of alsa-plugins.

When using the alsa option the audio didn't worked for me until alsa-plugins got installed. 

If the package alsa-plugins is not installed then no audio device will appear for Play and Rec and you will get the message below:

ALSA lib dlmisc.c:285:(snd_dlobj_cache_get0) Cannot open shared library /usr/local/lib/alsa-lib/libasound_module_pcm_oss.so ((null): Cannot open "/usr/local/lib/alsa-lib/libasound_module_pcm_oss.so"

I do only have applied the fix it for wine and wine7 so far, didnt tested it in for wine-devel (but the same case i guess)

2. Fix 32-Bit pkg messages for wine7 and wine-devel
Comment 1 Alexander Vereeken freebsd_triage 2023-08-14 17:06:03 UTC
Created attachment 244099 [details]
Fix untracked dependency of alsa-plugins and 32-Bit pkg messages for wine7 and wine-devel (V2)

Fix typo
Comment 2 Gerald Pfeifer freebsd_committer freebsd_triage 2023-08-15 13:52:51 UTC
Thank you, Alexander! Just to confirm, alsa-plugins is purely a run time
dependency, not required at build time?

It may take a few days, but I'll see to commit your changes to the (now)
three wine* ports!
Comment 3 Alexander Vereeken freebsd_triage 2023-08-15 16:42:16 UTC
Yes, the port alsa-plugins is not required for building.
Comment 4 commit-hook freebsd_committer freebsd_triage 2023-08-15 21:58:37 UTC
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=16d8a4b821d2db0134901589e22740290cc40020

commit 16d8a4b821d2db0134901589e22740290cc40020
Author:     Alexander Vereeken <Alexander88207@protonmail.com>
AuthorDate: 2023-08-15 21:57:36 +0000
Commit:     Gerald Pfeifer <gerald@FreeBSD.org>
CommitDate: 2023-08-15 21:57:36 +0000

    emulators/wine-devel: Fix 32-bit pkg invocation for WoW64

    This addresses a regression introduced with commit 3f914243d8179e0d
    on 2022-03-12 which was a forward port from emulators/wine and not
    sufficiently adjusted to this port).

    Do not bump PORTREVISION since 32-bit builds are broken right now.

    PR:     273137

 emulators/wine-devel/files/wine-wow64.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
Comment 5 commit-hook freebsd_committer freebsd_triage 2023-08-16 11:18:44 UTC
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=a26844393bb20a18203cc5c218d593244c7ab577

commit a26844393bb20a18203cc5c218d593244c7ab577
Author:     Alexander Vereeken <Alexander88207@protonmail.com>
AuthorDate: 2023-08-16 11:17:32 +0000
Commit:     Gerald Pfeifer <gerald@FreeBSD.org>
CommitDate: 2023-08-16 11:17:32 +0000

    emulators/wine: Add dependency on alsa-plugins

    When using the ALSA option (which is not the default) audio did not
    worke - no audio device appeared for playing and recording and the
    following message appeared:

      ALSA lib dlmisc.c:285:(snd_dlobj_cache_get0) Cannot open shared
      library /usr/local/lib/alsa-lib/libasound_module_pcm_oss.so

    Providing alsa-plugins at run time addresses this.

    Bump PORTREVSION accordingly.

    PR:             273137

 emulators/wine/Makefile | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
Comment 6 Alexander Vereeken freebsd_triage 2023-08-16 17:58:24 UTC
We have to change from 

ALSA_RUN_DEPENDS=	libasound_module_pcm_oss.so:audio/alsa-plugins

to ALSA_RUN_DEPENDS=	alsa-plugins>0:audio/alsa-plugins

I forgot that when using RUN_DEPENDS it will look for executables but not for libs that do only gets used for running.

Sorry for that...
Comment 7 commit-hook freebsd_committer freebsd_triage 2023-08-17 21:18:40 UTC
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=8cd3e4f0fc7a696019bd3cbeaa810679f23d1482

commit 8cd3e4f0fc7a696019bd3cbeaa810679f23d1482
Author:     Gerald Pfeifer <gerald@FreeBSD.org>
AuthorDate: 2023-08-16 11:41:53 +0000
Commit:     Gerald Pfeifer <gerald@FreeBSD.org>
CommitDate: 2023-08-17 21:08:39 +0000

    emulators/wine7: Adjust 32-bit pkg invocation for WoW64

    When cloning wine7 from emulators/wine we missed to update the
    wine-wow64.sh script with the new package name.

    PR:             273137

 emulators/wine7/files/wine-wow64.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
Comment 8 commit-hook freebsd_committer freebsd_triage 2023-08-18 21:59:26 UTC
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=3a2594ed6fc746dd3c5518c709d0d36e03b84bcd

commit 3a2594ed6fc746dd3c5518c709d0d36e03b84bcd
Author:     Alexander Vereeken <Alexander88207@protonmail.com>
AuthorDate: 2023-08-18 21:57:12 +0000
Commit:     Gerald Pfeifer <gerald@FreeBSD.org>
CommitDate: 2023-08-18 21:57:12 +0000

    emulators/wine: Fix ALSA run-time dependency

    The form libfoo.so:category/port only works for LIB_DEPENDS; for
    RUN_DEPENDS we need to depend on the port as such. This fixes up
    commit a26844393b.

    Bump PORTREVSION.

    PR:             273137

 emulators/wine/Makefile | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
Comment 9 commit-hook freebsd_committer freebsd_triage 2023-08-19 11:20:25 UTC
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=7044c9d91429af6b9f074cb577ebc500e5349cbb

commit 7044c9d91429af6b9f074cb577ebc500e5349cbb
Author:     Alexander Vereeken <Alexander88207@protonmail.com>
AuthorDate: 2023-08-19 11:19:32 +0000
Commit:     Gerald Pfeifer <gerald@FreeBSD.org>
CommitDate: 2023-08-19 11:19:32 +0000

    emulators/wine7: Add run-time dependency on alsa-plugins

    When using the ALSA option (which is not the default) audio did not
    work - no audio device appeared for playing and recording and the
    following message appeared:

      ALSA lib dlmisc.c:285:(snd_dlobj_cache_get0) Cannot open shared
      library /usr/local/lib/alsa-lib/libasound_module_pcm_oss.so

    Providing alsa-plugins at run time addresses this.

    (This is a backport of a26844393b and 3a2594ed6f from the primary
    wine port.)

    Bump PORTREVSION accordingly.

    PR:             273137

 emulators/wine7/Makefile | 2 ++
 1 file changed, 2 insertions(+)
Comment 10 Gerald Pfeifer freebsd_committer freebsd_triage 2023-08-19 11:32:04 UTC
Hi Alexander, as far as I can tell all the changes you submitted are
now in, is that correct?

If so, I am planning to make a similar change re ALSA to wine-devel
and then close this PR.

Any other thoughts/suggestions?
Comment 11 commit-hook freebsd_committer freebsd_triage 2023-08-20 08:08:59 UTC
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=ba0e1a78f3e1623b5bc1740b2afbc1d2075c4699

commit ba0e1a78f3e1623b5bc1740b2afbc1d2075c4699
Author:     Gerald Pfeifer <gerald@FreeBSD.org>
AuthorDate: 2023-08-20 08:07:09 +0000
Commit:     Gerald Pfeifer <gerald@FreeBSD.org>
CommitDate: 2023-08-20 08:07:09 +0000

    emulators/wine-devel: Add run-time dependency on alsa-plugins

    When using the ALSA option (which is not the default) audio did not
    work - no audio device appeared for playing and recording and the
    following message appeared:

      ALSA lib dlmisc.c:285:(snd_dlobj_cache_get0) Cannot open shared
      library /usr/local/lib/alsa-lib/libasound_module_pcm_oss.so

    Providing alsa-plugins at run time addresses this.

    (This is a forward port of a26844393b and 3a2594ed6f from the primary
    wine port.)

    Bump PORTREVSION accordingly.

    PR:             273137

 emulators/wine-devel/Makefile | 2 ++
 1 file changed, 2 insertions(+)
Comment 12 Gerald Pfeifer freebsd_committer freebsd_triage 2023-08-20 08:12:17 UTC
Voila! Please advise if this is alright or I missed anything.
Comment 13 Alexander Vereeken freebsd_triage 2023-08-20 12:01:50 UTC
All good, thanks for taking the time Gerald.