[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [lmi] [PATCH] C++ m11n: range-based for loops
From: |
Greg Chicares |
Subject: |
Re: [lmi] [PATCH] C++ m11n: range-based for loops |
Date: |
Fri, 13 Jan 2017 16:10:43 +0000 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Icedove/45.5.1 |
On 2017-01-13 15:51, Greg Chicares wrote:
[...]
> I think the first patch (with that one extra change, above) is
> okay: it passes all my tests. However, I reverted all changes and
> applied both patches in order, and they both applied perfectly, but
> now I seem to have a problem already with the smallest test, as
> follows; did you have the same problem, or have I broken something?
> I'm quite sure I haven't removed the chroot's /opt directory.
>
> /opt/lmi/src/lmi[0]$make $coefficiency cgi_tests cli_tests
> Test common gateway interface:
> make[2]: warning: -jN forced in submake: disabling jobserver mode.
> Test command line interface:
> boost::filesystem::is_directory: "opt": File not found.
> 172 errors
Similarly, 'make unit_tests" gives:
Running mc_enum_test:
** uncaught exception: std::exception: boost::filesystem::path: invalid name
"Z:" in path: "Z:"
Running input_test:
** uncaught exception: std::exception: boost::filesystem::is_directory: "opt":
File not found.
so maybe whatever I did with 'wine' didn't get completely undone;
but that seems implausible, because after confirming that all tests
including these passed with the first patch, eftsoons I applied the
second patch and ran the same tests again, without changing any wine
setting in the interim. The environment looks fine:
/opt/lmi/src/lmi[0]$echo $PATH
/opt/lmi/local/bin:/opt/lmi/local/lib:/usr/local/bin:/usr/bin:/bin
/opt/lmi/src/lmi[0]$echo $WINEPATH
Z:\opt\lmi\local\bin;Z:\opt\lmi\local\lib
and wine finds the special msw-binary md5sum that I had placed on
$WINEPATH:
/opt/lmi/src/lmi[0]$wine md5sum --version
md5sum (PCP patchlevel 2) (GNU textutils) 1.22
which, as expected, is rather older than debian's:
/opt/lmi/src/lmi[0]$md5sum --version
md5sum (GNU coreutils) 8.23
- [lmi] [PATCH] C++ m11n: range-based for loops, Vadim Zeitlin, 2017/01/11
- Re: [lmi] [PATCH] C++ m11n: range-based for loops, Greg Chicares, 2017/01/12
- Re: [lmi] [PATCH] C++ m11n: range-based for loops, Greg Chicares, 2017/01/13
- Re: [lmi] [PATCH] C++ m11n: range-based for loops, Greg Chicares, 2017/01/13
- Re: [lmi] [PATCH] C++ m11n: range-based for loops, Vadim Zeitlin, 2017/01/13
- Re: [lmi] [PATCH] C++ m11n: range-based for loops, Greg Chicares, 2017/01/13
- Re: [lmi] [PATCH] C++ m11n: range-based for loops,
Greg Chicares <=
- Re: [lmi] [PATCH] C++ m11n: range-based for loops, Vadim Zeitlin, 2017/01/13
- Re: [lmi] [PATCH] C++ m11n: range-based for loops, Vadim Zeitlin, 2017/01/13
- [lmi] Improving usability of automated tests [Was: [PATCH] C++ m11n: range-based for loops], Greg Chicares, 2017/01/13
- Re: [lmi] Improving usability of automated tests, Vadim Zeitlin, 2017/01/13
- Re: [lmi] [PATCH] C++ m11n: range-based for loops, Greg Chicares, 2017/01/13
- Re: [lmi] [PATCH] C++ m11n: range-based for loops, Greg Chicares, 2017/01/13
Re: [lmi] [PATCH] C++ m11n: range-based for loops, Greg Chicares, 2017/01/13
Re: [lmi] [PATCH] C++ m11n: range-based for loops, Greg Chicares, 2017/01/14