SFGUI icon indicating copy to clipboard operation
SFGUI copied to clipboard

Published 20 hours ago • verified publisher icon TankOs

Travis build + Conan dependencies managment

Open DrBarbare opened this issue 6 years ago • 5 comments

Build Status

IMPORTANT: A Travis account will be needed to use the CI pipeline.

Features

  • Added a Conan configuration file to easily get SFML
  • Added Travis configuration (Ubuntu, OSX and Windows) with scripts to configure and build the project (no test stage)
  • Update the Readme to reflect changes in workflow
  • Minor changes in CMake configuration to support Conan
  • Fixes in CMake configuration revealed by Travis
  • Upgraded .gitignore

Known issues

  • Travis job does not run any tests at the moment, only builds

To test

Everything should be in the update documentation. You should still be able to work on the project without the need of Conan, it does make it easier.

DrBarbare avatar Mar 02 '19 04:03 DrBarbare

@eXpl0it3r Sorry it took so long to do so simple things... How are the Travis scripts though? My coworkers say that I tend to over complicate things... Also is this account linked to Travis yet? @TankOs?

DrBarbare avatar Apr 21 '19 15:04 DrBarbare

Totally forgot to check back here, been a while since I've touched SFGUI... 😓

As Travis CI is kind of dead (30 day trial with 10'000 credits) and given that GitHub has since added GitHub Actions, I've gone ahead and added a very basic script with #91

Would you be interested to change this PR (or create a new one) to adapt the conan integration with GitHub Actions instead of Travis CI?

eXpl0it3r avatar Mar 23 '22 12:03 eXpl0it3r

Indeed. this has been quite a while. I didn't even realize Travis was not the hot stuff anymore (I mostly use Jenkins in professional settings). This PR should be closed then.

I don't think it would be worth my time to work on Conan integration for this project for a couple reason, first and foremost, my current occupation is very taxing, and I don't feel like spending extra time coding (or doing anything in front of a computer. Thanks to Zoomagedon). Second, I feel like using Conan directly in CMake is better when your project is the final item. For a project like SFGUI, it may be better to write a conanfile.py directly, that can then be used with conan create to easily make releases. I've had my team develop such packaging scripts for fairly complex neural network libraries with great success in deployment.

That being said, if there are other changes you think would benefit the project, and if SFGUI is getting some love back from you lovely maintainers, I will allocate a bit of my free time to port said changes.

Thank you for reaching out!

DrBarbare avatar Mar 24 '22 12:03 DrBarbare

Thanks for getting back to my comment. Might make sense to close the PR, but maybe you could PR the README changes (without the Conan bits), as I think it improves things quite a bit. Do you have time for that or shall I just copy it?

eXpl0it3r avatar Mar 25 '22 08:03 eXpl0it3r

I will see to do it after work today (5-6pm EDT).

On Fri, Mar 25, 2022 at 4:01 AM Lukas Dürrenberger @.***> wrote:

Thanks for getting back to my comment. Might make sense to close the PR, but maybe you could PR the README changes (without the Conan bits), as I think it improves things quite a bit. Do you have time for that or shall I just copy it?

— Reply to this email directly, view it on GitHub https://github.com/TankOs/SFGUI/pull/73#issuecomment-1078755059, or unsubscribe https://github.com/notifications/unsubscribe-auth/AJEP46UDYREWNEZWAFOANMDVBVXFDANCNFSM4G3IUQZA . You are receiving this because you authored the thread.Message ID: @.***>

DrBarbare avatar Mar 25 '22 13:03 DrBarbare