openfluid icon indicating copy to clipboard operation
openfluid copied to clipboard

Ware migration failing under Windows

Open Arthoni opened this issue 1 year ago • 0 comments

Piste d'une variable d'environnement mal transmise

  • Ajout contrôle variable OpenFLUID_HOME_DIR dans openfluid/cmake/OpenFLUIDConfig.cmake.in
-- OpenFLUID_HOME_DIR: _OpenFLUID_HOME_DIR-NOTFOUND
-- OpenFLUID_INCLUDE_DIRS: C:/LocalInstallAT_msys2/include
-- ENV OpenFLUID_MIGRATION_INCLUDE_PATHS:

Remplacement à la main

  • modification C:\LocalInstallAT_msys2\share\openfluid\waresdev\migration\templates\CMakeLists.txt pour affichage variables mobilisées et remplacement local d'OpenFLUID_HOME_DIR en tant que var d'env
$ more /home/OpenFLUID/TEST/_migration/signature/configure_log.out
-- OpenFLUID_HOME_DIR: C:/LocalInstallAT_msys2/
-- OpenFLUID_INCLUDE_DIRS: C:/LocalInstallAT_msys2/include
-- ENV OpenFLUID_MIGRATION_INCLUDE_PATHS: C:/_brli_at_msys2/src/
  • Test va plus loin, erreur inconnue :
-- Processing CMake configuration [OK]
-- Creating new ware structure [OK]
-- Dispatching
[ERROR] unknown error
  • Relancé en verbeux :
-- Processing CMake configuration : processing CMakeLists.txt, processing CMake.in.config [OK]
-- Creating new ware structure : creating new source tree, applying CMake configuration, preparing src directory [OK]
-- Dispatching : standard doc directory
[ERROR] unknown error
  • [ ] Semble bien indiquer que 2 variables manquaient

Analyse erreur résultante

  • Pas une histoire de répertoire de destination différent de la source, vérifié avec même commit sous Ubuntu 22.04
  • [ ] Rendre code plus explicite et recompiler pour préciser l'erreur
    • [x] try catch autour de la ligne de copie ? ✅ 2023-12-01
    • [x] Changer dans vrai code l'env var, voire écrire en dur le chemin ✅ 2023-12-01

Le catch indique une filesystem error : la copie échoue parce que le répertoire de destination existe.

  • Spécificité windows ? Non, plutôt le fait que plusieurs options donnent un comportement indéfini ?

Arthoni avatar May 17 '24 09:05 Arthoni