Skip to content

Pgsql cube type compile fail#3459

Merged
abonander merged 3 commits intolaunchbadge:mainfrom
kdesjard:pgsql_cube_type_compile_fail
Aug 26, 2024
Merged

Pgsql cube type compile fail#3459
abonander merged 3 commits intolaunchbadge:mainfrom
kdesjard:pgsql_cube_type_compile_fail

Conversation

@kdesjard
Copy link
Copy Markdown
Contributor

@kdesjard kdesjard commented Aug 24, 2024

Does your PR solve an issue?

Resolves #3461
Resolves #3460
Replaces #3457
Replaces #3456

The Postgresql cube type fails to compile in version 0.8.1:

.../sqlx-postgres-0.8.1/src/types/cube.rs:307:33
    |
307 |     const PACKED_WIDTH: usize = size_of::<u32>();
    |                                 ^^^^^^^ not found in this scope
    |
help: consider importing one of these items
    |
1   + use core::mem::size_of;
    |
1   + use std::mem::size_of;

The PR includes a patch for this as well as a scoping change for the hstore type to keep it consistent with the others.

@CommanderStorm
Copy link
Copy Markdown
Contributor

CommanderStorm commented Aug 24, 2024

Could you add the following to your PR-description? (this way, other users have a link to follow)

Resolves #3461
Resolves #3460
Replaces #3457
Replaces #3456

@abonander
Copy link
Copy Markdown
Collaborator

@kdesjard cargo fmt needs to be run.

@imor
Copy link
Copy Markdown

imor commented Aug 26, 2024

@kdesjard could you please run cargo fmt and push a commit. Without this fix pg_replicate's MSRV needs to be bumped which we'd like to avoid.

@kdesjard
Copy link
Copy Markdown
Contributor Author

I did push a commit f47ce6b

Am I missing something?

@abonander abonander merged commit 371cf4a into launchbadge:main Aug 26, 2024
jrasanen pushed a commit to jrasanen/sqlx that referenced this pull request Oct 14, 2024
* fails to compile as size_of is not found in scope

* keep scoping consistent with other type modules

* fmt fixes

---------

Co-authored-by: kdesjard <kristian.desjardins@nrcan-rncan.gc.ca>
jrasanen pushed a commit to jrasanen/sqlx that referenced this pull request Oct 14, 2024
* fails to compile as size_of is not found in scope

* keep scoping consistent with other type modules

* fmt fixes

---------

Co-authored-by: kdesjard <kristian.desjardins@nrcan-rncan.gc.ca>
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

Successfully merging this pull request may close these issues.

const PACKED_WIDTH: usize = size_of::<u32>(); not found in this scope Building v0.8.1 fails with feature "postgres" enabled

5 participants