View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0004887 | libmicrohttpd | portability | public | 2017-02-10 13:28 | 2021-09-02 15:54 |
| Reporter | tomasheran | Assigned To | Christian Grothoff | Due Date | |
| Priority | normal | Severity | major | Reproducibility | always |
| Status | closed | Resolution | fixed | ||
| Platform | Any | OS | Solaris | OS Version | 11.next |
| Product Version | 0.9.52 | ||||
| Target Version | 0.9.53 | Fixed in Version | 0.9.53 | ||
| Summary | 0004887: _SET_INIT_AND_DEINIT_FUNCS isn't ready for Solaris Studio compiler | ||||
| Description | The way src/include/autoinit_funcs.h is structured is only around GNU C and MS Visual Studio compilers. Solaris Studio isn't supported in that header file and (by default) the fact that the header file "couldn't figure out" the way how to register the library constructors and destructors is silently ignored and so when the library is built using Solaris Studio, MHD_init() is never called (automatically upon library load). | ||||
| Additional Information | Luckily, Solaris Studio does understand GNU C's "__attribute__ ((constructor|destructor))" syntax and so this patch seems to fix the situation: diff -r d61abfd2e00c -r 035e9e268ad6 src/include/autoinit_funcs.h --- a/src/include/autoinit_funcs.h Wed Feb 08 12:57:20 2017 +0100 +++ b/src/include/autoinit_funcs.h Fri Feb 10 14:17:39 2017 +0100 @@ -67,7 +67,7 @@ */ #define AUTOINIT_FUNCS_VERSION 0x01000001 -#if defined(__GNUC__) +#if defined(__GNUC__) || defined(__SUNPRO_C) #/* if possible - check for supported attribute */ #ifdef __has_attribute #if !__has_attribute(constructor) || !__has_attribute(destructor) @@ -76,7 +76,7 @@ #endif /* __has_attribute */ #endif /* __GNUC__ */ -#if defined(__GNUC__) && !defined(_GNUC_ATTR_CONSTR_NOT_SUPPORTED) +#if (defined(__GNUC__) || defined(__SUNPRO_C)) && !defined(_GNUC_ATTR_CONSTR_NOT_SUPPORTED) #define GNUC_SET_INIT_AND_DEINIT(FI,FD) \ void __attribute__ ((constructor)) _GNUC_init_helper_##FI(void) \ | ||||
| Tags | No tags attached. | ||||
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2017-02-10 13:28 | tomasheran | New Issue | |
| 2017-02-12 08:39 | Christian Grothoff | Assigned To | => Christian Grothoff |
| 2017-02-12 08:39 | Christian Grothoff | Status | new => resolved |
| 2017-02-12 08:39 | Christian Grothoff | Resolution | open => fixed |
| 2017-02-12 08:39 | Christian Grothoff | Fixed in Version | => 0.9.53 |
| 2017-02-12 08:39 | Christian Grothoff | Note Added: 0011717 | |
| 2017-02-12 08:39 | Christian Grothoff | Target Version | => 0.9.53 |
| 2017-04-11 22:00 | Christian Grothoff | Status | resolved => closed |
| 2021-09-02 15:54 | Christian Grothoff | ||
| 2021-09-02 15:54 | Christian Grothoff | Note Added: 0018186 |