Skip to content

chore: remove ECM and KSH submodules#1323

Merged
ouuan merged 3 commits into
masterfrom
remove-ecm-ksh-submodules
Aug 11, 2024
Merged

chore: remove ECM and KSH submodules#1323
ouuan merged 3 commits into
masterfrom
remove-ecm-ksh-submodules

Conversation

@ouuan

@ouuan ouuan commented Feb 18, 2024

Copy link
Copy Markdown
Member

Description

Remove ECM and KSH submodules and clone them in CI.

Related Issues / Pull Requests

#1322

@ouuan ouuan changed the base branch from master to v7.0 February 18, 2024 15:21
@coder3101

coder3101 commented Jun 1, 2024

Copy link
Copy Markdown
Member

We should not build the KSH or ECM in CI rather use package managers to install them. We are going with upgrading to qt6 and KF5* packages such as syntax-higlighting needs qt5.

  • For macOS, we can use the official homebrew tap
  • For Ubuntu, libkf5syntaxhighlighting-dev package should work
  • For Windows, we can use craft but I haven't tried this nor do I have a windows machine.

@coder3101 coder3101 changed the base branch from v7.0 to master June 1, 2024 09:23
@coder3101 coder3101 force-pushed the remove-ecm-ksh-submodules branch from ee9ee7b to 391e076 Compare June 1, 2024 09:26
@coder3101

Copy link
Copy Markdown
Member

macOS builds are failing because of architecture mismatch. GH has rolled out arm64 with macos-latest

@coder3101

Copy link
Copy Markdown
Member

As first step it's good to just get rid of the submodules.

@ouuan ouuan enabled auto-merge (squash) August 11, 2024 07:43
@ouuan ouuan merged commit 228e0f3 into master Aug 11, 2024
@ouuan ouuan deleted the remove-ecm-ksh-submodules branch August 11, 2024 07:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants