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

CMake: USE_SWIG_DEPENDENCIES for CMake 3.20+ #691

Merged
merged 1 commit into from Jun 25, 2021

Conversation

ferdnyc
Copy link
Contributor

@ferdnyc ferdnyc commented Jun 21, 2021

(F---ING FINALLY!!!)

Mostly a developer convenience, when building using the newest CMake 3.20 or later, the SWIG targets will be configured with USE_SWIG_DEPENDENCIES set to TRUE, which causes the target to automatically set up dependencies on all of the files referenced by the SWIG bindings.

Which means, when you make changes to Frame.h or Clip.h or whatever, the bindings will be automatically regenerated, and you no longer have to rm -r build/bindings in order to force a re-sync with the header files.

@ferdnyc ferdnyc added build Issues related to compiling or installing libopenshot and its dependencies bindings libopenshot's Python or Ruby interface bindings labels Jun 21, 2021
@codecov
Copy link

codecov bot commented Jun 21, 2021

Codecov Report

Merging #691 (93cdfae) into develop (d9ea98e) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop     #691   +/-   ##
========================================
  Coverage    50.41%   50.41%           
========================================
  Files          155      155           
  Lines        13315    13315           
========================================
  Hits          6713     6713           
  Misses        6602     6602           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d9ea98e...93cdfae. Read the comment docs.

@ferdnyc
Copy link
Contributor Author

ferdnyc commented Jun 25, 2021

Merging, as this is a Good Thing™ for the very few developers it will currently affect (CMake 3.20 being quite new, at the moment).

@ferdnyc ferdnyc merged commit 09eb807 into OpenShot:develop Jun 25, 2021
@ferdnyc ferdnyc deleted the swig-deps branch June 25, 2021 05:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bindings libopenshot's Python or Ruby interface bindings build Issues related to compiling or installing libopenshot and its dependencies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant