Leo Famulari writes: > On Wed, Mar 01, 2017 at 12:20:41PM -0500, Kei Kebreau wrote: >> Marius Bakke writes: >> >> > Kei Kebreau writes: >> > >> >> * gnu/packages/telephony.scm (mumble)[source]: Add 'substitute*' calls >> >> to 'snippet' for src/Version.h and src/mumble/VersionCheck.cpp. >> > >> > Thanks a lot for this! The patch LGTM, but FYI won't be necessary once >> > 'core-updates' is merged. See this commit, which backports a gcc7 patch. >> > >> > d71d6fe85c4db778be6a2081cbbec6a3a494e4f9 >> > >> >> Yay, what a useful patch! >> >> Aren't we pretty close to merging core-updates? If that's the case, I >> won't worry about pushing this to master. > > Someone should try checking if they can reproduce mumble on > core-updates, to make sure the backported GCC 7 patch is working > correctly :) That's a good test. Unfortunately, mumble fails to build on 'core-updates'. starting phase `configure' Reading /tmp/guix-build-mumble-1.2.19.drv-0/mumble-1.2.19/celt-0.7.0-build/celt-0.7.0-build.pro Reading /tmp/guix-build-mumble-1.2.19.drv-0/mumble-1.2.19/celt-0.11.0-build/celt-0.11.0-build.pro Reading /tmp/guix-build-mumble-1.2.19.drv-0/mumble-1.2.19/src/mumble/mumble.pro Project ERROR: Package speexdsp not found phase `configure' failed after 0.2 seconds It seems to be related to the speex update; apparently the DSP component have been split out of speex into a separate package: https://github.com/xiph/speexdsp