search

FreeFem / FreeFem-sources

FreeFEM source code
https://freefem.org/
Other
770 stars 191 forks source link

FF_INCLUDEPATH is not split when it contains several paths #36

Closed cdoucet closed 6 years ago

cdoucet commented 6 years ago

Environment variable FF_INCLUDEPATH is not split when it contains several paths separated by ":" on Unix systems. Here is the error message coming from examples++/include.edp (take a look at the last try):

--lex open :funct.edp --lex open :/home/cdoucet/Depots/github/FreeFem-sources/examples++:/home/cdoucet/Depots/github/FreeFem-sources/examples++-3d:/home/cdoucet/Depots/github/FreeFem-sources/examples++-load/funct.edp Error openning file funct.edp in: -- try :"funct.edp" -- try :"/home/cdoucet/Depots/github/FreeFem-sources/examples++:/home/cdoucet/Depots/github/FreeFem-sources/examples++-3d:/home/cdoucet/Depots/github/FreeFem-sources/examples++-load/funct.edp"

when FF_INCLUDEPATH is defined by

export FF_INCLUDEPATH=$FF_INCLUDEPATH:/home/cdoucet/Depots/github/FreeFem-sources/examples++:/home/cdoucet/Depots/github/FreeFem-sources/examples++-3d:/home/cdoucet/Depots/github/FreeFem-sources/examples++-load

sgarnotel commented 6 years ago

You must separate paths using a ; not a :. See the doc

frederichecht commented 6 years ago

Il faut faire attention, j'ai mis un ";" pour windows car il y a des : dans path window C:\Users....

et je veux un truc assez universel

Le 27 juil. 2018 à 11:59, cdoucet notifications@github.com a écrit :

Environment variable FF_INCLUDEPATH is not split when it contains several paths separated by ":" on Unix systems. Here is the error message coming from examples++/include.edp (take a look at the last try):

--lex open :funct.edp --lex open :/home/cdoucet/Depots/github/FreeFem-sources/examples++:/home/cdoucet/Depots/github/FreeFem-sources/examples++-3d:/home/cdoucet/Depots/github/FreeFem-sources/examples++-load/funct.edp Error openning file funct.edp in: -- try :"funct.edp" -- try :"/home/cdoucet/Depots/github/FreeFem-sources/examples++:/home/cdoucet/Depots/github/FreeFem-sources/examples++-3d:/home/cdoucet/Depots/github/FreeFem-sources/examples++-load/funct.edp"

when FF_INCLUDEPATH is defined by

export FF_INCLUDEPATH=$FF_INCLUDEPATH:/home/cdoucet/Depots/github/FreeFem-sources/examples++:/home/cdoucet/Depots/github/FreeFem-sources/examples++-3d:/home/cdoucet/Depots/github/FreeFem-sources/examples++-load

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/FreeFem/FreeFem-sources/issues/36, or mute the thread https://github.com/notifications/unsubscribe-auth/AFb2jLQVY-_bziHGyIMh50_8qwJuwhj9ks5uKuRngaJpZM4VjPFN.

cdoucet commented 6 years ago

Ok I see. Maybe it could be an improvement of user experience to display an explanatory error message in some unambiguous situations, e.g. when FF_INCLUDEPATH contains at least two ":" but no ";". But it is not mandatory as it appears in the documentation. Thanks.