Bug 95727

Summary:	media-sound/gramofile-1.6 needs amd64-specific fixes (with patch)
Product:	Gentoo Linux	Reporter:	Uri Sivan <tartif>
Component:	Current packages	Assignee:	Gentoo Sound Team <sound>
Status:	RESOLVED FIXED
Severity:	normal	CC:	amd64
Priority:	High	Keywords:	Inclusion
Version:	unspecified
Hardware:	AMD64
OS:	Linux
Whiteboard:
Package list:		Runtime testing required:	---
Attachments:	Fix the WAV header structure to use 32-bit fields

Description Uri Sivan 2005-06-10 15:47:42 UTC

This package needs a patch to work on amd64; a patch is supplied.

The README for the package says:

Note for all 64-bit systems: when your `long'-type vars are 64 bit, .wav
headers are not read/written correctly (=bug). If you replace every `long'
with an `int' (or other 32-bit type), everything should be fine.

Therefor, a patch should be applied on amd64 machines. I attach what I think is
the appropriate patch.

Reproducible: Always
Steps to Reproduce:
How to test:
Run gramofile
Select "3. Locate Tracks"
Pick any WAV file, select "Start computation" on following screen.

Actual Results:  
Unpatched version: an error appears: "The source file is a .wav file with
unknown format"


Expected Results:  
Patched version: This error does not appear.

Comment 1 Uri Sivan 2005-06-10 15:49:08 UTC

Created attachment 61023 [details, diff]
Fix the WAV header structure to use 32-bit fields

Comment 2 raffaele_at_gentoo 2005-07-04 00:11:20 UTC

(In reply to comment #1)
> Created an attachment (id=61023) [edit]
> Fix the WAV header structure to use 32-bit fields
> 
The patch works for me, as Uri described I was unable to read WAV files without it.
Thanks Uri.

Comment 3 Diego Elio Pettenò (RETIRED) gentoo-dev

2005-09-01 16:52:05 UTC

That does not seems good, please use standard integer types.

Comment 4 Simon Stelling (RETIRED) gentoo-dev

2005-10-08 03:59:37 UTC

fixed, thanks