Skip to content

Add support for Motor OS#200

Draft
lasiotus wants to merge 1 commit into
nagisa:masterfrom
moturus:master
Draft

Add support for Motor OS#200
lasiotus wants to merge 1 commit into
nagisa:masterfrom
moturus:master

Conversation

@lasiotus

Copy link
Copy Markdown

This noop change makes rust_libloading compile for Motor OS target (see https://doc.rust-lang.org/rustc/platform-support/motor.html)

Comment thread src/os/unix/consts.rs
target_env = "uclibc",
all(target_os = "linux", target_arch = "mips"),
all(target_os = "linux", target_arch = "mips64"),
all(target_os = "linux", target_os = "motor", target_arch = "mips"),

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This ain't right.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Obviously. Sorry about that. Git merge error? Human error? Let me fix this (give me a couple of days - busy at my day job). I'll convert this into a draft.

Comment thread src/os/unix/consts.rs
pub(super) const RTLD_NOW: c_int = 1;
} else if #[cfg(any(
target_os = "linux",
target_os = "linux", target_os = "motor",

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

formatting

Comment thread src/os/mod.rs
//! ```ignore
//! # extern crate libloading;
//! #[cfg(unix)]
//! #[cfg(any(unix, target_os = "motor"))]

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure I want to pollute the docs with this; at the same time if motor is unix-like, why does it not have unix cfg set?

@lasiotus
lasiotus marked this pull request as draft July 21, 2026 02:23
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.

2 participants