Skip to content

Add support for more platforms - #61

Draft
xtqqczze wants to merge 10 commits into
djc:mainfrom
xtqqczze:target-aix
Draft

xtqqczze wants to merge 10 commits into
djc:mainfrom
xtqqczze:target-aix

Conversation

@xtqqczze

@xtqqczze xtqqczze commented Sep 17, 2026 •

Copy link
Copy Markdown

Closes #60

Verified locally with:

RUSTC_BOOTSTRAP=1 cargo clippy -q -Zbuild-std \
  --workspace --all-targets --all-features \
  --target powerpc64-ibm-aix \
  --target aarch64-apple-ios \
  --target aarch64-apple-ios-macabi \
  --target aarch64-apple-ios-sim \
  --target aarch64-apple-tvos \
  --target aarch64-apple-tvos-sim \
  --target aarch64-apple-visionos \
  --target aarch64-apple-visionos-sim \
  --target aarch64-apple-watchos \
  --target aarch64-apple-watchos-sim \
  --target x86_64-apple-ios \
  --target x86_64-apple-ios-macabi

@xtqqczze xtqqczze changed the title feat: add AIX support Add AIX support Sep 17, 2026
@xtqqczze xtqqczze changed the title Add AIX support Add support for more platforms Sep 17, 2026
@xtqqczze
xtqqczze force-pushed the target-aix branch 3 times, most recently from 83291ea to e5cd215 Compare September 17, 2026 01:51
@xtqqczze

Copy link
Copy Markdown
Author

@djc Hopefully this can be backported, I'd like to avoid taking a dependency on windows-link v0.100.0.

@djc

djc commented Sep 17, 2026

Copy link
Copy Markdown
Owner

@djc Hopefully this can be backported, I'd like to avoid taking a dependency on windows-link v0.100.0.

Why?

@xtqqczze

Copy link
Copy Markdown
Author

windows-link v0.100.0 was only released 14 days ago, and most crates haven't adopted it yet. Using it here would therefore introduce a duplicate dependency for uutils.

@djc

djc commented Sep 18, 2026

Copy link
Copy Markdown
Owner

Okay, I think we can get out a 0.4.3 release with this. Suggest you add commits (a) reverting the offending commits before your change, (b) bumping the version to 0.4.3, (c) adding back the reverted changes all on this branch. After merging this, I will tag and publish the version bump.

@xtqqczze
xtqqczze force-pushed the target-aix branch 2 times, most recently from b9266f6 to e942ee2 Compare September 18, 2026 11:56
@xtqqczze
xtqqczze marked this pull request as draft September 18, 2026 12:03
@xtqqczze

Copy link
Copy Markdown
Author

@djc Why isn't CI running?

@djc

djc commented Sep 18, 2026

Copy link
Copy Markdown
Owner

No clue -- probably a GitHub fluke?

@xtqqczze

Copy link
Copy Markdown
Author

I don't think it's a fluke, it consistently isn't running, so I suspect it's an issue with the repo (maybe permissions?).

@djc
djc marked this pull request as ready for review September 18, 2026 12:59
@djc

djc commented Sep 18, 2026

Copy link
Copy Markdown
Owner

Can you try another push?

@xtqqczze

Copy link
Copy Markdown
Author

I checked the Actions dashboard, no workflow runs were even created.

@djc

djc commented Sep 18, 2026

Copy link
Copy Markdown
Owner

Ugh, you're right. Fixing in

@djc

djc commented Sep 18, 2026

Copy link
Copy Markdown
Owner

Alright, please rebase on current main?

@xtqqczze
xtqqczze marked this pull request as draft September 18, 2026 13:59
@xtqqczze

Copy link
Copy Markdown
Author

@djc I'm working on some CI improvements.

Just to confirm, is it desirable to have CI for every Apple target listed in the Tier 2 with Host Tools documentation?

@xtqqczze
xtqqczze force-pushed the target-aix branch 2 times, most recently from 7c38eb4 to d139853 Compare September 18, 2026 14:37
@xtqqczze
xtqqczze force-pushed the target-aix branch 2 times, most recently from fc270c6 to 8b4f7d5 Compare September 18, 2026 14:43
@djc

djc commented Sep 18, 2026

Copy link
Copy Markdown
Owner

Just to confirm, is it desirable to have CI for every Apple target listed in the Tier 2 with Host Tools documentation?

I inherited the current matrix and have not spent much time on it. I'm not eager to expand it more for theoretical use cases but if you're building for a particular platform I'm happy to have it added. I expect it could be a fair bit simpler by, for example, collapsing all Apple targets into a single one.

```sh
RUSTC_BOOTSTRAP=1 cargo clippy -q -Zbuild-std \
  --workspace --all-targets --all-features \
  --target aarch64-apple-ios \
  --target aarch64-apple-ios-macabi \
  --target aarch64-apple-ios-sim \
  --target aarch64-apple-tvos \
  --target aarch64-apple-tvos-sim \
  --target aarch64-apple-visionos \
  --target aarch64-apple-visionos-sim \
  --target aarch64-apple-watchos \
  --target aarch64-apple-watchos-sim \
  --target x86_64-apple-ios \
  --target x86_64-apple-ios-macabi
```
@xtqqczze

xtqqczze commented Sep 18, 2026 •

Copy link
Copy Markdown
Author

sethostname and gethostname have the same definition in libc across all Apple targets. Since there’s no need to test each target purely for completeness, I’ve removed the additional targets like aarch64-apple-visionos-sim from CI.

@djc

djc commented Sep 18, 2026

Copy link
Copy Markdown
Owner

The "refactor MSRV job" reads like it's doing a whole bunch of stuff at the same time. Either split that out or clarify in the commit message what goals/concerns it's trying to address.

@xtqqczze

Copy link
Copy Markdown
Author

At this point, it may be worth splitting the PR to separate the CI changes from the additional target support?

@xtqqczze xtqqczze mentioned this pull request Sep 18, 2026
@djc

djc commented Sep 18, 2026

Copy link
Copy Markdown
Owner

At this point, it may be worth splitting the PR to separate the CI changes from the additional target support?

Yes, please!

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.

Fails to compile on AIX

2 participants