>>>>> "Rupert" == Rupert Swarbrick <rswarbrick at gmail.com> writes:
Rupert> Raymond Toy <toy.raymond at gmail.com> writes:
Rupert> Great! Embarrassingly, my searching of the clhs didn't find
Rupert> compile-file-pathname... Are you happy to push the relevant changes?
Rupert> Otherwise, I'll try and make time tomorrow.
Yes, I'm pretty happy with these changes. Except for the fgrep. We
should probably change common-lang-recode.mk to be created via
configure so we can do the right thing with fgrep (and iconv and
recode).
Oh, and one other comment. You create
lisp-utils/parse-info-$lisp.$fasl. Would it make sense to follow what
the rest of maxima does and put the compiled result in
lisp-utils/binary-$lisp/parse-info.$fasl? This might be easier if
parse-info.lisp were in src instead of lisp-utils, but lisp-utils
seems to be the right place for it.
Rupert> Thank you for going through the code so thoroughly. All of my Unix
Rupert> experience has been on Linux (and most of my developing, for that
Rupert> matter), so I find this sort of portability hard to get right.
I haven't actually reviewed all of your code. I just did enough to
make it work on solaris 10. :-)
Overall, I find the the parse-info code works fine except that things
like ? mnewton return two entries. The old system returned just one.
This needs to be fixed. And we also need to test this on Windows to
make sure it works there.
Ray