feature: add an unused-libs alias #12623
Conversation
rgrinberg
force-pushed
the
ocamlobjinfo-multi-files
branch
from
ff31b91 to
5ea10af
Compare
rgrinberg
changed the title
rgrinberg
force-pushed
the
ocamlobjinfo-multi-files
branch
5 times, most recently
from
49e8456 to
848e13e
Compare
rgrinberg
force-pushed
the
ocamlobjinfo-multi-files
branch
from
848e13e to
9f7438c
Compare
rgrinberg
force-pushed
the
ocamlobjinfo-multi-files
branch
from
9f7438c to
00742fd
Compare
|
@nojb this is ready now |
rgrinberg
force-pushed
the
ocamlobjinfo-multi-files
branch
from
00742fd to
1f46798
Compare
|
ping @nojb. We're preparing a release and I think users would be pleased to see this feature finally arrive. |
nojb
left a comment
nojb
left a comment
There was a problem hiding this comment.
LGTM
Sorry for the lag, been really busy, as usual.
Just to confirm: if one links an executable with -linkall then none of the libraries will be "unused", right?
nojb
left a comment
nojb
left a comment
There was a problem hiding this comment.
Sorry, a quick follow-up to the previous review (see question below). Also, I think the allow_unused_libraries field is not documented.
rgrinberg
force-pushed
the
ocamlobjinfo-multi-files
branch
from
1f46798 to
6f15c31
Compare
Introduce an alias to detect unused libraries in libraries and executable stanzas. This stanza relies on information extracted using ocamlobjinfo to detect which modules are actually used Signed-off-by: Rudi Grinberg <me@rgrinberg.com> feature: allow_unused_libraries Signed-off-by: Rudi Grinberg <me@rgrinberg.com>
rgrinberg
force-pushed
the
ocamlobjinfo-multi-files
branch
from
6f15c31 to
5f76e87
Compare
Been disabled now. It will need a bit more refactoring and thought to support without unnecessary duplication. |
3 participants
Introduce an alias
unused-libsthat will error out if it detects any used libraries.In the future, we could consider adding an alias per library or executable stanza if anybody wants such granularity.
Fixes #650