-
Notifications
You must be signed in to change notification settings - Fork 738
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
Update Spinnaker to v.1.19.0.22 #660
Conversation
jpsacha
commented
Dec 17, 2018
- Added QuickSpinC module
- Added SpinVideoC module
- Added TransportLayer*C modules
- Change Linux CI build to use Ubuntu to ease setup, Spinnaker is distributed for Ubuntu. Centos would require hacking to fix linking issues.
* Added QuickSpinC module * Added SpinVideoC module * Added TransportLayer*C modules * Change Linux CI build to use Ubuntu to ease setup, Spinnaker is distributed for Ubuntu.
I haven't tried it, but it shouldn't be too hard to install deb files on CentOS 7 using alien: /cc @vb216 |
I was trying with Centos6, was not able to install alien due to missing dependencies. May work with Centos7, but it requires additional configuration, compared to Ubuntu. |
I was able to install alien on Centos 7 and convert Spinnaker *.deb packages to *.rpm, but cannot install them due to numerous missing dependencies (manual editing using rpmrebuild is not a viable option). At this point I will make sure that current setup (with Ubuntu) works with full 'javacpp-presets' build (it does work if I just build Spinnaker). Hopefully last commit [a476f38] corrects the Travis configuration issue. Need to wait till regular CI build is finished. |
Since we're not actually using FlyCapture or Spinnaker during the builds, we don't need to install any of the dependencies. Anyway, I'll leave @vb216 decide what to do about this. If you're able to maintain the build with that version of Ubuntu, that's fine tough, but please get it working! I won't merge this until the builds pass. |
I've not got any strong opinion really.. I remember having similar problems with alien last time I tried it, so I guess maybe its logical to have a more 'reliable' setup on ubuntu than something pushed into centos, only downside I can see is the ci .sh files growing. But, I wouldn't be surprised if we have to revisit those at some point anyway if they grow to the point of becoming difficult to maintain. |
Looks to me that it will be easier to maintain Ubuntu build for Spinnaker, it may require less changes in the future, building on Centos may potentially require tweaks with each release. It may also make sense to move Ubuntu/Spinnaker part from I can do, in this PR, the basic split, that is, just move the parts needed to build Spinnaker to a separate script. |
Sounds alright sure, builds for |
Need to also change logic that determines if maven build is done in docker.
I changed Travis configuration to build without docker, similar to On other hand, I do not understand how FlyCapture is build on linux-armhf, the SDK archive is downloaded, copied, but never extracted. |
Thanks, but let's not do Bionic. Stay with Trusty and harmonize the build environment with For ARM, it gets extracted in cppbuild.sh because that's how they distribute those versions: |
I've just tried it on a machine running Trusty and it installs just fine. The only thing that doesn't work is the GUI, so we just need to comment out the relevant packages from
|
The current Spinnaker build setup is simple, easy to maintain, and builds really fast, only 2 min 22 seconds. Faster than any other job. I have put a lot of work in it trying to satisfy changing demands, first moving to Centos docker, then getting rid of docker, now back-porting to an obsolete distro. I am done tweaking it. Back-porting to an Trusty would make configuration more complex and more difficult to maintain. I rather move it to a separate repo, that is simpler to maintain, faster to release, and does not require days to complete CI build. Similar with FlyCapture (the PR #613 is waiting there for over 3 months to be merged). They are not needed by anything in |
Your fork is already on a separate repo! You can comment out everything you don't need in However, what I am offering is to maintain the builds you create as part of Bytedeco. Whenever something breaks with the builds, I'm the one that has to fix them, so I'm merely asking to make it easy to maintain. If you do not wish to have your work maintained and integrated into the distribution, that is fine. You don't need to do anything else than what you're doing now. |
If you simply do not have the time to work on this though, let's leave @vb216 do his job. :) We'll eventually get it working. |
@saudet I know you are doing a tremendous work maintaining javacpp projects. I really appreciate that. However it is quite difficult to contribute to javacpp-presets due to its complex build setup and extremely long CI builds. Overtime this thing become very large. Spinnaker (and FlyCapture) are only small pieces of javacpp-presets. Going forward, after this release, it really will be easier to keep them in a separate small repo, so they are easier to maintain and faster to release. I would prefer to keep it under Bytedeco. We can figure out details when there is a need for next release for Spinnaker/FlyCapture, after 1.19.0.22. I can fix FlyCapture, if there are issues with wrapper, but will leave CI build setup to @vb216. |
You don't need to keep everything as part of your fork or pull requests. Like I said, you can remove all other libraries from Anyway, as part of pull #613 we managed to extract without installing the DEB packages for FlyCapture on CentOS and |
Might be a quiet spell coming up from me again I'm afraid - job change on the horizon and usually takes a good few months to get up and running and back to having any spare capacity |
Ok, so with that we should be able to maintain with minimal effort until FlyCapture3, Spinnaker2, or something comes out. |