[Bug 28414] New: Have no sound in Live For Speed S2

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

Summary: Have no sound in Live For Speed S2
           Product: Wine
           Version: 1.3.28
          Platform: x86-64
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: directx-dsound
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]

Created an attachment (id=36422)
 --> (http://bugs.winehq.org/attachment.cgi?id=36422)
The sond stop works.

The sound works well in wine 1.3.24, but stop working in 1.3.25 and above. 
But when I test the sound in winecfg, it works. I use PulseAudio in my
ArchLinux x86_64.
When I see the logs, 1.3.24 and 1.3.28 both have some errors about alsa, but in
1.3.25 and above, it stop works.

[Bug 28414] Have no sound in Live For Speed S2

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

Austin English  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]
          Component|directx-dsound              |winmm&mci

--- Comment #1 from Austin English  2011-09-19 13:26:36 CDT ---
Please attach a +tid,+mmdevapi,+winmm,+dsound,+oss,+alsa,+coreaudio trace.

[Bug 28414] Have no sound in Live For Speed S2

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

--- Comment #2 from Biscuit  2011-09-22 03:37:57 CDT ---
Created an attachment (id=36507)
 --> (http://bugs.winehq.org/attachment.cgi?id=36507)
Terminal output with debug option

I have uploaded the new log with debug option

[Bug 28414] Have no sound in Live For Speed S2

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

--- Comment #3 from Andrew Eikum  2011-09-29 14:08:50 CDT ---
Created attachment 36605
  --> http://bugs.winehq.org/attachment.cgi?id=36605
(incomplete) winmm: Handle DRV_QUERYMAPPABLE in waveXxxMessage

Thanks for the log. Could you try building with the attached patch and see if
the sound works?  If not, can you attach a new log with the patch applied?

[Bug 28414] Have no sound in Live For Speed S2

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

--- Comment #4 from Biscuit  2011-10-02 10:43:01 CDT ---
Thanks for your quick reply. 
I'm sorry to say that, but I have some difficult to apply this patch to 1.3.28
and 1.3.29, it just give me an error as below.

patching file dlls/winmm/waveform.c
Hunk #1 FAILED at 2772.
Hunk #2 FAILED at 3143.
2 out of 2 hunks FAILED -- saving rejects to file dlls/winmm/waveform.c.rej

I don't know much about software development, and I only have one year Linux
experience. So, I don't what to do. And I apologize for my poor English, I'm
not an English-speaking person.

[Bug 28414] Have no sound in Live For Speed S2

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

--- Comment #5 from Andrew Eikum  2011-10-03 09:06:01 CDT ---
(In reply to comment #4)
> I'm sorry to say that, but I have some difficult to apply this patch to 1.3.28
> and 1.3.29, it just give me an error as below.
>
> patching file dlls/winmm/waveform.c
> Hunk #1 FAILED at 2772.
> Hunk #2 FAILED at 3143.
> 2 out of 2 hunks FAILED -- saving rejects to file dlls/winmm/waveform.c.rej
>
> I don't know much about software development, and I only have one year Linux
> experience. So, I don't what to do. And I apologize for my poor English, I'm
> not an English-speaking person.

Thanks for trying to test this, Biscuit. Here's a sequence of steps to try out
the patch:

# Clone the Wine repository. You can skip this step if you already have already
cloned Wine
git clone git://source.winehq.org/git/wine.git
cd wine

# Make sure you're working with the very latest version of Wine
git fetch
git checkout origin/master

# Apply the patch from Comment 3
git am /path/to/the/patch

# Build Wine:
./configure
make -j2

# Test your application with this version of Wine:
./wine ~/.wine/path/to/your/application.exe

If this is too much work, then I can just submit the patch and we can work
release-by-release. Obviously that will go much slower :) Let me know if you do
try the test.

[Bug 28414] Have no sound in Live For Speed S2

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

--- Comment #6 from Biscuit  2011-10-03 10:47:20 CDT ---
Thanks for your explanation, I'm cloning the repository now. I have a slow
Internet connection, and it well take a while to complete

[Bug 28414] Have no sound in Live For Speed S2

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

Biscuit  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|                            |WORKSFORME

--- Comment #7 from Biscuit  2011-10-03 11:23:30 CDT ---
(In reply to comment #5)
> (In reply to comment #4)
> > I'm sorry to say that, but I have some difficult to apply this patch to 1.3.28
> > and 1.3.29, it just give me an error as below.
> >
> > patching file dlls/winmm/waveform.c
> > Hunk #1 FAILED at 2772.
> > Hunk #2 FAILED at 3143.
> > 2 out of 2 hunks FAILED -- saving rejects to file dlls/winmm/waveform.c.rej
> >
> > I don't know much about software development, and I only have one year Linux
> > experience. So, I don't what to do. And I apologize for my poor English, I'm
> > not an English-speaking person.
>
> Thanks for trying to test this, Biscuit. Here's a sequence of steps to try out
> the patch:
>
> # Clone the Wine repository. You can skip this step if you already have already
> cloned Wine
> git clone git://source.winehq.org/git/wine.git
> cd wine
>
> # Make sure you're working with the very latest version of Wine
> git fetch
> git checkout origin/master
>
> # Apply the patch from Comment 3
> git am /path/to/the/patch
>
> # Build Wine:
> ./configure
> make -j2
>
> # Test your application with this version of Wine:
> ./wine ~/.wine/path/to/your/application.exe
>
> If this is too much work, then I can just submit the patch and we can work
> release-by-release. Obviously that will go much slower :) Let me know if you do
> try the test.

Oh! Thanks very much! It works! The sound is back.

[Bug 28414] Have no sound in Live For Speed S2

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

Andrew Eikum  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |REOPENED
         Resolution|WORKSFORME                  |
     Ever Confirmed|0                           |1

--- Comment #8 from Andrew Eikum  2011-10-03 11:26:17 CDT ---
I'm glad to hear it! I'll take care of resolving the bug once the patch makes
its way into Wine. Thanks for reporting and testing.

[Bug 28414] Have no sound in Live For Speed S2

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

Andrew Eikum  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
      Fixed by SHA1|                            |e19be58e90a5707acbb317af0ff
                   |                            |a6e7464d51ab6
             Status|REOPENED                    |RESOLVED
         Resolution|                            |FIXED

--- Comment #9 from Andrew Eikum  2011-10-24 10:45:19 CDT ---
Forgot to resolve this after the patch got in. Fixed by
e19be58e90a5707acbb317af0ffa6e7464d51ab6.

[Bug 28414] Have no sound in Live For Speed S2

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

Alexandre Julliard  changed:

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

--- Comment #10 from Alexandre Julliard  2011-11-04 14:15:32 CDT ---
Closing bugs fixed in 1.3.32.