I looked a little at the code. I don't think the separation is that clean at all, unfortunately. TBH, I think the way we handled the CUPS dependency in SMB is also a little unfortunate, and I'm not sure I want to make more cases like that.
In the case of wanboot, I don't think anyone besides Joerg is using it. And, as I've indicated already, neither Joerg nor I consider that his product is an illumos downstream, so he'll be unaffected. On Fri, Aug 8, 2014 at 8:14 AM, Gordon Ross <[email protected]> wrote: > I can predict that you all are not going to like this suggestion, > but why not just ifdef out the calls that pull in OpenSSL, > similar to how we did for the CUPS dependency in smb? > If I understand correctly, the thing would then still work > for plain http (just not https). > > Gordon > > > > On Fri, Aug 8, 2014 at 9:27 AM, Gary Mills via illumos-developer < > [email protected]> wrote: > >> On Thu, Aug 07, 2014 at 02:31:58PM +0100, Peter Tribble via >> illumos-developer wrote: >> > >> > Something else: SVR4 packaging uses wanboot code to provide its >> > http client support. I would be more than happy for that code and the >> > ability for pkgadd to directly retrieve packages via http to be >> > removed. >> > (In Tribblix, packages are retrieved using higher-level facilities - >> > the >> > builtin functionality in pkgadd is rubbish.) >> >> This comment led me to investigate further. The commands below I ran >> on a recent version of the illumos source built on an x86 server: >> >> $ find usr -name '*wanboot*' | wc -l >> 47 >> $ find proto -name '*wanboot*' | wc -l >> 27 >> >> All of those things called wanboot will be retained. The only thing >> that will disappear is the single stand-alone SPARC executable that is >> the first file sent to a SPARC machine when it's doing a network boot. >> >> Here are the libraries needed by a SVR4 package command, first on a >> SPARC machine that uses those packages: >> >> $ dump -Lv $(whence pkgadd) | grep 'NEED' >> [2] NEEDED libpkg.so.1 >> [4] NEEDED libinstzones.so.1 >> [6] NEEDED libadm.so.1 >> [8] NEEDED libcrypto.so.0.9.8 >> [10] NEEDED libwanboot.so.1 >> [11] NEEDED libc.so.1 >> [25] VERNEED 0x14180 >> [26] VERNEEDNUM 0x5 >> >> And now on an x86 machine that uses IPS packages: >> >> $ dump -Lv $(whence pkgadd) | grep 'NEED' >> [2] NEEDED libpkg.so.1 >> [4] NEEDED libinstzones.so.1 >> [6] NEEDED libadm.so.1 >> [8] NEEDED libcrypto.so.0.9.8 >> [10] NEEDED libwanboot.so.1 >> [11] NEEDED libc.so.1 >> [25] VERNEED 0x80540a8 >> [26] VERNEEDNUM 0x5 >> >> So, libwanboot.so.1 will be retained too. It's simply a library >> that makes calls into the two openssl libraries. >> >> -- >> -Gary Mills- -refurb- -Winnipeg, Manitoba, >> Canada- >> >> >> ------------------------------------------- >> illumos-developer >> Archives: https://www.listbox.com/member/archive/182179/=now >> RSS Feed: >> https://www.listbox.com/member/archive/rss/182179/21175074-7782178a >> Modify Your Subscription: >> https://www.listbox.com/member/?& >> Powered by Listbox: http://www.listbox.com >> > > ------------------------------------------- illumos-discuss Archives: https://www.listbox.com/member/archive/182180/=now RSS Feed: https://www.listbox.com/member/archive/rss/182180/21175430-2e6923be Modify Your Subscription: https://www.listbox.com/member/?member_id=21175430&id_secret=21175430-6a77cda4 Powered by Listbox: http://www.listbox.com
