Hi Grigory, Mildly off topic, but I was just checking release hmg-1.7-77-setup.zip, and I have a few points to make:
1) It was nice to see that hbmk2 is now included, but the build is a modified version which has some strange bugs (enough to run it like: 'hbmk2 a.prg' to see). Plus it has apparently been modified in a few other places. It's not indicated anywhere that it's a modified version, and even modified source is missing from the distribution, so it's not possible to tell what was exactly changed. Can you please include the modified source code in the distro? [ or even better would be to use an unmodified build, see below ] 2) Distributed Harbour COPYING file is outdated. Could you update it? 3) For some reason HMG Extended uses non-standard (old) names for Harbour libs, and I've already seen confusion in some forums because of that, plus also hbmk2 needs to be changed to accommodate to these non-standard names. Would it be possible to use standard names, so that the included Harbour wouldn't look like a specially modified version (in essence a fork) of Harbour, but rather resemble the standard distro? 4) Single threaded VM lib is not distributed, but -mt switch is missing from .hbc file. This looks like a bug unless hbmk2 was internally modified (which doesn't seem like a very good idea to me). IMO this could also be synced to work like Harbour, -mt switch added to minigui.hbc file. 5) We've once discussed TGET.prg and TGETINT.prg which is still kept as old versions and which needs special Harbour build. Maybe there is better solution for that which doesn't require that. I've proposed a few ones in the past. 6) I see some sources with '#ifdef _HBMK_' lines. I just can guess the reason for this solution, but it's probably easy to workaround with a local hbmk.hbm file or global hbmk2 option, -gtgui or -lgtgui. 7) Unless Win9x compatibility is still important, I'd suggest to change BATCH/hbmk2.bat file to be position independent. F.e. I installed it in F:\devl\minigui, so it doesn't work. Simple 'MG_ROOT=%~dp0..' seems enough to solve it. 8) Instead of '-prgflag=/q0' it's enough to write: -q0 I'd be interested in your opinion on above topics. Brgds, Viktor _______________________________________________ Harbour mailing list (attachment size limit: 40KB) Harbour@harbour-project.org http://lists.harbour-project.org/mailman/listinfo/harbour
