On Fri, Mar 22, 2013 at 1:05 AM, Dan Gora <d...@adax.com> wrote: > > The previous test to see if we need pragma pack(1) was failing if we > cross compile because there was no way to run the test. > > We now use AC_TRY_COMPILE and add the pragma pack(1) to the test code. > If it compiles, it must be good, so add the define. > > Signed-off-by: Dan Gora <d...@adax.com>
I'm ok with this one. Z. > --- > ipmitool/configure.in | 6 ++++-- > 1 files changed, 4 insertions(+), 2 deletions(-) > > diff --git a/ipmitool/configure.in b/ipmitool/configure.in > index a35cd24..e319ef2 100644 > --- a/ipmitool/configure.in > +++ b/ipmitool/configure.in > @@ -513,7 +513,7 @@ if test "x$xenable_file_security" != "xno"; then > fi > > > -AC_TRY_RUN([ > +AC_TRY_COMPILE([],[ > #include <stdio.h> > > struct packstruct { > @@ -532,7 +532,9 @@ AC_TRY_RUN([ > else > return(0); > } > - ],[],[AC_DEFINE(HAVE_PRAGMA_PACK,[1], > + ], > + [], > + [AC_DEFINE(HAVE_PRAGMA_PACK,[1], > [Define to 1 if you need to use #pragma pack instead of > __attribute__ ((packed))])] > ) > > -- > 1.7.7 > > > ------------------------------------------------------------------------------ > Everyone hates slow websites. So do we. > Make your web apps faster with AppDynamics > Download AppDynamics Lite for free today: > http://p.sf.net/sfu/appdyn_d2d_mar > _______________________________________________ > Ipmitool-devel mailing list > Ipmitool-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/ipmitool-devel ------------------------------------------------------------------------------ Minimize network downtime and maximize team effectiveness. Reduce network management and security costs.Learn how to hire the most talented Cisco Certified professionals. Visit the Employer Resources Portal http://www.cisco.com/web/learning/employer_resources/index.html _______________________________________________ Ipmitool-devel mailing list Ipmitool-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/ipmitool-devel