Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove antilib shared object because it's not really shared! #98

Closed
AriaMoradi opened this issue Nov 29, 2020 · 5 comments · Fixed by #107
Closed

Remove antilib shared object because it's not really shared! #98

AriaMoradi opened this issue Nov 29, 2020 · 5 comments · Fixed by #107

Comments

@AriaMoradi
Copy link
Contributor

AriaMoradi commented Nov 29, 2020

Original Issue:

I was browsing the file list of the built package. We are building all c++ stuff under libantilib.so.1 and linking libantilib.so to it.

That looks a little dumb. If it won't cause problems, we could name it libantimicro(x).so and no .1 file for more cleanness?

edit: numbers in the shared object name ending is for versioning the file and letting different programs link against different object versions. Since we are shipping the appropriate object with every version and we are the only program using it, it defeats the purpose.

And @gombosg said:

Libraries have to have soname fields. If we don't need it because it's not shared, then let's not ship a shared library.

So we should seek to completely remove it and statically link our code.

@pktiuk
Copy link
Member

pktiuk commented Nov 29, 2020

I also think so.

@gombosg
Copy link
Collaborator

gombosg commented Nov 29, 2020

This is because of Fedora packaging guidelines: https://docs.fedoraproject.org/en-US/packaging-guidelines/#_downstream_so_name_versioning

Libraries have to have soname fields. If we don't need it because it's not shared, then let's not ship a shared library.

I'm fine with removing this separate library and just compiling it as part of the main executable. I'd rather write a PR for that solution when I get the time to do so.

@AriaMoradi
Copy link
Contributor Author

AriaMoradi commented Nov 29, 2020

This is because of Fedora packaging guidelines: https://docs.fedoraproject.org/en-US/packaging-guidelines/#_downstream_so_name_versioning

This is not really a "shared" object. agreed.

I'm fine with removing this separate library and just compiling it as part of the main executable. I'd rather write a PR for that solution when I get the time to do so.

I'm going to try change my PR

@AriaMoradi AriaMoradi changed the title Shared object name ~Shared object name~ Nov 30, 2020
@AriaMoradi AriaMoradi changed the title ~Shared object name~ Shared object name Nov 30, 2020
@AriaMoradi AriaMoradi changed the title Shared object name Remove antilib shared object because it's not really shared! Nov 30, 2020
@pktiuk
Copy link
Member

pktiuk commented Nov 30, 2020

Okay, but don't merge it instantly when it will be ready, I think it would be good to at least think what was their original purpose.

@gombosg
Copy link
Collaborator

gombosg commented Nov 30, 2020

We had that conversation here: juliagoda/antimicroX#118 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants