|
From: | Juergen Sauermann |
Subject: | Re: [Bug-apl] AP100 problem |
Date: | Wed, 15 Feb 2017 14:51:20 +0100 |
User-agent: | Mozilla/5.0 (X11; Linux i686; rv:45.0) Gecko/20100101 Thunderbird/45.2.0 |
Hi Frederick, First of all, I believe the aclocal problem could be fixed with running autoreconf before ./configure in the same directory. Regarding the SVN version, i am currently fairly confused. I am working with two SVN repositories: a local one into which I commit even small changes (so that I revert changes easily) and a remote one at GNU/savannah into which I commit only versions that have been tested and are are (supposedly) working. The savannah repo is currently at SVN 885 while my local one is at SVN 12560. My local repository shows the latest change of AP100.cc as: address@hidden:~/projects/juergen/apl-1.6/src/APs$ svn diff -r12013 AP100.cc Index: AP100.cc =================================================================== --- AP100.cc (revision 12013) +++ AP100.cc (working copy) @@ -100,7 +100,8 @@ return; } - cmd = string((const char *)&cdr[20], cdr.size() - 20).c_str(); + cmd = string((const char *)cdr.get_items() + 20, + cdr.size() - 20).c_str(); if (verbose) get_CERR() << pref << " got command " << cmd << endl; @@ -170,7 +171,7 @@ { if (var.data == 0) return E_VALUE_ERROR; - data = "" char *)&var.data[0], var.data->size()); + data = "" char *)(var.data->get_items()), var.data->size()); error_loc = "no_error"; return E_NO_ERROR; } //----------------------------------------------------------------------------- address@hidden:~/projects/juergen/apl-1.6/src/APs$ That change was entered into the local repository on Thu, 08 Dec 2016, The reason for the change was an earlier change in CDR.hh which crewated a compile-error in AP100.cc. In theory, the change made in AP100.cc should have been included in the next commit into the savannah repository. But the savannah repo does not show anything after 2014: address@hidden:~/projects/juergen/savannah-repo-apl/trunk/src/APs$ svnversion 885 address@hidden:~/projects/juergen/savannah-repo-apl/trunk/src/APs$ svn log AP100.cc ------------------------------------------------------------------------ r370 | j_sauermann | 2014-07-14 12:32:54 +0200 (Mon, 14 Jul 2014) | 1 line fixed ∇ on non-empty line, more functions moved to APserver ------------------------------------------------------------------------ r91 | j_sauermann | 2014-01-13 17:23:10 +0100 (Mon, 13 Jan 2014) | 1 line finalized 10⎕CR and fixed recent bugs ... However, the latest version in savannah contains the last change in AP100.cc. That suggests that the SVN log is wrong, but the savannah SVN is up-to-date. To double-check that, please check that line 103 of AP100.cc reads: cmd = string((const char *)cdr.get_items() + 20, cdr.size() - 20).c_str(); and not: cmd = string((const char *)&cdr[20], cdr.size() - 20).c_str(); BTW line 103 is just before the wrong command is being printed. Another possible problem is that some .cc files may not have picked up changes in .hh files. The "normal" ./configure configures for a "fast" build which does not build dependency files. That build performs fewer compilations (and is therefore faster). But the price is that updates of header files (via SVN) may not be recognized (make itself compares only .cc files). So if your APs/AP100 binary is older than, say, 2 month, then this problem has most likely occurred. Unfortunately the changes that has made the change in AP100.cc necessary were mostly happening in header files. If you do make develop then the ./configure is done differently an dependencies will be detected properly. After doing make develop once, updates from SVN are usually safe (until the next ./configure which may override the proper settings). In any case 'make clean; sudo make install' is always a good idea to rule out problems caused by missing dependencies. At this point, I believe it is clear that the commands sent from APL to the AP100 processor are being corrupted, but the reason why is not yet clear. The date where the changes occurred was somewhere around Dec 8, 2016. Best Regards, Jürgen Sauermann On 02/15/2017 02:26 AM, Frederick Pitts
wrote:
|
[Prev in Thread] | Current Thread | [Next in Thread] |