monolithic-lib-op broken for ECL

[Daniel Kochmański]

It may be also broken for other linking implementations (mkcl, clasp). In ECL static library can't be build from other static libraries (because `ar' can work properly only on object files) and as far as I can see ECL has never planned to do that.

This effectively breaks also FASL bundles under certain conditions (monolithic-compile-bundle-op?).

For steps to reproduce see:
https://gitlab.com/embeddable-common-lisp/ecl/issues/274
https://gitlab.com/embeddable-common-lisp/ecl/issues/177

Since next release instead of allowing such combination[1] into C::BUILDER (which leads to broken binary) ECL will signal an error.

Proper solution is to use object files which were used to build static library used as input right now. Recently I've added documentation[2] for proper transition of input files into their build output.

[1] Static library being input file for :static-library target.
[2] https://common-lisp.net/project/ecl/static/ecldoc/Extensions.html#System-building

[Daniel Kochmański (jackdaniel)]

duplicate of #1626994

[Daniel Kochmański (jackdaniel)]

My bad, #1626994 is about dll-op, this one is still valid

[Faré (fahree)]

I remember hitting the very same issue long ago when I re-wrote asdf-ecl and wrote asdf/bundle, and when I ported it to MKCL, where indeed, a .so is built from the .o's not the .a's.

So, what behavior do you want? When shall .a's ever be used as input?

[Daniel Kochmański (jackdaniel)]

It's useful target for fasl and program (and of course monolithic static libraries are very useful as a final target in case of ECL if embedded in C application).

[Daniel Kochmański (jackdaniel)]

s/useful target/valid input/

[Daniel Kochmański (jackdaniel)]

Fix in the MR !59 seems to work for me.

[Faré (fahree)]

The fix in !59 eschews using libraries as input, except when linking an executable, at which point they use individual libraries and not a monolithic one — because that's how (so far) bundle.lisp determines whether uiop, asdf, cmp have already been linked (or not).

All the combinations are possible. If you tell me which ones you want, I'll make it happen.

[Daniel Kochmański (jackdaniel)]

The current behaviour looks alright.

[Faré (fahree)]

Slated for release 3.2