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

json2ts: fix build on darwin; allow case insensitive import #358398

Merged
merged 1 commit into from
Nov 23, 2024

Conversation

hsjobeki
Copy link
Contributor

@hsjobeki hsjobeki commented Nov 23, 2024

Darwins case (in-)sensitive file-system behavior of typescript causes this package to fail:

https://www.typescriptlang.org/tsconfig/#forceConsistentCasingInFileNames default: true

I used jq to set it to false in postConfigure.

Things done

  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
    • sandbox = relaxed
    • sandbox = true
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
  • Fits CONTRIBUTING.md.

Add a 👍 reaction to pull requests you find important.

#ZurichZHF

@hsjobeki hsjobeki added 0.kind: ZHF Fixes Fixes during the Zero Hydra Failures (ZHF) campaign backport release-24.11 Backport PR automatically labels Nov 23, 2024
@hsjobeki hsjobeki force-pushed the fix/json2ts branch 3 times, most recently from f33d55a to f07a550 Compare November 23, 2024 11:42
Copy link
Contributor

@imincik imincik left a comment

Choose a reason for hiding this comment

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

Diff looks good.

Manually tested on darwin

nix-build --argstr system aarch64-darwin -A json2ts
/nix/store/g6s4342v2n0x4bg88c6r1j343xdh5vkk-json2ts

@imincik imincik self-assigned this Nov 23, 2024
@hsjobeki
Copy link
Contributor Author

Fiy: bcherny/json-schema-to-typescript#644

We can remove the fix once this is fixed upstream.

Copy link
Member

@tricktron tricktron left a comment

Choose a reason for hiding this comment

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

Builds on aarch64-darwin and x86_64-darwin

@wegank
Copy link
Member

wegank commented Nov 23, 2024

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 358398


aarch64-darwin

✅ 1 package built:
  • json2ts

@wegank
Copy link
Member

wegank commented Nov 23, 2024

@ofborg build json2ts json2ts.passthru.tests

@wegank wegank merged commit 2960631 into NixOS:master Nov 23, 2024
19 of 21 checks passed
Copy link
Contributor

Successfully created backport PR for release-24.11:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
0.kind: ZHF Fixes Fixes during the Zero Hydra Failures (ZHF) campaign backport release-24.11 Backport PR automatically
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants