On Fri, Sep 02, 2016 at 10:08:43AM +0200, Antoine Jacoutot wrote: > On Wed, Aug 31, 2016 at 01:22:43AM -0600, Alexandre Ratchov wrote: > > CVSROOT: /cvs > > Module name: src > > Changes by: ratc...@cvs.openbsd.org 2016/08/31 01:22:43 > > > > Modified files: > > share/man/man4 : audio.4 > > sys/dev : audio.c audio_if.h > > sys/sys : audioio.h > > sys/kern : kern_pledge.c > > > > Log message: > > Delete unused ioctls and associated macros. Move macros that are still > > used internally by low-level drivers from sys/audioio.h to > > dev/audio_if.h instead of deleting them. > > This broke ports/x11/emiclock. > > error: 'AUDIO_DRAIN' undeclared (first use in this function) >
According to sources, sound never worked on modern machines. Given that last update of emiclock was in ~2000, $MASTER_SITES is dead, before I start fixing this, I'd prefer asking what's the most more appropriate: - fix sound? - disable sound? - delete emiclock? Meanwhile, below is a quick fix to keep the current behavior: AUDIO_DRAIN is useless in this case, because close() drains anyway. So just drop the calls to AUDIO_DRAIN. OK? (moving discutions to ports@) Index: Makefile =================================================================== RCS file: /cvs/ports/x11/emiclock/Makefile,v retrieving revision 1.26 diff -u -p -u -p -r1.26 Makefile --- Makefile 11 Oct 2013 21:21:00 -0000 1.26 +++ Makefile 2 Sep 2016 09:18:58 -0000 @@ -3,7 +3,7 @@ COMMENT= hyper-animated face analog clock for X11 DISTNAME= emiclock-2.0.2 -REVISION= 3 +REVISION= 4 CATEGORIES= x11 MASTER_SITES= http://download.vector.co.jp/pack/unix/personal/tokei/ Index: patches/patch-sys_FreeBSD_audio_c =================================================================== RCS file: /cvs/ports/x11/emiclock/patches/patch-sys_FreeBSD_audio_c,v retrieving revision 1.1 diff -u -p -u -p -r1.1 patch-sys_FreeBSD_audio_c --- patches/patch-sys_FreeBSD_audio_c 26 Oct 2007 20:44:07 -0000 1.1 +++ patches/patch-sys_FreeBSD_audio_c 2 Sep 2016 09:18:58 -0000 @@ -1,6 +1,6 @@ $OpenBSD$ --- sys/FreeBSD/audio.c.orig Thu Sep 2 16:34:34 1999 -+++ sys/FreeBSD/audio.c Fri Oct 26 22:33:32 2007 ++++ sys/FreeBSD/audio.c Fri Sep 2 11:13:33 2016 @@ -90,7 +90,9 @@ S_PlaySound(char *soundfile) return (SOUND_DEVICE_ERROR); } @@ -11,19 +11,17 @@ $OpenBSD$ fstat(sfile, &sstat); len = sstat.st_size; -@@ -100,7 +102,11 @@ S_PlaySound(char *soundfile) +@@ -100,7 +102,9 @@ S_PlaySound(char *soundfile) if ( (l = read(sfile, buffer, len)) > 0) write(sdev, buffer, l); +#ifdef __FreeBSD__ ioctl(sdev, SNDCTL_DSP_SYNC, 0); -+#elif defined(__OpenBSD__) -+ ioctl(sdev, AUDIO_DRAIN, 0); +#endif XtFree(buffer); close(sdev); -@@ -159,11 +165,17 @@ S_PlayCacheSound(SoundCacheBuffer *cacheBufferPtr) +@@ -159,11 +163,15 @@ S_PlayCacheSound(SoundCacheBuffer *cacheBufferPtr) return(SOUND_DEVICE_ERROR); } @@ -35,8 +33,6 @@ $OpenBSD$ +#ifdef __FreeBSD__ ioctl(sdev, SNDCTL_DSP_SYNC, 0); -+#elif defined(__OpenBSD__) -+ ioctl(sdev, AUDIO_DRAIN, 0); +#endif close(sdev);