Code review comment for lp:~widelands-dev/widelands/remove_duplicate_code

Revision history for this message
Benedikt Straub (nordfriese) wrote :

Um, the compile script no longer works for me on linux in trunk-8854?

[…]
-- Not using AddressSanitizer.
-- Version of Widelands Build is (Release)

CMake Warning (dev) at /usr/share/cmake-3.10/Modules/CTest.cmake:234 (if):
  Policy CMP0054 is not set: Only interpret if() arguments as variables or
  keywords when unquoted. Run "cmake --help-policy CMP0054" for policy
  details. Use the cmake_policy command to set the policy and suppress this
  warning.

  Quoted variables like "" will no longer be dereferenced when the policy is
  set to NEW. Since the policy is not set the OLD behavior will be used.
Call Stack (most recent call first):
  CMakeLists.txt:254 (include)
This warning is for project developers. Use -Wno-dev to suppress it.

-- Configuring done
-- Generating done
-- Build files have been written to: /home/benedikt/widelands-debug/widelands/build
[67/68] Linking CXX executable src/scripting/test/WL_VERSIONtest_scripting
src/third_party/libthird_party_eris.a(loslib.c.o): In Funktion »os_tmpname«:
loslib.c:(.text+0x467): Warnung: the use of `tmpnam' is dangerous, better use `mkstemp'
[68/68] cd /home/benedikt/widelands-debug/widelands/build && /usr/bin/ctest --output-on-failure
Test project /home/benedikt/widelands-debug/widelands/build
    Start 1: test_economy
1/4 Test #1: test_economy ..................... Passed 0.06 sec
    Start 2: test_io_filesystem
2/4 Test #2: test_io_filesystem ............... Passed 0.03 sec
    Start 3: notifications_test
3/4 Test #3: notifications_test ............... Passed 0.01 sec
    Start 4: test_scripting
4/4 Test #4: test_scripting ................... Passed 0.01 sec

100% tests passed, 0 tests failed out of 4

Total Test time (real) = 0.12 sec
mv: Aufruf von stat für 'src/widelands' nicht möglich: Datei oder Verzeichnis nicht gefunden

« Back to merge proposal