[Build-common-hackers] Bug#661962: Bug#661962: cdbs: PD class processes debian/tmp instead of debian/$package, causing multiple dpkg-shlibdeps invokations

[Jonas Smedegaard]

On 12-03-02 at 09:35pm, Felipe Sateler wrote:
> PD uses cdbs_curdestdir, but that is set to debian/tmp in multiple 
> binary packages. It should use debian/$package instead, to avoid 
> calling strip and dpkg-shlibdeps to be called multiple times.
> 
> This pollutes the build logs since pd externals usually have undefined 
> references (they use symbols from the main pd binary), so they appear 
> multiple times.

cdbs_curdestdir resolves to whatever is destdir for current package.

Yes, by default that is debian/tmp for multiple packages.  It is 
overridden like this:

  DEB_DESTDIR = $(CURDIR)/debian/$(cdbs_curpkg)

...but why do you file this as a bugreport?

Are you saying that the pd.mk snippet should _always_ use per-package 
destdir?  If so, are you certain that no existing package will break 
when doing such radical change?


 - Jonas

-- 
 * Jonas Smedegaard - idealist & Internet-arkitekt
 * Tlf.: +45 40843136  Website: http://dr.jones.dk/

 [x] quote me freely  [ ] ask before reusing  [ ] keep private
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.alioth.debian.org/pipermail/build-common-hackers/attachments/20120303/b563c200/attachment.pgp>