Skip to content

[kf5declarative] new port#19813

Merged
vicroms merged 34 commits intomicrosoft:masterfrom
wrobelda:kf5declarative
Oct 22, 2021
Merged

[kf5declarative] new port#19813
vicroms merged 34 commits intomicrosoft:masterfrom
wrobelda:kf5declarative

Conversation

@wrobelda
Copy link
Copy Markdown
Contributor

@wrobelda wrobelda commented Aug 31, 2021

Describe the pull request
Adds new port. Depends on:

Copy link
Copy Markdown

@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!

PRs must add only one version and must not modify any published versions

When making any changes to a library, the version or port-version in vcpkg.json or CONTROL must be modified.

Error: Local changes detected for kf5declarative but no changes to version or port version.
-- Version: 5.84.0
-- Old SHA: bce20c3b942e4f4403efcb239c0058955ac0c5dd
-- New SHA: c62865e53d9c10a985be2bb016df04aa9ba06968
-- Did you remember to update the version or port version?
-- Pass `--overwrite-version` to bypass this check.
***No files were updated.***
After committing all other changes, the version database must be updated
git add -u && git commit
git checkout a761a22de8a5b82503837e58630f9aec38f05d52 -- versions
./vcpkg x-add-version --all
Diff
diff --git a/versions/baseline.json b/versions/baseline.json
index cc146fd..166ad96 100644
--- a/versions/baseline.json
+++ b/versions/baseline.json
@@ -2964,6 +2964,10 @@
       "baseline": "5.84.0",
       "port-version": 0
     },
+    "kf5notifications": {
+      "baseline": "5.75.0",
+      "port-version": 0
+    },
     "kf5package": {
       "baseline": "5.84.0",
       "port-version": 0

@wrobelda wrobelda force-pushed the kf5declarative branch 2 times, most recently from 841f509 to 3469eff Compare August 31, 2021 02:36
Copy link
Copy Markdown

@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!

PRs must add only one version and must not modify any published versions

When making any changes to a library, the version or port-version in vcpkg.json or CONTROL must be modified.

Error: Local changes detected for kf5declarative but no changes to version or port version.
-- Version: 5.84.0
-- Old SHA: bce20c3b942e4f4403efcb239c0058955ac0c5dd
-- New SHA: c62865e53d9c10a985be2bb016df04aa9ba06968
-- Did you remember to update the version or port version?
-- Pass `--overwrite-version` to bypass this check.
***No files were updated.***

@NancyLi1013 NancyLi1013 added 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 labels Aug 31, 2021
Copy link
Copy Markdown

@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!

PRs must add only one version and must not modify any published versions

When making any changes to a library, the version or port-version in vcpkg.json or CONTROL must be modified.

Error: Local changes detected for kf5declarative but no changes to version or port version.
-- Version: 5.84.0
-- Old SHA: bce20c3b942e4f4403efcb239c0058955ac0c5dd
-- New SHA: c62865e53d9c10a985be2bb016df04aa9ba06968
-- Did you remember to update the version or port version?
-- Pass `--overwrite-version` to bypass this check.
***No files were updated.***
After committing all other changes, the version database must be updated
git add -u && git commit
git checkout 9fb1585e7a375cfa873697557ced116dfacbd1d7 -- versions
./vcpkg x-add-version --all
Diff
diff --git a/versions/baseline.json b/versions/baseline.json
index f7101cf..35a6f0f 100644
--- a/versions/baseline.json
+++ b/versions/baseline.json
@@ -2924,11 +2924,11 @@
       "baseline": "5.84.0",
       "port-version": 1
     },
-    "kf5globalaccel": {
+    "kf5declarative": {
       "baseline": "5.84.0",
       "port-version": 0
     },
-    "kf5declarative": {
+    "kf5globalaccel": {
       "baseline": "5.84.0",
       "port-version": 0
     },
@@ -2960,7 +2960,7 @@
       "baseline": "5.84.0",
       "port-version": 0
     },
-    "kf5package": {
+    "kf5kio": {
       "baseline": "5.84.0",
       "port-version": 0
     },
@@ -2968,7 +2968,7 @@
       "baseline": "5.84.0",
       "port-version": 0
     },
-    "kf5kio": {
+    "kf5package": {
       "baseline": "5.84.0",
       "port-version": 0
     },

Copy link
Copy Markdown

@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 9fb1585e7a375cfa873697557ced116dfacbd1d7 -- versions
./vcpkg x-add-version --all
Diff
diff --git a/versions/baseline.json b/versions/baseline.json
index f7101cf..35a6f0f 100644
--- a/versions/baseline.json
+++ b/versions/baseline.json
@@ -2924,11 +2924,11 @@
       "baseline": "5.84.0",
       "port-version": 1
     },
-    "kf5globalaccel": {
+    "kf5declarative": {
       "baseline": "5.84.0",
       "port-version": 0
     },
-    "kf5declarative": {
+    "kf5globalaccel": {
       "baseline": "5.84.0",
       "port-version": 0
     },
@@ -2960,7 +2960,7 @@
       "baseline": "5.84.0",
       "port-version": 0
     },
-    "kf5package": {
+    "kf5kio": {
       "baseline": "5.84.0",
       "port-version": 0
     },
@@ -2968,7 +2968,7 @@
       "baseline": "5.84.0",
       "port-version": 0
     },
-    "kf5kio": {
+    "kf5package": {
       "baseline": "5.84.0",
       "port-version": 0
     },

Copy link
Copy Markdown

@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 9fb1585e7a375cfa873697557ced116dfacbd1d7 -- versions
./vcpkg x-add-version --all
Diff
diff --git a/versions/baseline.json b/versions/baseline.json
index f7101cf..35a6f0f 100644
--- a/versions/baseline.json
+++ b/versions/baseline.json
@@ -2924,11 +2924,11 @@
       "baseline": "5.84.0",
       "port-version": 1
     },
-    "kf5globalaccel": {
+    "kf5declarative": {
       "baseline": "5.84.0",
       "port-version": 0
     },
-    "kf5declarative": {
+    "kf5globalaccel": {
       "baseline": "5.84.0",
       "port-version": 0
     },
@@ -2960,7 +2960,7 @@
       "baseline": "5.84.0",
       "port-version": 0
     },
-    "kf5package": {
+    "kf5kio": {
       "baseline": "5.84.0",
       "port-version": 0
     },
@@ -2968,7 +2968,7 @@
       "baseline": "5.84.0",
       "port-version": 0
     },
-    "kf5kio": {
+    "kf5package": {
       "baseline": "5.84.0",
       "port-version": 0
     },

Copy link
Copy Markdown

@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 9fb1585e7a375cfa873697557ced116dfacbd1d7 -- versions
./vcpkg x-add-version --all
Diff
diff --git a/versions/baseline.json b/versions/baseline.json
index f7101cf..35a6f0f 100644
--- a/versions/baseline.json
+++ b/versions/baseline.json
@@ -2924,11 +2924,11 @@
       "baseline": "5.84.0",
       "port-version": 1
     },
-    "kf5globalaccel": {
+    "kf5declarative": {
       "baseline": "5.84.0",
       "port-version": 0
     },
-    "kf5declarative": {
+    "kf5globalaccel": {
       "baseline": "5.84.0",
       "port-version": 0
     },
@@ -2960,7 +2960,7 @@
       "baseline": "5.84.0",
       "port-version": 0
     },
-    "kf5package": {
+    "kf5kio": {
       "baseline": "5.84.0",
       "port-version": 0
     },
@@ -2968,7 +2968,7 @@
       "baseline": "5.84.0",
       "port-version": 0
     },
-    "kf5kio": {
+    "kf5package": {
       "baseline": "5.84.0",
       "port-version": 0
     },

Copy link
Copy Markdown

@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 9fb1585e7a375cfa873697557ced116dfacbd1d7 -- versions
./vcpkg x-add-version --all
Diff
diff --git a/versions/baseline.json b/versions/baseline.json
index 10f578b..35a6f0f 100644
--- a/versions/baseline.json
+++ b/versions/baseline.json
@@ -2924,11 +2924,11 @@
       "baseline": "5.84.0",
       "port-version": 1
     },
-    "kf5globalaccel": {
+    "kf5declarative": {
       "baseline": "5.84.0",
       "port-version": 0
     },
-    "kf5declarative": {
+    "kf5globalaccel": {
       "baseline": "5.84.0",
       "port-version": 0
     },
@@ -2960,15 +2960,15 @@
       "baseline": "5.84.0",
       "port-version": 0
     },
-    "kf5package": {
+    "kf5kio": {
       "baseline": "5.84.0",
       "port-version": 0
     },
-    "kf5kio": {
+    "kf5notifications": {
       "baseline": "5.84.0",
       "port-version": 0
     },
-    "kf5notifications": {
+    "kf5package": {
       "baseline": "5.84.0",
       "port-version": 0
     },

Copy link
Copy Markdown

@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 5bc27e8a5073ba139e29432a1b0d8ced50794a41 -- versions
./vcpkg x-add-version --all
Diff
diff --git a/versions/baseline.json b/versions/baseline.json
index fd3a461..3eb38c8 100644
--- a/versions/baseline.json
+++ b/versions/baseline.json
@@ -2932,7 +2932,6 @@
       "baseline": "5.84.0",
       "port-version": 0
     },
-
     "kf5guiaddons": {
       "baseline": "5.84.0",
       "port-version": 2

Copy link
Copy Markdown

@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 5bc27e8a5073ba139e29432a1b0d8ced50794a41 -- versions
./vcpkg x-add-version --all
Diff
diff --git a/versions/baseline.json b/versions/baseline.json
index e1a3e04..feb5be0 100644
--- a/versions/baseline.json
+++ b/versions/baseline.json
@@ -2932,7 +2932,6 @@
       "baseline": "5.84.0",
       "port-version": 0
     },
-
     "kf5guiaddons": {
       "baseline": "5.84.0",
       "port-version": 2
@@ -2972,7 +2971,7 @@
     "kf5package": {
       "baseline": "5.84.0",
       "port-version": 0
-    },    
+    },
     "kf5plotting": {
       "baseline": "5.84.0",
       "port-version": 1

Copy link
Copy Markdown

@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 5d734efcef674dbf9b7b204484fc8f006101dcb5 -- versions
./vcpkg x-add-version --all
Diff
diff --git a/versions/baseline.json b/versions/baseline.json
index 783aad3..39de514 100644
--- a/versions/baseline.json
+++ b/versions/baseline.json
@@ -2992,7 +2992,6 @@
       "baseline": "5.84.0",
       "port-version": 0
     },
-    },
     "kf5globalaccel": {
       "baseline": "5.84.0",
       "port-version": 0
@@ -3025,11 +3024,11 @@
       "baseline": "5.84.0",
       "port-version": 0
     },
-    "kf5notifications": {
+    "kf5kio": {
       "baseline": "5.84.0",
       "port-version": 0
     },
-    "kf5kio": {
+    "kf5notifications": {
       "baseline": "5.84.0",
       "port-version": 0
     },

Copy link
Copy Markdown

@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 5d734efcef674dbf9b7b204484fc8f006101dcb5 -- versions
./vcpkg x-add-version --all
Diff
diff --git a/versions/baseline.json b/versions/baseline.json
index f30b096..39de514 100644
--- a/versions/baseline.json
+++ b/versions/baseline.json
@@ -3024,11 +3024,11 @@
       "baseline": "5.84.0",
       "port-version": 0
     },
-    "kf5notifications": {
+    "kf5kio": {
       "baseline": "5.84.0",
       "port-version": 0
     },
-    "kf5kio": {
+    "kf5notifications": {
       "baseline": "5.84.0",
       "port-version": 0
     },

Copy link
Copy Markdown

@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!

PRs must add only one version and must not modify any published versions

When making any changes to a library, the version or port-version in vcpkg.json or CONTROL must be modified.

Error: Local changes detected for kf5declarative but no changes to version or port version.
-- Version: 5.84.0
-- Old SHA: c62865e53d9c10a985be2bb016df04aa9ba06968
-- New SHA: dc898926b69ad5ef32655c80d3cc2acdf47d166f
-- Did you remember to update the version or port version?
-- Pass `--overwrite-version` to bypass this check.
***No files were updated.***
You have modified or added at least one portfile where deprecated functions are used.
Details

If you feel able to do so, please consider migrating them to the new functions:
vcpkg_install_cmake -> vcpkg_cmake_install (from port vcpkg-cmake)
vcpkg_build_cmake -> vcpkg_cmake_build (from port vcpkg-cmake)
vcpkg_configure_cmake -> vcpkg_cmake_configure (Please remove the option PREFER_NINJA) (from port vcpkg-cmake)
vcpkg_fixup_cmake_targets -> vcpkg_cmake_config_fixup (from port vcpkg-cmake-config)

In the ports that use the new function, you have to add the corresponding dependencies:

{
  "name": "vcpkg-cmake",
  "host": true
},
{
  "name": "vcpkg-cmake-config",
  "host": true
}

The following files are affected:

  • ports/ecm/portfile.cmake
  • ports/kf5bookmarks/portfile.cmake
  • ports/kf5declarative/portfile.cmake
  • ports/kf5kio/portfile.cmake
  • ports/kf5notifications/portfile.cmake
  • ports/kf5xmlgui/portfile.cmake
  • ports/libcanberra/portfile.cmake

@wrobelda
Copy link
Copy Markdown
Contributor Author

@NancyLi1013 no longer has dependencies, ready to be merged.

@NancyLi1013 NancyLi1013 removed the depends:different-pr This PR or Issue depends on a PR which has been filed label Oct 21, 2021
@NancyLi1013 NancyLi1013 added info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this. and removed requires:author-response labels Oct 22, 2021
@NancyLi1013
Copy link
Copy Markdown
Contributor

LGTM now, thanks for adding this new port. @wrobelda

@vicroms vicroms merged commit a3aef4e into microsoft:master Oct 22, 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! info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants