Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Can't install Tango server via 'make install' #25

Open
jkrueger1 opened this issue Aug 22, 2022 · 7 comments
Open

Can't install Tango server via 'make install' #25

jkrueger1 opened this issue Aug 22, 2022 · 7 comments

Comments

@jkrueger1
Copy link
Contributor

Durring the installation via:

make install

I got the following error:

CMake Error at applications/tango/python/cmake_install.cmake:54 (file):
  file INSTALL cannot find
  "/home/jkrueger/sources/LiMA/applications/tango/python/scripts/LimaCCDs":
  No such file or directory.
Call Stack (most recent call first):
  cmake_install.cmake:168 (include)

I assume, the corresponding cmake file is not correct, i.e. not really related to the python setup.py file.
There is no creation of the LimaCCDs file (at least I didn't find it)

@laurent-claustre
Copy link
Contributor

laurent-claustre commented Aug 23, 2022

Hi Jens,
you are right, we do not decide yet to either remove the CMakeLists.txt file or update it.
@sdebionne

@sdebionne
Copy link
Contributor

The file scripts/LimaCCDs was removed when the project was reorganized in a more pythonic way and the CMakeLists.txt is indeed broken. I guess we could easily restore scripts/LimaCCDs if installation with CMake are used... but honestly I would prefer to keep one single build / install script (the python one).

@jkrueger1
Copy link
Contributor Author

I prefer (at the moment) the installation via CMake (i have to develop and test some camera drivers) and so it would be nice (for me) if the CMake can install LimaCCDs. For me it doesn't matter if the CMake generates it via the setup.py or it is a file from the repo as it was before.

@sdebionne
Copy link
Contributor

Allright, the fix is being reviewed, should be available soon...

@laurent-claustre
Copy link
Contributor

Merged.

@jkrueger1
Copy link
Contributor Author

Thanx

@birkenfeld
Copy link

Unfortunately, the fix seems not to be present in the v1.9.18 tag of main Lima (or v1.9.19rc1 either). Probably the submodule needs to be updated there?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants