patchGNU Autoconf Archive - Patches: patch #7601, ax_lib_sqlite3 could use C...


patch #7601: ax_lib_sqlite3 could use C compiler to check for the library but not the C++ compiler

Submitted by:  Hong Xu <xuhdev>
Submitted on:  Wed Aug 24 10:25:49 2011  
Priority: 5 - NormalStatus: Done
Assigned to: Peter Simons <simons>Open/Closed: Closed

Add a New Comment (Rich MarkupRich Markup):

You are not logged in

Please log in, so followups can be emailed to you.


Mon Aug 29 10:13:49 2011, comment #3:

I made the change in 600c85a5bcf2eb9677bec12a2b6a6a3c5256424f.

Peter Simons <simons>
Project AdministratorIn charge of this item.
Wed Aug 24 12:20:14 2011, comment #2:

OK I was wrong about that. Just change the AC_LANG_PUSH and AC_LANG_POP and that'll be ok.

Hong Xu <xuhdev>
Wed Aug 24 11:52:37 2011, comment #1:

Thank you very much for the patch.

I noticed that it modifies use of CPPFLAGS to CFLAGS in a number of places, and I wonder whether that is really what you intented to do. CPPFLAGS is for flags that ought to be passed to pre-processor; those flags are used by both the C and the C++ compiler. (C++-specific flags are stored in CXXFLAGS). So, it shouldn't be necessary to use CFLAGS instead of CPPFLAGS to check for sqlite3 with the C compiler. Arguably, it's even wrong to put "-I" options into CFLAGS (although it is probably going to work).

In other words, switching use of CPPFLAGS to CFLAGS seems to be unnecessary for what your patch is trying to accomplish.

Peter Simons <simons>
Project AdministratorIn charge of this item.
Wed Aug 24 10:25:49 2011, original submission:

Currently, ax_lib_sqlite3 is using C++ compiler to check the library. In fact, C compiler is enough. The attached file is the patch to fix this.

Hong Xu <xuhdev>


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

Attach File(s):

Attached Files
file #23856:  ax_lib_sqlite3.m4.patch added by xuhdev (2kB - application/octet-stream)


Depends on the following items: None found

Items that depend on this one: None found


Carbon-Copy List
  • -unavailable- added by simons (Posted a comment)
  • -unavailable- added by simons
  • -unavailable- added by xuhdev (Submitted the item)

    Please enter the title of George Orwell's famous dystopian book (it's a date):



    Follow 7 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Mon Sep 5 20:03:42 2011simonsStatusReady For Test=>Done
    Mon Aug 29 10:13:49 2011simonsStatusNeed Info=>Ready For Test
    Wed Aug 24 11:52:37 2011simonsStatusNone=>Need Info
      Assigned toNone=>simons
      Carbon-Copy-=>Added mateusz loskot
    Wed Aug 24 10:25:49 2011xuhdevAttached File-=>Added ax_lib_sqlite3.m4.patch, #23856

    Back to the top

    Powered by Savane 3.1-cleanup1