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

Increase the MSRV presubmit checks to include multiple architectures #197

Merged
merged 8 commits into from
Feb 11, 2024

Conversation

tkaitchuck
Copy link
Owner

No description provided.

Signed-off-by: Tom Kaitchuck <tom.kaitchuck@emc.com>
Signed-off-by: Tom Kaitchuck <tom.kaitchuck@emc.com>
Signed-off-by: Tom Kaitchuck <tom.kaitchuck@emc.com>
@alamb
Copy link

alamb commented Jan 8, 2024

https://github.com/tkaitchuck/aHash/actions/runs/7449693417/job/20266892686?pr=197#step:5:1

Seems to imply it just installed 1.60 in addition to the master build already installed

Maybe you need to change

  cargo check --target armv7-unknown-linux-gnueabihf

To explicitly mention the tool chain like

  cargo +1.60 check --target armv7-unknown-linux-gnueabihf

FWIW we use cargo msrv to check this in datafusion

https://github.com/apache/arrow-datafusion/blob/746988a7e5c9f256c3b24ab7e3f30ffd90d542a0/.github/workflows/rust.yml#L546-L568

@tkaitchuck
Copy link
Owner Author

It runs rustup default 1.60.0 is this insufficient?

Signed-off-by: Tom Kaitchuck <tom.kaitchuck@emc.com>
@tkaitchuck
Copy link
Owner Author

I added this, everything is still passing...

tkaitchuck and others added 4 commits January 8, 2024 15:37
Signed-off-by: Tom Kaitchuck <tom.kaitchuck@emc.com>
Signed-off-by: Tom Kaitchuck <Tom.Kaitchuck@gmail.com>
Signed-off-by: Tom Kaitchuck <Tom.Kaitchuck@gmail.com>
@tkaitchuck
Copy link
Owner Author

It looks like it used to pass with 1.60 but then failed with 1.71
So the command was correct, it was just a discrepancy in what rust supported.

@tkaitchuck tkaitchuck merged commit 722ca8a into master Feb 11, 2024
18 checks passed
@tkaitchuck tkaitchuck deleted the presubmit branch February 11, 2024 05:06
tkaitchuck added a commit that referenced this pull request Feb 11, 2024
…197)

Increase the MSRV presubmit checks to include multiple architectures

Signed-off-by: Tom Kaitchuck <Tom.Kaitchuck@gmail.com>
@alamb
Copy link

alamb commented Feb 11, 2024

🎉

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.

None yet

2 participants