<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Aug 25, 2015 at 11:33 AM, Ryan Tandy <span dir="ltr"><<a href="mailto:ryan@nardis.ca" target="_blank">ryan@nardis.ca</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Daniel,<br>
<br>
Sorry for not responding to this in so long.<br>
<br>
On Tue, Sep 24, 2013 at 09:31:36AM -0700, Daniel Schepler wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
During bootstrapping, I need to break the Build-Depends cycle openldap -><br>
heimdal-dev -> libldap2-dev.  The attached patch allows this.<br>
</blockquote>
<br>
Thanks for the patch! Comments below.<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
(There's also a small part that's not qualified by ifeq ($(DEB_BUILD_PROFILE),stage1) as it seems like a minor bug to me that the build process depends on the previous shlibs file for libldap-2.4-2.)<br>
</blockquote>
<br>
Reported separately as #742841, fixed in 2.4.40-1.<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
+ifeq ($(DEB_BUILD_PROFILE),stage1)<br>
</blockquote>
<br>
In the time since your original message, it looks like this was renamed to DEB_BUILD_PROFILES (plural).<br></blockquote><div><br></div><div>Yup, so this would be translated to: ifneq (,$(filter stage1,$(DEB_BUILD_PROFILES))) </div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I guess we should also use 'filter' (as in the wiki example), in case multiple profiles (not just 'stage1') are active?<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
+DH_PACKAGE_EXCLUDES = -Nslapd-smbk5pwd<br>
</blockquote>
<br>
I don't think this is necessary. The Heimdal support in smbk5pwd is not mandatory; we just have to build it with DO_KRB5 undefined, and suppress linking against the Heimdal libs somehow.<br></blockquote><div><br></div><div>The new (much simpler) equivalent would be putting "Build-Profiles: <!stage1>" into the slapd-smbk5pwd debian/control stanza.  (I tend to put it right under the Architecture field.)  I would definitely want to exclude that package from the stage1 profile build as otherwise it might not provide all expected functionality of the package.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
+ifneq ($(DEB_BUILD_PROFILE),stage1)<br>
        $(MAKE) -C contrib/slapd-modules/smbk5pwd<br>
+endif<br>
</blockquote>
<br>
So maybe:<br>
<br>
        $(MAKE) -C contrib/slapd-modules/smbk5pwd DEFS='-DDO_SAMBA -DDO_SHADOW'<br>
<br>
HEIMDAL_{INC,LIB} are already empty since krb5-config.heimdal is not available, and the error from missing it is apparently not fatal.<br></blockquote><div><br></div><div>I don't see the point in building the addon if it's not going to be packaged in the stage1 profile anyway - so I would just update to</div><div><br></div><div>ifeq (,$(filter stage1,$(DEB_BUILD_PROFILES)))</div><div>        $(MAKE) -C contrib/slapd-modules/smbk5pwd</div><div>endif</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
I think we also want a debian/control hunk, right? Is the build-profiles syntax supported for uploading to the archive these days?<br></blockquote><div><br></div><div>Yup, for example nss and util-linux are already using it.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
What do you think about the attached patch?<br></blockquote><div><br></div><div>I'm at work at the moment so I don't have time to look at it in detail right now - I'll try to look at it this evening.</div><div>-- </div><div>Daniel</div></div></div></div>