-
-
Notifications
You must be signed in to change notification settings - Fork 792
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
Port code to INDI 2.0.2 and update CPM #3269
base: master
Are you sure you want to change the base?
Conversation
based on work of Chris Kuethe <chris.kuethe@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for adding your first pull request to Stellarium. If you have questions, please do not hesitate to contact us.
Looks interesting, thank you! Anybody with real hardware willing to test? |
for openSUSE TW users: I've created a test repo with Stellarium compiled using INDI 2.0. You can find it here: https://build.opensuse.org/package/show/home:polslinux:branches:Education/stellarium |
IIRC there were some problems only present in Qt6 but not Qt5. Did you check this? |
@10110111 the tests I did were with Stellarium compiled using QT6 |
macOS:
|
@alex-w thanks for checking macos! I'm gonna look into it |
please see our build bots also - looks like cmake has missing files + source code need to be update (check include# directives) for INDI |
That's weird though, I can't reproduce that issue locally 🤔 |
Probably you have locally installed INDI |
yep I do have INDI installed, but I'm running cmake with |
Thanks for following this up. I am super busy at the moment and got stuck in the CMake porting part. I wonder if it is really necessary to build INDI through |
I agree that bundling INDI is, IMHO, not the best solution. |
If the existence of compatible versions is guaranteed by simple dependencies, and we don't change anything in the INDI code, this would be OK for me. |
Wouldn't this go against the practice used for |
If INDI is available as standard package from default repositories just like Qt stuff, and people may have installed them already for other programs, it would be preferred over CPM and private compilations. But I had understood our first adoption of INDI had some non-standard edits in the INDI 0.* sources, so required a private build. The other packages for QXlsx and CMS are also still optional, but there are no apt packages from Debian, right? |
Right. BTW I just noticed that in Ubuntu, even in 23.04 and in the development version, Stellarium is still 0.22.2. Do you happen to know why it's so old? |
No, I don't know how the official Ubuntu maintainers upgrade their packages. We provide an Ubuntu repo with the latest version, see Guide 2.3.3. If INDI does the same, it should be fine for us. But CPM may be the solution if INDI is not available everywhere. |
Speaking of Linux, most of the widely used distros have INDI 2:
For MacOS, there may be a port or whatever. Still, compiling there it's a pretty trivial task. There are instructions on the INDI website. For Windows, I don't think it's gonna be an issue. There they have ASCOM, official software from hardware makers and imaging software that integrates pretty well with Stellarium. |
Hello 😄 do we have a final decision on what to do with INDI? Treat it as a dep or bundle it? |
INDI client within Telescope Control plugin expected to all platforms, so, e.g. Windows user can connect direct to INDI server onto Linux. The current branch has few |
I guess we better keep CPM then. Otherwise Windows support could be an issue 🤔 |
Any news? It seems other projects are expecting Stellarium finally moving to INDI 2.* 'really soon'. |
Any news here? |
I'm working on getting Stellarium running on Windows on ARM. I already got it working without CalcMySky and without the Telescope plugin (due to INDI). But I just packaged CalcMySky and I'm going to package libindiclient, but it will have to be the latest INDI (2.0.4 as of this writing). Support for INDI 2.x should happen soon™️ . |
Any progress? |
Well since I got Stellarium building for Windows on Arm with the current INDI I did not pursue this any further. It was not my main objective. I don't know this library nor how it's used by Stellarium so someone who knows this stuff should do this upgrade |
As already mentioned four months ago: if we didn't try to build INDI from Stellarium but treat it like any other dependency, the issue would have been long solved. Some distros, e.g. NixOS, apply the patch for new INDI support and just use the system-provided INDI. |
If we can depend on optionally system-installed stock INDI2 (build yourself where required), this would be the best solution IMO. When users don't need telescope control, there is no need for INDI either. That would make it a "recommended" package for apt-get or similar systems. We still need to know what "our own copy" has changed, and a maintainer who occasionally cares for INDI updates as years go by. |
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
What's the verdict here ? |
|
Anybody using INDI and interested to move forward? |
Is anybody out there using Stellarium with INDI interested in going forward? |
This PR is based on the work done by some people at #2891. In particular:
What was tested:
What was not tested: