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

CFIndex not found in this scope error #619

Open
ckampfe opened this issue Jul 24, 2023 · 5 comments
Open

CFIndex not found in this scope error #619

ckampfe opened this issue Jul 24, 2023 · 5 comments

Comments

@ckampfe
Copy link

ckampfe commented Jul 24, 2023

It's possible something in the recent core-foundation-sys update broke. I'm getting this on a compile of an application with core-foundation at 0.9.3 and core-foundation-sys at 0.8.5:

   Compiling core-foundation v0.9.3
error[E0412]: cannot find type `CFIndex` in this scope
  --> /Users/clark/.cargo/registry/src/index.crates.io-6f17d22bba15001f/core-foundation-0.9.3/src/mach_port.rs:17:16
   |
17 |         order: CFIndex,
   |                ^^^^^^^ not found in this scope
   |
help: consider importing this type alias
   |
1  + use base::CFIndex;
   |

I do not directly depend on either core-foundation or core-foundation-sys, they are pulled in by libraries I use.

@tonyalaribe
Copy link

I just started getting the same error.
core-foundation needs a new release. The latest release 0.9.3 is old and is missing an import for CFIndex. But this issue was already fixed on master, so a 0.9.4 release from latest master seems to be needed.

Here's the code with the issue:


That import line is missing CFIndex, which is present on master

@guglielmino
Copy link

Same problem here, with versione 0.9.3
Needed to add use base::CFIndex; in core-foundation-0.9.3/src/mach_port.rs to fix it.

@hjmallon
Copy link

We need a release of code-foundation to go with the core-foundation-sys release I think. This commit is the troublesome one. 6d00383

@jrmuizel
Copy link
Collaborator

I've yanked core-foundation-sys 0.8.5

jrmuizel added a commit to jrmuizel/core-foundation-rs that referenced this issue Jul 24, 2023
This reverts commit 6d00383.

It broke semver because it removed the export of CFIndex which
which older versions of the core-foundation crate are depending on.

Fixes servo#619
jrmuizel added a commit to jrmuizel/core-foundation-rs that referenced this issue Jul 24, 2023
This reverts commit 6d00383.

It broke semver because it removed the export of CFIndex which
which older versions of the core-foundation crate are depending on.

Fixes servo#619
github-merge-queue bot pushed a commit that referenced this issue Jul 24, 2023
This reverts commit 6d00383.

It broke semver because it removed the export of CFIndex which
which older versions of the core-foundation crate are depending on.

Fixes #619
@jrmuizel jrmuizel mentioned this issue Jul 24, 2023
@micolous
Copy link
Contributor

Given #620 is merged, can there be a new release again?

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

No branches or pull requests

6 participants