Changes between Version 4 and Version 5 of pm-switch


Ignore:
Timestamp:
Oct 27, 2010 2:02:37 AM (9 years ago)
Author:
stefan
Comment:

CFLAGS should also include all dependencies

Legend:

Unmodified
Added
Removed
Modified
  • pm-switch

    v4 v5  
    183183LDADD = $(COINDEPEND_LIBS)
    184184}}}
    185 Note that the names of the variables correspond exactly to the names given to the libraries and binaries in the third argument {{{AC_COIN_HAS_PACKAGE()}}}, as described in the section on the {{{Root/Xxx/configure.ac}}} file above. Hence, the command {{{AC_COIN_HAS_PACKAGE(CoinDepend, [coinutils = trunk osi = trunk alps = trunk], [XxxLib])}}} will result in the libraries and flags for each of those dependencies being put into variables called {{{XXXLIB_LIBS}}} and {{{XXXLIB_CFLAGS}}} respectively. One curcial difference between these two variables, though, is that the {{{LIBS}}} variable includes secondary and tertiary dependencies, whereas the {{{CFLAGS}}} variable does not. So there's a little more work involved in building up the flags than the libraries. We will try to change this.
     185Note that the names of the variables correspond exactly to the names given to the libraries and binaries in the third argument {{{AC_COIN_HAS_PACKAGE()}}}, as described in the section on the {{{Root/Xxx/configure.ac}}} file above. Hence, the command {{{AC_COIN_HAS_PACKAGE(CoinDepend, [coinutils = trunk osi = trunk alps = trunk], [XxxLib])}}} will result in the libraries and flags for each of those dependencies being put into variables called {{{XXXLIB_LIBS}}} and {{{XXXLIB_CFLAGS}}} respectively.
    186186
    187187A final different is that there is no longer any need to use the {{{ADDLIBS}}} variable at all (it is not even defined), as the additional libraries are all included as dependencies when the lists of libraries are built up.