Comment 5 for bug 1884758

Revision history for this message
Christian Ehrhardt  (paelzer) wrote :

debian/patches/enable-https.patch patches src/config/qemu/general.h, but export CONFIG=qemu makes it use src/config/qemu/general.h. Yet (as bug 1882671 indicates t is on) it is hierarchically included still.

at build time:
ar r bin-i386-pcbios/blib.a ... bin-i386-pcbios/https.o ...

Search:
^ar.*https.o
But that happens with & without the https config being enabled.

Yet it makes a difference (in the past we crossed size boundaries due to that):
With https:
-rw-r--r-- 1 paelzer paelzer 91136 Jun 24 08:27 ./src/bin-i386-pcbios/82540em.rom
Without:
-rw-r--r-- 1 paelzer paelzer 76288 Jun 24 08:39 ./src/bin-i386-pcbios/82540em.rom

I was able to debug it with a rule like:
  grep https ./src/$@*.tmp.map || /bin/true
That clearly shows A/B behavior and with that I feel safe to mess with it enabling it only for non-EFI and being able to check the result.

Almost as expected I see the size drop below 256k again.
We might let the size change and do the same twist as last time and that way (very) long term get rid of the -compat- package.