//--------------
Benjaming - June 13, 2012:

the ApplicationsSlicerAppData.py are with the other python files because we include it.
How can I include it if ApplicationsSlicerAppData.py is far away ?
Thanks,
Benjamin

//--------------
Jc - June 13, 2012:

Python path / sys.path is your friend. You could look at how things are done with python_unit_test.
Jc

//--------------
Benjaming - June 13, 2012:

What about adding the file into Base/Pyhton/Slicer ?
and after I should be able to do "from Slicer import ApplicationsSlicerAppData"

Make sens, I also propose the file is renamed:
ApplicationsSlicerAppDataPaths.py.in / ApplicationsSlicerAppDataPaths.py

every files are named lowercase in this directory.
Maybe we should name it : appdatapath.py or appdata.py instead

Indeed, the idea is to follow this convention [1]

slicerappdatapaths.py

May be we could add the file into a folder named "datapaths", it means we would have:

slicer/datapaths/slicerapp.py

Later we could then add file like these ones if it makes sens:

slicer/datapaths/baseqtcore.py
slicer/datapaths/baseqtgui.py
slicer/datapaths/mrml.py

What do you think ?

[1] http://www.python.org/dev/peps/pep-0008/#package-and-module-names

As long as we don't need more path, we should stay with slicerappdatapaths.py

Jc: "Indeed, the idea is to follow this convention [1]"
-> can you update the Slicer coding style guide ?

1) slicerappdatapaths.py sounds good to me. Make sure the file is not installed, it wouldn't make sens.

2) Python guidelines have been updated. See http://www.slicer.org/slicerWiki/index.php/Documentation/4.1/Developers/Style_Guide/Python#Guidelines

fixed in r20354:
http://viewvc.slicer.org/viewvc.cgi/Slicer4?view=revision&revision=20354

The file "Python/slicer/slicerappdatapaths.py" should NOT be committed, it currently contain hard coded value !

The file Applications/SlicerApp/Testing/Python/slicerappdatapaths.py.in should be configured into the the build directory:

${Slicer_BINARY_DIR}/bin/Python/slicer

Fixed in r20363
http://viewvc.slicer.org/viewvc.cgi/Slicer4?view=revision&revision=20363

I suggested the use of "${Slicer_BINARY_DIR}/bin/Python/slicer" because I anticipated that you would keep the file "slicerappdatapaths.py.in" in "pplications/SlicerApp/Testing/Python/"

I would recommend to move the file "slicerappdatapaths.py.in"

If you convince me that moving "slicerappdatapaths.py.in" is not what we should do, use ${CMAKE_CURRENT_BINARY_DIR}/slicer/slicerappdatapaths.py instead of "${Slicer_BINARY_DIR}/bin/Python/slicer/slicerappdatapaths.py". See http://viewvc.slicer.org/viewvc.cgi/Slicer4/trunk/Base/Python/CMakeLists.txt?view=markup&pathrev=20363#l59

Fixed in r20366
http://viewvc.slicer.org/viewvc.cgi/Slicer4?view=revision&revision=20366

Thanks :)

Make also sure to fix indent, it is missing two spaces. See http://viewvc.slicer.org/viewvc.cgi/Slicer4/trunk/Applications/SlicerApp/Testing/Python/CMakeLists.txt?view=markup&pathrev=20366#l166

Dear issue reporter,

Good news :) Slicer developers SOLVED the problem you reported - YOU now need to VERIFY and CLOSE this issue.