Skip to content

GitLab

  • Menu
Projects Groups Snippets
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • KDE Craft Blueprints KDE Craft Blueprints
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 2
    • Issues 2
    • List
    • Boards
    • Service Desk
    • Milestones
  • Bugzilla
    • Bugzilla
  • Merge requests 8
    • Merge requests 8
  • 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
  • Packaging
  • KDE Craft BlueprintsKDE Craft Blueprints
  • Merge requests
  • !399

Fix expat compilation on MSVC non-english language pack

  • Review changes

  • Download
  • Email patches
  • Plain diff
Open Christian Rendina requested to merge lakor/craft-blueprints-kde:master into master May 02, 2022
  • Overview 2
  • Commits 2
  • Changes 1

This PR fixes libexpat compilation on a specific situation (having cl.exe/Visual Studio output not being in english), already reported and discussed on the libexpat repository https://github.com/libexpat/libexpat/issues/418.

My solution to this is to apply the same virtual env solution used in libs/ghostscript, by forcing the cl.exe to output in English, altrogh this (while I have not tested it myself) should require the english language pack to be installed on the system.

The cmake regex are not aware of multilanguages in compilers in the last version of cmake (3.23.1 at this time), I am not aware of any way to dynamically set an environment variable without updating the system on cmake, which is why I have decided to fall back to this solution.

This issue can be reproduced if any default language is set other than English in Visual Studio.

Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: master