> everything's as you said.
> Can you please point me to a real source package using them that I can
> later use for testing it correctly and add it to the tests withouth me
> neededing to invent one?
>

Surprisingly enough:
$ grep -i build-depends-indep /var/lib/apt/lists/*_Sources | wc -l
11946
$ grep -i build-depends-arch /var/lib/apt/lists/*_Sources | wc -l
0

So, the mplayer package I have been testing Build-Depends-Arch on is
the only one I know of. I thought it was being used since using
Build-Depends-Indep without Build-Depends-Arch in general would
install a ton of packages that are not needed when doing arch all-only
builds. The best I could do to provide a test case is to provide the
attached patched that can be applied to the master branch of:
http://anonscm.debian.org/cgit/pkg-multimedia/mplayer.git

I guess the severity could be dropped since only packages not in the
archive are affected.

Cheers,
Miguel
From e85c9e2d8167bbc2bbbe6bd1b1b36e02709718cc Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Miguel=20A=2E=20Col=C3=B3n=20V=C3=A9lez?=
 <debian.mic...@gmail.com>
Date: Wed, 21 Oct 2015 04:49:22 -0400
Subject: [PATCH] Split B-D into Build-Depends-Arch and Build-Depends-Indep.

It avoids installing 350+ packages when only building mplayer-doc.
---
 debian/control                       | 10 ++++++----
 debian/patches/0202_doc_ffmpeg.patch | 21 +++++++++++++++++++++
 debian/patches/series                |  1 +
 debian/rules                         | 12 +++++++++++-
 4 files changed, 39 insertions(+), 5 deletions(-)
 create mode 100644 debian/patches/0202_doc_ffmpeg.patch

diff --git a/debian/control b/debian/control
index 1af31d4..75c97dc 100644
--- a/debian/control
+++ b/debian/control
@@ -11,9 +11,8 @@ Vcs-Git: git://anonscm.debian.org/pkg-multimedia/mplayer.git
 Vcs-Browser: https://anonscm.debian.org/cgit/pkg-multimedia/mplayer.git
 Homepage: https://www.mplayerhq.hu
 Build-Depends:
- debhelper (>= 9~),
- docbook-xml,
- docbook-xsl,
+ debhelper (>= 9~)
+Build-Depends-Arch:
  ladspa-sdk,
  liba52-dev,
  libaa1-dev,
@@ -86,9 +85,12 @@ Build-Depends:
  pkg-config,
  vstream-client-dev,
  x11proto-core-dev,
- xsltproc,
  yasm,
  zlib1g-dev
+Build-Depends-Indep:
+ docbook-xml,
+ docbook-xsl,
+ xsltproc
 
 Package: mplayer-gui
 Architecture: any
diff --git a/debian/patches/0202_doc_ffmpeg.patch b/debian/patches/0202_doc_ffmpeg.patch
new file mode 100644
index 0000000..c557f34
--- /dev/null
+++ b/debian/patches/0202_doc_ffmpeg.patch
@@ -0,0 +1,21 @@
+Description: Allow the documentation to build without FFmpeg.
+Author: Miguel A. Colón Vélez <debian.mic...@gmail.com>
+Forwarded: yes
+
+--- a/configure
++++ b/configure
+@@ -7119,9 +7119,11 @@
+   def_ffmpeg_a='#undef CONFIG_FFMPEG_A'
+   def_ffmpeg_so='#undef CONFIG_FFMPEG_SO'
+   nocodecmodules="ffmpeg $nocodecmodules"
+-  echo "Compiling without FFmpeg is currently not supported/working."
+-  echo "Please contact us if that is an issue for you."
+-  die "both internal and external FFmpeg missing"
++  if test "$_mplayer" = yes || test "$_mencoder" = yes ; then
++    echo "Compiling without FFmpeg is currently not supported/working."
++    echo "Please contact us if that is an issue for you."
++    die "both internal and external FFmpeg missing"
++  fi
+ fi
+ test "$_vf_lavfi" = yes && def_vf_lavfi='#define CONFIG_VF_LAVFI 1'|| libavfilters=''
+ test "$_libavcodec_mpegaudio_hp" = yes &&
diff --git a/debian/patches/series b/debian/patches/series
index 06c73b9..782a280 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -4,3 +4,4 @@
 0101_svn37548.patch
 0200_Hurd_PATH_MAX.patch
 0201_PATH_MAX_HURD.patch
+0202_doc_ffmpeg.patch
diff --git a/debian/rules b/debian/rules
index eb1836b..42d196c 100755
--- a/debian/rules
+++ b/debian/rules
@@ -27,6 +27,13 @@ CONFIGURE_FLAGS = \
 	--extra-ldflags="${LDFLAGS}" \
 	$(archconf)
 
+CONFIGURE_DOC_FLAGS = \
+	--disable-ffmpeg_a \
+	--disable-mencoder \
+	--disable-mplayer \
+	--yasm='' \
+	--language=all
+
 # Runtime detections is available for i386, amd64 and ppc.
 ifneq (,$(filter $(DEB_HOST_ARCH),i386 hurd-i386 kfreebsd-i386 amd64 kfreebsd-amd64 powerpc))
   CONFIGURE_FLAGS += --enable-runtime-cpudetection
@@ -82,9 +89,12 @@ override_dh_auto_build-indep:
 override_dh_auto_clean:
 	[ -f config.mak ] && $(MAKE) distclean || true
 
-override_dh_auto_configure:
+override_dh_auto_configure-arch:
 	./configure $(CONFIGURE_FLAGS) --enable-gui
 
+override_dh_auto_configure-indep:
+	[ -f config.mak ] || ./configure $(CONFIGURE_DOC_FLAGS)
+
 override_dh_auto_install-arch:
 	$(MAKE) install-mplayer DESTDIR=$(CURDIR)/debian/mplayer
 	$(MAKE) install-mencoder DESTDIR=$(CURDIR)/debian/mencoder
-- 
2.6.1

Reply via email to