rnpgp / rnp

RNP: high performance C++ OpenPGP library used by Mozilla Thunderbird
https://www.rnpgp.org
Other
199 stars 55 forks source link

Portability issues #122

Closed dewyatt closed 4 years ago

dewyatt commented 7 years ago

See discussion in: https://github.com/riboseinc/rnp/pull/117

I guess the proper way to handle this is to update the autotools bits. My knowledge there is lacking. It defaults to --std=gnu99 locally for me, which is not quite enough for clang (gnu11 would probably work).

I should probably avoid overriding --std and defining _GNU_SOURCE explicitly in the CI scripts.

Some of the portability concerns I see are:

ronaldtse commented 7 years ago

Aha so this is the list of portability concerns!

Let me add the following too mentioned in #9 .

UPDATED: all asprintf calls were already removed by @shaunwheelhouse .

dewyatt commented 7 years ago

@ronaldtse This reminds me, do you have an official list of platforms we should aim to support?

ronaldtse commented 7 years ago

@dewyatt the official list of platforms is listed in the README now.

ni4 commented 4 years ago

@dewyatt looks like these functions are removed or handled in other way, so should we close this now?

dewyatt commented 4 years ago

@ni4 Yes indeed