Re: [lmi] [PATCH] C++ m11n: range-based for loops

[Vadim Zeitlin]

On Fri, 13 Jan 2017 16:10:43 +0000 Greg Chicares <address@hidden> wrote:

GC> On 2017-01-13 15:51, Greg Chicares wrote:
GC> [...]
GC> > I think the first patch (with that one extra change, above) is
GC> > okay: it passes all my tests. However, I reverted all changes and
GC> > applied both patches in order, and they both applied perfectly, but
GC> > now I seem to have a problem already with the smallest test, as
GC> > follows; did you have the same problem, or have I broken something?
GC> > I'm quite sure I haven't removed the chroot's /opt directory.
GC> > 
GC> > /opt/lmi/src/lmi[0]$make $coefficiency cgi_tests cli_tests              
GC> > Test common gateway interface:
GC> > make[2]: warning: -jN forced in submake: disabling jobserver mode.
GC> > Test command line interface:
GC> > boost::filesystem::is_directory: "opt": File not found.
GC> >   172 errors
GC> 
GC> Similarly, 'make unit_tests" gives:
GC> 
GC> Running mc_enum_test:
GC> 
GC> ** uncaught exception: std::exception: boost::filesystem::path: invalid 
name "Z:" in path: "Z:"
GC> 
GC> Running input_test:
GC> 
GC> ** uncaught exception: std::exception: boost::filesystem::is_directory: 
"opt": File not found.

 I see it too now and I have no idea how did I manage to not see these
errors during my testing before :-( I'm looking into this and will post a
patch fixing the problem a.s.a.p., sorry again.

VZ