RISC OS Open: Forum: !Builder fails with missing Sound swis

Jul 22, 2016 1:49pm

I couldn’t get the Disc build to work due to missing Sound_InstallVoice (and a few others). The Sound switch definitions were not in swis.h.
I believe I’ve tracked down the problem to the export header phase.
From the log:

SoundDMA (castle.RiscOS.Sources.HWSupport.Sound.Sound0)…
amu -E -k export PHASE=hdrs COMPONENT=SoundDMA TARGET=Sound0
do <Perl$Dir>.perl Build:Hdr2H hdr.Sound <cexport$dir>.Interface.h.Sound
hdr.Sound:84: Unrecognised line: SoundLevel1VoiceTable # 4 * (1 + MaxNVoices) at Build:Hdr2H line 332, line 84.
hdr.Sound:85: Bad space definition: ‘SoundPhysChannels’
AMU: * Return code too large *

The file h.Sound file has been changed in the last 6 weeks and my previous builds were before this so it does look suspicious.

Can someone have a look to see if I’m correct (and possibly fix) this?

Jul 22, 2016 2:38pm

Jeffrey Lee (213) 6048 posts

When you updated your sources, did you re-run the Prepare script? When those changes to the sound header went in I also updated the Hdr2H scriot so that it can skip over blocks which are irrelevant/confusing (you should be able to see the "NoHdr2H’ markers in hdr.Sound). If you didn’t re-run Prepare then you’ll be using the old version of the script still (since it gets copied from mixed.Library to the root Library directory)

Jul 22, 2016 2:43pm

David Pitt (102) 743 posts

Just to confirm that the Disc build completed here. That is from scratch.

Jul 27, 2016 1:32pm

Alan Buckley (167) 232 posts

I deleted everything and updated the sources and ran Prepare. I think I must have misunderstood something about the local cvs setup.

On a Ubuntu virtual machine, I downloaded the CVS tar, unpacked it and used cvs init to set it up as a local repository.
cvs checkout castle admin scripts
used script checkout to check out the Disc component.
This doesn’t checkout the latest version as I was expecting so it didn’t include the updated Hdr2H script.

I instead changed this to checkout DiscDev and then could build the Disc component on RPCEmu OK.

What’s the difference between Disc and DiscDev?

Jul 27, 2016 3:37pm

David Pitt (102) 743 posts

What’s the difference between Disc and DiscDev?

OS5.22 stable and OS5.23 beta?

Jul 27, 2016 4:15pm

Jeffrey Lee (213) 6048 posts

What’s the difference between Disc and DiscDev?

OS5.22 stable and OS5.23 beta?

Correct.

Looks like this was my fault then; since it was “just” a header update I retagged the module with the same version, not thinking that the Hdr2H issue might affect products files that are locked to older versions. I’ll reset the tags when I get home.

!Builder fails with missing Sound swis

Reply

Search forums

Social

ROOL Store

Donate! Why?

RISC OS IPR

Description

Voices

Options

Jul 22, 2016 1:49pm Alan Buckley (167) 232 posts	I couldn’t get the Disc build to work due to missing Sound_InstallVoice (and a few others). The Sound switch definitions were not in swis.h. I believe I’ve tracked down the problem to the export header phase. From the log: SoundDMA (castle.RiscOS.Sources.HWSupport.Sound.Sound0)… amu -E -k export PHASE=hdrs COMPONENT=SoundDMA TARGET=Sound0 do <Perl$Dir>.perl Build:Hdr2H hdr.Sound <cexport$dir>.Interface.h.Sound hdr.Sound:84: Unrecognised line: SoundLevel1VoiceTable # 4 * (1 + MaxNVoices) at Build:Hdr2H line 332, line 84. hdr.Sound:85: Bad space definition: ‘SoundPhysChannels’ AMU: * Return code too large * The file h.Sound file has been changed in the last 6 weeks and my previous builds were before this so it does look suspicious. Can someone have a look to see if I’m correct (and possibly fix) this?

Jul 22, 2016 2:38pm Jeffrey Lee (213) 6048 posts	When you updated your sources, did you re-run the Prepare script? When those changes to the sound header went in I also updated the Hdr2H scriot so that it can skip over blocks which are irrelevant/confusing (you should be able to see the "NoHdr2H’ markers in hdr.Sound). If you didn’t re-run Prepare then you’ll be using the old version of the script still (since it gets copied from mixed.Library to the root Library directory)

Jul 22, 2016 2:43pm David Pitt (102) 743 posts	Just to confirm that the Disc build completed here. That is from scratch.

Jul 27, 2016 1:32pm Alan Buckley (167) 232 posts	I deleted everything and updated the sources and ran Prepare. I think I must have misunderstood something about the local cvs setup. On a Ubuntu virtual machine, I downloaded the CVS tar, unpacked it and used cvs init to set it up as a local repository. cvs checkout castle admin scripts used script checkout to check out the Disc component. This doesn’t checkout the latest version as I was expecting so it didn’t include the updated Hdr2H script. I instead changed this to checkout DiscDev and then could build the Disc component on RPCEmu OK. What’s the difference between Disc and DiscDev?

Jul 27, 2016 3:37pm David Pitt (102) 743 posts	What’s the difference between Disc and DiscDev? OS5.22 stable and OS5.23 beta?

Jul 27, 2016 4:15pm Jeffrey Lee (213) 6048 posts	What’s the difference between Disc and DiscDev? OS5.22 stable and OS5.23 beta? Correct. Looks like this was my fault then; since it was “just” a header update I retagged the module with the same version, not thinking that the Hdr2H issue might affect products files that are locked to older versions. I’ll reset the tags when I get home.