Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • Kig Kig
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 0
    • Issues 0
    • List
    • Boards
    • Service Desk
    • Milestones
  • Bugzilla
    • Bugzilla
  • Merge requests 1
    • Merge requests 1
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Releases
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • EducationEducation
  • KigKig
  • Merge requests
  • !12

Fix compilation of the Python scripting code

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Albert Astals Cid requested to merge work/aacid/fix_python_stuff into master Aug 24, 2022
  • Overview 2
  • Commits 1
  • Pipelines 3
  • Changes 4

3 changes are needed:

  • Fix boost/python detection code in cmake

We can't detect the header without having detected python first because when linking the app that uses the header we need to link to python, so move the header to the code that we're trying to compile

  • Fix ObjectImpType constructor, 0 is not a valid KLazyLocalizedString

  • Set QT_NO_KEYWORDS since the new python has a variable called slots, we need to tell Qt to not use that

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: work/aacid/fix_python_stuff