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

doesn't compile on Ubuntu installed on Jetson #185

Closed
agiordana opened this issue Jan 6, 2023 · 5 comments
Closed

doesn't compile on Ubuntu installed on Jetson #185

agiordana opened this issue Jan 6, 2023 · 5 comments

Comments

@agiordana
Copy link

I'm trying to use mqtt on jetson nano. Cargo fail running cmake because of the flag --parallel not supported.
looking in the code I've found paho-mqtt-sys/cCargo.toml, where there is a comment about using dependencies from cmake crate v0.1.45 because the fleg --parallel is not supported everywery.
As a matter of fact the dependency in the build-dependencies is set to v0.1.48!

@fpagliughi
Copy link
Contributor

fpagliughi commented Jan 6, 2023

Hey! Coincidentally, I just received two Nano's in the post, so I can try this out.

IIRC, the cmake crate was changed in v0.1.46 to use the --parallel flag, but that only appeared in the CMake executable as of v3.12. So this instantly broke the Paho build for slightly older distro's. I had pinned the crate to v0.1.45 for a while to work around this.

But supposedly this was fixed:
rust-lang/cmake-rs#146

(ironically, the fix for v0.1.46 was in PR 146?!?)

Maybe that PR was merged, but not released?

I took it on faith that this was no longer an issue and upgraded to the latest release of the cmake crate, but perhaps my assumption was incorrect. I can check. But I wasn't able to test it because I had upgraded CMake on all my development machines to something like v3.25.

So... I will look to get it back working with some older systems, but really, my advice it to upgrade the CMake executable on your Nano to something more recent. Certainly something >= v3.12.

@fpagliughi
Copy link
Contributor

OK. It looks like that PR 146 didn't make it into a release until cmake crate v0.1.49. But I had accidentally let the version pinned to the one specified even after I updated it.

I will do a new release of the -sys crate later today, with the fix, and then it should build on your Nano.

@agiordana
Copy link
Author

agiordana commented Jan 6, 2023 via email

@fpagliughi
Copy link
Contributor

fpagliughi commented Jan 7, 2023

You should also be able to move it forward to the latest release:

cmake = { version = "0.1.49", optional = true }

(get rid of the equals while you’re at it) and that will work. That’s in the release I just put out, so just run cargo update and that should work,too.

I’ll close this, but feel free to re-open if there’s any other problem.

@agiordana
Copy link
Author

agiordana commented Jan 7, 2023 via email

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

2 participants