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

[speex] Build every triplets with CMake and added CMake config files #26487

Closed
wants to merge 2 commits into from

Conversation

Sharpe49
Copy link
Contributor

Builds the package with CMake for every triplets
Added CMake config files

  • What does your PR fix?

    None

  • Which triplets are supported/not supported? Have you updated the CI baseline?

    all, No

  • Does your PR follow the maintainer guide?

    Yes

  • If you have added/updated a port: Have you run ./vcpkg x-add-version --all and committed the result?

    I am still working on this PR

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 88b1071e39f13b632644d9d953738d345a4ac055 -- versions
./vcpkg x-add-version --all
Diff
diff --git a/versions/baseline.json b/versions/baseline.json
index ae7869d..c7c153f 100644
--- a/versions/baseline.json
+++ b/versions/baseline.json
@@ -6838,7 +6838,7 @@
     },
     "speex": {
       "baseline": "1.2.1",
-      "port-version": 0
+      "port-version": 1
     },
     "speexdsp": {
       "baseline": "1.2.1",
diff --git a/versions/s-/speex.json b/versions/s-/speex.json
index 092587c..21b8755 100644
--- a/versions/s-/speex.json
+++ b/versions/s-/speex.json
@@ -1,5 +1,10 @@
 {
   "versions": [
+    {
+      "git-tree": "64f8faf78a155f020441207440e5da22aaa43081",
+      "version": "1.2.1",
+      "port-version": 1
+    },
     {
       "git-tree": "0520509fd60e20c5362959a7549dd29f7d04d54b",
       "version": "1.2.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 88b1071e39f13b632644d9d953738d345a4ac055 -- versions
./vcpkg x-add-version --all
Diff
diff --git a/versions/baseline.json b/versions/baseline.json
index ae7869d..c7c153f 100644
--- a/versions/baseline.json
+++ b/versions/baseline.json
@@ -6838,7 +6838,7 @@
     },
     "speex": {
       "baseline": "1.2.1",
-      "port-version": 0
+      "port-version": 1
     },
     "speexdsp": {
       "baseline": "1.2.1",
diff --git a/versions/s-/speex.json b/versions/s-/speex.json
index 092587c..a59f784 100644
--- a/versions/s-/speex.json
+++ b/versions/s-/speex.json
@@ -1,5 +1,10 @@
 {
   "versions": [
+    {
+      "git-tree": "394913cea6fddf43847d42fa73eb731cce1442eb",
+      "version": "1.2.1",
+      "port-version": 1
+    },
     {
       "git-tree": "0520509fd60e20c5362959a7549dd29f7d04d54b",
       "version": "1.2.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 88b1071e39f13b632644d9d953738d345a4ac055 -- versions
./vcpkg x-add-version --all
Diff
diff --git a/versions/baseline.json b/versions/baseline.json
index ae7869d..c7c153f 100644
--- a/versions/baseline.json
+++ b/versions/baseline.json
@@ -6838,7 +6838,7 @@
     },
     "speex": {
       "baseline": "1.2.1",
-      "port-version": 0
+      "port-version": 1
     },
     "speexdsp": {
       "baseline": "1.2.1",
diff --git a/versions/s-/speex.json b/versions/s-/speex.json
index 092587c..2547b14 100644
--- a/versions/s-/speex.json
+++ b/versions/s-/speex.json
@@ -1,5 +1,10 @@
 {
   "versions": [
+    {
+      "git-tree": "4c4de325e799efb2d5d857c389475a99cba9e80a",
+      "version": "1.2.1",
+      "port-version": 1
+    },
     {
       "git-tree": "0520509fd60e20c5362959a7549dd29f7d04d54b",
       "version": "1.2.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 88b1071e39f13b632644d9d953738d345a4ac055 -- versions
./vcpkg x-add-version --all
Diff
diff --git a/versions/baseline.json b/versions/baseline.json
index ae7869d..c7c153f 100644
--- a/versions/baseline.json
+++ b/versions/baseline.json
@@ -6838,7 +6838,7 @@
     },
     "speex": {
       "baseline": "1.2.1",
-      "port-version": 0
+      "port-version": 1
     },
     "speexdsp": {
       "baseline": "1.2.1",
diff --git a/versions/s-/speex.json b/versions/s-/speex.json
index 092587c..fdddc46 100644
--- a/versions/s-/speex.json
+++ b/versions/s-/speex.json
@@ -1,5 +1,10 @@
 {
   "versions": [
+    {
+      "git-tree": "c9e494dbe25cfa472e9f89b216f7e1b28993225c",
+      "version": "1.2.1",
+      "port-version": 1
+    },
     {
       "git-tree": "0520509fd60e20c5362959a7549dd29f7d04d54b",
       "version": "1.2.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 88b1071e39f13b632644d9d953738d345a4ac055 -- versions
./vcpkg x-add-version --all
Diff
diff --git a/versions/baseline.json b/versions/baseline.json
index ae7869d..c7c153f 100644
--- a/versions/baseline.json
+++ b/versions/baseline.json
@@ -6838,7 +6838,7 @@
     },
     "speex": {
       "baseline": "1.2.1",
-      "port-version": 0
+      "port-version": 1
     },
     "speexdsp": {
       "baseline": "1.2.1",
diff --git a/versions/s-/speex.json b/versions/s-/speex.json
index 092587c..dc38ed6 100644
--- a/versions/s-/speex.json
+++ b/versions/s-/speex.json
@@ -1,5 +1,10 @@
 {
   "versions": [
+    {
+      "git-tree": "cc743c1d22cec3808f7c55da9bfae913cf78e065",
+      "version": "1.2.1",
+      "port-version": 1
+    },
     {
       "git-tree": "0520509fd60e20c5362959a7549dd29f7d04d54b",
       "version": "1.2.1",

@Adela0814 Adela0814 added the category:port-feature The issue is with a library, which is requesting new capabilities that didn’t exist label Aug 24, 2022
@dg0yt
Copy link
Contributor

dg0yt commented Aug 24, 2022

Did you consider using the original build system for all triplets instead? I know it is slower on Windows, but having the buildsystem maintenance in vcpkg (and no tests!) is a problem in some other ports already. vcpkg_configure_make has improved a lot since this port was added, and pkg-config support, too.

@Sharpe49
Copy link
Contributor Author

Sharpe49 commented Aug 24, 2022

I have just tried it : it's completely broken with MSVC. I get build errors coming from everywhere.
And I am really not used to autoconf and automake, so I can't try and fix those files.

@dg0yt
Copy link
Contributor

dg0yt commented Aug 24, 2022

Looks like there was just a tiny showstopper: xiph/speex#20

Full alternative PR: #26502

@Sharpe49
Copy link
Contributor Author

Ok, closing this pull request.
I would have prefered to use unofficial config files instead of pkg-config, but anyway...

@Sharpe49 Sharpe49 closed this Aug 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:port-feature The issue is with a library, which is requesting new capabilities that didn’t exist
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants