Project

General

Profile

Actions

Support Request #544

closed

ADTFS-46508 Improve documentaion regarding CMake Environment

Added by hidden about 7 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
Normal
Customer:
ELEKTROBIT
Department:
SUPPORT
Requester's Priority:
Normal
Support Level:
2nd Level
Resolution:
Product Issue Opened
Affected Products:
Platform:
Ubuntu 16.04 64bit, Windows 7 64bit
Topic:
ADTF::Doc
FAQ Links:

Description

Supportanfrage:

Beim bauen der Examples ist mir einiges aufgefallen, was verbessert werden könnte:

ADTFQTMacros.cmake

  • CMAKE_PREFIX_PATH sollte durch QT_DIR oder ADTF_QT_DIR oder ähnliches ersetzt werden.
    Rationale: CMake-gui versteckt den Eintrag defaultmäßig und der Name sollte gleich klar machen, dass es sich um QT handelt.

Examples projekt:

  • AEV_ENABLE_EXAMPLES_INSTALLATION, sollte durch ADTF_ENABLE_EXAMPLES_INSTALLATION ersetzt werden
  • ADTF_ENABLE_QT sollte als cache variable angelegt werden oder ganz entfernt werden, da die Variable eh nur bei den Qt-Filtern geprüft wird
  • SDLMAIN_LIBRARY sollte durch SDL_MAIN_LIBRARY ersetzt werden
  • CMAKE_INSTALL_PREFIX sollte defaultmäßig auf ADTF_DIR gesetzt werden und nicht auf C:/Program Files/adtf_examples

Zum SDL Demo: Hier finde ich etwas unglücklich dass der Pfad zum SDL.dll abhängig vom SDL_INCLUDE_DIR angegeben wird, dass passt nicht bei jeder Installation. Vielleicht kann man hier auch besser mit einer Cache Variablen wie unter Linux arbeiten.
install(FILES ${SDL_INCLUDE_DIR}/../lib/SDL.dll DESTINATION src/examples/bin/debug CONFIGURATIONS Debug)
Generell sollte irgendwo auch die Info zur verwendeten SDL-Version erscheinen, lediglich das build_examples.bat liefert dazu eine Info.

Verbesserungsvorschlage für die Dokumentation:

Die generellen Infos zum CMake Build System sollten aus dem Abschnitt "A_UTILS Library" in den Abschnitt "The ADTF CMake Environment" wandern. Auch sollte die Versionsinformation für CMake von 3.1.0 auf 3.5.2 geändert werden.

Best regards - Beste Grüße
Jochen Messner
EB Assist ADTF Support-Team

Lösung:

Zu dem Verbesserungsvorschlägen wurde ein Produktticket angelegt:

http://km-aev.in.audi.vwg/redmine/issues/39587

Actions

Also available in: Atom PDF