Toninho: Auto-detect isn't mandatory. I've been testing Viktor's changes and they work just fine once you do the minor adjustments to you compile environment.
ADS_REQUIRE_VERSION becomes obsolete so you should not define it. Instead, you would use ADS_LIB_VERSION as in SET ADS_LIB_VERSION=700 or SET ADS_LIB_VERSION=710 depending if your users have 7.0 or 7.1 And then build rddads.lib The only minor thing I found is that the ADS_DIR #define won't work with long path names so in my case instead of doing this: SET ADS_DIR=C:\Program Files\Extended Systems\Advantage 8.1\acesdk I just had to convert it to the short path name equivalent SET ADS_DIR=C:\PROGRA~1\EXTEND~1\ADVANT~1.1\acesdk and problem solved. So I think we should go ahead with the changes since users building xHarbour will only need to set 1 new envvar (ADS_DIR) if they want to go with the auto-detect feature, or a 2nd one (ADS_LIB_VERSION) if they want to build to a specific ads version or even build for all the version they require. Regards, [EMAIL PROTECTED] wrote: >> Brian, >> I completely trust what ever your decision will be. >> Ron >> > > Brian, Thanks for your great job. I agree with Ron, but I have one > doubt about: > > >> We should start a separate thread on the >> ADS_REQUIRE_VERSION issue; Viktor has replaced it with a better >> #define as part of his "auto-detecting" the installed version >> of ACE--his proposal is that developers always have a real >> installation of the aceapi, and the (x)Harbour build will look >> there for headers and to determine dll versions and even create >> ace32.lib for Borland users. >> > > I, for example: have ADS 9.0 installed in my machine for tests > pourposes, but I always compile RDDADS with ADS_REQUIRE_VERSION=7 > because all my custommers have ADS 7. So IMO autodetec is not a good > idea. > -- Luis Krause Mantilla lkrausem at shaw dot ca luis_krause at hotmail dot com "May the Source be with GNU" ------------------------------------------------------------------------- This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/ _______________________________________________ xHarbour-developers mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/xharbour-developers
