Skip to content

GitLab

  • Menu
Projects Groups Snippets
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • C cantarell-fonts
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 17
    • Issues 17
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 1
    • Merge requests 1
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Releases
  • Packages & Registries
    • Packages & Registries
    • Container Registry
  • Wiki
    • Wiki
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • GNOME
  • cantarell-fonts
  • Issues
  • #55

Closed
Open
Created Dec 07, 2021 by Fabian Greffrath@fabiangreffrathContributor

please make skia-pathops dependency optional

Hi there,

skia-pathops is quite a complex dependency to add to a font package. Since it is not yet packaged in Debian, I had to patch its explicit usage out of the make-static-fonts.py build script in order to rebuild the static fonts [1]. The build thus falls back to using the default "booleanOperations" algorithm for overlap removal. I have checked the resulting fonts in Gnome Shell, Gnome Control Center and some other apps and haven't encountered any obvious issues yet. Do you know of any issues with using the default overlap removal algorithm that I have missed so far, or what were the reasons for unconditionally choosing the "pathops" algorithm?

If you prefer, I can provide a merge request - either using skia-pathops only if it's available or using it by default but adding a choice not to use it and fall back to compileOTF's default.

[1] https://salsa.debian.org/fabian/fonts-cantarell/-/blob/master/debian/patches/01-no-pathops.patch

Assignee
Assign to
Time tracking