Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[kf5kio] new port #19779

Merged
merged 33 commits into from
Oct 11, 2021
Merged

[kf5kio] new port #19779

merged 33 commits into from
Oct 11, 2021

Conversation

wrobelda
Copy link
Contributor

@wrobelda wrobelda commented Aug 28, 2021

Describe the pull request
Adds new kf5kio port.
Depends on following ports:

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a new experimental fast check for PR issues. Please let us know if this bot is helpful!

After committing all other changes, the version database must be updated
git add -u && git commit
git checkout 21935b744dc411e0fb1926618fdfb18c67d4b188 -- versions
./vcpkg x-add-version --all
Diff
diff --git a/versions/baseline.json b/versions/baseline.json
index 2c5a139..1d6df47 100644
--- a/versions/baseline.json
+++ b/versions/baseline.json
@@ -2892,6 +2892,10 @@
       "baseline": "5.84.0",
       "port-version": 1
     },
+    "kf5bookmarks": {
+      "baseline": "5.75.0",
+      "port-version": 0
+    },
     "kf5codecs": {
       "baseline": "5.84.0",
       "port-version": 1
@@ -2952,6 +2956,10 @@
       "baseline": "5.84.0",
       "port-version": 0
     },
+    "kf5kio": {
+      "baseline": "5.75.0",
+      "port-version": 0
+    },
     "kf5plotting": {
       "baseline": "5.84.0",
       "port-version": 1
@@ -2959,7 +2967,7 @@
     "kf5service": {
       "baseline": "5.84.0",
       "port-version": 0
-    },    
+    },
     "kf5solid": {
       "baseline": "5.84.0",
       "port-version": 0

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a new experimental fast check for PR issues. Please let us know if this bot is helpful!

After committing all other changes, the version database must be updated
git add -u && git commit
git checkout 21935b744dc411e0fb1926618fdfb18c67d4b188 -- versions
./vcpkg x-add-version --all
Diff
diff --git a/versions/baseline.json b/versions/baseline.json
index 9d34a3d..1d6df47 100644
--- a/versions/baseline.json
+++ b/versions/baseline.json
@@ -2892,6 +2892,10 @@
       "baseline": "5.84.0",
       "port-version": 1
     },
+    "kf5bookmarks": {
+      "baseline": "5.75.0",
+      "port-version": 0
+    },
     "kf5codecs": {
       "baseline": "5.84.0",
       "port-version": 1

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a new experimental fast check for PR issues. Please let us know if this bot is helpful!

After committing all other changes, the version database must be updated
git add -u && git commit
git checkout 21935b744dc411e0fb1926618fdfb18c67d4b188 -- versions
./vcpkg x-add-version --all
Diff
diff --git a/versions/baseline.json b/versions/baseline.json
index 9d34a3d..1d6df47 100644
--- a/versions/baseline.json
+++ b/versions/baseline.json
@@ -2892,6 +2892,10 @@
       "baseline": "5.84.0",
       "port-version": 1
     },
+    "kf5bookmarks": {
+      "baseline": "5.75.0",
+      "port-version": 0
+    },
     "kf5codecs": {
       "baseline": "5.84.0",
       "port-version": 1

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a new experimental fast check for PR issues. Please let us know if this bot is helpful!

After committing all other changes, the version database must be updated
git add -u && git commit
git checkout 21935b744dc411e0fb1926618fdfb18c67d4b188 -- versions
./vcpkg x-add-version --all
Diff
diff --git a/versions/baseline.json b/versions/baseline.json
index 9d34a3d..1d6df47 100644
--- a/versions/baseline.json
+++ b/versions/baseline.json
@@ -2892,6 +2892,10 @@
       "baseline": "5.84.0",
       "port-version": 1
     },
+    "kf5bookmarks": {
+      "baseline": "5.75.0",
+      "port-version": 0
+    },
     "kf5codecs": {
       "baseline": "5.84.0",
       "port-version": 1

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a new experimental fast check for PR issues. Please let us know if this bot is helpful!

After committing all other changes, the version database must be updated
git add -u && git commit
git checkout 21935b744dc411e0fb1926618fdfb18c67d4b188 -- versions
./vcpkg x-add-version --all
Diff
diff --git a/versions/baseline.json b/versions/baseline.json
index 9d34a3d..b2a0138 100644
--- a/versions/baseline.json
+++ b/versions/baseline.json
@@ -2892,6 +2892,10 @@
       "baseline": "5.84.0",
       "port-version": 1
     },
+    "kf5bookmarks": {
+      "baseline": "5.84.0",
+      "port-version": 0
+    },
     "kf5codecs": {
       "baseline": "5.84.0",
       "port-version": 1

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a new experimental fast check for PR issues. Please let us know if this bot is helpful!

After committing all other changes, the version database must be updated
git add -u && git commit
git checkout 21935b744dc411e0fb1926618fdfb18c67d4b188 -- versions
./vcpkg x-add-version --all
Diff
diff --git a/versions/baseline.json b/versions/baseline.json
index 03527e4..b2a0138 100644
--- a/versions/baseline.json
+++ b/versions/baseline.json
@@ -2967,7 +2967,7 @@
     "kf5service": {
       "baseline": "5.84.0",
       "port-version": 0
-    },    
+    },
     "kf5solid": {
       "baseline": "5.84.0",
       "port-version": 0

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a new experimental fast check for PR issues. Please let us know if this bot is helpful!

After committing all other changes, the version database must be updated
git add -u && git commit
git checkout 21935b744dc411e0fb1926618fdfb18c67d4b188 -- versions
./vcpkg x-add-version --all
Diff
diff --git a/versions/baseline.json b/versions/baseline.json
index 03527e4..b2a0138 100644
--- a/versions/baseline.json
+++ b/versions/baseline.json
@@ -2967,7 +2967,7 @@
     "kf5service": {
       "baseline": "5.84.0",
       "port-version": 0
-    },    
+    },
     "kf5solid": {
       "baseline": "5.84.0",
       "port-version": 0

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a new experimental fast check for PR issues. Please let us know if this bot is helpful!

After committing all other changes, the version database must be updated
git add -u && git commit
git checkout 21935b744dc411e0fb1926618fdfb18c67d4b188 -- versions
./vcpkg x-add-version --all
Diff
diff --git a/versions/baseline.json b/versions/baseline.json
index 03527e4..b2a0138 100644
--- a/versions/baseline.json
+++ b/versions/baseline.json
@@ -2967,7 +2967,7 @@
     "kf5service": {
       "baseline": "5.84.0",
       "port-version": 0
-    },    
+    },
     "kf5solid": {
       "baseline": "5.84.0",
       "port-version": 0

@JonLiu1993 JonLiu1993 added the category:new-port The issue is requesting a new library to be added; consider making a PR! label Aug 30, 2021
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a new experimental fast check for PR issues. Please let us know if this bot is helpful!

After committing all other changes, the version database must be updated
git add -u && git commit
git checkout de623488c7d9ab90328ec447f241dfff172b1c45 -- versions
./vcpkg x-add-version --all
Diff
diff --git a/versions/baseline.json b/versions/baseline.json
index ed76e99..94084ad 100644
--- a/versions/baseline.json
+++ b/versions/baseline.json
@@ -2967,7 +2967,7 @@
     "kf5service": {
       "baseline": "5.84.0",
       "port-version": 0
-    },    
+    },
     "kf5solid": {
       "baseline": "5.84.0",
       "port-version": 0

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a new experimental fast check for PR issues. Please let us know if this bot is helpful!

After committing all other changes, the version database must be updated
git add -u && git commit
git checkout de623488c7d9ab90328ec447f241dfff172b1c45 -- versions
./vcpkg x-add-version --all
Diff
diff --git a/versions/baseline.json b/versions/baseline.json
index ed76e99..94084ad 100644
--- a/versions/baseline.json
+++ b/versions/baseline.json
@@ -2967,7 +2967,7 @@
     "kf5service": {
       "baseline": "5.84.0",
       "port-version": 0
-    },    
+    },
     "kf5solid": {
       "baseline": "5.84.0",
       "port-version": 0

@wrobelda wrobelda force-pushed the kf5kio branch 2 times, most recently from bc90758 to 5b424cd Compare August 30, 2021 20:16
@wrobelda
Copy link
Contributor Author

@JackBoosY all the dependencies are now in master, ready to merge.

@JackBoosY JackBoosY removed the depends:different-pr This PR or Issue depends on a PR which has been filed label Sep 30, 2021
@wrobelda
Copy link
Contributor Author

wrobelda commented Oct 4, 2021

@JackBoosY can this be merged already?

@wrobelda
Copy link
Contributor Author

wrobelda commented Oct 6, 2021

@JonLiu1993 is there anything that's left to be done here? It is a blocker for a bunch of other PRs of mine, hence me pinging.

@JackBoosY
Copy link
Contributor

We got back from our vacations, sorry for late.

@JonLiu1993
Copy link
Member

@wrobelda ,I am testing feature locally,sorry for late.

versions/k-/kf5kio.json Outdated Show resolved Hide resolved
@JonLiu1993
Copy link
Member

All feature are tested successfully in the following triplet :

  • x86-windows
  • x64-windows
  • x64-windows-static

@JonLiu1993 JonLiu1993 removed the requires:all-feature-testing vcpkg install port[all features supported by that port] needs to be demonstrated to function label Oct 8, 2021
@JonLiu1993 JonLiu1993 added the info:reviewed Pull Request changes follow basic guidelines label Oct 9, 2021
@BillyONeal BillyONeal merged commit 6ff166c into microsoft:master Oct 11, 2021
@BillyONeal
Copy link
Member

Thanks for the new port :)

@wrobelda wrobelda deleted the kf5kio branch October 12, 2021 07:01
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! info:reviewed Pull Request changes follow basic guidelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants