helpAutoconf - Support: sr #110367, AC_FUNC_STRERROR_R regression on...

 
 

sr #110367: AC_FUNC_STRERROR_R regression on autoconf-2.69d compared to 2.69

Submitter:  Sergei Trofimovich <slyfox>
Submitted:  Sun 08 Nov 2020 12:21:29 PM UTC
 
Category:  None Priority:  5 - Normal
Severity:  4 - Important Status:  Done
Privacy:  Public Assigned to:  None
Open/Closed:  Closed Operating System:  GNU/Linux
* Mandatory Fields

Add a New Comment (Rich Markup)
   

Comment Type & Canned Response:
       

 

Sun 08 Nov 2020 02:05:27 PM UTC, comment #2: 

комментарий №1:

> This should have been fixed by https://git.savannah.gnu.org/cgit/autoconf.git/commit/?id=c8b3f6987a7bbfe15ce2c2f04f308a6abc263d04 . Please try again with git trunk. If that doesn't fix it, please upload the config.log file generated by configure generated by git trunk.


The patch worked. Thank you!

Sergei Trofimovich <slyfox>
Sun 08 Nov 2020 01:56:31 PM UTC, comment #1: 

This should have been fixed by https://git.savannah.gnu.org/cgit/autoconf.git/commit/?id=c8b3f6987a7bbfe15ce2c2f04f308a6abc263d04 . Please try again with git trunk. If that doesn't fix it, please upload the config.log file generated by configure generated by git trunk.

Zack Weinberg <zackw>
Project Administrator
Sun 08 Nov 2020 12:21:29 PM UTC, original submission:  

Initially the bug is observed on thin-provisioning-tools-0.9.0 package where build started failing due to mis-detection of `strerror_r` prototype.

Probably related to C++. Minimal configure snippet that detects different prototypes:

$ cat configure.ac
AC_PREREQ(2.61)
AC_INIT
AC_PROG_CXX([g++])
AC_PROG_CC([gcc])
AC_LANG(C++)

AC_FUNC_STRERROR_R

AC_OUTPUT

$ autoconf-2.69 && ./configure --host=x86_64-pc-linux-gnu
checking for x86_64-pc-linux-gnu-g++... x86_64-pc-linux-gnu-g++
checking whether the C++ compiler works... yes
checking for C++ compiler default output file name... a.out
checking for suffix of executables...
checking whether we are cross compiling... no
checking for suffix of object files... o
checking whether we are using the GNU C++ compiler... yes
checking whether x86_64-pc-linux-gnu-g++ accepts -g... yes
checking for x86_64-pc-linux-gnu-gcc... x86_64-pc-linux-gnu-gcc
checking whether we are using the GNU C compiler... yes
checking whether x86_64-pc-linux-gnu-gcc accepts -g... yes
checking for x86_64-pc-linux-gnu-gcc option to accept ISO C89... none needed
checking how to run the C++ preprocessor... x86_64-pc-linux-gnu-g++ -E
checking for grep that handles long lines and -e... /bin/grep
checking for egrep... /bin/grep -E
checking for ANSI C header files... yes
checking for sys/types.h... yes
checking for sys/stat.h... yes
checking for stdlib.h... yes
checking for string.h... yes
checking for memory.h... yes
checking for strings.h... yes
checking for inttypes.h... yes
checking for stdint.h... yes
checking for unistd.h... yes
checking whether strerror_r is declared... yes
checking for strerror_r... yes
checking whether strerror_r returns char *... yes
configure: creating ./config.status

$ autoconf-2.70_beta3 && ./configure --host=x86_64-pc-linux-gnu
checking for x86_64-pc-linux-gnu-g++... x86_64-pc-linux-gnu-g++
checking whether the C++ compiler works... yes
checking for C++ compiler default output file name... a.out
checking for suffix of executables...
checking whether we are cross compiling... no
checking for suffix of object files... o
checking whether the compiler supports GNU C++... yes
checking whether x86_64-pc-linux-gnu-g++ accepts -g... yes
checking for x86_64-pc-linux-gnu-g++ option to enable C++11 features... none needed
checking for x86_64-pc-linux-gnu-gcc... x86_64-pc-linux-gnu-gcc
checking whether the compiler supports GNU C... yes
checking whether x86_64-pc-linux-gnu-gcc accepts -g... yes
checking for x86_64-pc-linux-gnu-gcc option to enable C11 features... none needed
checking how x86_64-pc-linux-gnu-g++ reports undeclared, standard C functions... error
checking for sys/types.h... yes
checking for sys/stat.h... yes
checking for strings.h... yes
checking for inttypes.h... yes
checking for stdint.h... yes
checking for unistd.h... yes
checking whether strerror_r is declared... yes
checking whether strerror_r returns char *... no
configure: creating ./config.status

Note, the final problem here is the difference in:

checking whether strerror_r returns char *... yes
checking whether strerror_r returns char *... no

In progobal project it get used in https://github.com/jthornber/thin-provisioning-tools/blob/main/configure.ac#L63 and fails to build later.

Should AC_FUNC_STRERROR_R pull in more headers (or defines?) explicitly?

Sergei Trofimovich <slyfox>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by zackw (Posted a comment)
  • -email is unavailable- added by slyfox (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

    Only logged-in users can vote.

     

     

     

     

    Follow 3 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2020-11-08 zackw StatusNeed Info Done
        Open/ClosedOpen Closed
    2020-11-08 zackw StatusNone Need Info

    Back to the top


    Powered by Savane 3.9