library built via make (not cmake) lack of SONAME

[Yanhao]

this following command show nothing while libcjson.so.1.7.5 was built via make.
readelf -d libcjson.so.1.7.5 | grep SONAME

It's google while build with cmake.

Please either fix this problem or delete Makefile only reserving CMakeLists.txt.

[FSMaxB]

I'm not sure how to set the so name when building via a makefile. Is there a compiler flag that I can set?

[FSMaxB]

(up until now I wasn't even aware of the fact that the SONAME exists in ELF files)

[Yanhao]

I'm not sure how to set the so name when building via a makefile. Is there a compiler flag that I can set?

that easy, according to this link, you could simply add soname by "gcc -Wl,-soname,libfoo.so.0"

although I think just deleting the Makefile would be a better solution.

[FSMaxB]

Great, I'll fix the Makefile in the next few days.

I won't remove it yet. That will happen with the next breaking release (2.0). But I think I will change the README that using the Makefile is not recommended.

[Tangerino]

There are awesome software out there (sqlite3 amalgamation for instance) that is easy to integrate, cJSON is one of them. Why not just drop the .c and .h in your project and move on? My 2 cents

…

On Thu, Mar 29, 2018 at 7:49 PM, Yanhao Mo ***@***.***> wrote: I'm not sure how to set the so name when building via a makefile. Is there a compiler flag that I can set? that easy, according to this link <https://lintian.debian.org/tags/sharedobject-in-library-directory-missing-soname.html>, you could simply add soname by "gcc -Wl,-soname,libfoo.so.0" although I think just deleting the Makefile would be a better solution. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#252 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ADBHxD8vk9MZEHHDvvE2RWWiqY8bFP4Hks5tjR63gaJpZM4TAngu> .

-- *Merci beaucoup, Muito obrigado, Thanks* *Carlos TangerinoMobile: +33 6 82 05 55 18*

[Yanhao]

@Tangerino Actually I am packaging this library for debian cause I believe a simple single line command "apt install libcjson-dev" is more elegant solution than copying *.c and *.h to mix them with your current sources.

[FSMaxB]

Fixed in 1.7.6