Re: Bug#942795: Acknowledgement (libsoxr: Building with "nocheck" results in a broken soxr.pc pkg-config file)

Mattia Rizzolo <[email protected]>
Newsgroups gmane.linux.debian.ports.superh,gmane.linux.debian.ports.68k
Message-ID <[email protected]>
On Mon, Oct 21, 2019 at 08:39:17PM +0200, John Paul Adrian Glaubitz wrote:
> ifneq ($(filter nocheck,$(DEB_BUILD_OPTIONS)),)
> CONFIGURE_FLAGS += -DBUILD_TESTS=0 -DBUILD_EXAMPLES=1
> endif
> 
> With the paragraph removed, "nocheck" still works as expected and building
> with "nocheck" no longer results in a broken soxr.pc.

This was added by Helmut to fix a FTCBFS.  Are you sure removing the
-DBUILD_TESTS=0 doesn't break cross builds?

Also, the fact that a thing such as BUILD_EXAMPLE affects the content of
the .pc files sounds wrong on many levels, so I'd rather see it fixed
some other way.

I haven't tested, but in particular I noticed this oddity in the
top-level CMakeFile.txt:
103 if (${BUILD_EXAMPLES})
104   project (${PROJECT_NAME}) # Adds c++ compiler
105 endif ()
that looks very suspicious.

So, I wrote a different patch (that one day should be forwarded
upstream) and committed it.


BTW, Adrian: autopkgtest is reporting a new error, specifically a
segfault in amd64.  That looks incredibly suspicious, could you please
look at it?  I'd like to see a fix for that before doing a new upload.

-- 
regards,
                        Mattia Rizzolo

GPG Key: 66AE 2B4A FCCF 3F52 DA18  4D18 4B04 3FCD B944 4540      .''`.
More about me:  https://mapreri.org                             : :'  :
Launchpad user: https://launchpad.net/~mapreri                  `. `'`
Debian QA page: https://qa.debian.org/developer.php?login=mattia  `-
signature.asc (application/pgp-signature, 833 B)
-----BEGIN PGP SIGNATURE-----

iQIzBAEBCgAdFiEEi3hoeGwz5cZMTQpICBa54Yx2K60FAl2wcHIACgkQCBa54Yx2
K63YUg//S168+SK6w4z/cAOJ3e1JpFH+mEmB/6V9F4i7Z+F6caQN+tBrNX36J2A8
Zb8e8Jsz//SRhpxYsHWJIcDOLLNmWgJ1fdHxwzWBrMXeDc6OYLPgsgyMtEFaqEnf
SONOL3w4L018FhBVeF4w49agyID5AW1NgyPVtV+Gj+ZtpKpq1BTpSRltScM6s0qb
VEU79Cb19NTapMTDq94jeJQikr55UZVkkYfdPPI1VFcWYEy50FP0bGuVHvi8PUHi
m0DJ+d6Bf2xEkEWMaGZZz3MskA3ZeVHJ5U6r9k2aCciW8ptVdhmDwZs9qMfox4/C
EPQWJxyzZ24UCHDzyt1poesMvsipmgm2ZAqu9/4G9lsOPQUew0e/vML/rw+20awn
+b8BqGFvmtynIB5ZKqfg1C5E7Cos0pvjQCFqQX144l8xqUvMMTxEzk/vXNbaN7h1
5VTfnFQW6ZPheLquGyuncfvmJTem8j9oDmOsg5ieXjNwKLSkHipG8Xgrq78F+w1H
pBXQgLsc0bVtrZqSVqStQe3PF/H6+u2PDUF37of56kJUv6xb60kTYD0YlMMHppi9
+2nL3ZYzl2wxOYbV5UeInNdcf/vaVdW57fn1pNB1d4VIg8xfX3xAFE59kwCIazYJ
xeTg+rtIEavY0MgVwP+Xn8WGWnEFPayPhMGbobfUIcxxIZj0w/w=
=fv6a
-----END PGP SIGNATURE-----
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.