Skip to content

[many ports] Add more KF5 frameworks#13469

Closed
wrobelda wants to merge 161 commits intomicrosoft:masterfrom
wrobelda:new_kf5_ports
Closed

[many ports] Add more KF5 frameworks#13469
wrobelda wants to merge 161 commits intomicrosoft:masterfrom
wrobelda:new_kf5_ports

Conversation

@wrobelda
Copy link
Contributor

@wrobelda wrobelda commented Sep 11, 2020

This adds ports for a whole bunch more KDE Frameworks

  • Which triplets are supported/not supported? Have you updated the CI baseline?
    All default triplets should be supported

  • Does your PR follow the maintainer guide?
    Yes, except the requirement to publish the ports separately. However, these ports are for the most part a copy-and-paste, with some exceptions.

wrobelda and others added 30 commits September 10, 2020 20:57
kf5crash
  - make x11 patch exclusive to linux
  - enable win/macOS builds
kf5syntaxhighlighting
  - fix building on non-Win by removing the cli tool from all platforms
This reverts commit 8703c24.
Co-authored-by: NancyLi1013 <46708020+NancyLi1013@users.noreply.github.com>
@NancyLi1013
Copy link
Contributor

Hi @wrobelda

Could you please spilt these changes in this PR into several PRs? Since there are a large amount of files here, which is more difficult to review.

You can add 2 new ports in one PR.

@wrobelda
Copy link
Contributor Author

wrobelda commented Nov 12, 2020

Could you please spilt these changes in this PR into several PRs? Since there are a large amount of files here, which is more difficult to review.

@NancyLi1013 Can do. However, all these depend on the other PR (#13467) so I would appreciate if you could review and hopefully merge that one first.

@NancyLi1013
Copy link
Contributor

Thanks for your kindly reminder.
I will try to review PR #13467 today.

@NancyLi1013 NancyLi1013 added the depends:different-pr This PR or Issue depends on a PR which has been filed label Feb 8, 2021
@wrobelda
Copy link
Contributor Author

wrobelda commented Mar 6, 2021

As requested by @NancyLi1013, I will create separate PRs for these ports.

@wrobelda wrobelda closed this Mar 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

category:new-port The issue is requesting a new library to be added; consider making a PR! depends:different-pr This PR or Issue depends on a PR which has been filed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants